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

    • ava Abductio 24.12.2009 12:01 # −1

      В чем говнокод?
      Ответить

    • ava guest 24.12.2009 12:06 # −1

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


      Хотя, кто в это поверит при таких именах ключей.
      Ответить

      • ava guest 24.12.2009 21:09 # 0

        Как насчет empty()?
        Ответить

      • ava guest 24.12.2009 21:13 # 0

        > и при этом он типа
        > boolean
        > и не равен
        > false

        тогда уж надо было сразу писать &&$arParams["FILE_NAME_".$i]===true
        Ответить

    • ava Yozheg 24.12.2009 12:20 # +3

      есть замечательная функция empty(), которая возвращает true если переменная (или элемент массива) не существует, или пуст (0, null, FALSE, ''). Также, кстати возвращает тру для массива, не содержащего ни одного элемента
      Ответить

      • ava guest 24.12.2009 20:14 # −2

        ... И не эквивалентна указанному в говнокоде условию.
        А вот ключи действительно странные.
        Ответить

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