Лучший говнокод за всё время

Лучший говнокод

В номинации:

Лучший рейтинг
Самый комментируемый

За время:

Java / Говнокод #6089

+78

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

public Boolean isEqual(RepJoinCondition targetJoin )
	{
		if (targetJoin == null) return false;

		Boolean isLeftSideEqual ;
		Boolean isRightSideEqual ;
					
		isLeftSideEqual = ((this.leftJoinFiled.fieldTableName.toUpperCase() == targetJoin.leftJoinFiled.fieldTableName.toUpperCase()) &&
							(this.leftJoinFiled.fieldName.toUpperCase() == targetJoin.leftJoinFiled.fieldName.toUpperCase()));
							
		isRightSideEqual = 	((this.rightJoinFiled.fieldTableName.toUpperCase() == targetJoin.rightJoinFiled.fieldTableName.toUpperCase()) &&
							(this.rightJoinFiled.fieldName.toUpperCase() == targetJoin.rightJoinFiled.fieldName.toUpperCase()));
							
		if (isLeftSideEqual && isRightSideEqual) return true;
		
		isLeftSideEqual = ((this.leftJoinFiled.fieldTableName.toUpperCase() == targetJoin.rightJoinFiled.fieldTableName.toUpperCase()) &&
							(this.leftJoinFiled.fieldName.toUpperCase() == targetJoin.rightJoinFiled.fieldName.toUpperCase()));
							
		isRightSideEqual = 	((this.rightJoinFiled.fieldTableName.toUpperCase() == targetJoin.leftJoinFiled.fieldTableName.toUpperCase()) &&
							(this.rightJoinFiled.fieldName.toUpperCase() == targetJoin.leftJoinFiled.fieldName.toUpperCase())); 					
		
		if (isLeftSideEqual && isRightSideEqual) return true;					
		
		return false;
	}

Весьма сложнная проверка равенства, изпользуется некорректно ==

vraptor, 25 Марта 2011

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

SQL / Говнокод #6074

−860

1
2
3
4

update dv_main set dv_main.tp_id = 2 where dv_main.uid in
(SELECT dv.uid FROM (select * from dv_main) as dv
LEFT JOIN users u ON (u.uid = dv.uid)
WHERE dv.tp_id = 1);

Вот так вот саппорт одного биллинга посоветовал массово сменить тарифный план пользователям...

NiTr0man, 23 Марта 2011

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

Куча / Говнокод #6073

+122

1
2
3
4
5

http://developers.face.com/docs/api/faces-detect/
Обратите внимание:
confirmed: false
но
value: "false"

Душевно так, не? :)

wvxvw, 23 Марта 2011

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

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

+157

01
02
03
04
05
06
07
08
09
10

if ( $arOrderProps['CODE'] == 'BIRTHDATE_MONTH'
	  || $arOrderProps['CODE'] == 'DOCEXPIRATION_MONTH'
	  || $arOrderProps['CODE'] == 'BIRTHDATE_DAY'
	  || $arOrderProps['CODE'] == 'DOCEXPIRATION_DAY' )
{
	if ( $curVal[0] == '0' )
	{
	$curVal = $curVal[1];
	}
}

Преобразование месяца или дня, например 05 => 5. var_dump( $curVal ) => string(2) "20"

wwwguru, 23 Марта 2011

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

SQL / Говнокод #6065

−855

1
2

$order_by="`name` ASC";
    echo mysql_result(mysql_query("SELECT CONCAT('<ul><li>',GROUP_CONCAT(CONCAT('<a href=',`parents`.`id`,'>',`parents`.`name`,'</a>', IFNULL(`children`.`a`,'')) ORDER BY $order_by SEPARATOR '</li><li>'),'</li><ul>') FROM (SELECT `id`,`name` FROM `cats` WHERE `parent_id`=0) parents LEFT JOIN (SELECT `id`, CONCAT('<ul><li>',GROUP_CONCAT(CONCAT('<a href=',`id`,'>',`name`,'</a>') ORDER BY $order_by SEPARATOR '</li><li>'),'</li></ul>') a, `parent_id` FROM cats GROUP BY `parent_id` HAVING `parent_id`<>0) children ON `parents`.`id`=`children`.`parent_id` ;",$cn),0);

Выборка категорий и подкатегорий, рабочая ссылка
http://fradvice.com/test.php

thefatfox, 22 Марта 2011

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

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

+156

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

if (!isset($_SERVER['PHP_AUTH_USER']))
 
{
        Header ("WWW-Authenticate: Basic realm=\"Admin Page\"");
        Header ("HTTP/1.0 401 Unauthorized");
        exit();
}
 
