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

    +163

    1. 1
    2. 2
    3. 3
    if (is_null($var) === false) {
    // не важно что
    }

    Встретил такой код в примерах одного Merchant-сервиса.
    Вот что значит простое сделать сложным.

    Запостил: darth_ixis, 21 Сентября 2011

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

    • а вдруг нет?
      Ответить
    • А что такого страшного?

      Рефакторинг сводится к if(isset($var))
      Ответить
      • После рефакторинга всё красиво.

        А если у вас все пустяковые IF-ы в проекте будут так написаны, я посмотрю на ваше "А что такого страшного?" :)
        Ответить
        • Рефакторинг сокращает условие на одну операцию.

          Лично мне, приведенная запись ни мозг ни глаз не ломает.
          Хотя я и не такого насмотрелся :)
          Ответить
    • Ну так PHP выдрессировал программистов, что вместо if(!something) нужно писать if(something === false). Спасибо strpos и проч.
      Ответить
    • боян
      Ответить

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