1. PHP / Говнокод #739

    +142

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

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

    Запостил: guest, 18 Марта 2009

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

    • ava guest 19.03.2009 10:46 # 0

      Тролль:
      А, кстати, нормальная проверка. Что не так?
      Ответить

    • ava guest 19.03.2009 11:12 # 0

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

    • ava guest 19.03.2009 16:04 # 0

      что не нравится то?
      вроде ничего такого говняного и нет
      Ответить

    • ava guest 19.03.2009 16:21 # 0

      недоТролль:
      2 Говногости,

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

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

    • ava guest 19.03.2009 17:01 # 0

      [quote]
      2 Говногости,

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

      давайте ещё булевы значения, как в древнем бояне, tostring().length() < 5 проверять
      [/quote]
      эмпти вернет фолс если пароль есть один или несколько пробелов. А пароли из пробелов это нифига некошерно.
      Вполне нормальное сравнение имхо
      Ответить

    • ava guest 19.03.2009 17:28 # 0

      Иван:
      А использовать empty дело вкуса. Просто надо знать как PHP приводит разные значения к булевому типу. Лично я никогда не пользовался empty
      Ответить

    • ava guest 19.03.2009 18:26 # 0

      аноним:
      я бы сделал проверку таким же образом.
      Ответить

    • ava guest 19.03.2009 22:47 # 0

      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)).

      НеДотроль СХ
      Ответить

    • ava guest 19.03.2009 23:40 # 0

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

    • ava guest 20.03.2009 00:18 # +1

      вяк!
      Ответить

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