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

    +5

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    if ($template)             // BAD
if (isset($template))      // GOOD
if ($template !== NULL))   // GOOD
if ($template !== ''))     // GOOD

if (strlen($template) > 0) // BAD! strlen("-1") is greater than 0
if (is_string($template) && strlen($template) > 0) // BETTER

if ($foo == $bar)          // BAD, avoid truthy comparisons
if ($foo != $bar)          // BAD, avoid falsy comparisons
if ($foo === $bar))        // GOOD
if ($foo !== $bar))        // GOOD

 // We only allow valid persons
if (is_object($p) && strlen($p->lastN) > 0 && $p->hidden === FALSE && $this->environment->moonPhase === MOON_LIB::CRESCENT) {
 $xmM = $thd;
}

if ($this->isValidPerson($person) {
  $xmM = $thd;
}

    Решил погуглить гайдлайны по PHP: http://flowframework.readthedocs.org/en/stable/TheDefinitiveGuide/PartV/CodingGuideLines/PHP.html

    Запостил: HiNeX, 07 Декабря 2015

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

    • ava someone 07.12.2015 09:36 # +2 

      if ($this->isValidPerson($person) {


      Скобку забыли.

      $this->environment->moonPhase === MOON_LIB::CRESCENT


      Интересно, есть ли в $this->environment свойство weatherOnMars.
      Ответить

      • ava HiNeX 07.12.2015 09:38 # 0

        Не только в этом месте. 
        if ($foo === $bar))        // GOOD
if ($foo !== $bar))        // GOOD
        Ответить

    • ava gost 07.12.2015 10:45 # 0 

      if ($foo == $bar)          // BAD, avoid truthy comparisons
if ($foo != $bar)          // BAD, avoid falsy comparisons
if ($foo === $bar))        // GOOD
if ($foo !== $bar))        // GOOD

      А нахуя тогда "==" и "!=" нужны?
      Ответить

      • ava imihajlov 07.12.2015 11:14 # +3

        ЧТОБЫ БЫЛО
        Ответить

      • ava kgm-rj 07.12.2015 13:08 # 0

        А людям удобно было писать 1 == '1', а не 1 === intval('1')
        Ответить

        • ava TarasB 07.12.2015 13:10 # +4

          А я в ужасе от таких "удобств".
          Да, я не скриптовик, я привык к строгим языкам.
          Ответить

          • ava kgm-rj 07.12.2015 15:28 # 0

            Да я тоже люблю строгие языки. А то приходится самому следить за типами
            Ответить

      • ava dxd 07.12.2015 13:33 # +1

        Это ошибки юности, за которые теперь придётся вечно расплачиваться.
        Ответить

        • ava imihajlov 07.12.2015 15:37 # 0

          В JS тоже ошибки юности? Его же серьезные дяди делали, а не какой-то школьник, в отличии от PHP.
          Ответить

          • ava guest 08.12.2015 01:58 # +3

            Js это бред наркомана, его надо было еще в 96м зарыть
            Ответить

          • ava FlySnake 08.12.2015 10:43 # +2

            на коленке за 10 дней
            Ответить

          • ava guest 08.12.2015 16:42 # 0

            Ну послушайте, никто же не знал что из этого эксперементального язычка для небольшой динамичности вырастит ВОТ ЭТО

            HTML тоже не лучший язык для рич веба
            И HTTP был не лучшим протоколом
            Ответить

            • ava HiNeX 08.12.2015 16:43 # 0

              колесо не лучшим изобретением
              Ответить

              • ava guest 08.12.2015 16:44 # +1

                круглое колесо годное

                квадратное не очень
                Ответить

                • ava HiNeX 08.12.2015 16:49 # 0

                  дурак
                  Ответить

                  • ava guest 08.12.2015 17:28 # +2

                    Гость
                    Ответить

                    • ava HiNeX 08.12.2015 18:44 # −1

                      верно, гость - дурак.
                      Ответить

                      • ava guest 08.12.2015 19:42 # +1

                        ты постоянно несешь какую-то чушь

                        ты часом не PHP программист-ли?
                        Ответить

                        • ava HiNeX 08.12.2015 20:18 # −1

                          я всего лишь твой отец
                          Ответить

                          • ava guest 08.12.2015 20:37 # +1

                            ты болен, тяжело болен
                            может быть даже неизлечимо

                            и мне тебя жаль
                            Ответить

                            • ava HiNeX 08.12.2015 20:40 # −2

                              > и мне тебя жаль

                              обычно такое можно встретить у тп баб в вк)
                              Ответить

                              • ava bormand 08.12.2015 20:41 # +1

                                У баб из техподдержки?
                                Ответить

                                • ava HiNeX 08.12.2015 20:43 # 0

                                  в поддержку еще писать не пробовал) там правда такое выдать могут?
                                  Ответить

      • ava 3_14dar 07.12.2015 16:01 # 0

        ОБРАТНАЯ СОВМЕСТИМОСТЬ
        Ответить

    • ava guest 25.02.2017 14:32 # 0

      vanished
      Ответить

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