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

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

В номинации:

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

За время:

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

+80

1
2
3
4
5
6
7

try {
     while (sender.isAlive()) {
         this.sleep(100);//static field usage(!)
     }
} catch (java.lang.InterruptedException e) {
     Log.log(Log.ERROR, this, e);
}

типа Thread.join() не наш метод.

qnikst, 04 Сентября 2010

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

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

+173

1
2
3

for ($i = 1; $i <= 1; $i++) {
    echo $top_stats;
}

Найден в одном из популярных дополнений для vbulletin.
На вопрос - "Зачем for ($i = 1; $i <= 1; $i++)", получил ответ - "А ты подставь 5 заместо второй единицы, увидишь как всё криво станет"

hcy321, 01 Сентября 2010

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

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

+166

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

<!-- Begin
var months=new Array(13);
months[1]="января";
months[2]="февраля";
months[3]="марта";
months[4]="апреля";
months[5]="мая";
months[6]="июня";
months[7]="июля";
months[8]="августа";
months[9]="сентября";
months[10]="октября";
months[11]="ноября";
months[12]="декабря";
var time=new Date();
var date=time.getDate();
var lmonth=months[time.getMonth() + 1];
var year=time.getYear();
if (year < 2000)    // Y2K Fix, Isaac Powell
year = year + 1900; // http://onyx.idbsu.edu/~ipowell
document.write(date + " ");
document.write( lmonth + " "+ year + " года");
// End -->

Приспешник Паскаля.
http://money.webdom.biz/

Alfred, 24 Августа 2010

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

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

+165

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

<?php

$luser=$this->get_authorized_user();
if (!$luser) retuen -101;

// ...

  if (!$lpwd) 
  {
   $db->disconnect();
   return -102;
  }

// ...

 if (!is_array($array)){$db->disconnect();return -103;}
  
 if (0 == count($array)){$db->disconnect();return -104;}

// ...

 if (!is_array($array)){$db->disconnect();return -105;}
  
 if (0 == count($array)){$db->disconnect();return -106;}

орфография и правописание сохранены

вот таких отрицательных "статусов" по всему проекту просто миллион. Только одному автору известно, что они значат.

и работает это все с неправильно написанным "return" в первом блоке

user654321, 23 Августа 2010

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

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

+165

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

<?php
//...
 $GrabURL = "http://www.google.com/search?q=".$url."&sourceid=opera&num=100&ie=utf-8&oe=utf-8&start=".$page;
  $x=getfile($GrabURL); //автор сделал велосипед-аналог file_get_contents
  $handle = fopen("rip_google.txt","w+");
  fwrite($handle, $x);
  fclose($handle);
  $screen = file("rip_google.txt");
  $content = implode("",$screen);
  // вытягиваем линк
  preg_match_all("!\<h2 class=r\>\<a href=\"(http:.*?)[\"|&]!si" ,$content, $out1, PREG_PATTERN_ORDER);
//...
?>

Код из парсера гугла. Файл rip_google.txt больше реально нигде не используется. Так же порадовала комбинация строк 8-9.

mr.The, 22 Августа 2010

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

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

+163

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

class A
{
public:
	~A();
};

void A::!A()
{
	//destructor
}

int WINAPI WinMain(HINSTANCE, HINSTANCE, LPSTR commandLine, int)
{
	A* a = new A();
	delete a;

	return 0;
}

Компилируется и РАБОТАЕТ (заходит в этот "деструктор" при удалении объекта) под Visual C++ 2008.
Перестаёт компилироваться после любого малейшего изменения (например, если убрать void перед реализацией "деструктора").
КАК?!

Kirinyale, 21 Августа 2010

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

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

+166

1
2
3
4
5
6
7

до ревью:

abcForm.Controls.FindControl<AbcDateControl>("MyDate").SetValue(DateTime.Parse(DateTime.Now.AddDays(1).ToString()));

после

abcForm.Controls.FindControl<AbcDateControl>("MyDate").SetValue(new DateTime(DateTime.Now.AddDays(1).ToString()));

Всегда думал, что ревью улучшает код... Похоже народ хотел сделать вот это:

abcForm.MyDate.SetValue(DateTime.Now.Add Days(1));

:)

pdk, 18 Августа 2010

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

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

+155

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

<div id="header">
<script>
    function mShow (mEl) {
      if (document.getElementById(mEl)) {
        el = document.getElementById(mEl);
        if (el.style.display=='block') el.style.display = 'none';
        else el.style.display ='block';
        if (mEl != 'mFoto') document.getElementById('mFoto').style.display = 'none';
        if (mEl != 'mFriend') document.getElementById('mFriend').style.display = 'none';
        if (mEl != 'mMessage') document.getElementById('mMessage').style.display = 'none';
      }
    }
    function sOff (e) {
        if(!e){e=window.event;}
        var message="";
        document.getElementById('mFoto').style.display = 'none';
        document.getElementById('mFriend').style.display = 'none';
        document.getElementById('mMessage').style.display = 'none';
        
        function clickIE() {if (document.all) {(message);return false;}}
        function clickNS(e) {if (document.layers||(document.getElementById&&!document.all)) {
          if (e.which==2||e.which==3) {
            (message);
          return false;}}}
            if (document.layers) {
              document.captureEvents(Event.MOUSEDOWN);
              document.onmousedown=clickNS;
            }else{
              document.onmouseup=clickNS;
              document.oncontextmenu=clickIE;
            }
  //document.oncontextmenu=new Function("return false")
    }

if(document.addEventListener){
  document.addEventListener("click",sOff,true);
}else if(document.attachEvent){
  document.attachEvent("onclick",sOff);
}else{
  document.onclick = sOff;
}

  </script>
<ul class="menu_top">

<li><a href="/users/555355" style="font-weight:bold;">inkanus</a> (<a href="/logout">выйти</a>)
</li>
<li>
<a href="#" onClick="mShow('mFoto');return false;">Мои фотографии <small>▼</small></a>
  <ul id='mFoto' class="submenu_top" style="right:280px;">
  <li><a href="/users/555355">Моя страница</a></li>
    <li><a href="/upload">Загрузить фотографии</a></li>
    <li><a href="/create">Создать альбом</a></li>
    <li><a href="/comments">Последние комментарии</a></li>
    <li><a href="/options">Настройки</a></li>
  </ul>
</li>
<li>
<a href="#" onClick="mShow('mFriend');return false;">Мои друзья <small>▼</small></a>
  <ul id='mFriend' class="submenu_top" style="right:166px;">
    <li><a href="/friends">Фотки друзей</a></li>
    <li><a href="/friends/control">Друзья</a></li>
    <li><a href="/friends/banned">Игнорируемые</a></li>
  </ul>
</li>
<li>
<a href="#" onClick="mShow('mMessage');return false;">Мои сообщения <small>▼</small></a>
  <ul id='mMessage' class="submenu_top" style="right:5px;">
    <li><a href="/messages">Входящие</a></li>
    <li><a href="/messages/sent">Исходящие</a></li>
    <li><a href="/messages/send">Отправить сообщение</a></li>
  </ul>
</li>
</ul>
  </div>

Взято с одного фотосайта.

inkanus-gray, 17 Августа 2010

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

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