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

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

В номинации:

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

За время:

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

+160.1

1
2
3
4
5
6
7

class Company extends BaseCompany {
//...
    public function setName($v){
        $this->setName(String::mb_ucfirst($v));
    }
//...
}

Мегаметодика переопределения сеттеров. Symfony, Propel.

guest, 29 Января 2009

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

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

+156.7

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

<?php

$mov   = new ffmpeg_movie ( '1.avi' );
$frame = $mov -> GetFrame ( 26 ):
$gd    = ToGd ( $frame );

header ( "Content-type: image/gif" );
imageGif ( $gd );

function ToGd ( $frame )
{
  RETURN $frame -> toGDimage ();
}

?>

Говнокод? =)

guest, 28 Января 2009

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

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

+19

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

// то что было

/************************************
return TRUE if version1 >= version2
version1 = "XX.XX.XX"
version2 = "XX.XX.XX"
************************************/
function CheckVersion($version1, $version2)
{
	$arr1 = explode(".",$version1);
	$arr2 = explode(".",$version2);
	if (intval($arr2[0])>intval($arr1[0])) return false;
	elseif (intval($arr2[0])<intval($arr1[0])) return true;
	else
	{
		if (intval($arr2[1])>intval($arr1[1])) return false;
		elseif (intval($arr2[1])<intval($arr1[1])) return true;
		else
		{
			if (intval($arr2[2])>intval($arr1[2])) return false;
			elseif (intval($arr2[2])<intval($arr1[2])) return true;
			else return true;
		}
	}
}

// а я бы сделал вот так
// причем при добавлении новой подверсии работоспособность не изменится

function CheckVersion($version1, $version2)
{
	return (intval(str_replace('.','',$version1))>=intval(str_replace('.','',$version2))) ? true : false;
}

Просто шикарное решение от умов битрикса

guest, 23 Января 2009

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

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

+34.2

1
2
3

if (preg_match("/^([zZ][aA]|[зЗ][аА])$/", $text)) {
		//za		
} elseif (preg_match("/^([pP][rR][oO][tT][iI][vV]|[пП][рР][оО][тТ]][иИ][вВ])$/", $text)){

Составил кто-то регулярку на проверку в сообщении слов за/za, против/protiv

Обратите внимание как элегантно решена проблемма регистра символов

guest, 16 Января 2009

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

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

+29

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

if (searchBox.isFocused()) {
			addressBookPopupMenu.enableItem(0, false);//view
			addressBookPopupMenu.enableItem(1, true);//search
			addressBookPopupMenu.enableItem(2, true);//add contact
			addressBookPopupMenu.enableItem(3, false);//edit contact
			addressBookPopupMenu.enableItem(4, false);//delete contact
			addressBookPopupMenu.enableItem(5, false);            
		}
		else {
			addressBookPopupMenu.enableItem(0,true);
			addressBookPopupMenu.enableItem(1,false);
			addressBookPopupMenu.enableItem(2,true);
			addressBookPopupMenu.enableItem(3,true);
			addressBookPopupMenu.enableItem(4,true);
			addressBookPopupMenu.enableItem(5, true);           
		}

		if(isSocialNWportal()) {
			addressBookPopupMenu.enableItem(2, false);//add contact
			addressBookPopupMenu.enableItem(3, false);//edit contact
			addressBookPopupMenu.enableItem(4, false);//delete contact
		}

индусская логика vs Российская. Чуть ни упал пацтол когда соптимизячил и сравнил со своим кодом:

boolean allowFlag = !(searchBox.isFocused() || contactListBox.isEmpty());
boolean portalFlag = !isSocialNWportal();
addressBookPopupMenu.enableItem(0, allowFlag);
addressBookPopupMenu.enableItem(1, !allowFlag);
addressBookPopupMenu.enableItem(2, portalFlag);//add contact
addressBookPopupMenu.enableItem(3, allowFlag && portalFlag);
addressBookPopupMenu.enableItem(4, allowFlag && portalFlag);
addressBookPopupMenu.enableItem(5, !searchBox.isFocused());

guest, 14 Января 2009

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

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

+37.1

01
02
03
04
05
06
07
08
09
10

function check_email($address)
{
	$regex = '/^((\"[^\"\f\n\r\t\v\b]+\")|([\w\!\#\$\%\&\'\*\+\-\~\/\^\`\|\{\}]+(\.[\w\!\#\$\%\&\'\*\+\-\~\/\^\`\|\{\}]+)*))@((\[(((25[0-5])|(2[0-4][0-9])|([0-1]?[0-9]?[0-9]))\.((25[0-5])|(2[0-4][0-9])|([0-1]?[0-9]?[0-9]))\.((25[0-5])|(2[0-4][0-9])|([0-1]?[0-9]?[0-9]))\.((25[0-5])|(2[0-4][0-9])|([0-1]?[0-9]?[0-9])))\])|(((25[0-5])|(2[0-4][0-9])|([0-1]?[0-9]?[0-9]))\.((25[0-5])|(2[0-4][0-9])|([0-1]?[0-9]?[0-9]))\.((25[0-5])|(2[0-4][0-9])|([0-1]?[0-9]?[0-9]))\.((25[0-5])|(2[0-4][0-9])|([0-1]?[0-9]?[0-9])))|((([A-Za-z0-9\-])+\.)+[A-Za-z\-]+))$/';
	if (preg_match($regex, $address)) {
		return true;
	}
	else {
		return false;
	}
}

Функция для проверки e-mail'а
Очень скромный РегЕксп

(с) n3m0

guest, 12 Января 2009

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

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

+31.7

1
2

$uin=$_POST['uin'];
$uin=$uin+1-1;

И такое сплошь и рядом... А intval() ведь зачем-то придумали...
by nod

guest, 11 Января 2009

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

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

+51.6

1

if ($name == "ryuzaki") {die("Вы заблокированы");}

guest, 10 Января 2009

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

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