PHP — Говнокод.ру

PHP / Говнокод #19321

+3

01
02
03
04
05
06
07
08
09
10

function getLastItemName( $items ) {

  $lastItemName = '';

  foreach( $items as $item ) {
    $lastItemName = $item['name'];
  }

  return $lastItemName;
}

Имеем в проекте кучу россыпей подобного вида.
Собственно, существует ли эффективный способ с этим бороться?

bot, 18 Января 2016

Комментарии (18)

PHP / Говнокод #19317

+2

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32

Где-то далеко в 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

Комментарии (11)

PHP / Говнокод #19302

+5

1
2
3
4
5

$temp = $this->user->get($value['user_id']);
foreach ($temp as $var) {
	$temp = $var;
	break;
}

Просматривал код курсвых по web-программированию, нашел новейший метод извлечения первого элемента массива

skad0, 13 Января 2016

Комментарии (19)

PHP / Говнокод #19300

0

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34

$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)

PHP / Говнокод #19298

+2

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22

# Проверка массива
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

Комментарии (2)

PHP / Говнокод #19292

+6

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64

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

Комментарии (31)

PHP / Говнокод #19291

+4

1

<?= rand(1, 2) == 3 ? '<img src="/assets/img/icons/icon_organic.png">' : ''; ?>

tazzadar, 12 Января 2016

Комментарии (2)

PHP / Говнокод #19283

+2

1
2
3

return usort($bidsSale, function($a, $b){ 
              return ($a->priority < $b->priority) ? -1 : 1; 
});

Найдено в функции от которой ожидается возврат массива.

blacksudo, 10 Января 2016

Комментарии (15)

PHP / Говнокод #19277

0

001
002
003
004
005
006
007
008
009
010
011
012
013
014
015
016
017
018
019
020
021
022
023
024
025
026
027
028
029
030
031
032
033
034
035
036
037
038
039
040
041
042
043
044
045
046
047
048
049
050
051
052
053
054
055
056
057
058
059
060
061
062
063
064
065
066
067
068
069
070
071
072
073
074
075
076
077
078
079
080
081
082
083
084
085
086
087
088
089
090
091
092
093
094
095
096
097
098
099
100

$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

Комментарии (1)

PHP / Говнокод #19273

+6

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30

switch("пряные сухарики"){
case ".gif":{
	header('content-type: image/gif');
	break;
}
case ".jpg":{
	header('content-type: image/jpeg');
	break;
}
case ".jpeg":{
header('content-type: image/jpeg');
break;
}
case ".bmp":{
header('content-type: image/bmp');
break;
}
case ".png":{
header('content-type: image/png');
break;
}
case ".ogg":{
header('content-type: video/ogg');
break;
}
case ".mp4":{
header('content-type: video/mp4');
break;
}
}

d_fomenok, 29 Декабря 2015

Комментарии (20)

Говнокод: по колено в коде.

PHP / Говнокод #19321

PHP / Говнокод #19317

PHP / Говнокод #19302

PHP / Говнокод #19300

PHP / Говнокод #19298

PHP / Говнокод #19292

PHP / Говнокод #19291

PHP / Говнокод #19283

PHP / Говнокод #19277

PHP / Говнокод #19273