- 1
if ($sess_gr != 7 || $sess_gr == 4 || $sess_gr == 14 || $sess_gr == 15) { // ... }
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+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;
+163
//проверяем, не извлечены ли данные пользователя из базы. Если нет, то он не вошел, либо пароль в сессии неверный. Выводим окно для входа. Но мы не будем его выводить для вошедших, им оно уже не нужно.
Не замысловатый комментарий из истории одного проекта
+158
$dbhost = DB_HOSTNAME;
$dbuser = DB_USERNAME;
$dbpass = DB_PASSWORD;
$dbdatabase = DB_DATABASE;
$dbprefix = DB_PREFIX;
Из платного модуля для опенкарта.
+155
if($s["type"] == 9){
if(in_array($ch_newprice, array("1220","1185","1244"))){
if(in_array($ch_oldprice,array("1237","1239","1238"))){
$status = delete_DW_Customer($s["login"]);
if($status == 0) {
echo "ERROR!!! Can't delete user in server DrWEB\n";
//break;
}
if(in_array($ch_oldprice,array("1239"))){
delete_DW_Customer($s["login"]."_android");
}
}
}
}
IF Driven Development + Magic Numbers
+154
for ($i = 0; $i < count($total_data); $i++) {
if ($total_data[$i]['title'] == "Сумма") {
$sum = (float) $total_data[$i]['value'];
$tdata = array();
$tdata[count($total_data)]['title'] = "Сумма со скидкой";
$tdata[count($total_data)]['code'] = "discount_total";
$tdata[count($total_data)]['value'] = $sum;
$tdata[count($total_data)]['text'] = $this->currency->format($sum);
$tdata[count($total_data)]['sort_order'] = "2";
$total_data = array_merge($total_data, $tdata);
$id_sum = $i;
$sub_total = $sum;
}
if ($total_data[$i]['title'] == "Итого") {
$id_discount = count($total_data) - 1;
$total_data[$i]['value'] = (float) $total_data[$i]['value'] - (float) $total_data[$id_sum]['value'] + (float) $total_data[$id_discount]['value'];
$total_data[$i]['text'] = $this->currency->format($total_data[$i]['value']);
}
}
+160
while ($rown2 = mysql_fetch_array($nresult2)) {
// Несколько строк кода
$znap = $rown2['znap'];
// И ещё немного
// А теперь надо узнать название шоссе, на котором расположен коттедж:
if ($znap == "1") {
$napr = "Алтуфьевское";
} elseif ($znap == "2") {
$napr = "Дмитровское";
} elseif ($znap == "3") {
$napr = "Куркинское";
} elseif ($znap == "4") {
$napr = "Ленинградское";
} elseif ($znap == "5") {
$napr = "Новосходненское";
} elseif ($znap == "6") {
$napr = "Рогачевское";
} elseif ($znap == "7") {
$napr = "Савеловское";
} elseif ($znap == "8") {
$napr = "Боровское";
} elseif ($znap == "9") {
$napr = "Калужское";
} elseif ($znap == "10") {
$napr = "Киевское";
} elseif ($znap == "11") {
$napr = "Старокалужское";
} elseif ($znap == "12") {
$napr = "Варшавское";
} elseif ($znap == "13") {
$napr = "Новокаширское";
} elseif ($znap == "14") {
$napr = "Симферопольское";
} elseif ($znap == "15") {
$napr = "Волоколамское";
} elseif ($znap == "16") {
$napr = "Пятницкое";
} elseif ($znap == "17") {
$napr = "Горьковское";
} elseif ($znap == "18") {
$napr = "Домодедовское";
} elseif ($znap == "19") {
$napr = "Егорьевское";
} elseif ($znap == "20") {
$napr = "Носовихинское";
} elseif ($znap == "21") {
$napr = "Рязанское";
} elseif ($znap == "22") {
$napr = "Ильинское";
} elseif ($znap == "23") {
$napr = "Можайское / Минское";
} elseif ($znap == "25") {
$napr = "Новорижское";
} elseif ($znap == "26") {
$napr = "Рижское";
} elseif ($znap == "27") {
$napr = "Рублево-Успенское";
} elseif ($znap == "28") {
$napr = "Сколковское";
} elseif ($znap == "29") {
$napr = "Успенское";
} elseif ($znap == "30") {
$napr = "Каширское";
} elseif ($znap == "31") {
$napr = "Новорязанское";
} elseif ($znap == "32") {
$napr = "Осташковское";
} elseif ($znap == "33") {
$napr = "Щелковское";
} elseif ($znap == "35") {
$napr = "Подушкинское";
} elseif ($znap == "34") {
$napr = "Ярославское";
} else {
$napr = "";
}
// Дальше ещё куча кода
Во, заказали доработку проекта....
+155
if(isset($_POST['tg'])){$tg=mysql_real_escape_string(htmlspecialchars($_POST['tg']));}else{$tg=null;}
if(isset($_POST['sess'])){$sessid=mysql_real_escape_string(htmlspecialchars($_POST['sess']));}else{$sessid=null;}
if($sessid==null){$sessid=session_id();}
$str=null;
if($v>0){
// Проверяем сумму заказа для доставки
$dRzdId=$dbRes->show1Var("s_rzd","page='".PAGE_DOSTAVKA."'","id");
$dCntId=$dbRes->show1Var("s_cnt","rzdId='".$dRzdId."'","id");
$dostavkaId=$dbRes->show1Var("s_imag","cntRowId='".$dCntId."'","id");
$cprice=0;
$zakazPreStr=$res->showTabAsPage("i_zakaz","sess='".$sessid."' AND imagId<>'".$dostavkaId."'","imagId,imagCol,zakazStr","id",0);
foreach($zakazPreStr as $zakazStr){
foreach($zakazStr as $ke=>$va){
$$ke=stripcslashes($va);
}
$price=$res->show1Var("s_imag","id='".$imagId."'","tovPrice");
$tovCol=1;
if($zakazStr>0){
$tovCol=$res->show1Var("i_zakaz","imagId='".$zakazStr."' AND sess='".$sessid."'","imagCol");
}
$cprice+=$price*$imagCol*$tovCol;
}
if($v==$dostavkaId&&$c>0){
if($cprice>=MAX_DOSTAVKA){
$c=0;
}
}else{
if($cprice>=MAX_DOSTAVKA){
$query="UPDATE i_zakaz SET imagCol='0', color='".$color."' WHERE sess='".$sessid."' AND imagId='".$dostavkaId."'";
@$insres=mysql_query($query);
}
}
//if($_SERVER['REMOTE_ADDR']=="87.224.169.4"){echo $str.=$v."--";}
switch($to){
case "ins":
// Сначала проверим, существует ли товар, а потом либо добавим к существующему нужное количество, либо добавим
$res=mysql_query("SELECT imagCol FROM i_zakaz WHERE sess='".$sessid."' AND imagId='".$v."' AND zakazStr='".$p."'");
$col=mysql_num_rows($res);
if($col==0){
@$insres=mysql_query("INSERT INTO i_zakaz SET sess='".$sessid."',imagId='".$v."',imagCol='".$c."',color='".$color."',zakazStr='".$p."',adttm='".date("YmdHis")."'");
}else{
while($row=mysql_fetch_assoc($res)){
/*
* Убрал суммирование количества заказа
$imagCol=$row['imagCol'];
$imagCol=$imagCol+$c;
*/
@$insres=mysql_query("UPDATE i_zakaz SET imagCol='".$c."',color='".$color."' WHERE sess='".$sessid."' AND imagId='".$v."' AND zakazStr='".$p."'");
}
}
mysql_free_result($res);
$str.=$imagRes->pageKorzina($sessid);
break;
case "del":
if($p==null){
// В этом случае удаляем все внутреннее содержание
@$res=mysql_query("DELETE FROM i_zakaz WHERE sess='".$sessid."' AND imagId='".$v."' AND zakazStr='".$p."'");
@$res=mysql_query("DELETE FROM i_zakaz WHERE sess='".$sessid."' zakazStr='".$v."'");
}else{
@$res=mysql_query("DELETE FROM i_zakaz WHERE sess='".$sessid."' AND imagId='".$v."' AND zakazStr='".$p."'");
}
switch($tg){
case 0:
$str.=$imagRes->pageKorzina($sessid);
break;
case 1:
$str.=$imagRes->kz($sessid,"imag");
break;
}
break;
case "update":
$res=mysql_query("SELECT imagCol FROM i_zakaz WHERE sess='".$sessid."' AND imagId='".$v."' AND zakazStr='".$p."'");
$col=mysql_num_rows($res);
if($col==0){
@$insres=mysql_query("INSERT INTO i_zakaz SET sess='".$sessid."',imagId='".$v."',imagCol='".$c."',zakazStr='".$p."',color='".$color."'");
}else{
@$res=mysql_query("UPDATE i_zakaz SET imagCol='".$c."',color='".$color."' WHERE sess='".$sessid."' AND imagId='".$v."' AND zakazStr='".$p."'");
}
switch($tg){
case 0:
$imagRes->pageKorzina($sessid);
//$str.=$imagRes->kz($sessid,"imag");
break;
case 1:
// Ищем id заказа
$res=new db_func();
$zakazid=$res->show1Var("i_bill","sessid='".$sessid."'","id","id",1);
if($zakazid>0){
$sessid=$res->show1Var("i_bill","id='".$zakazid."'","sessid","id",1);
}
$str.=$imagRes->kz($sessid,"imag");
break;
}
break;
}
}
$str=str_replace("{ROOTPATH}",$rootpath,$str);
Сегодня в нашей программе: November.CMS и ее "Корзина" =))
+157
function commandLineArgsToGet() {
global $argv;
foreach ($argv as $arg) {
$param = explode('=', $arg);
if (count($param) == 2) {
$_GET[$param[0]] = $param[1];
}
else {
$_GET[$param[0]] = null;
}
}
}
Встретил функцию для перевода аргументов командной строки в $_GET-массив.