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

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

В номинации:

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

За время:

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

+57

1
2
3
4
5
6
7

function attribute( $attr ) {
    if( method_exists( $this, $attr ) ) {
        return $this->$attr();
    }

    return $this->$attr();
}

Видимо сначало было без проверки.

enures, 25 Декабря 2012

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

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

+57

1
2
3
4
5
6
7
8
9

// Просто удаляет элемент из массива
  function delElement($arr, $index)
  {
    $result = Array();
    for($i=0;$i<count($arr);$i++)
      if($index != $i) $result[] = $arr[$i];
    
    return $result;
  }

- Ваше мнение, коллега?
- Кал молодой особи пыхаписта, коллега. Очень ярко выражены типичные признаки.

clauclauclau, 14 Декабря 2012

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

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

+57

1
2
3
4
5
6
7

if($atributId){
      $sql = "UPDATE directory_atributes SET name = '$newName' WHERE id = $atributId LIMIT 1";
      $db-> Query($sql);
      die();
 } else{
      die();
 }

Депрессивное программирование. В любом случае ты умрёшь.

somnambulism, 01 Октября 2012

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

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

+57

1

if( $data['id_Account'] =! $this->user->Account )

somnambulism, 17 Сентября 2012

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

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

+57

1
2
3
4
5
6
7
8
9

$aMethods[] = array(
			'call' => 'getNewCount',
			'requires' => array(
				'user_id' => 'user_id'
			),
			'detail' => Phpfox::getPhrase('notification.get_the_total_number_of_unseen_notifications_if_you_do_not_pass_the_user_id_we_will_return_information_about_the_user_that_is_currently_logged_in'),
			'type' => 'GET',			
			'response' => '{"api":{"total":5,"pages":0,"current_page":0},"output":5}'			 
		);

Движок Phpfox 3.3. Самая длинная фраза сообщения :))

xakip, 11 Сентября 2012

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

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

+57

1
2
3
4
5
6

echo '<strong>'.$amount.'</strong>';
if($amount > 1) {
 echo $VM_LANG->_('PHPSHOP_PRODUCTS_LBL');
}else{
  echo $VM_LANG->_('PHPSHOP_PRODUCTS_LBL');
}

Virtuemart forever!
А может просто блондинка там код пишет?

virtual_cia, 07 Сентября 2012

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

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

+57

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

function GetOrderSkidka(&$arrData)
    	{
	    	if($this->flag_opt){
	    	 	$arrData['skidka']  = ($this->admin_mode) ? $arrData['skidka'] : 0;
	    	 	$arrData['allsum']  = $arrData['sum'] - $arrData['skidka'];
	    	 	return;
	        }
	    	if(!$this->flag_in_action){
	    		$arrData['cnt_s_prod'] = $arrData['cnt'];
	    	}elseif(in_array($this->flag_action_type,array(2,3))){
	    		$this->calcCntProd($arrData);
	    	}else{
	    		$arrData['cnt_s_prod'] = 0;
	    	}
	    	$this->discount->GetOrderSkidka($arrData);
    	}

Работаю с сайтом, в котором все методы классов работают со своими параметрами таким образом.
Метод может ничего не возвращать, а вызывать другие методы (которые также могут вызывать какие-то методы),
которые в зависимости от множества условий меняют переданные по ссылке параметры.
Итог работы модифицированный параметр- массив. Только XDebug выручает.

minramilka, 31 Августа 2012

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

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

+57

1
2
3
4
5
6
7

function SetImage($row=array())
	{
     foreach($row as $property=>$value)
     	{
     	eval("\$this->".$property." = '".$value."';");
     	}
	}

minramilka, 30 Августа 2012

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

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

+57

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

