Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
0
public static function create($siteId)
{
$basket = new static();
$basket->setSiteId($siteId);
// if ($fuserId !== null)
// $basket->setFUserId($fuserId);
return $basket;
}
public static function loadItemsForFUser($fUserId, $siteId)
{
/** @var Basket $basket */
$basket = static::create($siteId);
$basket->setFUserId($fUserId);
$basket->setSiteId($siteId);
/********/
}
Новый говнокод в новой версии популярного framework'а среди бизнесменов
Вопрос, зачем надо было столько раз делать одно и тоже?
Угадай framework по говнокоду, и в качестве подарка, ты ничего не получишь =)
turbosnail,
19 Января 2016
+3
function getLastItemName( $items ) {
$lastItemName = '';
foreach( $items as $item ) {
$lastItemName = $item['name'];
}
return $lastItemName;
}
Имеем в проекте кучу россыпей подобного вида.
Собственно, существует ли эффективный способ с этим бороться?
bot,
18 Января 2016
+2
Где-то далеко в index.php:
require_once 'face.php';
А где-то внутри face.php :
require_once $_SERVER['DOCUMENT_ROOT'].'/includes/sidebar.html'
А в середине sidebar.html вот такое:
<div class="sideBoxContent" style="font-size:1.2em;">
<ul class="list">
<?php
require_once $_SERVER['DOCUMENT_ROOT'].'/config/config.php';
$dbt=db_connect();
$dbt->query("SET NAMES 'utf8'");
$resultt = $dbt->query("SELECT * FROM a_news ORDER BY date DESC LIMIT 3");
while ($myrowt = $resultt->fetch_array(MYSQLI_ASSOC))
{
echo "
<li><script>addLink('icon go-clear','/b2b/news/news.html','".$myrowt["title"]."')</script></li>";
}
$resultt->close();
$dbt->close();
?>
</ul>
<script>addLink('icon go','/b2b/news/news.html','Все новости')</script>
</div>
</div>
P.S. addLink(c, h, t) генерит <a class="c" href="h">t</a>
Господи, господи, зачем???
Нет, возможно это не говнокод. Возможно, я чего-то не понимаю в этой жизни. Но чего???
P.S. Я даже не знаю какой язык ставить. Тут и PHP, и JS...
Arris,
17 Января 2016
+5
$temp = $this->user->get($value['user_id']);
foreach ($temp as $var) {
$temp = $var;
break;
}Просматривал код курсвых по web-программированию, нашел новейший метод извлечения первого элемента массива
skad0,
13 Января 2016
0
$l_data["comm_text"] = $mail_body;
$l_var = implode(",", $l_var);
$l_path = "root->mail->".$l_group_id."->".$l_uuid."->[".$l_var.",group_name,comm_text]";
$this->engine->smartquery->create( $l_path );
$this->engine->smartquery->setdata( $l_path, $l_data );
if ($this->engine->var->get("comm_subscribe") == 1 && defvar($this->engine->var->get("comm_email"),"") != "")
{
$l_result = $this->engine->smartquery->getdata( "root->clients->(list)->[*]"," and `comm_email`='".$this->engine->var->get("comm_email")."' ");
$l_result = $l_result->getall();
if (count($l_result) == 0)
{
//Создать группу если отсутствует
$l_data = array();
$l_data["comm_fio"] = $this->engine->var->get("comm_fio");
$l_data["comm_email"] = defvar($this->engine->var->get("comm_email"),"");
$l_data["comm_phone"] = defvar($this->engine->var->get("comm_phone"),"");
$l_data["comm_formid"] = $l_group_id;
$l_data["comm_formname"] = defvar($config["mail.communication".$l_group_id.".subj"],"");
$l_var = implode(",", array_keys($l_data));
$l_uuid = mktime();
$l_path = "root->clients->".$l_uuid."->[".$l_var."]";
$this->engine->smartquery->create( $l_path );
$this->engine->smartquery->setdata( $l_path, $l_data );
}
}
Это код из одного "супер" движка мало того что тут используется дибильный шаблонизатор, тут используется просто еб*нутый класс для работы с базой данных....
Не знаю что автор хотел оптимизировать этим классом, но простая вставка данных в таблицу тут занимает целых два метода 1-ый($this->engine->smartquery->create) создает таблицу(если ее нет и пустую строку), второй только собственно осуществляет вставку в эту строку данных($this->engine->smartquery->setdata)
cowrvalera,
13 Января 2016
+2
# Проверка массива
if ($_POST) {
# Проверка Категории
if ($_POST['category'] > 0) {
# Отображение на главной
if ($_POST['main'] == "0" || $_POST['main'] == "1") {
# Имя товара
if ($_POST['item'] != "") {
# Описание товара
if ($_POST['body'] != "") {
# Картинка товара
if ($_POST['image'] != "") {
# Цены на товар
if ($_POST['price'] != "") {
...
}
}
}
}
}
}
}Когда ну очень хочется комментировать и оператор AND бессилен.
mikamika83,
13 Января 2016
+6
if($_GET['menu_id']==5){header("Location:index.php?link=pages_blog&menu_id=101");}
if($_GET['menu_id']==2){header("Location:index.php?link=pages_blog&menu_id=102");}
if($_GET['menu_id']==9){header("Location:index.php?link=pages&id=113");}
if($_GET['menu_id']==16){header("Location:index.php?link=pages&id=114");}
if($_GET['menu_id']==11){header("Location:index.php?link=pages&id=115");}
if($_GET['menu_id']==10){header("Location:index.php?link=pages&id=116");}
if($_GET['id']==22){header("Location:index.php?link=pages&id=129");}
if($_GET['id']==23){header("Location:index.php?link=pages&id=130");}
if($_GET['id']==27){header("Location:index.php?link=pages&id=135");}
if($_GET['id']==19){header("Location:index.php?link=pages&id=125");}
if($_GET['id']==28){header("Location:index.php?link=pages&id=127");}
if($_GET['id']==30){header("Location:index.php?link=pages&id=132");}
if($_GET['id']==29){header("Location:index.php?link=pages&id=132");}
if($_GET['id']==26){header("Location:index.php?link=pages&id=134");}
if($_GET['id']==24){header("Location:index.php?link=pages&id=103");}
if($_GET['id']==1){ header("Location:index.php");}
if($_GET['id']==32){ header("Location:index.php?link=pages&id=116#1");}
if($_GET['id']==33){ header("Location:index.php?link=pages&id=116#2");}
if($_GET['id']==34){ header("Location:index.php?link=pages&id=116#3");}
if($_GET['id']==35){ header("Location:index.php?link=pages&id=116#4");}
if($_GET['id']==36){ header("Location:index.php?link=pages&id=116#5");}
if($_GET['id']==37){ header("Location:index.php?link=pages&id=116#6");}
if($_GET['id']==38){ header("Location:index.php?link=pages&id=116#7");}
if($_GET['id']==39){ header("Location:index.php?link=pages&id=116#8");}
if($_GET['menu_id']==12){header("Location:index.php?link=pages&id=117");}
if($_GET['menu_id']==14){header("Location:index.php?link=pages&id=118");}
if($_GET['menu_id']==13){header("Location:index.php?link=pages&id=119");}
if($_GET['menu_id']==34){header("Location:index.php?link=pages&id=119");}
if($_GET['menu_id']==15){header("Location:index.php?link=pages&id=120");}
if($_GET['menu_id']==31){header("Location:index.php?link=pages&id=121");}
if($_GET['menu_id']==3){header("Location:index.php?link=pages_blog&menu_id=103");}
if($_GET['id']==17){ header("Location:index.php?link=pages&id=122");}
if($_GET['menu_id']==17){ header("Location:index.php?link=pages&id=122");}
if($_GET['menu_id']==35){header("Location:index.php?link=pages&id=123");}
if($_GET['id']==18){ header("Location:index.php?link=pages&id=124");}
if($_GET['menu_id']==18){ header("Location:index.php?link=pages&id=124");}
if($_GET['id']==19){ header("Location:index.php?link=pages&id=125");}
if($_GET['id']==20){ header("Location:index.php?link=pages&id=126");}
if($_GET['id']==28){ header("Location:index.php?link=pages&id=127");}
if($_GET['menu_id']==32){header("Location:index.php?link=pages_blog&menu_id=103");}
if($_GET['menu_id']==7){header("Location:index.php?link=pages&menu_id=104");}
if($_GET['menu_id']==36){header("Location:index.php?link=pages&menu_id=105");}
if($_GET['menu_id']==6){header("Location:index.php?link=pages_blog&menu_id=106");}
if($_GET['id']==61){ header("Location:index.php?link=pages&id=143");}
if($_GET['id']==60){ header("Location:index.php?link=pages&id=112");}
if($_GET['id']==57){ header("Location:index.php?link=pages&id=111");}
if($_GET['id']==59){ header("Location:index.php?link=pages&id=110");}
if($_GET['id']==56){ header("Location:index.php?link=pages&id=109");}
if($_GET['id']==55){ header("Location:index.php?link=pages&id=108");}
if($_GET['id']==54){ header("Location:index.php?link=pages&id=107");}
if($_GET['id']==53){ header("Location:index.php?link=pages&id=106");}
if($_GET['id']==52){ header("Location:index.php?link=pages&id=105");}
if($_GET['id']==50){ header("Location:index.php?link=pages&id=103");}
if($_GET['id']==44){ header("Location:index.php?link=pages&id=102");}$_GET марафет
shklovsky,
12 Января 2016
+4
<?= rand(1, 2) == 3 ? '<img src="/assets/img/icons/icon_organic.png">' : ''; ?>
tazzadar,
12 Января 2016
+2
return usort($bidsSale, function($a, $b){
return ($a->priority < $b->priority) ? -1 : 1;
});Найдено в функции от которой ожидается возврат массива.
blacksudo,
10 Января 2016
0
$rsm = mysql_query("select * from ussersmatrices where ID=$mid");
$arrm = mysql_fetch_array($rsm);
$forcedussersmatrix = $arrm[5];
$refbonus = $arrm[84];
$refbonuspaid = $arrm[83];
$payouttype = $arrm[6];
$btlevsbonus = $arrm[7];
$matchingbonus = $arrm[8];
$level1 = $arrm[9];
$level2 = $arrm[10];
$level3 = $arrm[11];
$level4 = $arrm[12];
$level5 = $arrm[13];
$level6 = $arrm[14];
$level7 = $arrm[15];
$level8 = $arrm[16];
$level9 = $arrm[17];
$level10 = $arrm[18];
$level1m = $arrm[19];
$level2m = $arrm[20];
$level3m = $arrm[21];
$level4m = $arrm[22];
$level5m = $arrm[23];
$level6m = $arrm[24];
$level7m = $arrm[25];
$level8m = $arrm[26];
$level9m = $arrm[27];
$level10m = $arrm[28];
$level1c = $arrm[29];
$level2c = $arrm[30];
$level3c = $arrm[31];
$level4c = $arrm[32];
$level5c = $arrm[33];
$level6c = $arrm[34];
$level7c = $arrm[35];
$level8c = $arrm[36];
$level9c = $arrm[37];
$level10c = $arrm[38];
$level1cm = $arrm[39];
$level2cm = $arrm[40];
$level3cm = $arrm[41];
$level4cm = $arrm[42];
$level5cm = $arrm[43];
$level6cm = $arrm[44];
$level7cm = $arrm[45];
$level8cm = $arrm[46];
$level9cm = $arrm[47];
$level10cm = $arrm[48];
$textcreditsentry = $arrm[49];
$bannercreditsentry = $arrm[50];
$textcreditscycle = $arrm[51];
$bannercreditscycle = $arrm[52];
$reentry = $arrm[53];
$reentrynum = $arrm[54];
$entry1 = $arrm[55];
$entry1num = $arrm[56];
$btlevsid1 = $arrm[57];
$entry2 = $arrm[58];
$entry2num = $arrm[59];
$btlevsid2 = $arrm[60];
$entry3 = $arrm[61];
$entry3num = $arrm[62];
$btlevsid3 = $arrm[63];
$entry4 = $arrm[64];
$entry4num = $arrm[65];
$btlevsid4 = $arrm[66];
$entry5 = $arrm[67];
$entry5num = $arrm[68];
$btlevsid5 = $arrm[69];
$welcomemail = $arrm[70];
$subject1 = stripslashes($arrm[71]);
$message1 = stripslashes($arrm[72]);
$eformat1 = $arrm[73];
$cyclemail = $arrm[74];
$subject2 = stripslashes($arrm[75]);
$message2 = stripslashes($arrm[76]);
$eformat2 = $arrm[77];
$cyclemailsponsor = $arrm[78];
$subject3 = stripslashes($arrm[79]);
$message3 = stripslashes($arrm[80]);
$eformat3 = $arrm[81];
$f1 = $forcedussersmatrix;
$f2 = $forcedussersmatrix * $forcedussersmatrix;
$f3 = $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix;
$f4 = $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix;
$f5 = $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix;
$f6 = $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix;
$f7 = $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix;
$f8 = $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix;
$f9 = $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix;
$f10 = $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix * $forcedussersmatrix;
if ($levels == 1) $fquery = "Level1<$forcedussersmatrix";
elseif ($levels == 2) $fquery = "Level2<$f2";
elseif ($levels == 3) $fquery = "Level3<$f3";
elseif ($levels == 4) $fquery = "Level4<$f4";
elseif ($levels == 5) $fquery = "Level5<$f5";
elseif ($levels == 6) $fquery = "Level6<$f6";
elseif ($levels == 7) $fquery = "Level7<$f7";
elseif ($levels == 8) $fquery = "Level8<$f8";
elseif ($levels == 9) $fquery = "Level9<$f9";100 строк кода из 4008 файлика admin.php.
PavelDevelop,
31 Декабря 2015
Follow us!