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

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

В номинации:

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

За время:

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

+155.4

1
2
3
4
5

function strtolowerex($str)
{
	$str = strtr($str, "АБВГДЕЁЖЗИЙКЛМНОПРСТУФХЦЧШЩЪЫЬЭЮЯ", "абвгдеёжзийклмнопрстуфхцчшщъыьэюя");
	return strtolower($str);
}

я может чего-то не понимаю? 0_о

gorky, 23 Сентября 2009

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

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

+55.9

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

while (true)
  {
    m_CurrentDevice += delta;

    if (m_CurrentDevice < 0)
      m_CurrentDevice = devices[DEV_PLATE]-1;

    if (m_CurrentDevice > devices[DEV_PLATE]-1)
      m_CurrentDevice = 0;

    break;
  }

Это конечно написано не специально - в результат целой кучи итераций и переделок. но результата это не отменяет

TObject, 21 Сентября 2009

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

ActionScript / Говнокод #1861

−90.9

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

function getActiveRoot():int
{
	for(var c:int=0;c<i;c++) if(r_arr[c].isActive) return c;
	return -1;
}





public function addEntry():void
{
	if(getActiveRoot()<0)
	{
		r_arr.push(new Roots(i,in_txt.text,""));
		r_arr[i].addEventListener(MouseEvent.MOUSE_DOWN, downHandler);
		r_arr[i].addEventListener(MouseEvent.MOUSE_UP, upHandler);
				
		addChild(r_arr[i++]);
	}
	else
	{
	        r_arr[getActiveRoot()].w_arr.push(new Words(getActiveRoot(),r_arr[getActiveRoot()].wi,in_txt.text,""));
		r_arr[getActiveRoot()].w_arr[r_arr[getActiveRoot()].wi].addEventListener(MouseEvent.MOUSE_DOWN, WdownHandler);
		r_arr[getActiveRoot()].w_arr[r_arr[getActiveRoot()].wi].addEventListener(MouseEvent.MOUSE_UP, WupHandler);
				
		var ls:Shape=new Shape();
		ls.graphics.lineStyle(1, 0xFF0000,0.2);
		ls.graphics.moveTo(r_arr[getActiveRoot()].x,r_arr[getActiveRoot()].y);
	        ls.graphics.lineTo(r_arr[getActiveRoot()].w_arr[r_arr[getActiveRoot()].wi].x,r_arr[getActiveRoot()].w_arr[r_arr[getActiveRoot()].wi].y);
		addChildAt(ls,++wd);
		addChildAt(r_arr[getActiveRoot()].w_arr[r_arr[getActiveRoot()].wi++],++wd);
	}
}

Ой. А давайте проц погоняем! Чтоб не скучал. Отовсюду будем запускать getActiveRoot() чтобы узнать индекс! Ха-ха-ха..

t8apb, 17 Сентября 2009

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

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

+59.9

1
2
3
4
5
6
7
8

void someMethod(const std::string& name,  std::autp_ptr<SomeClass> p);

{
    ...
    std::auto_ptr<SomeClass> p(new SomeClass());
    someMethod(p->GetName(), p);
    ...
}

Самое забавное, что при определенных условиях оно может работать.

Lexey, 17 Сентября 2009

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

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

+52.3

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

void GAMEPLAYER::cCar::abandonCar(GAMEPLAYER::cPlayer* player){
	for(unsigned int i=0;i<size();++i){
		cBaseAnimManager * bam = get(i);
		if(!bam->isCarRider())
			continue;
		cCarRider* rdr = (cCarRider*)bam;
		if(rdr->gScript()==player->gScript()){
			delete rdr;
			storage.erase(i);
			--i;
		}
	}
}

НЕ time-critical код из некоторой игры
Наблюдаем:
1) непониманием того, зачем нужно наследование
2) плохо пахнущие имена переменных

generalgda, 16 Сентября 2009

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

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

+147.7

1

