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

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

В номинации:

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

За время:

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

+147

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

<script type="text/javascript" >
    var str = window.location + "";
    var t = str.substr(7, 2);
    if (t=="40") {document.write("(4842)<span style=\"font-size:30px;font-weight:bold;color:white;\">562–003</span>");}
    if (t=="36") {document.write("(473)<span style=\"font-size:30px;font-weight:bold;color:white;\">233-03-20</span>");}
    if (t=="62") {document.write("(4912)<span style=\"font-size:30px;font-weight:bold;color:white;\">300-305</span>");}
    if (t=="ww") {document.write("(4872)<span style=\"font-size:30px;font-weight:bold;color:white;\">39-03-06</span>");}
    if (t=="po") {document.write("(4872)<span style=\"font-size:30px;font-weight:bold;color:white;\">39-03-06</span>");}
</script>
<script type="text/javascript" >
   var str = window.location + "";
   var t = str.substr(7, 2);
   if (t=="40") {document.write("Калуга");}
   if (t=="36") {document.write("Воронеж");}
   if (t=="62") {document.write("Рязань");}
   if (t=="ww") {document.write("Тула");}
   if (t=="po") {document.write("Тула");}
</script>

Вот так упыри меняют телефоны на сайте в зависимости от региона, при том, что сайт использует PHP на сервере.

demitriy_, 12 Августа 2011

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

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

+144

1

<head> <body bgcolor="00FF00"></head>

прислали в вопросом "будет ли работать" :)

viktore4ka, 12 Августа 2011

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

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

+165

1
2

foreach ($_POST as $kp => $post)
	$_POST2[$kp] = $post;

Это написано в самом начале того скрипта, что мне дали фиксить, в коде я встретил строчки описанные выше, а также деление на 0 и неправильно раставленные скобки в условиях с && и ||

DarkThinker, 11 Августа 2011

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

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

+84

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

