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

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

В номинации:

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

За время:

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

+135

1

<SCRIPT LANGUAGE=JavaScript src="../../../../../vstavka.txt"></script>

http://www.helloworld.ru/texts/comp/lang/php/spravscript/stranica13.html что статьи говно что сайт.

Vasiliy, 02 Марта 2011

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

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

+115

1
2
3
4
5
6
7

RepaymentEvent e = DatabaseHelper.GetString("event_type", pReader) == "RBLE"
	                ? new BadLoanRepaymentEvent {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}
	                : (DatabaseHelper.GetString("event_type", pReader) == "RRLE"
	                       ? new RescheduledLoanRepaymentEvent {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}
	                       :(DatabaseHelper.GetString("event_type", pReader).StartsWith("P") 
                                ? new PendingRepaymentEvent (DatabaseHelper.GetString("event_type", pReader)) {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}
                                : new RepaymentEvent {Id = DatabaseHelper.GetInt32("rpe_id", pReader)}));

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

_Ru55_, 02 Марта 2011

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

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

+164

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

var s = setUpOmnitureVariable();
s.pageName="LO-20110214-EE-VQP-203";
s.server="www2.experts-exchange.com";
s.channel="Web_Development/Web_Languages-Standards/PHP";
s.pageType="";
s.prop1=document.title;
s.prop2="";
s.prop3="";
s.prop4="";
s.prop5="";
s.prop6="";
s.prop7=""; // TODO THIS IS NO LONGER PROP 7, SHOULD BE BLOG AUTHOR
s.prop8="";
s.prop9="";
s.prop10="";
s.prop11="";
s.prop12="";
s.prop14="";
s.prop15="50";
s.prop16="";

<...>

s.prop27="";
s.prop28="";

<...>
s.prop41="";
s.prop42="";
s.prop43="";
s.prop44="";
s.prop45="";
s.prop46="";
s.prop47="";
s.prop48="";
s.prop49="";
s.prop50="";
/* E-commerce Variables */
s.campaign="";
s.events="event4,event74";
s.products="";
s.state="";
s.zip="";
s.purchaseID="";
s.eVar1=document.title;
s.eVar2="";
s.eVar3="AD #3 - Scroll, bullet, no steps";
s.eVar4="";
s.eVar5="";
s.eVar6="Free Trial Guest";
s.eVar7="";
s.eVar8="";
s.eVar9="";
s.eVar10="";
s.eVar14="";

<...>
s.eVar26="";
s.eVar27="";
s.eVar28="";
s.eVar29="5AM";
s.eVar30="Sunday";
s.eVar31="77.243.99.37";
s.eVar32="";
s.eVar33="Web_Development/Web_Languages-Standards/PHP";
s.eVar34="3";
s.eVar35="20110214-EE-VQP-203-FREE-TRIAL";
s.eVar36="Header AD";

<...>

s.eVar44="Accepted Solution";
s.eVar45="EE_Level3 - 2";
s.eVar46="";
s.eVar47="";
s.eVar48="";
s.eVar49="rating9.6";

Источник виден в коде. Неужели нельзя было массив сделать?! Или это такой странный вид обфускации?

RaZeR, 27 Февраля 2011

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

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

+164

1
2
3
4
5

$this->db->Query("DELETE FROM `u_proposal`
				WHERE `id` = '%d' AND `user` = '%d'", $item, $this->user->id);
$this->db->Query("DELETE FROM `p_options`
				LEFT JOIN `u_proposal` ON `p_options`.`portfolio` = `u_proposal`.`id`
				WHERE `p_options`.`portfolio` = '%d' AND `p_options`.`type` = 'project' AND `u_proposal`.`user` = '%d'", $item, $this->user->id);

3 часа еб**ся не мог понять почему записи из p_options не удаляются ))

Sulik78, 25 Февраля 2011

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

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

+164

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

$zzz1="<table width=\"100%\" border=\"1\" cellspacing=\"0\" cellpadding=\"0\" class=\"tsa\">";
$zzz3="</table>";
        if($str==1){
                $str=0;
        }
        for($lk=$str;$lk<$str+10;$lk++){
        $kod=$pv;
        $kod=$this->sql("SELECT name_saita, url_saita, opis FROM $this->sait WHERE id='{$kod[$lk][id]}'");
        $kod=$this->mb($kod);
        if($kod[0]['name_saita']==NULL){
                break;
        }
$zzz2=$zzz2."<tr>
   <td align=\"center\" width=\"20%\">{$kod[0][nov_saita]}</td>
   <td width=\"80%\" class=\"op\">{$kod[0][opis]}</td>
   </tr>";
}
$zzz=$zzz1.$zzz2.$zzz3;

Классика жанра

Uchkuma, 21 Февраля 2011

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

SQL / Говнокод #5713

−858

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

INSERT INTO actpr (userid,orgid,grupid,grupa, sum,abo_sum,;
	req,atbilde,kods,time,requestid,pakalpojum);
	value(Grozs .userid, Grozs .orgid,Grozs .grupid,Grozs .grupa,Grozs.sum ,0, ;
	Grozs .req,Grozs .atbilde, Grozs .kods,;
	this.time,this.requestId ,Grozs .pakalpojum)

