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

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

В номинации:

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

За время:

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

−7

1
2
3
4
5
6

int CompareTo (A a1, A a2)
{
   if (a1.Year>a2.Year) return 1;
   if (a1.Year<a2.Year) return -1;
    return 0;
}

реализация интерфейса IComparable, чем не понравилось стандартное норминрование: return a1.Year - a2.Year; так и не понял

guest, 28 Декабря 2008

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

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

+13.4

1

prNewProduct.Description = Convert.ToString(tbDescription.Text);

А мы её ещё стрингее сделаем!

guest, 26 Декабря 2008

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

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

−421

1
2

def list2str(lst):
	return reduce(lambda x,y:x+y,str(list(lst)).split("]")[0].split("[")[1].split(", "))

Функция, переводящая список в строку

guest, 26 Декабря 2008

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

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)

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