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

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

В номинации:

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

За время:

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

+127

1

(&goodsBuff[k].name==&vmass[p].name)

Сравнение строк

GrayCardinal, 25 Октября 2010

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

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

+135

1
2
3

do {
                 data = get_stat();
        } while ((data & (1 << stat_smth_ok_bit_n)) != (1 << stat_smth_ok_bit_n));

Суровый педантичный немецкий код.

minixoid, 22 Октября 2010

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

1C / Говнокод #3880

−128

1
2
3
4

ПолучитьСтроку();
Пока КоличествоСтрок() <> 0 Цикл
	УдалитьСтроку();
КонецЦикла;

Новый способ удаления всех строк из документа. Похоже, что разработчик не ведает о существовании метода УдалитьСтроки();

SSNick, 05 Августа 2010

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

JavaScript / Говнокод #3879

+161

1
2
3
4

var s = '';
	s += (hours)  ? ( (hours>9)  ? hours  + ':' : '0' + hours  + ':') : '00:';
	s += (minute) ? ( (minute>9) ? minute + ':' : '0' + minute + ':') : '00:';
	s += (second) ? ( (second>9) ? second + '.' : '0' + second + '.') : '00.';

почему бы не сделать так:
var s = '';
s += (hours<10) ? '0' + hours : hours) + ':';
// и так далее...

rudolff, 05 Августа 2010

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

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

+165

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

if ($row = mysql_fetch_array($result))
{
	$obegin = $row[0];
	$vbegin = $row[1];
	$lbegin = $row[2];
	$oclose = $row[3];
	$vclose = $row[4];
	$opodk = $row[5];
	$vpodk = $row[6];
	$lpodk = $row[7];
	$oend = $row[8];
	$vend = $row[9];
	$lend = $row[10];
	$ozatr = $row[11];
	$vzatr = $row[12];
	$lzatr = $row[13];
	$ooplat = $row[14];
	$voplat = $row[15];
	$loplat = $row[16];
	$opay = $row[17];
	$vpay = $row[18];
	$lpay = $row[19];
	$otraf = $row[20];
	$vtraf = $row[21];
	$ltraf = $row[22];
}

Даже не смешно

Werdn, 30 Июля 2010

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

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

+166

1
2
3
4
5
6

function ostatok($chislo1, $chislo2){
	$delit=$chislo1/$chislo2;
	$celoe=round($delit);
	$ostatok=$chislo1-$chislo2*$celoe;
	return $ostatok;
}

1. Всё можно было уместить в 1 строчку.
2. Когда аффтору кода сказали о существовании функции fmod - он был немного в шоке.

XyHb, 27 Июля 2010

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

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

+168

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

function endPanel()
	{
		if($this->type=="common" || $this->type=="common_blank")
		{
				?>
								</div>
						</div>
				</div>
				<?php
		}
		elseif($this->type=="common_short")
		{
				?>
								</div>
						</div>
				</div>
				<?php
		}
		elseif($this->type=="main" || $this->type=="main_small")
		{
			?>
					</div>
				</div>
				<div class="bottom">
					<div class="bottom-right"></div>
				</div>
			</div>
			<?php
		}
		elseif($this->type=="light" || $this->type=="light_blank" || $this->type=="light_narrow")
		{
			?>
					</div>
				</div>
				<div class="bottom">
					<div class="bottom-right"></div>
				</div>
			</div>
			<?php
		}
	}

Прислали на работе проект на доработку, мало того предыдущий программер не знает про разделение логики и представления, дак еще и такие штуки на каждом шагу встречаются

xoros, 22 Июля 2010

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

JavaScript / Говнокод #3618

+162

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

var titleAlt = String(dbField("title").value);
if (titleAlt.toLowerCase().indexOf("<b>") != -1) { titleAlt = titleAlt.replace(/<b>/gi, "").replace(/<\/b>/gi, ""); }
if (titleAlt.toLowerCase().indexOf("<i>") != -1) { titleAlt = titleAlt.replace(/<i>/gi, "").replace(/<\/i>/gi, ""); }
if (titleAlt.toLowerCase().indexOf("<small>") != -1) { titleAlt = titleAlt.replace(/<small>/gi, "").replace(/<\/small>/gi, ""); }
if (titleAlt.toLowerCase().indexOf("<br>") != -1) { titleAlt = titleAlt.replace(/<br>/gi, ""); }
if (titleAlt.toLowerCase().indexOf("<p>") != -1) { titleAlt = titleAlt.replace(/<p>/gi, "").replace(/<\/p>/gi, ""); }
f = 0;
while (f != -1) {
    f = titleAlt.indexOf("<img ");
    if (f == -1) { f = titleAlt.indexOf("< img "); }
    if (f == -1) { f = titleAlt.indexOf("<  img "); }
    if (f == -1) { f = titleAlt.indexOf("<IMG "); }
    if (f == -1) { f = titleAlt.indexOf("< IMG "); }
    if (f == -1) { f = titleAlt.indexOf("<  IMG "); }
    if (f != -1) {
        t = -1;
        for (i = f; i < titleAlt.length; i++) {
            if (titleAlt.substr(i, 1) == ">") { t = i; i = titleAlt.length; }
        }
        if (t != -1) {
            ttl = titleAlt;
            titleAlt = ttl.substring(0, f) + ttl.substring(t + 1, titleAlt.length);
        }
    }
}

это не что иное, как попытка очистить строку от html
код как бы говорит нам - "регулярные выражения - зло!"
цикл while - это просто шедевр зодчества эпохи просвещения. он ищет тег img и закрывающую скобку, и если они есть - присваивает результату подстроку от начала строки до первого вхождения img и от последнего вхождения угловой скобки

найдено в серверном JavaScripte на одном из зарубежных сайтов

mozg_raka, 01 Июля 2010

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

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

+67

1
2
3
4

String a = 1234567890
String b = 4
def c = a.split(b)
if c.size() > 1 //...

Замена indexOf

robin, 30 Июня 2010

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

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