- 1
- 2
- 3
- 4
- 5
- 6
- 7
- 8
- 9
if(count($colors)>1){
if($i==2){
$col = $colors[1];
}else{
$col = $colors[0];
}
}else{
$col=$colors[0];
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 15
+153
if(count($colors)>1){
if($i==2){
$col = $colors[1];
}else{
$col = $colors[0];
}
}else{
$col=$colors[0];
}
Вот где не надо - тьма тернарных операторов, а где их логичнее всего ожидать - вот такой огород из IF/ELSE
+158
$q= mysql_query("select * from BillingParam where Name like ('% %')");
while($r=mysql_fetch_object($q)){
$name=str_replace(" ", " ", $r->Name);
$que="update BillingParam set Name='".$name."' where ID='".$r->ID."'";
echo $que."<br>";
$qq=mysql_query($que)or die(mysql_error());
Нужно больше, ещё больше запросов к базе!
+163
<?
$text[5] .="</body>";
$text[5] .="</html>";
echo $text[5];
?>
И вот таким вот макаром
Наша рота солдат
Обходит ровно за сутки
Квадрат, квадрат, квадрат...
+164
echo "Дней до освобождения: " . $freedays, "<br>";
В жопу унификацию. Программист не справился с задачей поставить два одинаковых знака - точки или запятые - в одной строке.
+158
if ($qproizv_r->s > 0 && $qpriem_n > 0) {
if ($qproizv_r->s >= $row->Kol) {
$Status = 6;
}
} else {
if ($qpriem_n > 0) {
$Status = 6;
}
}
Так и живем.
+155
if($numr>1){
$i=0;
while($row=mysql_fetch_object($q)){++$i;
if($num==$i){
return $row->id;
break;
}
}
}elseif($numr==1){
$row=mysql_fetch_object($q);
return $row->id;
}else{
return 0;
}
А на случай, если return() не сработает, у нас есть break
+156
$c=count($arr1);
for($i=0;$i<$c;$i++){
$key=key($arr1);
ksort($arr1[$key]);
next($arr1);
}
foreach() чем-то не угодил
+154
<option<?=($qqrow->ServiceID==$row->ID&&!isset($servid))?" selected":""?><?=(isset($servid))?($servid==$row->ID)?" selected":"":""?> value="<?=$row->ID?>"><?=$row->Name?>
+155
$query = "SELECT DATE_FORMAT(`Date`, '%d.%m.%Y') 'dat',
DATE_FORMAT(`Date`, '%Y-%m-%d') 'd'
FROM Sklad
WHERE OrderID = '{$row->ID}'
AND OrderStatusID = 6
AND Kol < 0";
$qsklad_data = mysql_fetch_object(mysql_query($query));
if ($row->DateFakt == "00.00.0000") {
$query = "UPDATE Orders
SET DateFakt = '{$qsklad_data->d}'
WHERE ID = '{$row->ID}'";
$qupddd = mysql_query($query);
}
Паттерн программирования: сначала сделать запрос к БД, а потом проверить, нужны ли нам его результаты. $qsklad_data используется только здесь. И опять же два SQL-запроса там, где мог быть один.
−116
$qkolsum = mysql_fetch_object(mysql_query("SELECT SUM(Kol) 's' FROM PostavkiDetail WHERE PostavkiID = '{$qspec->PostavkiID}'"));
$qkolsumitog = mysql_fetch_object(mysql_query("SELECT SUM(SumItog) 's' FROM PostavkiDetail WHERE PostavkiID = '{$qspec->PostavkiID}'"));
Тут и PHP не особо хороший, но соль все-таки в двух запросах к одной таблице. Посчитать суммы в двух колонках с помощью одного запроса - это, видимо, что-то запредельное.