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

    • Хард-код, да еще и с ошибками
      Ответить
      • Молодец, теперь весь говнокод знает, какой ты умный.
        А где здесь хардкод? И где здесь ошибка номер 2 и далее, если имеются?
        Ответить
        • Хардкоды:
          $row['status_id'] != 6
          $row['assist_type'] == 0
          $row['assist_type'] == 3
          Ответить
        • Ржу нимагу) Так и знал, что на такой коммент, кто-нибудь да ответит нечто подобное. А для чего нужны комментарии, раз пост сам не описан? Для того, чтобы придираться ко множественному числу?
          Ответить
          • Мне интересно, что вы понимаете под словом "хардкод" и где это здесь присутствует.
            Ответить
            • Hard code - это привязка в коде к каким-либо реальным значениям из вашей системы.
              У вас есть база продуктов. Заказчик попросил сделать вывод какой-нибудь надписи у конкретных продуктов. Вы вместо того, чтобы добавить новое свойство для продукта, фигачите что-то типа:
              if($product_id == 9231 || $product_id == ...) {
              ....
              }
              Да, это будет быстрее, но такой код не будет динамическим и читабельным для вас и для тех, кто будет потом в нем разбираться. Например кого-то другого попросили очистить вашу базу. Когда она заполнится товаром у части из них будет ваша надпись.

              В этом куске кода хардкод лезет со всех щелей.
              Ответить
          • Тут и так все ясно, без какого-либо описания.
            Ответить
    • бред...
      Ответить
      • По-моему это и есть определение говнокода.
        Для тех, кто не понял, часть этого кода не несет никакого смысла:
        if($row['assist_type'] == 0){
        $row['assist_type'] == 3; <- в сравнении делаем сравнение.
        }
        Я сначала подумал, что человек описался, но там по всему коду такие вещи...
        Ответить
        • > сначала подумал, что человек описался
          С кем не бывает!
          Ответить
          • "но там по всему коду такие вещи" и в разных вариациях.
            Ответить
        • > Для тех, кто не понял, часть этого кода не несет никакого смысла:
          Сильно сказал, да...
          Ответить
    • - Остин, я тебя знаю очень давно. Думаешь, я бы это не заметил?
      Ответить

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