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

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

В номинации:

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

За время:

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

+75

1
2
3
4
5
6
7

private boolean isShowPosition(List<Integer> lists, int p) {
        for (int l : lists) {
            if (p == l)
                return true;
        }
        return false;
    }

Вьетнамское творчество

AAverin, 04 Марта 2014

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

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

+139

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

#include <stdio.h>
#include <stdlib.h>
#define value_i int
#define gen_program main()
#define begin {
#define end }
#define sep ;
#define back_val return
#define vis(a) printf ("%d", a)
#define ln printf ("\n")
#define ln3 printf ("\n\n\n")
#define cycle_count for
#define rand rand()
#define gen_rand srand((unsigned)time(NULL))
#define value_c char
#define vis_c(a) printf("%c", a)
#define eq =
#define uneq !=
#define check ==
#define inc(a) a++
#define res %
#define add +
#define sub -
#define blank void
#define small <
#define big >

blank line(blank) sep

value_i gen_program
begin
    value_i x, i                                sep
    gen_rand                                    sep
    value_c c eq ' '                            sep
    cycle_count(i eq 1 sep i small 10 sep inc(i))
    begin
        x eq rand res 10 add 1                  sep
        if (x uneq 10)
        begin
            vis(x)                              sep
            vis_c(c)                            sep
        end
    end
    ln3                                         sep
    line()                                      sep
    ln                                          sep
    back_val 0                                  sep

end

blank line(blank)
begin
    value_i i sep
    cycle_count (i eq 0 sep i small 10 sep inc(i) )
    begin
    printf("-/*\\")                                sep
    end
    printf("-")                                    sep
end

Можно рефакторить.

Hammer, 01 Марта 2014

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

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

+67

1
2
3
4
5
6
7
8

public class LoaderForMongoCompanies extends AbstractJdbcLoader {

    @Override
    protected String getTableSuffix() {
        return "mongo";
    }

}

нужно больше классов

roman-kashitsyn, 28 Февраля 2014

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

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

+153

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

/*где-то в начале: error_reporting(0);*/

class PhocaGalleryFile{
	function getCSSFile($id = 0, $fullPath = 0) {
		if ((int)$id > 0) {
			$db = &Factory::getDBO();
			$query = 'SELECT a.filename as filename, a.type as type'
				.' FROM #__styles AS a'
			    .' WHERE a.id = '.(int) $id;
			$db->setQuery($query, 0, 1);
			$filename = $db->loadObject();
			if (isset($filename->filename) && $filename->filename != '') {
				if ($fullPath == 1 && isset($filename->type)) {
					return self::getCSSPath($filename->type). $filename->filename;
				} else {
					return $filename->filename;
				}
			}
		}

		return false;
	}

 . . .
}

Поубивал пустые строки (по 2 штуки между каждой строкой кода) и комменты вида:
/* ===============
== This function is geting a CSSFile fron database .....
===============
*/
Гы! Код для заказчика должен выглядеть наукообразно и непонятно.
И обязательно выдавать Notices и Warnings.

virtual_cia, 19 Декабря 2013

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

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

+132

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

procedure tnewthread.checkfiles; // процедура выполняется в потоке
var
  i:integer;
  status:tstatus;
  ptmp:array of char;
  temp:string;
  len:integer;
  fstream:tfilestream;
begin
  flist.Clear;
  findfiles(findpath);
  for i:=flist.Count-1 downto 0 do
  begin
    status:=s_ok;
    try
      try
        fstream:=tfilestream.Create(flist[i],fmopenread);
        fstream.Position:=0;
        setlength(ptmp,fstream.size);
        fstream.Read(pointer(ptmp)^,fstream.size);
      except
        status:=s_error;
      end;
    finally
      fstream.free;
    end;
    temp:=string(pchar(ptmp));
    temp:=stringreplace(temp,'&nbsp;',' ',[rfreplaceall]);
    temp:=stringreplace(temp,'&gt;','>',[rfreplaceall]);
    temp:=stringreplace(temp,'&nbsp;',' ',[rfreplaceall]);
    temp:=stringreplace(temp,'&lt;','<',[rfreplaceall]);
    temp:=stringreplace(temp,'&amp;','&',[rfreplaceall]);
    temp:=stringreplace(temp,'&quot;','"',[rfreplaceall]);
    temp:=stringreplace(temp,'&copy;',#169,[rfreplaceall]);
    temp:=stringreplace(temp,#10,#13#10,[rfreplaceall]);
    Len := Length(temp);
    try
      try
        fstream:=tfilestream.Create('C:\1.txt',fmcreate); // заменил в целях теста, не помогает.
        fstream.Position:=0;
          fstream.WriteBuffer(temp[1], Len); // в этом  месте поток вылетает с ошибкой "Range check error"
      except
        status:=s_error;
      end;
    finally
      fstream.free;
    end;
    if status=s_ok then
    begin
      addfileinfo(flist[i]); 
      shrecyclefile(flist[i]);
    end
    else
    begin
      adderrinfo(flist[i]); // синхронизируемся с мемо и добавляем в него красную строчку с именем файла
      shmovefile(flist[i],erroroutputpath +'\' + extractfilename(flist[i])); // перемещаем файл в директорию с файлами, при обр. которых произошла ошибка
    end;
  end;
end;

Процедура для обработки текстовых файлов. Имеем дремлющий поток, залоченный waitsingleobject, который будит
таймерная функция, если в папке есть по крайней мере 1 файл. т.е. одновременно к файлам обращается 1 поток.
При разлочивании поток немедленно начинает заполнять лист именами файлов, после чего начинает прогонять их
через процедуру-обработчик. Но вот беда - возникает ошибка range check error. причем возникает только в доп.потоке -
вне потока все работает нормально. Товарищи ,не подскажете, в чем лажа? (

Stertor, 24 Ноября 2013

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

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

+101

1

if( a.GetPixel(i, k).Name.ToString().Substring(0,6)=="ff0404")

http://www.cyberforum.ru/csharp-net/thread954611.html#post5057105

Qwertiy, 15 Сентября 2013

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

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

+133

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

private string LineTabsCRLFToSpaces(string line)
        {
            string ret_line = "";
            if (line != null)
            {
                //    throw new ArgumentNullException("line");
                if (line.Length == 0) return string.Empty;  // Пустая входная строка
                for (int i = 0; i < line.Length; i++)
                {
                    char ch = line[i];
                    if (ch == '\t' || ch == '\r' || ch == '\n')
                        ret_line = ret_line + " ";
                    else
                        ret_line = ret_line + ch;
                }
            }
            return ret_line;
        }

- line.Replce()?
- Не, не слышал!

nolka4, 12 Августа 2013

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

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

+14

1

std::set_unexpected( [] () {} );

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

maksim_ovcharik, 29 Мая 2013

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

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

+133

1
2
3
4

while (Process.GetProcesses().Where(x => x.ProcessName == "FREngine").Count() >= 2)
{
 Thread.Sleep(5000);
}

ждём-с

taburetka, 22 Марта 2013

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

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

+129

1
2
3
4
5
6
7
8

> echo 1 >1.cp
> echo 1 >1.cpp
> echo 1 >1.cppp
> dir /b *.cp
1.cp
> dir /b *.cpp
1.cpp
1.cppp

Баг в FindFirstFile/FindNextFile, показан на виндовом dir для простоты демонстрации. Призрак DOS'а жив даже в семерке...

bormand, 18 Января 2013

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

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