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

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

+165

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17

$fileSelect = array();
if (is_dir($filefolder)) {
    if ($dh = @opendir($filefolder)) {
        while (($file = readdir($dh)) !== false) {
            if (filetype($filefolder . $file) == "file" and $file != "0.txt") {
                $sql  = 'SELECT * FROM bd_file WHERE file_file = ?';
                $stmt = db_prepare($sql, $file); $res  = db_query($stmt);
                if (!db_num_rows($res)) {
                    $fileSelect[$file] = $file;
                }
            }
        }
    }
    closedir($dh);
}
...
unset($fileSelect);

Сервер медленно умирал, так как данная конструкция вызывалась при каждом удобном случае, будь то добавление нового файла или скачивание. Самое поразительное, что с полученным массивом нечего не делали, а тупо уничтожили

P.S. минимальная стоимость сайта у этой фирмы 3.000.000 рублей

MoLe-X, 05 Апреля 2010

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

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

+159.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

<?php 
include( "dbopen.php" ); 

function ShowTree($ParentID, $lvl) { 

global $link; 
global $lvl; 
$lvl++; 

$sSQL="SELECT id,title,pid FROM catalogue WHERE pid=".$ParentID." ORDER BY title";
$result=mysql_query($sSQL, $link);

if (mysql_num_rows($result) > 0) {
echo("<UL>\n");
while ( $row = mysql_fetch_array($result) ) {
$ID1 = $row["id"];
echo("<LI>\n");
echo("<A HREF=\""."?ID=".$ID1."\">".$row["title"]."</A>"."  \n");
ShowTree($ID1, $lvl); 
$lvl--;
}
echo("</UL>\n");
}

}

ShowTree(0, 0); 

mysql_close($link); 

?>

Построение дерева с запросом в цикле:)))

http://www.codenet.ru/webmast/php/tree.php

ar4ibal, 05 Апреля 2010

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

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

+167.8

1
2
3

if($_GET['admin']=='da'){
	echo 'adminka_true';
}

А самое страшное, что писал это я...

Armanio, 03 Апреля 2010

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

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

+165

1

$result = mysql_query("SELECT `id` FROM `cats` WHERE `id` = (SELECT `pid` FROM `tovary` WHERE `id` = '".$this->id."' LIMIT 1)");

В один запрос получать id не кошерно.

Было найдено в этом же проекте - http://www.govnokod.ru/2897

Rpsl, 02 Апреля 2010

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

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

+166

1

$limit = (isset($this->params['pass'][1]) == 'news')?null:5;

балина, и это работает :D

lener, 02 Апреля 2010

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

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

+162.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
65
66
67
68
69
70
71
72
73
74
75
76
77
78

<?php


if($_GET['id'] == 10 ){

$location = "/articles/lyustry/lustra-na-kuhne/";
}elseif($_GET['id'] == 13 and $_GET['id_type'] == 17){
$location = "/market/interernye_svetilniki/lyustry/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 22 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/bolshie_lyustry/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 34 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/lyustry/bolee_6_rozhkov/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 2 ){
$location = "/market/interernye_svetilniki/podvesnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 40 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/podvesnye/odinochnyj_podves/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 41 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/podvesnye/trojnoj_i_bolee/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 5 ){
$location = "/market/interernye_svetilniki/nastenno-potolochnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 35 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/potolochnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 52 ){
$location = "/market/interernye_svetilniki/vlagozawitnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 51 ){
$location = "/market/interernye_svetilniki/podsvetka_kartin/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 7 ){
$location = "/market/interernye_svetilniki/bra/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 37 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/bra/odnorozhkovye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 38 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/bra/dvuhrozhkovye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 39 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/bra/treh_i_bolee_rozhkovye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 1 ){
$location = "/market/interernye_svetilniki/tochechnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 42 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/tochechnye/odinochnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 43 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/tochechnye/dvojnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 44 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/tochechnye/tri_i_bolee/";
}
elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 8){
$location = "/market/interernye_svetilniki/nastolnye_lampy/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 46 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/nastolnye_lampy/ofisnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 47 and $_GET['act'] == 'sub_
type'){
$location = "/market/interernye_svetilniki/nastolnye_lampy/prikrovatnye_domashnie_ili_interernye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 6 ){
$location = "/market/interernye_svetilniki/torshery/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 9 ){
$location = "/market/ulichnye_svetilniki/";
}...
И так далее..

Клиент поставил нашу CMS, на старой были ссылки типа /?id=10 и т.д. Сделал ему для примера скрипт редиректа с использованием switch case и вот что я увидел через две недели 0_0 Неприкрытый говнокод.

VeroLom, 01 Апреля 2010

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

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

+159.4

01
02
03
04
05
06
07
08
09
10
11
12
13

//возвращает массив без повторяющихся элементов (обязательно должен быть элемент [id])
function arrayUnique($arr)
	{	
		$aResult = array();
		$aIDs = array();
		foreach($arr as $val)
			if (!in_array($val['id'], $aIDs))
			{
				array_push($aIDs, $val['id']);
				array_push($aResult, $val);
			}
		return $aResult;
	}

На входе массив массивов.
Массивы равны, если все элементы одного массива равны элементам другого.
На выходе должен быть массив уникальных массивов.
По моему это попахивает.

nechin, 31 Марта 2010

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

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

+180

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

function writeTotalEntries()
    {
        $cnt = $this->totalentries;
        if ($cnt == 0) {
            return "записей нет";
        } elseif ($cnt == 1)
        {
            return "1 запись";
        } elseif ($cnt == 2)
        {
            return "2 записи";
        } elseif ($cnt == 3)
        {
            return "3 записи";
        } elseif ($cnt == 4)
        {
            return "4 записи";
        } elseif ($cnt == 5)
        {
            return "5 записей";
        } elseif ($cnt == 6)
        {
            return "6 записей";
        } elseif ($cnt == 7)
        {
            return "7 записей";
        } elseif ($cnt == 8)
        {
            return "8 записей";
        } elseif ($cnt == 9)
        {
            return "9 записей";
        } elseif (($cnt >= 10) && ($cnt <= 20))
        {
            return $cnt." записей";
        } elseif ($cnt%10 == 1)
        {
            return $cnt." запись";
        } else
        {
            return $cnt." записей";
        }
    }

Было найдено в одном из проектов.

Rpsl, 30 Марта 2010

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

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

+182

1

$submitted=trSubmit($dir,$db,$c,$tries,$dTracker,$Accs,$box,$dTkey,$writelog,$current_agent,$progressId,$prox_url,$prox_pwd,$max_tries_proxy,$max_tries_login,$max_tries_submit,$max_tries_captcha,$froot,$cats,$ts[$tkey],$cat[$tkey],$descr[$tkey],$shot[$tkey],$skips[$ts[$tkey]],$auth,$cookie,$ref,$check_enabled,$checks,$noProx,$cordebug,$debug);

Спихнули тут на меня код мальчика, попросили "чуть чуть переписать". Типичный вызов функции.
Хочется взять за такое и ударить по голове.

dzugaru, 30 Марта 2010

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

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

+185.4

1
2
3
4

function epic_die(){
    mysql_close();
    die();
}

Автор не я)

a13ks3y, 29 Марта 2010

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

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