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

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

В номинации:

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

За время:

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

+92

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

protected IOException copyRange(InputStream istream, ServletOutputStream ostream) {

        // Copy the input stream to the output stream
        IOException exception = null;
        byte buffer[] = new byte[input];
        int len = buffer.length;
        while (true) {
            try {
                len = istream.read(buffer);
                if (len == -1) {
                    break;
                }
                ostream.write(buffer, 0, len);
            } catch (IOException e) {
                exception = e;
                len = -1;
                break;
            }
        }
        return exception;

    }

Си-стайл в исходниках Tomcat. Зачем кидать исключения, если их можно возвращать вместо кода ошибки?

Eyeless, 01 Августа 2011

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

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)

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)

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

+92

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

var
    CommHandle:integer;
    DCB:TDCB;
    Stat:TComStat;
    Kols,TransMask,Errs:DWord;
    Ovr:TOverlapped;
    s:string;
    i:integer;

{Чтение канала n}
function ReadChannel(n:integer):string;
begin
inc(n,32);                                                                                         //код запроса для прибора (это по спецификации прибора)
s:=chr(n);                                                                                         //символ запроса
WriteFile(CommHandle,s[1],length(s),Kols,@Ovr);                   //оправили этот код в прибор
TransMask:=0;                                                                               //маска любого события в порте
Ovr.hEvent:=CreateEvent(nil, True, False, nil);                          //хз
repeat
if not WaitCommEvent(CommHandle, TransMask, @Ovr) then                                                       //если приём закончен не до конца, то...
  if GetLastError = ERROR_IO_PENDING then WaitForSingleObject(Ovr.hEvent, INFINITE);           //при условии "хз" подождать ещё чуть-чуть

ClearCommError(CommHandle,Errs,@Stat);                                          //скинуть длину байт из буфера порта в переменную Stat
Kols:=Stat.cbInQue;                                                                                    //выяснить длину символов
until (Kols=1) or (Kols=14);                                                                        //закончить, если в канале пусто (1) или данные есть (14)

SetLength(s,Kols);                                                                                       //установить переменную s длиной Kols
ReadFile(CommHandle,s[1],Kols,Kols,@Ovr);                                         //прочитать данные из порта в s, обнулив буфер
if length(s)=1 then ReadChannel:='Пусто';                                             //дальше просто идёт обработка полученных данных
if length(s)=14 then ReadChannel:=copy(s,5,7);
end;



{Основной код}
for i:=1 to 60 do Value:=ReadChannel(i);                                         //ПОЛУЧИТЬ ДАННЫЕ ИЗ ПОРТА n

Имеется прибор с хранением данных в 60 каналах. Связь осуществляется по RS-232.
Чтобы получить данные, нужно отправить № кода нужного канала. В ответ придут данные этого канала.
Если канал пустой, то приходит 1 байт, если данные есть, то 14 байт.
Создана функция ReadChannel(n) с запросом данных из канала n.

ПРОБЛЕМА:
Прочитать все 60 каналов удаётся иногда 3 раза, иногда 15 раз. Но обязательно когда-нибудь программа зависнет на
WaitCommEvent(CommHandle, TransMask, @Ovr), так ничего и не получив из порта.
Если между командами
WriteFile и WaitCommEvent
поставить sleep(50) //9600 бод = 1200 б/с, то бишь 12 мс на 1 байт
то всё пашет. Но это не дело же привязываться ко времени?
Как избавиться от зависания на WaitCommEvent, чтобы процесс продолжился?
Если прибор не получил байт, из-за чего не ответил, как это проверить? "Данных нет уже 0,5 с, значит и не будет; отправлю этот байт ещё раз".

sbs, 10 Июня 2011

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

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

+92

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

procedure TForm1.Button1Click(Sender: TObject);
var
outfi:string;
begin
  if OpenDialog1.Execute then
  begin
  outfi:=sysutils.ExtractFilePath((OpenDialog1.FileName))+'/program.asm';
  windows.CreateFile(PChar(outfi),0,0,0,0,0,0);
  AssignFile(outPutFile,outfi);
  Rewrite(outPutFile);
  Append(outPutFile);
  AssignFile(inPutFile,OpenDialog1.FileName);
  Reset(inPutFile);
  init();
  writeLn(outPutFile,prologMemo.Lines.GetText());
 
  doprogram();
 
  writeLn(outPutFile,epilogMemo.Lines.GetText());
 
  CloseFile(outPutFile);
  CloseFile(inPutFile);
  ShellExecute( 0, 'open', PChar(outfi), nil, nil, SW_SHOWNORMAL);
  end;
end;

bugmenot, 06 Марта 2011

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

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

+92

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
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95

type BOOK= record
      Name:string[20];
      Avtor:string[20];
      Price:integer;
      end;

 BOOKS=Array[1..100] of BOOK;
 F= File of BOOK;
