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

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

В номинации:

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

За время:

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)

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

+171

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

// Файл settings.cpp
#define SETTINGS_READ
#include "Settings.h" 

#undef SETTINGS_READ
#include "Settings.h" 
#endif

// ----------------------------------
// Файл settings.h
#ifdef SETTINGS_READ
        #define _XML_OPTION_VALUE(name,value) if (XML.FindElem(_CRT_WIDE(#name))) {/*XML.IntoElem();*/ XML.GetData(value);  /*ShowVar(value);ShowVar(_CRT_WIDE(#name));*/  /* XML.OutOfElem();*/}
        #define _XML_OPTION_ATTR(name,value) XML.GetAttrib(_CRT_WIDE(#name),value) 
        #define XML_NODE_START(name,value) if (XML.FindElem(_CRT_WIDE(#name))) {XML.IntoElem();
        #define XML_NODE_START_WITHATTRIB(name,value) if (XML.FindElem(_CRT_WIDE(#name))) {
        #define XML_NODE_STARTS(name,value) if (XML.FindElem(name)) {
        #define XML_NODE_START_VALUE(name,value) if (XML.FindElem(_CRT_WIDE(#name))) { XML.GetData(value);XML.IntoElem();
        #define XML_NODE_START_VALUE_WITHATTRIB(name,value) if (XML.FindElem(_CRT_WIDE(#name))) { XML.GetData(value);     
        #define XML_NODE_END()  \
                XML.OutOfElem(); \
        }
        #define XML_OPTION_VALUE(name) _XML_OPTION_VALUE(name,name)
        #define XML_OPTION_MEMBER_VALUE(prefix, name) _XML_OPTION_VALUE(name, prefix##.name)
        #define XML_OPTION_ATTR(name) _XML_OPTION_ATTR(name,name)
        #define XML_OPTION_MEMBER_ATTR(prefix, name) _XML_OPTION_ATTR(name, prefix##.name)
        #define COLOR_TO_PINT(c) (*(int*)&##c)
#else
        // SAVING MACROSES
        #define _XML_OPTION_VALUE(name,value) XML.AddElem(_CRT_WIDE(#name),value) ; /*XML.OutOfElem();*/
        #define _XML_OPTION_ATTR(name,value) XML.SetAttrib(_CRT_WIDE(#name),value) 
        #define XML_NODE_START(name,value) XML.AddElem(_CRT_WIDE(#name)); {XML.IntoElem();
        // ( еще десяток макросов)
#endif

#ifdef SETTINGS_READ
bool CSettings::MacroLoadSettings(CMyXml &XML)
{
#else
bool CSettings::MacroSaveSettings(CMyXml &XML)
{
#endif
        TCHAR szFont[1024];
	CString Font;
        XML_NODE_START(Settings);   
        XML_NODE_START(General);
                XML_OPTION_MEMBER_VALUE(Settings,Language); 
                XML_OPTION_VALUE(ExplorerContextMenu);

                #ifndef  SETTINGS_READ
                         HotkeysStr= Settings.Hotkeys.toString();
                         FontToString(&LogoSettings.Font,Font);                        
                #endif
                        
                XML_OPTION_VALUE(HotkeysStr);
                        #ifdef  SETTINGS_READ
                                Settings.Hotkeys.DeSerialize(HotkeysStr);
                                StringToFont(Font, &LogoSettings.Font);
                        #endif
                #endif
        XML_NODE_END();
       // ..............................................
        XML_NODE_END(); // end of settings
        XML_NODE_END(); // 
        return true;
}

Фееричный пример повторного использования кода с помощью директив препроцессора.

Создано с целью совместить в одном месте чтение/запись настроек программы (с риском что-нибудь забыть) из XML файла.

Kernel32, 13 Июня 2011

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

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