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

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

В номинации:

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

За время:

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

+145

1
2
3
4
5
6

function Redirect($url) {
  print "<html><head>\n";
  print "<meta http-equiv='Refresh' content='0; url=$url'>\n";
  print "</head></html>\n";
  exit();
}

Во как делают переадресацию нереальные пацаны!

dwinner, 17 Ноября 2010

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

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

+157

01
02
03
04
05
06
07
08
09
10
11

void Scene3D::DrawTriangle(const Point3D &A, const Point3D &B, const Point3D &C, const Color& color)
{
        // ---------------------------------------------------------------------------------------
        // вспомогательные вычисления
        // нормаль
        const Vector3D& n = (B - A) ^ (C - A);

        // ...

        // центр треугольника
        const Point3D& medium = (A + B + C) / 3.0;

Руки как-то привыкли const TypeName& variableName набирать в определении параметров методов.
И случайно набралось такое (строки 6, 11)

Операторы (+, -, ^, /) над векторами возвращают Vector3D, не const Vector3D&.

Заметил только через полгода, и всё это время оно почему-то работало, и даже ворнингов не было.

Но такое ведь не должно работать!
Результат вычисления в правой части присваивания структура, то есть она возвращается в стеке. Если бы я присвоил её какой-то локальной переменной, для которой выделена память в стеке текущей функции, то она бы перед удалением скопировалась в локальную переменную. А так получается что ссылка (n, medium) указывает куда-то на стек, где временно хранится возвращенное оператором значение. И при следующем вызове любой функции эта область стека должна перезаписаться.

burdakovd, 12 Ноября 2010

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

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

+144

1
2
3
4

if (!bNextPageVisited)
this->button(QWizard::NextButton)->setEnabled(false);
if (bNextPageVisited)
this->button(QWizard::NextButton)->setEnabled(true);

Упячка мозга?..

des-1008d, 12 Ноября 2010

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

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

+164

1
2
3
4

double *ptr = new (nothrow) double[n];
if (ptr) {
//....
} else throw bad_alloc();

Sanya_M, 11 Ноября 2010

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

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

+79

01
02
03
04
05
06
07
08
09
10

public class Daemon {
   //...

   /**
   * Constant defines url pattern for register
   */
   private static final String S_ID_D_URL_S_STRATEGY_CLASS_S_STATE_S = "%s?id=%d&url=%s&strategyClass=%s&state=%s";
 
   //...
}

....

nikelin, 11 Ноября 2010

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

Куча / Говнокод #4561

+138

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

function StripLeadingComma(str)
  str = Ltrim(str)
  if len(str) > 0 then
    if Left(str,1) = "," then  str = Mid(str,2)
  end if
  if len(str) > 0 then
    if Left(str,1) = "," then  str = Mid(str,2)
  end if
  if len(str) > 0 then
    if Left(str,1) = "," then  str = Mid(str,2)
  end if
  if len(str) > 0 then
    if Left(str,1) = "," then  str = Mid(str,2)
  end if
  StripLeadingComma = str
end function

function StripTrailingComma(str)
  str = rtrim(str)
  if len(str) > 0 then
    if Right(str,1) = "," then  str = Left(str,len(str)-1)
  end if
  if len(str) > 0 then
    if Right(str,1) = "," then  str = Left(str,len(str)-1)
  end if
  if len(str) > 0 then
    if Right(str,1) = "," then  str = Left(str,len(str)-1)
  end if
  if len(str) > 0 then
    if Right(str,1) = "," then  str = Left(str,len(str)-1)
  end if
  if len(str) > 0 then
    if Right(str,1) = "," then  str = Left(str,len(str)-1)
  end if
  if len(str) > 0 then
    if Right(str,1) = "," then  str = Left(str,len(str)-1)
  end if
  StripTrailingComma = str
end function

Классика жанра на production. ASP, VBScript.

Seth, 08 Ноября 2010

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

C# / Говнокод #4455

+122

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

public static bool IsConnectedToInternet
{
    get
    {
        // сервер для пинга
        Uri url = new Uri("www.google.com"); 
        // его перевод в строку
        string pingurl = string.Format("{0}", url.Host); 
        string host = pingurl;
        // возвращаемое значение
        bool result = false; 
        Ping p = new Ping();
        try
        {
            PingReply reply = p.Send(host, 3000);
            if (reply.Status == IPStatus.Success)
            // если все ок, то вернем true                
               return true;
        }
        catch { }
        // в этом случае возвращается false
        return result;
    }
}

Взято с http://msgeeks.ru/?artid=42 - "Пинг сервера на С#".
Особенно порадовал перевод url.Host в строку))

inso, 26 Октября 2010

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

Python / Говнокод #4451

−100

1
2
3

s = 'blablabla'
if s[:7] <> 'http://':
   s = 'http://%s' % s

blednolicui, 26 Октября 2010

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

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

+168

1
2

$Y=date('Y'); $m=date('m'); $d=date('d');$G=date('G');$i=date('i');$s=date('s'); 
$tdate=mktime($G,$i,$s,$m,$d,$Y);

fork, 26 Октября 2010

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

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

+157

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

...

if ($kpp['id_complect'] == '14') {
$smarty->assign('sost', $kpp['tit']);

}
if ($kpp['id_complect'] == '54') {
$smarty->assign('dvig', $kpp['tit']);

}

if ($kpp['id_complect'] == '17') {
$smarty->assign('kpp', $kpp['tit']);

}
if ($kpp['id_complect'] == '16') {
$smarty->assign('priv', $kpp['tit']);

}
if ($kpp['id_complect'] == '18') {
$smarty->assign('weel', $kpp['tit']);

}
if ($kpp['id_complect'] == '19') {
$smarty->assign('ras', $kpp['tit']);

}
if ($kpp['id_complect'] == '101') {
$smarty->assign('color', $kpp['tit']);

}
if ($kpp['id_complect'] == '15') {
$smarty->assign('torg', $kpp['tit']);

}

if ($kpp['id_complect'] == '96') {
$smarty->assign('ls', $kpp['tit']);

}

if ($kpp['id_complect'] == '100') {
$smarty->assign('vin', $kpp['tit']);

}


...

и так ооочень много...
гениальный прогер наверно писал.

yezhoff, 25 Октября 2010

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

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