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

    +147

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    if($row['status_id'] != 6){
    if($row['assist_type'] == 0){
        $row['assist_type'] == 3;
    }
    ...
}

    :-|

    Запостил: temka, 13 Января 2012

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

    • ava Dmios 13.01.2012 23:08 # 0

      Хард-код, да еще и с ошибками
      Ответить

      • ava 7ion 14.01.2012 00:31 # −1

        Молодец, теперь весь говнокод знает, какой ты умный.
        А где здесь хардкод? И где здесь ошибка номер 2 и далее, если имеются?
        Ответить

        • ava guest 14.01.2012 01:31 # 0

          Хардкоды:
          $row['status_id'] != 6
          $row['assist_type'] == 0
          $row['assist_type'] == 3
          Ответить

        • ava Dmios 14.01.2012 11:26 # 0

          Ржу нимагу) Так и знал, что на такой коммент, кто-нибудь да ответит нечто подобное. А для чего нужны комментарии, раз пост сам не описан? Для того, чтобы придираться ко множественному числу?
          Ответить

          • ava 7ion 15.01.2012 00:05 # −1

            Мне интересно, что вы понимаете под словом "хардкод" и где это здесь присутствует.
            Ответить

            • ava temka 15.01.2012 05:21 # +1

              Hard code - это привязка в коде к каким-либо реальным значениям из вашей системы.
              У вас есть база продуктов. Заказчик попросил сделать вывод какой-нибудь надписи у конкретных продуктов. Вы вместо того, чтобы добавить новое свойство для продукта, фигачите что-то типа:
              if($product_id == 9231 || $product_id == ...) {
              ....
              }
              Да, это будет быстрее, но такой код не будет динамическим и читабельным для вас и для тех, кто будет потом в нем разбираться. Например кого-то другого попросили очистить вашу базу. Когда она заполнится товаром у части из них будет ваша надпись.

              В этом куске кода хардкод лезет со всех щелей.
              Ответить

          • ava temka 15.01.2012 05:24 # 0

            Тут и так все ясно, без какого-либо описания.
            Ответить

    • ava istem 15.01.2012 00:11 # −1

      бред...
      Ответить

      • ava temka 15.01.2012 05:27 # 0

        По-моему это и есть определение говнокода.
        Для тех, кто не понял, часть этого кода не несет никакого смысла:
        if($row['assist_type'] == 0){
        $row['assist_type'] == 3; <- в сравнении делаем сравнение.
        }
        Я сначала подумал, что человек описался, но там по всему коду такие вещи...
        Ответить

        • ava bugmenot 15.01.2012 06:46 # 0

          > сначала подумал, что человек описался
          С кем не бывает!
          Ответить

          • ava temka 15.01.2012 06:50 # 0

            "но там по всему коду такие вещи" и в разных вариациях.
            Ответить

        • ava istem 15.01.2012 23:08 # 0

          > Для тех, кто не понял, часть этого кода не несет никакого смысла:
          Сильно сказал, да...
          Ответить

    • ava blackray 25.08.2021 02:08 # 0

      - Остин, я тебя знаю очень давно. Думаешь, я бы это не заметил?
      Ответить

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