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

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

В номинации:

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

За время:

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

+73

1
2
3
4
5
6
7

int[] indexes = list.getSelectedIndices();
        if (indexes.length == 0) {
            indexes = getAllIndexes();
        }
        if (indexes.length == 0) {
            return false;
        }

индуссы атакуют

Dexter, 15 Августа 2011

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

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

+180

1

http://i.imgur.com/K9ZM6.png

А вот и сам гет.

bugmenot, 09 Августа 2011

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

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

+183

1
2
3
4
5
6
7
8

if (is_file('img/maps/'.$user_id.'.jpg'))
{
$photo = 1;
} elseif(!is_file('img/maps/'.$user_id.'.jpg')) {
$photo = 0;
} else {
$photo = 0;
}

кто сказал что третьего не дано?))))

invision70, 07 Августа 2011

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

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

+170

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

$q = mysql_query("SELECT * FROM ".$prefix."_users WHERE id='$user_id'");

$org = mysql_result($q,0,"org");
$orginfo = mysql_result($q,0,"orginfo");
$orgsite = mysql_result($q,0,"orgsite");
$orgaddr = mysql_result($q,0,"orgaddr");
$orgcity = mysql_result($q,0,"orgcity");
$orgname = mysql_result($q,0,"orgname");
$sendme = mysql_result($q,0,"sendme");
$news_sendme = mysql_result($q,0,"news_sendme");
$arend = mysql_result($q,0,"arend");
$seller = mysql_result($q,0,"seller");
$manufacter = mysql_result($q,0,"manufacter");
$bank = mysql_result($q,0,"bank");
$leasing = mysql_result($q,0,"leasing");
$a_bus = mysql_result($q,0,"a_bus");
$a_gruz = mysql_result($q,0,"a_gruz");
$a_prec = mysql_result($q,0,"a_prec");
$a_po_prec = mysql_result($q,0,"a_po_prec");
$a_sr_tg = mysql_result($q,0,"a_sr_tg");
...

В бесконечность и далее!!!

invision70, 03 Августа 2011

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

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

+78

1
2
3
4
5
6
7
8

public Boolean isOdd(int par) {
  Boolean ret=true;
  if(par%2==0)
   ret=false;
  else
   ret=true;
  return ret;
 }

Narekgevorgyan90, 02 Августа 2011

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

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

+146

1
2
3
4

// поменяем местами get и post
$_REQUEST=$_GET;
$_POST=$_GET;
$_GET=$_REQUEST;

хорошее начало. Уже после комментария я стал бояться.
Нет, ну блять,
1. нахуя пытаться менять местами _GET и _POST
2. через _REQUEST (что, переменных что ли мало?)
3. Попутно перетирая нахуй его и чем-то провинившийся _POST со всеми остановками по дороге.

PS каким-то образом кары избежал _GET. Порадуемся хоть за его.

alazutov, 28 Июля 2011

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

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

+174

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

<?php
function htmlspecialchars($a) {
	return htmlspecialchars($a);
	return preg_replace('/</','<',htmlspecialchars($a));
        echo htmlspecialchars($a);
}
echo htmlspecialchars('<a href="http://">линка</a>');  // мы пришли к тому, 
что заменили <a href="http://">линка</a>, 
то есть оно выведет прям ссылку, 
а мы сделали так, 
чтобы оно вывело <a href="http://">линка</a>, 
то есть выведит само <a href="http://">линка</a>, 
без всяких ссылок, прям как текст. 
Внести в функцию изменения, 
можно, 
но осторожно. 
ТОЛЬКО ДЛЯ ЗНОЮЩИХ ПИХХИПИ.
?>

мозготрах

substr, 13 Июля 2011

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

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

+92

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

// к говнокоду 7101:

interface

type TObjectAuto = class;
TSmartPtr = packed record
  data: TObjectAuto;