$this->view->isAdmin = ($auth->getIdentity()->role_id == 1) ? true : false;

Называется: "заработался".
Написал и только потом дошло.

oldfornit, 16 Сентября 2009

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

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

+157.4

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

....
$str = str_replace(',', ' ', $this->searchString);
$str = str_replace(';', ' ', $str);
$str = str_replace('.', ' ', $str);
$str = str_replace(':', ' ', $str);
$str = str_replace('+', ' ', $str);
$str = str_replace('-', ' ', $str);
$str = str_replace('"', ' ', $str);
$str = str_replace("'", ' ', $str);
		
$str = trim(preg_replace('!\s{2,}!', ' ', $str));
.....

вот что я наклал как-то

paranoid, 11 Сентября 2009

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

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

+154.8

1
2
3
4
5
6
7

function get_($what)
	{
		global $parser;
		$lang = $parser->lang;
		$lang = $this->$lang;
		return $lang[$what];
	}

random2, 09 Сентября 2009

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

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

+166.8

1

unset($keys[count($keys)-1][count($keys[count($keys)-1])-1]);

stepushyn, 07 Сентября 2009

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

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

+75.3

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

public static ru.project.subpackage.PersonDTO convertOshPersonDtoToPersonDto(OshPersonDTO person){
        ru.project.subpackage.PersonDTO dto = new ru.project.subpackage.PersonDTO();
        dto.setPersonid(person.getPersonid().longValue());
        dto.setNamelast(person.getNamelast());
        dto.setNamefirst            (person.getNamefirst            ());
        dto.setNamesec              (person.getNamesec              ());
        dto.setInitials             (person.getInitials             ());
        dto.setSex                  (convertSkVocValue(person.getSex()));
        dto.setReason               (person.getReason               ());
        dto.setWorkphone            (person.getWorkphone            ());
        dto.setWorkphonedigit       (person.getWorkphonedigit       ());
        dto.setLocalphone           (person.getLocalphone           ());
        dto.setLocalphonedigit      (person.getLocalphonedigit      ());
        dto.setHomephone            (person.getHomephone            ());
        dto.setHomephonedigit       (person.getHomephonedigit       ());
        dto.setMobilephone          (person.getMobilephone          ());
        dto.setMobilephonedigit     (person.getMobilephonedigit     ());
        dto.setFax                  (person.getFax                  ());
        dto.setFaxdigit             (person.getFaxdigit             ());
        dto.setPager                (person.getPager                ());
        dto.setEmail                (person.getEmail                ());
        dto.setWeb                  (person.getWeb                  ());
        dto.setNamelastdative       (person.getNamelastdative       ());
        dto.setNamefirstdative      (person.getNamefirstdative      ());
        dto.setNamesecdative        (person.getNamesecdative        ());
        dto.setNamelastaccusative   (person.getNamelastaccusative   ());
        dto.setNamefirstaccusative  (person.getNamefirstaccusative  ());
        dto.setNamesecaccusative    (person.getNamesecaccusative    ());
        dto.setNamelastgenitive     (person.getNamelastgenitive     ());
        dto.setNamefirstgenitive    (person.getNamefirstgenitive    ());
        dto.setNamesecgenitive      (person.getNamesecgenitive      ());
        dto.setNamelastinstrumental (person.getNamelastinstrumental ());
        dto.setNamefirstinstrumental(person.getNamefirstinstrumental());
        dto.setNamesecinstrumental  (person.getNamesecinstrumental  ());
        dto.setNamelastprepositional(person.getNamelastprepositional());
        dto.setNamefirstprepositional(person.getNamefirstprepositional());
        dto.setNamesecprepositional (person.getNamesecprepositional ());
        return dto;
    }

И так далее еще несколько сотен строк. А главное переупаковка из одних объектов в другие и обратно бессмысленна, т.к. можно использовать исходные (они доступны в приложении)

johnsoft, 07 Сентября 2009

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

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