- 1
- 2
- 3
- 4
- 5
- 6
if($row['status_id'] != 6){
if($row['assist_type'] == 0){
$row['assist_type'] == 3;
}
...
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+147
if($row['status_id'] != 6){
if($row['assist_type'] == 0){
$row['assist_type'] == 3;
}
...
}
:-|
Dmios 13.01.2012 23:08 # 0
7ion 14.01.2012 00:31 # −1
А где здесь хардкод? И где здесь ошибка номер 2 и далее, если имеются?
guest 14.01.2012 01:31 # 0
$row['status_id'] != 6
$row['assist_type'] == 0
$row['assist_type'] == 3
sectus 14.01.2012 08:31 # 0
guest 14.01.2012 11:08 # −6
Это когда мамаша за 9 месяцев до твоего рождения участвовала в gangbang, бгггг
Dmios 14.01.2012 11:26 # 0
7ion 15.01.2012 00:05 # −1
temka 15.01.2012 05:21 # +1
У вас есть база продуктов. Заказчик попросил сделать вывод какой-нибудь надписи у конкретных продуктов. Вы вместо того, чтобы добавить новое свойство для продукта, фигачите что-то типа:
if($product_id == 9231 || $product_id == ...) {
....
}
Да, это будет быстрее, но такой код не будет динамическим и читабельным для вас и для тех, кто будет потом в нем разбираться. Например кого-то другого попросили очистить вашу базу. Когда она заполнится товаром у части из них будет ваша надпись.
В этом куске кода хардкод лезет со всех щелей.
temka 15.01.2012 05:24 # 0
istem 15.01.2012 00:11 # −1
temka 15.01.2012 05:27 # 0
Для тех, кто не понял, часть этого кода не несет никакого смысла:
if($row['assist_type'] == 0){
$row['assist_type'] == 3; <- в сравнении делаем сравнение.
}
Я сначала подумал, что человек описался, но там по всему коду такие вещи...
bugmenot 15.01.2012 06:46 # 0
С кем не бывает!
temka 15.01.2012 06:50 # 0
istem 15.01.2012 23:08 # 0
Сильно сказал, да...
blackray 25.08.2021 02:08 # 0