procedure Vvod(var biblioteka: books; var File_tBIBL:F; var n:integer);
Procedure AVTOR_KN(VAR File_tBIBl:F; VAR File_tBIBl_AVTOR:text);
Procedure TIP_TEXT(VAR File_tBIBl:F; VAR File_tBIBl_text:text);
Function MIN_CENA( var File_tBIBl:F ; n:integer ):integer;
procedure Vvod(var biblioteka: books; var File_tBIBL:F; var n:integer);
  var i:integer;
  begin
   rewrite(File_tBIBL);

  FOR i:=1 to n do begin
                 Writeln('  Vvedite nazvanie knigi');
                   readln(biblioteka[i].Name);
                 Writeln('  Vvedite avtora');
                   readln(biblioteka[i].Avtor);
                 Writeln('  Vvedite cenu');
                   readln(biblioteka[i].Price);
                 Write(File_tBIBL,biblioteka[i]);
                 writeln;
                   end;
  close(File_tBIBL);
  end;

  Procedure TIP_TEXT(VAR File_tBIBl:F; VAR File_tBIBl_text:text);
   var BIBLIOTEKA:BOOk;

     BEGIN

   reset(File_tBIBl);
   rewrite(File_tBIBl_text);

   WHILE not eof(File_tBIBl) do begin
                                read(File_tBIBl,BIBLIOTEKA);
                                writeln(File_tBIBl_text,BIBLIOTEKA.Name);
                                writeln(File_tBIBl_text,BIBLIOTEKA.Avtor);
                                writeln(File_tBIBl_text,BIBLIOTEKA.Price);
                                writeln(File_tBIBl_text);
                                writeln(File_tBIBl_text);
                                end;
   close(File_tBIBl);
   close(File_tBIBl_text);
   end;

  Procedure AVTOR_KN(VAR File_tBIBl:F; VAR File_tBIBl_AVTOR:text);
   var BIBLIOTEKA:BOOk; Av:string;

     BEGIN


   reset(File_tBIBl);
   rewrite(File_tBIBl_AVTOR);

   Writeln('Vvedite iskomogo avtora');
   readln(Av);

 WHILE not eof(File_tBIBl) do   begin
                                read(File_tBIBl,BIBLIOTEKA);
                                if BIBLIOTEKA.Avtor = Av then begin
                                                             writeln(File_tBIBl_AVTOR,BIBLIOTEKA.Name);
                                                             writeln(File_tBIBl_AVTOR,BIBLIOTEKA.Avtor);
                                                             writeln(File_tBIBl_AVTOR,BIBLIOTEKA.Price:4);
                                                             writeln(File_tBIBl_AVTOR);
                                                             end;
                                end;
   close(File_tBIBl);
   close(File_tBIBl_AVTOR);
    end;

var nomer,i,min:integer;BIBLIOTEKA:BOOks; SIZE:integer;
begin
  reset(File_tBIBl);
 size:=1;
    while  not eof(File_tBIBl) do
        with BIBLIOTEKA[size] do
    begin
        readln(File_tBIBl , BIBLIOTEKA.name);
         Readln(File_tBIBl,BIBLIOTEKA.Avtor);
        readln(File_tBIBl,BIBLIOTEKA.Price);

        inc(size);
    end;
    dec(size);
 min:=BIBLIOTEKA[1].Price;
for i:=1 to size  do
if BIBLIOTEKA[i].Price< min then        begin
                                        min:=BIBLIOTEKA[i].Price;
                                        nomer:=i;
                                        end;

Juris_Kabanis, 27 Февраля 2011

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

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

+92

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

I:=1;
    while I<=High(Sockets) do
     begin
      if FD_IsSet(Sockets[I],FDSet) then
       if Recv(Sockets[I],…)<=0 then
        begin
         // Связь разорвана, надо закрыть сокет
         // и удалить его из массива
         CloseSocket(Sockets[I]);
         for J:=I to High(Sockets)-1 do
          Sockets[J]:=Sockets[J+1];
         Dec(I);
         SetLength(Sockets,Length(Sockets)-1)
        end
       else
        begin
         // Получены данные от клиента, надо ответить
         Send(Sockets[I],…)
        end;
      Inc(I)
     end;

На первый взгляд может показаться странным, почему для перебора элементов массива используется цикл while, а не for. Но в дальнейшем мы увидим, что размер массива во время выполнения цикла может изменяться. Особенность же цикла for заключается в том, что его границы вычисляются один раз и запоминаются в отдельных ячейках памяти, и дальнейшее изменение значений выражений, задающих эти границы, не изменяет эти границы. В нашем примере это приведёт к тому, что в случае уменьшения массива цикл for не остановится на реальной уменьшившейся длине, а продолжит цикл по уже не существующим элементам, что приведёт к трудно предсказуемым последствиям. Поэтому в данном случае лучше использовать цикл while, в котором условие продолжения цикла полностью вычисляется при каждой его итерации.

TarasB, 09 Февраля 2011

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

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

+92

1
2

Result := not FExecuting;
if not Result then Exit;

сейчас работаю над правкой компонента доставшегося от другого разработчика :), причем эта вершина мысли была обнаружена в недрах левого компонента, используемого моим компонентом :) ,который использовал этот разработчик :), Сам код соправождаемого мною компонента не далеко по стилю ушел от этого куска

ageron, 23 Декабря 2010

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

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

+92

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

unit uncrt;

interface

procedure ClrScr;
procedure Sound(Hz: Word);
procedure NoSound;

procedure TextColor(Color: Byte);
procedure TextBackground(Color: Byte);

var
  TextAttr: Byte; { Для особо продвинутых случаев }

implementation

procedure ClrScr;
begin end;

procedure Sound(Hz: Word);
begin end;
procedure NoSound;
begin end;

procedure TextColor(Color: Byte);
begin end;
procedure TextBackground(Color: Byte);
begin end;

end.

Вас достала школота, раскрашивающая свои бибикающие программы?
Вам поможет этот модуль!

TarasB, 16 Декабря 2010

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

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

+92

1
2
3
4

if (x>13)or(x<13)or(x=13) then begin writeln('введено невероное значение X');halt end else begin
//Расчёт уравнения
..........
end;

в лабе в студента увидел o_O
конечно он получил 2 за лабу
но ка пример какокода супер

adli__82, 17 Августа 2010

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

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