- 1
- 2
- 3
if($servid>0){
$servid=$servid;
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 15
+153
if($servid>0){
$servid=$servid;
}
А если $servid == 0?
+155
$q=mysql_query($que.$limit)or die(mysql_error());
$numr=mysql_num_rows($q);
$q2 = mysql_query($que)or die(mysql_error());
$numrows = mysql_num_rows ($q2);
$pages = ceil($numrows/$step);
Здесь: $que - тяжелый SQL-запрос, который собирался в течение предыдущих сотен строк (десятки IF... ELSE...). $limit - лимит: выбрать 30 записей с такой по этакую. Чтобы орагнизовать на сайте пагинацию, автор выполняет запрос $que дважды. Во второй раз - чтобы просто посчитать общее число записей. При этом в MySQL второй запрос убивается парой SELECT SQL_CALC_FOUND_ROWS... / SELECT FOUND_ROWS().
+160
if ($sess_gr != 7 || $sess_gr == 4 || $sess_gr == 14 || $sess_gr == 15) { // ... }
Избыточное условие.
+155
<? if($sess_gr==1||$sess_gr==7||$sess_gr==11||$sess_gr==2||$sess_gr==3){?>
<? if($sess_gr==1||$sess_gr==7||$sess_gr==11||$sess_gr==2||$sess_gr==3){?><td rowspan="2"></td><?}?><td rowspan="2"></td>
<? }?>
Контрольный IF, на случай, если PHP с первого раза не понял
+157
$PriceProduct = $Price;
$Skidka = $Skidka;
$PriceSkidka = $PriceSkidka;
$Price = $PriceItog;
Самописная CRM. Коллега метко назвал этот кусок кода считалочкой. Для полноты картины осталось заменить последнюю строку на $Price = $PriceProduct;