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

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

В номинации:

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

За время:

Python / Говнокод #2922

−129.4

1

s = ''.join([s[i] for i in range(len(s)-1,-1,-1)])

Найдено, как вариант реверсирования строки

arcanum, 02 Апреля 2010

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

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 / Говнокод #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 / Говнокод #2889

+167.2

1
2
3
4

<?php
class Xml {
//Эта библиотека УГ, она не требуется
}

Волею судеб взялся за поддержку "сайтега", написанного каким-то малограмотным студентом.
В процессе изучению сорцов я увидел много новых гениальных паттернов и практик, phpMyAdmin без пароля, который лежал прямо в корне и был доступен извне, полное отсутствие ясной мысли и трезого ума.

Но сегодня наткнулся на класс, который меня просто добил.

ps. Не формат, просто наболело.

dig, 29 Марта 2010

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

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

+165.4

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

<form style="height:15px ; width:100px"method=post name='ChangeCurrencyForm'>
{literal}
<script language='JavaScript'>
function _changeCurrency()
{
document.ChangeCurrencyForm.submit();
}
</script>
{/literal}

<select name='current_currency' onchange='JavaScript:_changeCurrency()'>
{section name=i loop=$currencies}
{if $log eq "" && $currencies[i].CID eq 2 | $currencies[i].CID eq 3 | $currencies[i].CID eq 4 | $currencies[i].CID eq 5 | $currencies[i].CID eq 6}
{*розница*} {elseif ($log eq "" or $log eq "" or $log eq "tigrakiev") && $currencies[i].CID eq 2 | $currencies[i].CID eq 3 | $currencies[i].CID eq 5 | $currencies[i].CID eq 6}
{*мелкий опт*} {elseif ($log eq "" or $log eq "") && $currencies[i].CID eq 2 | $currencies[i].CID eq 3 | $currencies[i].CID eq 4 | $currencies[i].CID eq 6}
{*опт*} {elseif ($log eq "") && $currencies[i].CID eq 2 | $currencies[i].CID eq 3 | $currencies[i].CID eq 4 | $currencies[i].CID eq 5}
{*плейлист*} {elseif ($log ne "" && $log ne "" && $log ne "" && $log ne "" && $log ne "" && $log ne "" && $log ne "") && $currencies[i].CID eq 2 | $currencies[i].CID eq 3 | $currencies[i].CID eq 4 | $currencies[i].CID eq 5 | $currencies[i].CID eq 6}
{else}
<option value='{$currencies[i].CID}'
{if $currencies[i].CID eq $current_currency}
selected
{/if}
>
{$currencies[i].Name}
</option>
{/if}
{/section}
</select>

KAMA3 HABO3A, 21 Марта 2010

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

Си / Говнокод #2823

+144.8

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

// C:\Program Files\Microsoft Visual Studio 9.0\VC\crt\src\dbghook.c

int __cdecl _CrtDefaultAllocHook(
        int nAllocType,
        void * pvData,
        size_t nSize,
        int nBlockUse,
        long lRequest,
        const unsigned char * szFileName,
        int nLine
        )
{
        return 1; /* allow all allocs/reallocs/frees */
}


// C:\Program Files\Microsoft Visual Studio 9.0\VC\crt\src\dbgheap.c

extern "C" _CRTIMP int __cdecl _CrtIsValidPointer(
        const void * pv,
        unsigned int nBytes,
        int bReadWrite
        )
{
        return (pv != NULL);
}

вот, в Microsoft Visual Studio 2008 нашёл такие вот перлы...

glook, 19 Марта 2010

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

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

+98.7

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

Procedure Halt;
begin
if atomTest<>0 then GlobalDeleteAtom(atomTest);
TerminateProcess(GetCurrentProcess,0);
end;
function RegisterSceenTimer(proc:tproc):dword;
var i:dword;
begin
result:=0;
for i:=1 to sMaxSceenGradient do
if  not SceenGradient[i].isNotEmpty then break ;
if i=sMaxSceenGradient then exit;
SceenTimers[i]:=proc;
result:=i;
end;

:D

Говногость, 13 Марта 2010

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

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

+73.4

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

public class SaveBlankElementException extends RuntimeException{
    public SaveBlankElementException(Throwable cause){
        super(cause);
    }
    public SaveBlankElementException(String message,Throwable cause){
        super(message,cause);
    }
    public SaveBlankElementException(String message){
        super(message);
    }
    public SaveBlankElementException(){

    }
    @Override
    public String getMessage(){
        return "Попытка сохранения пустого элемента\nНе вызван prepareCreate()";
    }
  
}

Зачем, спрашивается было создавать этот класс, когда можно было бы
вызвать исключение
throw new UnsupportedOperationException("Попытка сохранения пустого элемента\nНе вызван prepareCreate()");

maxt, 10 Марта 2010

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

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

+73.8

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

public class Date extends java.util.Date{
    @Override
    public String toString(){
        return toLocaleString();
    }
    public Date(String s){
        super(s);
    }
    public Date(int year,int month,int date,int hrs,int min,int sec){
        super(year,month,date,hrs,min,sec);
    }
    public Date(int year,int month,int date,int hrs,int min){
        super(year,month,date,hrs,min);
    }
    public Date(int year,int month,int date){
        super(year,month,date);
    }
    public Date(long date){
        super(date);
    }
    public Date(){
    }
    public java.sql.Date getSQLDate(){
        return new java.sql.Date(getTime());
    }
}

из за переопределения и еще какого, а главное как:
@Override
public String toString(){
return toLocaleString();
}
писать целый класс, это не каждый сможет.

maxt, 09 Марта 2010

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

C++ / Говнокод #2736

+59.4

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

m = 1;

label:

...block1;

if (m == 1)
{
	m++;
	goto label;
}
else if (m == 2)
{
	m++;
	...block2;
	goto label;
else
{
	...block3;
}
if (m == 3)
	m = 1;
else
	m--;

из реально работающей математической проги

будет ли выполнено m--? ))

glook, 08 Марта 2010

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

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