Помогите исправить собственное быдлокодерство

[craz]

Доброе утро. 5.3.

а 5.3 когда блин появилось? предлагаешь все написанное с 4 до 5.2 портировать что ль?

 

[gray]

Так лучше не писать

return $truefalse[$id]

если я передам туда к примеру число 10000, а массив

$truefalse

будет размером 10 элементов мы получим Notice: Undefined offset.Использовать isset в данном случае не лучший вариант,если передам в вашу функцию isAdmin,в место числа null, то меня пустят внутрь блока if.

Попробовал подавать на вход моей функции числа 10К и 100К и т.п. Все нормально работает. Нотайс не было замечено.
Попробовал на вход подать null. Да, в if пускает, но результат то всеравно получается false, т.к. в бд не может быть id с таким значением.

С empty поработаю, спасибо.

 

[Василий М.]

craz
ему лишь бы до"баться да поспорить, не обращай внимания )

 

[Фанат]

кстати да. обсуждение своих фреймворков переносите в отдельный топик.

 

[craz]

Ну почему же, это же примеры "хорошего" кода)

 

[AmdY]

Нормально делает я считаю, я вот никак не наберусь смелости) прилепил какой нить Craz_ClassName - и все пипец теперь это ТВОЙ код)

Очень зря этого не делаешь. Как только добавишь, появится стимул писать лучший реиспользуемый код.

 

[craz]

Очень зря этого не делаешь. Как только добавишь, появится стимул писать лучший реиспользуемый код.

боюсь пока не соответствовать, это как автограф художника.

 

[WMix]

Вурдалак

Доброе утро. 5.3.

и все бросились переписывать классы.....

 

[AmdY]

боюсь пока не соответствовать, это как автограф художника.

вот поэтому столько плюшек и получишь, потому как будешь стараться писать лучше, чтобы было не стыдно.

 

[weregod]

Доброе утро. 5.3.

уже писал про префиксы и пространства имён: при разработке продукта, который пишется со времён PHP 3 и гоняется на куче платформ:
1. сужать кол-во площадок через требование ">= PHP 5.3" глупо;
2. менять API вообще некошерно.

рекомендую префиксовать утверждения подобного типа чем-то вроде "новый проект"

 

[WMix]

боюсь пока не соответствовать, это как автограф художника.

а код всегда хочется доработать... нет времени вылизывать, или когда все вылизал вышла новая версия, или появился експириенс и перешел на новый левел... сколько не пишешь, тут чтот добавить, ах тут чтот проиппал, и там былоб лучше еслиб не скаляр а масив... бесконечно...

посморишь на прогу которой 2 года волосы дыбом...

 

[Вурдалак]

рекомендую префиксовать утверждения подобного типа чем-то вроде "новый проект"

О каком утверждении речь?

 

[Вурдалак]

Василий М., у тебя синглтон головного мозга.

 

[gray]

У меня вопрос к по продолжению темыФанат.
Собственно вопрос первого поста уже раскрыт, но заголовок "помогите исправить собственное быдлокодерство" актуален.
У меня имеются вопросы, которые не относятся к первоначальному коду.

Например. Как лучше че можно записать такую конструкцию

$meta = (isset($meta))?$meta:"";

php 5.2.1, потому конструкция

$meta = (isset($meta))?:"";

не работает.

Или такие вопросы задавать уже в новых топиках или на http://stackoverflow.com/ и им подобным?

 

[gray]

Уже нашел решение.
$meta=($meta?$meta:'');

 

[Фанат]

Как лучше че можно записать такую конструкцию

Я не люблю тернарный оператор, и по возможности стараюсь избегать его. Поэтому пишу так

if (!isset($meta)) {
    $meta = "";
}

Это немного длиннее писать, но (по моему мнению) на порядок легче читать.

Я, правда, не понял, к какому куску кода это относится.

А вот это

Уже нашел решение.
$meta=($meta?$meta:'');

и вовсе бессмысленная строчка. Наличие её в коде не несет ни малейшего смысла

 

[gray]

Я, правда, не понял, к какому куску кода это относится.

В том то и дело ни к какому из вышеприведенных. Вопрос состоял в том, задавать ли подобные вопросы в этой же теме "помогите исправить собственное быдлокодерство" или плодить новые? И привел пример того, какого рода вопросы у меня возникают.
Т.е. вот сейчас у меня целая серия вопросы про notice. Эти оповещения противоречат тому что указано в доках и разных сниппетах на php.net. Как в этом случае поступать, забивать болт на нотайсы или нет? Собственно отсюда и этот кусок кода:

$meta=($meta?$meta:'');

. И возмущением php что не на всех страницах программы я создал этот самый $meta для вывода.
Впрочем эту проблему я уже решил.
Но есть и другие.

Например есть конструкция:

switch ($_GET["f"])
{
  case "a":/*...*/break;
  case "b":/*...*/break;
  case "c":/*...*/break;
  default:/*действие по-умолчанию*/
}

На неё php ругается так:

Notice: Undefined index: f

Ругается на несуществующую переменную $_GET['f'].
Как в этом случае правильно оформить конструкцию, чтобы ругани не было или просто плюнуть?
Вариант сделать так:

if(isset($_GET["f"])){switch ($_GET["f"])
{
  case "a":/*...*/break;
  case "b":/*...*/break;
  case "c":/*...*/break;
  default:/*действие по-умолчанию*/
}
}
else{/*действие по-умолчанию*/}

 

[Фанат]

Собственно отсюда и этот кусок кода:
PHP:
$meta=($meta?$meta:'');

Не знаю, где ты взял этот кусок кода и как егого творчески переработал, но смысла он как не имел, так и не имеет.
Либо у тебя работает не он, либо эта переменная всегда определена, либо отключены нотисы, либо они таки сыпятся.

 

[WMix]

/**
 * Get an _GET parameter
 *
 * @param string $key
 * @param mixed $default Default value to use if key not found
 * @return mixed
 */
function getParam($key, $default = null){
    $key = (string) $key;
    if (isset($_GET[$key])) {
        return $_GET[$key];
    }
    return $default;
}

switch (getParam('f','default')){
  case "a":/*...*/break;
  case "b":/*...*/break;
  case "c":/*...*/break;
  default:/*действие по-умолчанию*/
}

 

[gray]

Не знаю, где ты взял этот кусок кода и как егого творчески переработал, но смысла он как не имел, так и не имеет.
Либо у тебя работает не он, либо эта переменная всегда определена, либо отключены нотисы, либо они таки сыпятся.

Всё верно, сыпятся notice. Написал глупость.

switch (getParam('f','default'))

Соображалку пока сложно перестроить на другую плоскость. Спасибо и за метод и за код.