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

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

В номинации:

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

За время:

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

+160

01
02
03
04
05
06
07
08
09
10
11

<?php
$countH =1;
$T = 0;
foreach ($aSelectedUnits as $v) {
  if ($T != $v['UnitType']) {
    $countH = 1;
  }
  $countHired[$v['UnitNameId']] = $countH ++; // calculate hired units by type
  $T = $v['UnitNameId'];
}
?>

всего навсего нужно было подсчитать количество юнитов каждого типа.
Как ни странно этот кусок кода считает юнытов, но не всегда верно.

Sergunik, 22 Февраля 2011

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

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

+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

<?php
//дата и время.
$date1 = date('d.m.Y');
$date2 = date('H:i:s');
//$i - счётчик пользователей, $cnt - то, что будем выводить (в завершении)
//$uall - Ссылки на пользователей онлайн
$i = 0;
$cnt = NULL;
$uall = NULL;
$db->query = "SELECT id, nick, timeof, online FROM ".$GLOBALS['pref']."users";
$qonl = $db->query();
$cnt .= "Всего участников ";

while ($usronl = $db->farray($qonl)) {
if ($usronl['online'] == TRUE) {
$ar = explode("|:|", $usronl['timeof']);

//Если сегодняшняя дата
if ($ar[0] === $date1) {

$est = explode(":", $ar[1]);
$tm = explode(":", $date2);
$m4 = $est[0] - $tm[0];
$m5 = $tm[1] - $est[1];
$m6 = "$m5";
if (($est[0] === $tm[0]) && (($m5 <= 15))) {
if ($i > 0) { $p = ","; } else { $p = NULL; }
if ($i == 1) { $s = ","; } else { $s = NULL; }
$i++;
$uall .= $s."<a href=\"http://".$GLOBALS['url']."/users/
profile/".$usronl['id']."/\" title=\"Посмотреть профиль\">".$usronl['nick']."<
/a>".$p." ";

 } elseif (($m4 == -1) && ($m6{0} === "-" && $m5 <= -45)) {
if ($i > 0) { $p = ","; } else { $p = NULL; }
if ($i == 1) { $s = ","; } else { $s = NULL; }
$i++;
$uall .= $s."<a href=\"http://".$GLOBALS['url']."/users/
profile/".$usronl['id']."/\" title=\"Посмотреть профиль\">".$usronl['nick']."<
/a>".$p." ";
 }
  }
   } 
	} //end while
if ($i == 0) {
$uall .= "никого";
} else {

$co = strlen($uall);
$uall = substr($uall, 0, $co - 7);
$uall .= ".";
}
$cnt .= "(".$i."):<BR>".$uall;

?>

Писал вывод онлайн посетителей на сайте в 2009 году.

Furry, 21 Февраля 2011

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

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

+160

1
2
3

if (RENAME_FILE) {
      $dest_filename = md5(uniqid(rand(), true)) . '.' . $file_ext;
    }

Рандом, рандом, еще рандом!

7ion, 12 Февраля 2011

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

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)

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

+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

<script language="javascript" type="text/javascript">
<!--
var ie=document.all?1:0;
var ns=document.getElementById&&!document.all?1:0;
 
function InsertSmile(SmileId)
{
    if(ie)
    {
    document.all.message.focus();
    document.all.message.value+=" "+SmileId+" ";
    }
 
    else if(ns)
    {
    document.forms['guestbook'].elements['message'].focus();
    document.forms['guestbook'].elements['message'].value+=" "+SmileId+" ";
    }
 
    else
    alert("Ваш браузер не поддерживается!");
}
// -->
</script>

qbasic, 08 Февраля 2011

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

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

+160

01
02
03
04
05
06
07
08
09
10

if ($this->Session->check('_userJustRegistered')) {
	switch (($tmpJR = $this->Session->read('_userJustRegistered'))) {
		case 'confirmed':
			$this->Session->delete('_userJustRegistered');
			break;
		case 'not_confirmed_first':
			$this->Session->write('_userJustRegistered', 'not_confirmed');
			break;
	}
}

Кусочек многоступенчатой системы по определению статуса юзера.

quall, 08 Февраля 2011

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

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

+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

QString convToHex( unsigned char *bytes, int size )
{
	if( size > 16 )
		size = 16;
	QString ret;
	for( int i = 0; i < size; i++ )
	{
		char buf[64];
		::snprintf( buf, sizeof(buf), "%02x", (unsigned int) bytes[i] );
		ret += buf;
	}	
	return ret;
}

QString TarCreator::generateGuid( const std::string &tDeviceSerial,
		const std::string &nDatetime, unsigned long long int id )
{
	std::ostringstream s;
	s << tDeviceSerial;
	s << nDatetime;
	s << id;
	
	QCryptographicHash hash( QCryptographicHash::Md5 );
	hash.addData( QByteArray( s.str().c_str() ) );
	QByteArray result = hash.result();
	return convToHex( (unsigned char*)result.data(), result.size() );
}

panter_dsd, 08 Февраля 2011

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

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

+160

1
2
3
4

void IRQHandler(void)
{
...
WaitSemaphore(semH);

Из кода для встроенного устройства реального времени.

Говногость, 08 Февраля 2011

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

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