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

    +162

    1. 1
    2. 2
    3. 3
    if ((int)$id == '') {
        throw new Engine_Exception('Нверный идентификатор организации', 403);
}

    O_o а это когда-нибудь сработает :)

    Запостил: lstaticl, 08 Октября 2010

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

    • ava zxm 08.10.2010 11:01 # 0

      Это сработает так как задумывалось. Когда id 0 или false условие выполнится.
      Ну да, == '' ничего не значит, можно выпилить
      Ответить

      • ava sectus 08.10.2010 11:50 # +1

        Всё равно говнокод, хоть автор поста и не уловил в чём именно.
        Ответить

    • ava Lure Of Chaos 08.10.2010 18:51 # 0

      строго типизированные языки нас учат верить тому, что написано. == '' подразумевает, что мы сравниваем со строкой. Но пхп коварен: он реагирует как привычно только на ===. В итоге кодомартышкам слишком многое сходит с рук
      Ответить

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