else {
        if (!get_magic_quotes_gpc()) {
                $_SERVER['PHP_AUTH_USER'] = mysql_escape_string($_SERVER['PHP_AUTH_USER']);
                $_SERVER['PHP_AUTH_PW'] = mysql_escape_string($_SERVER['PHP_AUTH_PW']);
        }
 
        $query = "SELECT pass FROM userlist WHERE user='".$_SERVER['PHP_AUTH_USER']."'";
        $lst = @mysql_query($query);
 
        if (!$lst)
        {
            Header ("WWW-Authenticate: Basic realm=\"Admin Page\"");
        Header ("HTTP/1.0 401 Unauthorized");
        exit();
        }
 
        if (mysql_num_rows($lst) == 0)
        {
           Header ("WWW-Authenticate: Basic realm=\"Admin Page\"");
           Header ("HTTP/1.0 401 Unauthorized");
           exit();
        }
 
        $pass =  @mysql_fetch_array($lst);
        if ($_SERVER['PHP_AUTH_PW']!= $pass['pass'])
        {
            Header ("WWW-Authenticate: Basic realm=\"Admin Page\"");
           Header ("HTTP/1.0 401 Unauthorized");
           exit();
        }
 
 
}

Авторизация

nepster, 20 Марта 2011

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

Pascal / Говнокод #6023

+99

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

var
  i,c,b,f:integer;
  str:string;
procedure TForm1.codir;
begin
b:=1;
f:=1;
c:=length(edit1.Text);
str:=edit1.Text;
repeat
i:=ord(str[f]);
case i of
0:inc(i);
1:inc(i);
2:inc(i);
3:inc(i);
4:inc(i);
5:inc(i);
6:inc(i);
7:inc(i);
8:inc(i);
9:i:=0;
end;

Delete(str, b, 1);
Insert(inttostr(i),str,b);
inc(b);
inc(f);
until b=c;
edit2.Text:=str;
end;

bugmenot, 19 Марта 2011

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

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

+161

01
02
03
04
05
06
07
08
09
10
11

header('Content-Type: text/plain; charset=windows-1251'); // текстовый формат... что бы переносы строки нормально отображались и ширина букв была одинаковая

function CheckDir($s,$step)
{
        echo str_repeat('-',$step).basename($s)."\r\n"; 
        $a = glob($s.'/*',GLOB_ONLYDIR);
        foreach($a as $v)
        {
                CheckDir($v,$step+1); 
        }
}

qbasic, 16 Марта 2011

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

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

+163

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

function price_filter_form()
{
    // Хуярим неиибически сложную систему фильтрации по цене:
    $sql = "
SELECT
Max(c_good.price) AS `max`,
Min(c_good.price) AS `min`,
Count(c_good.price) AS `count`
FROM
c_good
WHERE
c_good.podcat_id =  '".mysql_real_escape_string($_GET['id'])."' AND
c_good.onoff =  '1' AND c_good.`check` = '1'";
    $sql = mysql_query($sql) OR DIE (log_error('sql'));
    $price = mysql_fetch_assoc($sql);
    #echo ("<pre>");
    #print_r($price);
    #echo ("</pre>");
    if ($price['count'] < '10') // если товаров меньше 10, выводим 1 диапазон цен
    {
     $price_form = "<option value=\"".$price['min'].":".$price['max']."\">".$price['min']."-".$price['max']."руб.</option>";    
    }
    else
    {
        // Далее идёт непонятный говнокод, который к удивлению работает
        $step = (($price['max']-$price['min'])/11);
        $step = ceil($step);
        $step_len = strlen($step);
        $step = round($step, -($step_len-1));
        $first_step = ($price['min']+$step);
        $first_step_len = strlen($first_step);
        $first_step = (round($first_step, -($first_step_len-2))-1);
        #if ($first_step < $price['min']) $first_step = $price['min']+$step;
        $price_form = "<option value=\"".$price['min'].":".$first_step."\">".$price['min']."-".$first_step."руб.</option>";
        $price_form .= "<option value=\"".($first_step+1).":".($first_step+$step)."\">".($first_step+1)."-".($first_step+$step)."руб.</option>";
        
        #echo $step;
        for ($i=1;$i<8;$i++)
        {
         $price_form .= "<option value=\"".($first_step+($step*$i)+1).":".($first_step+($step*($i+1)))."\">
         ".($first_step+($step*$i)+1)."-".($first_step+($step*($i+1)))."руб.</option>";    
        }
        $price_form .= "<option value=\"".($first_step+($step*8)+1).":".$price['max']."\">
        ".($first_step+($step*8)+1)."-".$price['max']."руб.</option>";
    }
    $price_form = str_replace("\"".$defult_price['1']."\"", "\"".$defult_price['1']."\" selected", $price_form);
    return $price_form;
}
?>

вот с таким ГК пришлось столкнуться, после профилирования. Как Вам? Или - это нормально для PHP???
P.S. комментарии сохранены авторские.

ZeiZ, 16 Марта 2011

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

SQL / Говнокод #5978

−864

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

