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

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

В номинации:

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

За время:

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)

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

+163

1

if (isset($SAN) and $SAN<>"" and $SA>0) {

пиздец проверочка, и так почти везде

ferry-very-good, 17 Августа 2010

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

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

+151

1
2
3
4
5
6
7
8

function del()
{
    temp = window.confirm("Вы действительно хотите удалить?");
    if (temp == true) 
        return true;
    else 
        return false;
}

Из реальной системы

Werdn, 17 Августа 2010

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

Assembler / Говнокод #3988

+231

1
2
3
4

cmp [wparam],BN_CLICKED shl 16 + ID_HOTKEY
        je .sethotkey
        jmp .processed
.sethotkey:

O_o

Xekep, 15 Августа 2010

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

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

+157

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

$q = mysql_query("... /* тут мог быть ваш код */") or die(mysql_error());
      $nr = mysql_num_rows($q);
      $contor=0;
      if($nr<1) {
        echo "No notes found";
        $contor=0;
      } else {
        $contor=1;
        $cc = 0;
        while($d = mysql_fetch_assoc($q)) {
          ... /* тут мог быть ваш код */
          $cc++;
          $contor++;
        }
        if (!$contor) $contor = 0;
      }
      if (!$contor) $contor = 0;

для уверенности.

n0ne, 15 Августа 2010

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

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