search = 0;
try {
	search = search + wdContext.currentSearchElement().getLifnr().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getBahns().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getINN().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getKPP().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getKtokk().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getLand1().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getBanks().length();
} catch (Exception e) {
	// TODO: handle exception
}
try {
	search = search + wdContext.currentSearchElement().getBankl().length();
} catch (Exception ex) {
	// TODO: handle exception
}
try {
	search = search + wdContext.currentSearchElement().getBankn().length();
} catch (Exception ex) {
//	TODO: handle exception
}
try {
	search = search + wdContext.currentSearchElement().getLocco().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	//	search = search + wdContext.currentSearchElement().getLoevm().length();
	search = search + wdContext.currentSearchElement().getName().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getOKPO().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getOrt01().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getSortl().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
if (search == 0)
	wdComponentAPI.getMessageManager().reportException("Введите критерии поиска", true);
else {
	//Далее идет поиск
}

Вот такой вот была забабахана проверка входных параметров перед вызовом функции поиска (мною добавлен только последний комментарий, все остальное оригинальное)

foGa, 11 Августа 2011

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

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

−86

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

def MonthsListGen(*args, **kwargs):
	from datetime import datetime

	if kwargs.has_key('month') and kwargs.has_key('year'):
		return "%s-%s"%(kwargs['year'], "0%d"%kwargs['month'] if kwargs['month']<10 else str(kwargs['month']))

	if kwargs.has_key('date'): return MonthsListGen(year = kwargs['date'].year, month = kwargs['date'].month)

	if kwargs.has_key('decodeName'): return MONTH_NAMES[int(kwargs['decodeName'].split('-')[1])]
	if kwargs.has_key('decodeYear'): return int(kwargs['decodeYear'].split('-')[0])
	if kwargs.has_key('startDate') and kwargs.has_key('endDate'):
		startDate = kwargs['startDate']
		endDate = kwargs['endDate']
		monthList = []
		if startDate.year < endDate.year:
			startDate1 = startDate
			endDate1 = datetime.strptime('%d.%d.%d'%(DAYS_IN_MONTH[12], 12, startDate1.year) , '%d.%m.%Y')
			monthList = MonthsListGen(startDate = startDate1, endDate = endDate1)

			startDate2 = datetime.strptime('%d.%d.%d'%(1, 1, startDate1.year+1) , '%d.%m.%Y')
			endDate2 = endDate
			monthList += MonthsListGen(startDate = startDate2, endDate = endDate2)
			return monthList
		if startDate.year == endDate.year:
			monthRange = range(startDate.month, endDate.month+1)
			year = startDate.year
			for monthNo in monthRange:
				monthList.append(MonthsListGen(year = year, month = monthNo))
			return monthList
	return False

Вместо того, чтобы писать несколько разных функций, решил сделать одну, которая почти во всех случаях вызывает сама себя с разными параметрами.

hakimovis, 10 Августа 2011

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

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

+158

01
02
03
04
05
06
07
08
09
10
11
12

$max = 150;

$dlja_poiska = explode(' ', $_GET['name']);
$num = count($dlja_poiska);

if ( $num > $max ) $num = $max;
$qPart = '';

foreach( $dlja_poiska as $param )
   $qPart .= ' AND (system_menu_paragraph.name RLIKE \''.addslashes().'\') ';

// далее запрос к база без mysql_real_escape_string()

http://forum.codeby.net/topic42345.html -- чел с завышенной самооценкой...
Код писал я исходя из найденой дырки на сайте
Вот url на скрипт из которого вы можете догадаться что код его действительно примерно такой...
http://alexanderc.org/uploads/scripts/expl.php

AlexanderC, 06 Августа 2011

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

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

+179

1
2
3
4
5
6

jQuery.atmosphere.request.transport = 'polling';
if (jQuery.atmosphere.request.transport != 'websocket') {
	jQuery.atmosphere.executeRequest();
} else if (jQuery.atmosphere.request.transport == 'websocket') {
	// ..
}

Сразу удар по двум яйцам.
Сначала говорим, что transport = 'polling', а потом проверяем, не равен ли он случайно 'websocket'
Второй удар - обычный else использовать, видимо, было не прикольно. Или у транспорта кроме "равен websocket" и "не равен websocket" может быть третье состояние? ;)

Код актуальный и на данный момент присутствует в официальном репозитории, пруф: https://github.com/jfarcand/atmosphere/blob/master/modules/jquery/src/main/webapp/jquery/jquery.atmosphere.js#L493-503

пс. А мы на работе удивляемся, почему 'websocket' у них не работает)

Shock, 03 Августа 2011

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

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

+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

/**
     * Xml Request
     *
     * @param int $hotel_id
     * @param int $hotel_id
     * @param int $hotel_id
     * @return array $cc_type of
     */
public function XmlStringForBokingFactura($transaction_id, $block, $begin_date, $end_date, $hotel_code, $guest_qty, $smoking, $cc_number, $cc_cardholder, $cc_type, $cc_expiration_date, $cc_cvc, $NameBooker, $LastNameBooker, $guestLastname, $guest_city, $guest_country, $guest_telephone, $email, $additionalEmail, $fake=1, $affiliate_id, $affiliate_pswd, $is_hash = null, $arrival_time, $dialog) {
    ...
    ...
    ...
    $mail = new MailMime($build_params);
    $mail->setHeader($from_email, $subject, array('charset' => 'UTF-8'));
    $mail->setTXTBody($url);
    $mail->setHTMLBody($url);
    $mail->send($to);
    $mail->setHeader($from_email, $subject, array('charset' => 'UTF-8'));
    $mail->setTXTBody($data);
    $mail->setHTMLBody($data);
    $mail->send($to2);

    ...
}

переменные $to и $to2 нигде в функции никак не объявляются. да и вообще функция шикарная.

shmaltorhbooks, 03 Августа 2011

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

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

+124

1

.signup-block { display: clock }

random2, 01 Августа 2011

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

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