SELECT foo.oper_id,
	                     foo.client_id,
			     base_clients.name,
			     base_clients.options,
			     base_providers.name,
	  		     base_subproviders.name,
			     base_clients.address_post,
			     base_clients.balance,
			     base_currency_types.descr,
			     base_clients.descr,
			     base_clients.client_type,
			     base_clients.pdn_permission_taken,
			     base_clients.person_use_srv_as_org,
	             base_clients.none_client
		      FROM   (SELECT oper_id,client_id FROM base_clients WHERE  (  oper_id = 'TT'  OR oper_id = 'SC'  OR oper_id = 'TT2'  OR oper_id = 'RC'  )   AND ( base_clients.client_id = '681843' OR base_clients.personal_id = '681843' OR base_clients.name ~ '[66][88][11][88][44][33]' OR base_clients.address_post ILIKE  '%681843%'OR base_clients.descr ILIKE '%681843%' )  UNION SELECT DISTINCT oper_id,client_id FROM telephone_services WHERE  (  oper_id = 'TT'  OR oper_id = 'SC'  OR oper_id = 'TT2'  OR oper_id = 'RC'  )  AND user_id ILIKE '%681843%'   UNION SELECT DISTINCT oper_id,client_id FROM channel_services WHERE  (  oper_id = 'TT'  OR oper_id = 'SC'  OR oper_id = 'TT2'  OR oper_id = 'RC'  )  AND user_id ILIKE '%681843%'   UNION SELECT DISTINCT oper_id,client_id FROM dry_pair_services WHERE  (  oper_id = 'TT'  OR oper_id = 'SC'  OR oper_id = 'TT2'  OR oper_id = 'RC'  )  AND user_id ILIKE '%681843%'   UNION SELECT DISTINCT oper_id,client_id FROM inet_dynamic_ip_services WHERE  (  oper_id = 'TT'  OR oper_id = 'SC'  OR oper_id = 'TT2'  OR oper_id = 'RC'  )  AND user_id ILIKE '%681843%'   
				UNION 
				    SELECT DISTINCT sss.oper_id, sss.client_id 
					FROM inet_fixed_ip_services as sss
					    left join inet_fixed_ip_group_items as itms on (sss.oper_id = itms.oper_id and sss.user_id = itms.ip_group )
				    WHERE 
					 (  sss.oper_id = 'TT'  OR sss.oper_id = 'SC'  OR sss.oper_id = 'TT2'  OR sss.oper_id = 'RC'  )  AND 
					(  
					    sss.user_id ILIKE '%681843%'  
						OR
					    itms.ip ILIKE '%681843%'  
					)
			     UNION SELECT DISTINCT oper_id,client_id FROM unix_shell_services WHERE  (  oper_id = 'TT'  OR oper_id = 'SC'  OR oper_id = 'TT2'  OR oper_id = 'RC'  )  AND user_id ILIKE '%681843%'   UNION SELECT DISTINCT oper_id,client_id FROM voip_ip_services WHERE  (  oper_id = 'TT'  OR oper_id = 'SC'  OR oper_id = 'TT2'  OR oper_id = 'RC'  )  AND user_id ILIKE '%681843%'   UNION SELECT DISTINCT oper_id,client_id FROM voip_card_services WHERE  (  oper_id = 'TT'  OR oper_id = 'SC'  OR oper_id = 'TT2'  OR oper_id = 'RC'  )  AND user_id ILIKE '%681843%'   UNION SELECT DISTINCT oper_id,client_id FROM ani_phone_services WHERE  (  oper_id = 'TT'  OR oper_id = 'SC'  OR oper_id = 'TT2'  OR oper_id = 'RC'  )  AND user_id ILIKE '%681843%'   UNION SELECT DISTINCT oper_id,client_id FROM cable_tv_services WHERE  (  oper_id = 'TT'  OR oper_id = 'SC'  OR oper_id = 'TT2'  OR oper_id = 'RC'  )  AND user_id ILIKE '%681843%'  UNION select distinct oper_id,client_id from base_client_aon_list where caller_phone = '681843' AND  (  oper_id = 'TT'  OR oper_id = 'SC'  OR oper_id = 'TT2'  OR oper_id = 'RC'  )  ) AS foo 
		             JOIN 
			     base_clients ON foo.oper_id=base_clients.oper_id AND 
			                     foo.client_id=base_clients.client_id 
			     JOIN 
			     base_providers ON base_clients.oper_id=base_providers.oper_id 
			     JOIN 
			     base_currency_types ON base_clients.oper_id=base_currency_types.oper_id AND 
			                            base_clients.currency_type=base_currency_types.currency_type
	                     LEFT JOIN
			     base_subproviders ON base_clients.oper_id=base_subproviders.oper_id AND
	                                          base_clients.suboper_id=base_subproviders.suboper_id 
	              ORDER BY to_number(base_clients.client_id,'999999999') DESC

Это запрос на простой поиск клиента в биллинге.

spaceoflabview, 14 Марта 2011

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

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