<?php
//...
foreach ($month as $key=>$value)
	{
		if($value!=0)
		{
			$leased[$year[$key]] = $pay_leased->get_leased_pay($year[$key],$value);
			$dialup[$year[$key]] = $pay_dialup->get_billing_pay($year[$key],$value);
			$domain[$year[$key]] = $pay_domain->get_domain_pay($year[$key],$value);
			$web[$year[$key]] = $pay_web->get_hosting_pay($year[$key],$value);

			for ($d=1;$d<=31;$d++) // во всех месяцах по 31 дню, да
			{
				$leased[$year[$key]][$value][$d] = isset($leased[$year[$key]][$value][$d]) ? $leased[$year[$key]][$value][$d] : 0;
				$dialup[$year[$key]][$value][$d] = isset($dialup[$year[$key]][$value][$d]) ? $dialup[$year[$key]][$value][$d] : 0;
				$domain[$year[$key]][$value][$d] = isset($domain[$year[$key]][$value][$d]) ? $domain[$year[$key]][$value][$d] : 0;
				$web[$year[$key]][$value][$d] = isset($web[$year[$key]][$value][$d]) ? $web[$year[$key]][$value][$d] : 0;
			}
			ksort($leased[$year[$key]][$value]);
			ksort($dialup[$year[$key]][$value]);
			ksort($domain[$year[$key]][$value]);
			ksort($web[$year[$key]][$value]);

			for ($c=1;$c<=31;$c++) // и еще
			{
				$leased[$year[$key]][$value][$c] = $leased[$year[$key]][$value][$c-1]+$leased[$year[$key]][$value][$c];
				$dialup[$year[$key]][$value][$c] = $dialup[$year[$key]][$value][$c-1]+$dialup[$year[$key]][$value][$c];
				$domain[$year[$key]][$value][$c] = $domain[$year[$key]][$value][$c-1]+$domain[$year[$key]][$value][$c];
				$web[$year[$key]][$value][$c] = $web[$year[$key]][$value][$c-1]+$web[$year[$key]][$value][$c];
				$sum[$year[$key]][$value][$c] = $leased[$year[$key]][$value][$c]+$dialup[$year[$key]][$value][$c]+$domain[$year[$key]][$value][$c]+$web[$year[$key]][$value][$c]; // вау, какая большая строка
			}
		}
	}
//...

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

rsvasilyev, 28 Августа 2012

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

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

+57

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

class GetParameter
	{
		var $dbhost = '';
		var $dbname = '';
		var $dblogin = '';
		var $dbpassword = '';

		var $deepmenu1itemstable = '`deepmenu1items`';
		var $deepmenu2itemstable = '`deepmenu2items`';
		var $topmenu2itemstable = '`topmenu2items`';
		var $topmenu3itemstable = '`topmenu3items`';
		var $topmenu4itemstable = '`topmenu4items`';
		var $news1table = '`news1`';
		var $news2table = '`news2`';
		var $news3table = '`news3`';
		var $news4table = '`news4`';
		var $sites2table = '`sites2`';
		var $chattable = '`chat`';
		var $votestable = '`votes`';
		

		var $varstable = '`vars`';
		var $sitestable = '`sites`';
		var $orderstable = '`orders`';
		var $fotoreporttable = '`fotoreport`';
		var $fotortable = '`fotor`';
		var $articlestable = '`articles`';
		var $menuitemstable = '`menuitems`';
		var $topmenuitemstable = '`topmenuitems`';
		var $streetstable = '`streets`';
		var $regionstable = '`regions`';
		var $zastrojstable = '`zastrojs`';
		var $naznachofisstable = '`naznachofiss`';
		var $managerstable = '`managers`';
		var $opensellstable = '`opensells`';
		var $objectstable = '`objects`';
		var $kodstable = '`kods`';
		var $namestable = '`names`';
		var $adminstable = '`admins`';
		var $peoplestable = '`peoples`';

		var $userstable = '`users`';
}

Сайт написан за 1500$
молчу про остальные классы и функции.

Samedi, 24 Июня 2012

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

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