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

    +156.9

    1. 1
    2. 2
    3. 3
    4. 4
    elseif(isset($arParams["FILE_NAME_".$i])&&$arParams["FILE_NAME_".$i])
    {
    ....
    }

    Запостил: atarix12, 24 Декабря 2009

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

    • В чем говнокод?
      Ответить
    • А вдруг автор действительно хотел проверить, что
      $arParams["FILE_NAME_".$i]
      существует и не равен
      NULL
      и при этом он типа
      boolean
      и не равен
      false
      вместо того чтобы просто
      array_key_exists();


      Хотя, кто в это поверит при таких именах ключей.
      Ответить
      • Как насчет empty()?
        Ответить
      • > и при этом он типа
        > boolean
        > и не равен
        > false

        тогда уж надо было сразу писать &&$arParams["FILE_NAME_".$i]===true
        Ответить
    • есть замечательная функция empty(), которая возвращает true если переменная (или элемент массива) не существует, или пуст (0, null, FALSE, ''). Также, кстати возвращает тру для массива, не содержащего ни одного элемента
      Ответить
      • ... И не эквивалентна указанному в говнокоде условию.
        А вот ключи действительно странные.
        Ответить

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