end;
TSmartPtrA = array of TSmartPtr;
//-------------------------------------------------------------------------------------------
// класс с "авто"-деструктором
TObjectAuto = class(TObject)
  n: integer; // для тестов
  constructor Create(var ptr: TSmartPtrA);
  destructor Destroy(); override;
end;

implementation

uses Windows;

var winheap: Cardinal;
var savedlinks: array of integer; // в тестовом примере сойдет, а вообще надо хеш-таблицу
//-------------------------------------------------------------------------------------------
constructor TObjectAuto.Create(var ptr: TSmartPtrA);
begin
  inherited Create();
  SetLength(ptr, 1);
  ptr[0].data := self;
  // сохраняем адрес выделенной памяти под массив (у него еще есть длина и счетчик ссылок)
  SetLength(savedlinks, Length(savedlinks) + 1);
  savedlinks[Length(savedlinks) - 1] := integer(ptr) - 2 * sizeof(integer);
end;
//-------------------------------------------------------------------------------------------
destructor TObjectAuto.Destroy();
begin
  n := 0; // сюда брякпойнт поставим:)
  inherited;
end;
//-------------------------------------------------------------------------------------------
function WinGetMem(Size: Integer): Pointer;
begin
  Result := HeapAlloc(winheap, 0, Size);
end;
//-------------------------------------------------------------------------------------------
function WinFreeMem(P: Pointer): Integer;
var i, j: integer;
begin
  // ищем адрес освобождаемой памяти среди сохраненных
  i := 0; j := 0; while(i < Length(savedlinks))do begin
    // если нашли, то вызываем "авто"-деструктор
    if (savedlinks[i] = integer(P)) then 
      TSmartPtrA(integer(p) + 2 * sizeof(integer))[0].data.Free()
    else begin
      savedlinks[j] := savedlinks[i];
      inc(j);
    end;
    inc(i);
  end;
  SetLength(savedlinks, j);
  HeapFree(winheap, 0, P);
  Result := 0;
end;
//-------------------------------------------------------------------------------------------
function WinReallocMem(P: Pointer; Size: Integer): Pointer;
begin
  Result := HeapReAlloc(winheap, 0, P, Size);
end;
//-------------------------------------------------------------------------------------------
var winmem: TMemoryManager = (
            GetMem: WinGetMem;
            FreeMem: WinFreeMem;
            ReallocMem: WinReallocMem);
    oldmem: TMemoryManager;
//-------------------------------------------------------------------------------------------
initialization
begin
  winheap := GetProcessHeap();
  GetMemoryManager(oldmem);
  SetMemoryManager(winmem);
  SetLength(savedlinks, 0);
end;
//-------------------------------------------------------------------------------------------
finalization
begin
  SetLength(savedlinks, 0);
  SetMemoryManager(oldmem);
end;
//-------------------------------------------------------------------------------------------
end.

// пример использования:
procedure TfrmTest.Button1Click(Sender: TObject);
var ptr: TSmartPtrA;
    obj: TObjectAuto;
begin
  obj := TObjectAuto.Create(ptr);
  obj.n := 222; // ptr[0].data.n := 222;
  // тут obj удалится сам
end;

примерно так можно реализовать автодеструктор в delphi
для передачи в функцию нужно использовать ptr и работать с ним как ptr[0].data - неудобно конечно.
ЗЫ: код тестовый - в нем полно кривостей.

ctm, 30 Июня 2011

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

C# / Говнокод #6999

+132

1

for(dynamic o=0; o!=237; o++){

Ох, что мне досталось поддерживать... dynamic почти везде... :( Кто это написал??? Хочу этого человека взять и @#$%^&... :(

Говногость, 19 Июня 2011

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

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

+92

1
2
3
4
5

public Boolean checkPermission(String login) {
    if (login == null && login.trim().length() == 0)
        throw new Exception("Не указан логин пользователя.");
    /* ... */
}

Чудесное условие. Есть в этом Exception что-то неуловимое - его никто никогда не поймает.

roman-kashitsyn, 14 Июня 2011

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

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