1. JavaScript / Говнокод #15508

    +166

    1. 1
    2. 2
    if (value) return (value != null)
           else return false;

    Из кода проекта.... Автор уволился

    Запостил: alex123098, 17 Марта 2014

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

    • ava guest 18.03.2014 20:29 # +2

      Видимо написал он этот шедевр, осознал всю свою никчемность и уволился. Что за проект то?
      Ответить

      • ava alex123098 20.03.2014 12:24 # 0

        Да так, малое кредитование. Нет, автор этого еще много чего в таком роде сочинил. Надо будет выложить еще че-нибудь
        Ответить

    • ava Yuuri 20.03.2014 18:27 # −1

      Я бы посмеялся, если бы не был наслышан про причуды JS-ного приведения типов и выкрутасы с false/null/undefined — не шибко удивлюсь, если этот код делает что-то полезное.
      Ответить

      • ava 1024-- 20.03.2014 18:34 # +2

        Если справедливо (value), то value != null. Может быть, этот код делал что-то полезное до переписывания. Может быть, там когда-то было if(value != false), которое заменили на if(value).
        Ответить

        • ava alex123098 20.03.2014 20:41 # 0

          Не делал этот код ничего полезного, просто автор не знал, что его можно написать как 
          return !!value;

          :)
          Ответить

          • ava bormand 20.03.2014 20:45 # 0 

            return value;
            Ответить

            • ava alex123098 20.03.2014 20:46 # 0

              нет, если value === null или value === undefined :)
              Ответить

              • ava bormand 20.03.2014 20:49 # 0

                null и undefined в условиях разве не зачтутся за false?

                Я тут пока вижу две проблемы:
                1) Сборщик мусора не сможет удалить то, на что ссылается value, что неприятно, если результат функции куда-то надолго сохранят, а value было каким-то огромным массивом/объектом.
                2) Кто-то может додуматься сравнивать на === false или === true. Но это уже ССЗБ, а не проблема.
                Ответить

                • ava alex123098 20.03.2014 20:53 # 0

                  в условиях зачтутся, если не напорятся на явное сравнение типа value === false
                  Ответить

                  • ava bormand 20.03.2014 20:54 # 0

                    > value === false
                    А зачем так писать? Ну кроме сериализации во всякие json'ы.
                    Ответить

          • ava roman-kashitsyn 20.03.2014 20:46 # +3

            Простые смертные любят писать Boolean(value)
            Ответить

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