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

    +163

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

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

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

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

    • ava Lure Of Chaos 21.09.2011 15:09 # +1

      а вдруг нет?
      Ответить

    • ava i-- 22.09.2011 00:16 # 0

      А что такого страшного?

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

      • ava darth_ixis 23.09.2011 14:03 # 0

        После рефакторинга всё красиво.

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

        • ava i-- 23.09.2011 14:49 # 0

          Рефакторинг сокращает условие на одну операцию.

          Лично мне, приведенная запись ни мозг ни глаз не ломает.
          Хотя я и не такого насмотрелся :)
          Ответить

    • ava gegMOPO4 22.09.2011 12:47 # +3

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

    • ava guest 23.09.2011 14:06 # 0

      боян
      Ответить

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