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

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

В номинации:

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

За время:

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

+154

1

http://habrahabr.ru/blogs/web_security/137196/

Нанотехнологии в действии

Spider, 28 Января 2012

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

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

+166

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
96

if ($id == 1) {
    $source = 'GallerySkins/s/skin_01.png';
    $dest = "MinecraftSkins/$user.png";
	
	if(!copy($source, $dest)) {
        echo "Ошибка<br />\n";
    }
}
elseif ($id == 2) {
    $source = 'GallerySkins/s/skin_02.png';
    $dest = "MinecraftSkins/$user.png";
	
	if(!copy($source, $dest)) {
        echo "Ошибка<br />\n";
    }
}
elseif ($id == 3) {
    $source = 'GallerySkins/s/skin_03.png';
    $dest = "MinecraftSkins/$user.png";
	
	if(!copy($source, $dest)) {
        echo "Ошибка<br />\n";
    }
}
elseif ($id == 4) {
    $source = 'GallerySkins/s/skin_04.png';
    $dest = "MinecraftSkins/$user.png";
	
	if(!copy($source, $dest)) {
        echo "Ошибка<br />\n";
    }
}
elseif ($id == 5) {
    $source = 'GallerySkins/s/skin_05.png';
    $dest = "MinecraftSkins/$user.png";
	
	if(!copy($source, $dest)) {
        echo "Ошибка<br />\n";
    }
}
elseif ($id == 6) {
    $source = 'GallerySkins/s/skin_06.png';
    $dest = "MinecraftSkins/$user.png";
	
	if(!copy($source, $dest)) {
        echo "Ошибка<br />\n";
    }
}
elseif ($id == 7) {
    $source = 'GallerySkins/s/skin_07.png';
    $dest = "MinecraftSkins/$user.png";
	
	if(!copy($source, $dest)) {
        echo "Ошибка<br />\n";
    }
}
elseif ($id == 8) {
    $source = 'GallerySkins/s/skin_08.png';
    $dest = "MinecraftSkins/$user.png";
	
	if(!copy($source, $dest)) {
        echo "Ошибка<br />\n";
    }
}
elseif ($id == 9) {
    $source = 'GallerySkins/s/skin_09.png';
    $dest = "MinecraftSkins/$user.png";
	
	if(!copy($source, $dest)) {
        echo "Ошибка<br />\n";
    }
}
elseif ($id == 10) {
    $source = 'GallerySkins/s/skin_10.png';
    $dest = "MinecraftSkins/$user.png";
	
	if(!copy($source, $dest)) {
        echo "Ошибка<br />\n";
    }
}
elseif ($id == 11) {
    $source = 'GallerySkins/s/skin_11.png';
    $dest = "MinecraftSkins/$user.png";
	
	if(!copy($source, $dest)) {
        echo "Ошибка<br />\n";
    }
}
elseif ($id == 12) {
    $source = 'GallerySkins/s/skin_12.png';
    $dest = "MinecraftSkins/$user.png";
	
	if(!copy($source, $dest)) {
        echo "Ошибка<br />\n";
    }
}

Разгребал систему скинов и наткнулся на это... даже не знаю, как это чудо назвать.
Заменяется элементарным
$source = 'GallerySkins/s/skin_'.$id.'.png'; // Исходная директория файла
$dest = "MinecraftSkins/$user.png"; // Конечная директория файла

if(!copy($source, $dest)) // Если файл не скопирован:
{
echo "Ошибка<br />\n";
}

FanAs, 28 Января 2012

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

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

+158

1
2
3
4

if(strlen(trim($row['article']))!=0)
{
 // Some action
}

empty? не, это из будущего.
Суки! Сколько можно это писать!

Zerstoren, 26 Января 2012

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

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

+150

1
2
3

http://php.net/include/layout.inc
http://php.net/include/site.inc
http://php.net/include/languages.inc

без комментариев :)

Beat, 24 Января 2012

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

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

+147

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

#include <string>
using namespace std;
int main();
void pushvalues(const string&, const string&, string&);

int main {
	string lol1;

	//OUTPUT
	
	if (!(lol1))
		return 1;
	string lol2;
	string *word1, *word2 = new string;
	
	for (int i = 0; i != '101'; ++i) {
		//Насколько я помню pop_back и pop_front возвращает удаленное значение, в противном случае сам допиши
		word1.push_back(lol1.pop_front); // 123456 <- 7 <- 89 :)
		if (word1[i-1] == ' ') {
			word1.pop_back;
			i = 101;
		}
	
	}
	for (int i = 0; i != '101'; ++i) {
		word2.push_back(lol1.pop_front);
		if (word2[i-1] == ' ') {
			word2.pop_back;
			i = 101;
		}

	}
	
		pushvalues(&word1, &word2, &lol2); // (word2 word1) (word3 word4 word5)

	while (lol1) // word1 <-> word3 | while anything in lol1 (1. Recognise word3 2. wrap with word1 3. ??? 4. PROFIT!)
	{
		for (int i = 0; i != '101'; ++i) {
			word2.push_back(lol1.pop_front);
			if (word2[i-1] == ' ') { // WORD3 RECOGNISED FUCK YEAH ;[
				word2.pop_back;
				i = 101;
			}
		}
	
		pushvalues(&word1, &word2, &lol2);
	}

	delete word1, word2;
//OUTPUT
return 0; }

/*void change(string& word1, string& word2) {
string temp;
temp = word1;
word1 = word2;
word2 = temp;
return; }*/

void pushvalues(const string& word1, const string& word2, string& container) {
	container.push_back(word2);
	container.push_back(' ');
	container.push_back(word1);
return; }

