"PHP" / Говнокод #739 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
if ('' == trim($params['password'])) {
    echo 'Password cannot be empty';
    ..............

проверка на пустоту

Запостил: guest guest, (Updated )

Комментарии (10) RSS

  • Тролль:
    А, кстати, нормальная проверка. Что не так?
    Ответить
  • обоссаться, блин, не переменная слева, а сравниваемое значение.
    если не знаешь что есть говнокод - не высасывай из пальца.
    Ответить
  • что не нравится то?
    вроде ничего такого говняного и нет
    Ответить
  • недоТролль:
    2 Говногости,

    а как нащщёт empty($params['password']) ?

    давайте ещё булевы значения, как в древнем бояне, tostring().length() < 5 проверять
    Ответить
  • [quote]
    2 Говногости,

    а как нащщёт empty($params['password']) ?

    давайте ещё булевы значения, как в древнем бояне, tostring().length() < 5 проверять
    [/quote]
    эмпти вернет фолс если пароль есть один или несколько пробелов. А пароли из пробелов это нифига некошерно.
    Вполне нормальное сравнение имхо
    Ответить
  • Иван:
    А использовать empty дело вкуса. Просто надо знать как PHP приводит разные значения к булевому типу. Лично я никогда не пользовался empty
    Ответить
  • аноним:
    я бы сделал проверку таким же образом.
    Ответить
  • Graff:
    Проверка полностью корректна. У ДЭбилов возможно вызвал удивление факт нахождения сравниваемого значения СЛЕВА = это стандарт для старых сишников (которым как и мне 0коло 40 лет).

    Выше должно быть еще приведение $_POST или $_GET переменных к этому $params...

    Замечания комментаторов по empty совершенно корректны...

    Для сведения
    empty() only checks variables as anything else will result in a parse error. In other words, the following will not work: empty(trim($name)).

    НеДотроль СХ
    Ответить
  • Тролль:
    Так что, аффтар учи мат.часть или убей себя ап стенку!
    Ответить

Добавить комментарий

Где здесь C++, guest?!

    А не использовать ли нам bbcode?


    8