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

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

В номинации:

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

За время:

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

+37.9

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

<?php
if (!$ncat) {
    $sql="select * from articles_cat where status=1  order by  `name` ASC limit 0,1";
    $res = m_qr($sql);
    if (m_rows($res)) {
        while ($row = m_arr2($res)) {
            $ncat=$row[id];
            ... 10 строк кода ...
        }
    }
} else {
    $sql="select * from articles_cat where id='$ncat'";
    $res = m_qr($sql);
    if (m_rows($res)) {
        while ($row = m_arr2($res)) {
            $ncat=$row[id];
            ... те же 10 строк кода ...
        }
    }
}

Найди десять отличий...

guest, 26 Декабря 2008

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

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

+24

1
2
3
4
5

if (!($conf_content = file_get_contents($data_path."voc.conf"))) {
        die("Cannot open voc.conf file, please check your DATA PATH ($data_path) parameter");
    }

eval($conf_content);

при условии что voc.conf является пхпфайлом, глупо делать такую мутотень когда можно просто обойдись file_exists и include

guest, 24 Декабря 2008

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

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

+12

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

[Test]
public void UserHasNoPermissionToDeleteTransactionWithEditPermission()
{
  var user = CreatetestUser("test")
    .WithPermissions()
    .Allow("Transaction/Edit").Do();
  TestCurrentUser = user;

  var trn = CreateTestTransaction(120, typeof(CreditCardPayment));

  try
  {    
    // Who cares about delete, we never do it.
    // I need this test passed just NOW.

    // trn.AsIObject().Delete();
    // Assert.Fail("Exception expected");
  }
  catch (PermissionException ex)
  {
    Assert.IsTrue(true, "Cool we got an exception");
  }
  catch (Exception generic)
  {
    Assert.Fail("PermissionException expected.");
  }
}

Ничего не тестирующий тест

guest, 24 Декабря 2008

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

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

+28.1

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

if ( 0 == 0 ) 
{ 
  $x += 1; 
  $yellow = ""; 
  if ( $myhighlight_unit[$i_high] == $row[$i]['ID'] ) 
  { 
    //echo $myhighlight_unit[$i_high]." == ".$i_high."<br />";   
    $yellow = "background-color: #ffcc33;"; 
  } 
  if ( $row[$i]['approved'] > 0 && $_SESSION['userid'] > 3 ) 
  { 
    $yellow = "background-color: #ff0000;"; 
  } 
  if ( $i_tr == 0 ) 
  { 
    echo "<tr>"; 
  }
}

Очередной шедевр от автора #66, #68 и #69

guest, 24 Декабря 2008

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

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

+24.9

01
02
03
04
05
06
07
08
09
10

if(empty($real_name)) $real_name = $file_name; 		
if(empty($real_name)) $real_name="fileid_".$id;

.... 

if($mode=="send") {
	$head[] = array("link" => "customer.php?area=$area&mode=send", "title" => func_get_langvar_by_name("lbl_send_to_mail"), active=>"Y");
}	else	{
	$head[] = array("link" => "customer.php?area=$area&mode=send", "title" => func_get_langvar_by_name("lbl_send_to_mail"), active=>"n");
}

Разбирая один из скриптов на досуге, обнаружил, что некоторые программисты уж совсем с головой не дружат.

1. Думаю, все тут ясно.
2. Чувак явно не знает что такое логика... Мало того, что в одну строку ложится массив, так еще и ВЕЗДЕ строковые 'Y'/'N' использует вместо булевых труфолсов. И такого говна - тыщи 2 строк. Ы =)

Не позавидую тем, кто будет дописывать / переделывать.
ЗЫ: Скрипт - рип корпоративной системы.

guest, 16 Декабря 2008

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

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

+21

1
2
3

TabToShow currentTabToShowInShortForm = tab;
string currentTemplateNameToShowInShortForm = templateName;
currentShortFormToShow = new ShortForm(currentTabToShowInShortForm, currentTemplateNameToShowInShortForm);

Чем длиннее названия переменных, тем солиднее выглядит код
[Ф]

guest, 16 Декабря 2008

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

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

+60

1
2
3

if (isset($param) && $param!=null && $param!=0 && $param>1) {
  sendRequest($param);
}

guest, 12 Декабря 2008

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

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

−60

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

if (!account_number.equals("") && cycleT != null) {
    if (!account_number.equals(num)) {
        doWrite(bufferedOut);
        account_number = num;
        cycleT = c;
        vT = v;
    } else if (!cycleT.equals(c)) {
        doWrite(bufferedOut);
        account_number = num;
        cycleT = c;
        vT = v;
    } else if (vT != v) {
        doWrite(bufferedOut);
        account_number = num;
        cycleT = c;
        vT = v;
    }
} else {
    account_number = num;
    cycleT = c;
    vT = v;
}

guest, 12 Декабря 2008

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

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

+27.7

1
2
3
4

$int = (int) $int;
if (is_int($int)) { // на всякий случай
   // ...
}

Идея где-то валяется в залежах башорга.

guest, 11 Декабря 2008

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

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