replace ;
	tarifs WITH  Grozs .tarifs ,;
	aid with Grozs .aid,;
	param with Grozs .param,;
	ip with this.ip,;
	atlikum WITH konti.extra_sum - Grozs.sum ,;
	abo_atlik WITH konti.abo_sum, ;
	old_atlik WITH konti.old_sum ,;
	sessionid  WITH this.sessionid,;
	veids with Grozs .veids,;
	stan_cena WITH Grozs .stan_cena,;
	pvn WITH Grozs .pvn,;
	next_sum WITH 0,;
	next_atlik WITH konti.next_sum;
	 IN  actpr

Написано на FoxPro (!)
Незнаю даже что написать...

lurseg, 18 Февраля 2011

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

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

+160

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

<?php

class UserSessionComponent extends Component {
	//...
	protected function loginByUserId($user_id, $is_autologin = false) {
		$list = $this->_loadComponents($this->components, array('UserAuth'));
		$userAuth = $list['UserAuth'];
		$userAuth->startup($this->controller); # !!! WORKFLOW VIOLATION !!!
		$method = $is_autologin ? 'autologin' : 'login';
		if ($userAuth->$method($user_id)) {
			return $this->controller->Session->read('User.data');
		}
	}
	//...
}

class UserAuthComponent extends BaseMagicComponent {
	protected $loginType = 'explicit';

	//...

	public function autologin($params = array()) {
		$this->loginType = 'implicit';
		return $this->_login($params);
	}

	public function login($params = array()) {
		$this->loginType = 'explicit';
		return $this->_login($params);
	}

	//...
	protected function _login($params = array()) {
		switch (true) {
			case (is_array($params) && isset($params['user']) && isset($params['password'])):
				$result = $this->loginByUserAndPassword($params['user'], $params['password']);
				break;
			case (is_numeric($params) && $params > 0):
				$result = $this->loginUser($params);
				break;
			default:
				if (empty($params)) {
					$result = false;
				} else {
					$result = $this->loginUser($params);
				}
		}
		// ...
	}

	// ...
	public function loginUser($user) {
		switch (true) {
			case is_object($user):
				if (isset($user->data)) {
					$a_user = $user->data;
				} else {
					return false;
				}
				break;
			case (is_numeric($user) && ($user > 0)):
				if (!($a_user = $this->User->findById($user))) {
					return false;
				}
				break;
			case (is_array($user) && (isset($user['User']['id']))):
				$a_user = $user;
				break;
			default:
				return false;
		}
		// ...
		CFL::using('log');
		if (empty($GLOBALS['loginIsLogged'])) {
			$loginLog = new cflLoginTrackingLogger();
			if ($this->loginType == 'implicit') {
				$loginLog->implicit_login(array('user_id' => $a_user['User']['id']));
			} else {
				$loginLog->explicit_login(array('user_id' => $a_user['User']['id']));
			}
			$GLOBALS['loginIsLogged'] = true;
		}
		// ...
	}
}

Сорри за много-много кода, еле-еле вписался в лимит. Но пришлось запостить все. Логическая цепочка совершенно потрясающая (смотрим с самого верхнего метода и идем дальше к самому нижнему методу). Тут у нас кода просто по колено: сумасшедший пахлава-код, использование глобальной переменной флага, особое извращенное использование конструкции switch, фактическое дублирование этого самого switch в двух последних методах, нейминг переменных и методов...
Казалось бы, как все это вообще могло быть написано? Но использование компонента с говорящим именем BaseMagic все ставит на свои места.

quall, 09 Февраля 2011

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

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

+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

// 0. original version:
struct Subscriber {
	ServiceConfig *services;
};

struct ServiceConfig {
};

// reported problem: as per doc, Customer may have multiple service configurations, yet we can configure only one.

// 1. next "fixed" version, as delivered to the customer:
struct Subscriber {
};

struct ServiceConfig {
	Subscriber *subs;
};

// ticket reopened: wtf. changing services for one customer, affects others!!

// 2. next "fixed" version, as delivered to the customer:
struct Subscriber {
	ServiceConfig *services;
};

struct ServiceConfig {
};

// ticket reopened: @#$%^& [*BEEP*] @#$%^& [*BEEP*] @#$%^&*

// 3. finally:
struct Subscriber {
};

struct ServiceConfig {
};

struct SubscriberServiceConfig {
	Subscriber *subs;
	ServiceConfig *services;
};

если бы лично не лицезрел эту переписку (навечно задокументированую в аналах вершн контрол) между нашими разработчиками и консультантами работающими на инсталяции у кастомера, нивжисть бы не поверил что такое даже может иметь место.

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

Dummy00001, 07 Февраля 2011

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

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

+146

1
2
3

char odin=1,odin2=1;
if (&odin!=&odin2)
        Memo1->Lines->Add("FUCK!");

я новичок, и после 3-й надписи FUCK! в Memo1 мой моск взорвалсо.
помните меня таким, каким я был...

myrik, 04 Февраля 2011

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

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