Кодил ночью в режиме потока сознания :)
Input и output не запилил тогда, попросили помощи, писал в общих чертах
Этот код, кстати вылетает с ошибками компиляции.
Плюсы вообще в последнее время начали страдать, ибо взялся за другой язык и бросил учить плюсы

ReckO, 12 Января 2012

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

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

+1008

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

QPainter painter(this);
int a1;
int a2;
int a3;
int a4;
//.......
int a999;
int a1000;

a1 = 1;
a2 = 1;
a3 = 1;
a4 = 1;
//.......
a999 = 1;
a1000 = 1;

while (a1 != 500)
{         painter.drawLine(a1, a1, a1, a1); a1 ++; 
}

while (a2 != 500)
{         painter.drawLine(a2, a2, a2, a2); a2 ++; 
}
while (a3 != 500)
{         painter.drawLine(a3, a3, a3, a3); a3 ++; 
}
while (a4 != 500)
{         painter.drawLine(a4, a4, a4, a4); a1 ++; 
}
//........
while (a999 != 500)
{         painter.drawLine(a999, a999, a999, a999); a999 ++; }
}
while (a1000 != 500)
{         painter.drawLine(a1000, a1000, a1000, a1000); a1000 ++; }
}

ПРИКИНЬТЕ такое в paintevente!!!!!!! Qt

VProger, 05 Января 2012

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

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

+104

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

// старый "медленый" код, проверяем размеры по именам файлов (последний параметр):

  if((checkFileLimits(_logTimeLimit,_logSizeLimit,_logStartTime,_traceFile1)>0) ||
     (checkFileLimits(_logTimeLimit,_logSizeLimit,_logStartTime,_traceFile2)>0) ||
     (checkFileLimits(_logTimeLimit,_logSizeLimit,_logStartTime,_traceFile3)>0) ||
     (checkFileLimits(_logTimeLimit,_logSizeLimit,_logStartTime,_traceFile) >0) )

// новый "быстрый" код, проверяем размеры по файл хэндлам:

  FILE* fp1 = fopen(_traceFile1, "r");
  FILE* fp2 = fopen(_traceFile2, "r");
  FILE* fp3 = fopen(_traceFile3, "r");
  FILE* fp4 = fopen(_traceFile, "r");
  
  if((checkFileLimitsHandle(_logTimeLimit,_logSizeLimit,_logStartTime,fp1)>0) ||
     (checkFileLimitsHandle(_logTimeLimit,_logSizeLimit,_logStartTime,fp2)>0) ||
     (checkFileLimitsHandle(_logTimeLimit,_logSizeLimit,_logStartTime,fp3)>0) ||
     (checkFileLimitsHandle(_logTimeLimit,_logSizeLimit,_logStartTime,fp4) >0) )
         setTraceFile(NULL);

  fclose(fp1);
  fclose(fp2);
  fclose(fp3);
  fclose(fp4);

наши бенчмаркеры чего-то там тестировали (на NFS!!!) и нашли что некоторые модули/библиотеки используют stat() вместо fstat()/ftell() для определения размера лог/трейс файлов (для ротации этих файлов). stat() берет как параметр не хэндл, а имя файла и поэтому дороже с точки зрения производительности. в особенности на NFS. ну начальник R&D и постановил: все stat()ы заменить на fstat()/ftell(). сказано - сделано. кусок сверху из модуля который пользуется внешней либой для логов и трейсов и у которого доступа к хэндлам нету. но герои не ищут легких путей: открываем файлы, получаем хэндлы, проверяем оптимальным образом размер файлов по хэндлам, закрываем файлы, гатова!

Dummy00001, 14 Декабря 2011

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

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

+165

1
2
3
4
5

$Drv=$pvu->driver_restriction;
        $Drv1=$pvu->driver_restriction;

        if ($Drv=="on") {$d='Y';}
        if ($Drv1=="off") {$d='N';}

Обрабатываем входные параметры перед сохранением.

denis0, 25 Ноября 2011

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

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

−92

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

#! /usr/bin/python3.1
import tarfile, os, datetime, time
tudate = datetime.datetime.now()
tud_date = list(tudate.timetuple())
day = tud_date[2]; month = tud_date[1]; year = tud_date[0]; dyear = tud_date[7]
tar_pwd = '/home/fess/Desktop/Server/arch/%d.%d.%d.tar.gz' % (day,month,year)
pwd_bases = '/home/fess/Desktop/Server/arch'; archives = os.listdir(pwd_bases) # получаем список всех файлов в дирректории
os.chdir(pwd_bases) # move arround the directory with the bases
if os.path.exists(tar_pwd): pass # если архив с именем сегоднешней даты есть в папке, то ничего не делает
else: # в противном случае создает архив            
        tar = tarfile.open(tar_pwd,'w:gz')
        tar.add('/home/fess/Desktop/Server/base/','bases')
        tar.close()
for archive in archives: # Проверка каждого архива в папке с архивами
        stf = os.stat(archive); dayy = time.localtime(stf.st_mtime).tm_yday; m = dyear-dayy # узнаем дату создания арх и узнаем сколько ему дней (m)
        if m >= 50: # если архиву больше или 50 дней
                os.remove(archive); print(archive,'was removed')# тогда удаляем архив которому больше 50 дней 
        if dyear < dayy: # Если сегодняшний день меньше чем дата создания файла(т.е. наступил новый год)
                os.remove(archive); print(archive,'was removed')# удалить 49 оставшихся архивов с прошлого года

зачетная катяшенция по моему вышла, бэкапчеГ ;)

fess, 10 Ноября 2011

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

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

+161

1
2
3
4

if ($_POST["SAVE"]!="OK")
        $this->IncludeComponentTemplate();
else
        $_POST=Array()

оп оп

atarix12, 10 Ноября 2011

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

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