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

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

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