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

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

В номинации:

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

За время:

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

+146

01
02
03
04
05
06
07
08
09
10

class GridPanel extends JPanel {

  protected Point myLocation;

  @Override
  public Point location() {
    return this.myLocation;
  }

  }

убивал бы за такое.

Lure Of Chaos, 07 Февраля 2011

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

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

+159

1
2
3
4
5
6
7
8
9

std::ostringstream s;
for( std::list< Candidate* >::iterator it = order_by_priority.begin(); 
		it != order_by_priority.end(); ){
	s << (*it)->id << "/" << priorityDesc((*it)->priority());
	it++;
	if( it!= order_by_priority.end() )
		s << ", ";
}
return s.str();

Странное обращение с циклом for.

panter_dsd, 04 Февраля 2011

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

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

+118

01
02
03
04
05
06
07
08
09
10

public class TWriteableForEach
	{
		public delegate void TForEachDelegate<TItem>(TItem item);

		public static void Exec<TItem>(IList<TItem> itemsCollection, TForEachDelegate<TItem> forEachDelegate)
		{
			for (int i = 0; i < itemsCollection.Count(); ++i)
				forEachDelegate(itemsCollection[i]);
		}
	} ;

Велосипед с квадратными колёсами?

Говногость, 03 Февраля 2011

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

ActionScript / Говнокод #5334

−117

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

private function formatCallResult(data : String) : URLVariables 
{
	try
	{
		var urlVariables 	: URLVariables  = new URLVariables(data);
		MonsterDebugger.trace(this,[urlVariables.msgdesc,urlVariables.msgbody]);
		var pattern			: RegExp 		= /\[s](.+)\[\/s\]/ig  //*new RegExp("\\[s\\]([\\w\\s]+)\\[/s\\]", "ig");*/
		var msgdescReady	: String 		= urlVariables.msgdesc.replace(pattern, "<b>$1</b>");
		var msgbodyReady	: String 		= urlVariables.msgbody.replace(pattern, "<b>$1</b>");
		var pattern2		: RegExp 		= /\[n](.+)\[\/n\]/ig;
		//TODO Write nice regexp instead !
		urlVariables.msgdesc = msgdescReady;				
		urlVariables.msgbody = msgbodyReady;
		msgdescReady 		= urlVariables.msgdesc.replace(pattern2, "<b>$1</b>");	
		msgbodyReady 		= urlVariables.msgbody.replace(pattern2, "<b>$1</b>");	
		urlVariables.msgdesc = msgdescReady;				
		urlVariables.msgbody = msgbodyReady;				
		MonsterDebugger.trace(this,[urlVariables.msgdesc,urlVariables.msgbody]);
	}
	catch (e : Error)
	{
		
	}
	return urlVariables;
}

Я уже минут 10 силюсь понять, что же оно должно было делать...

wvxvw, 19 Января 2011

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

SQL / Говнокод #5321

−858

01
02
03
04
05
06
07
08
09
10
11

FUNCTION get_yesterday
      RETURN DATE
   AS
      dd   VARCHAR2 (2);
      mm   VARCHAR2 (2);
      yy   VARCHAR2 (4);
   BEGIN
      SELECT TO_CHAR (SYSDATE - 1, 'yyyy')
        INTO yy
        FROM DUAL;
   END get_yesterday;

Вычисляем вчерашнюю дату на PL/SQL

mshogin, 18 Января 2011

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

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

+74

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

@Name("solr")
@AutoCreate
public class SolrClient {
    @Create
    public void init() {
        try {
            server = startRemoteSolr();
        } catch (Exception e) {
            log.error(e, e);
        }
    }

    public SolrServer startRemoteSolr() throws MalformedURLException, SolrServerException {
        CommonsHttpSolrServer solr = new CommonsHttpSolrServer("http://localhost:8983/solr");
        solr.setRequestWriter(new BinaryRequestWriter());
        return solr;
    }

}

Seam. Перевод для непричастного: на старте сайта вызывается метод startRemoteSolr(). Solr - это поисковый движок. Если он не запустится, на сайте не будет работать поиск. И если на запуске Solr будет ошибка, то в лог она попадёт просто как exception + stack trace.

Что не так в коде? Очевидно, его писали люди, не задумываясь об эксплуатации, что свойственно новичкам. А ведь устранение проблем в рабочем сайте - это приоритет номер один. Сама разработка сайта, сидя в удобном кресле с кашкой кофе - где-то сильно ниже.

Так вот, что не так?

1. При выводе ошибок надо писать - что это за ошибка. В данном случае: "Запуск поисковика неудался, ПОИСК НА САЙТЕ РАБОТАТЬ НЕ БУДЕТ." И уже потом - stack trace.

2. Если некая ошибка связана с ресурсами - сетевыми, или на файловой системе, в логе надо также описывать этот ресурс. Т.е. "Не удалось запустить клиент поисковика по адресу http://localhost:8983/solr". Что бы из логов сразу было видно, что порт указан правильно или нет, хост и тд.

Но выпускникам ВМК МГУ такие вещи неведомы, они слишком крутые что бы опускаться до таких мелочей. Чайники, хуле.

yvu, 12 Января 2011

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

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

+160

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

<?
$ux=0; 
$array = file( 'logins.TXT' );
$count = count( $array );
for ( $i = 0; $i < $count; $i++ )
{
    If (trim($array[$i],"\n\r")==$Text1.$Password1) {$ux=1;};}
If ($ux==0) {
header('Location: index1.html');}
If ($ux==1) {$logins=$Text1;
$formers=<<<HERE
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=windows-1251">
<title>Перенаправление</title>
<script language="JavaScript">
var seconds=3;
remain();
function remain(){
seconds--;
document.timer.clock.value=seconds;
if(seconds>0)
{window.setTimeout('remain()',3000)}
else
{
location.replace("insystem/1.php");}
}
 
</script>
</head>
 
<body onLoad="remain()">
<p>Вы авторизированы!!! Сейчас вы будите перенаправлены в систему. Щёлкните по ссылке, если ваш браузер не поддерживает автопереадресацию.</p>
<p><a href="insystem/insystem.php">Ссылка</a></p>
<p>Через 3секунды переход произойдет автоматически.</p>
<form action="" method="get" name="timer">
Переадресация через:
<input name="clock" type="text" size=2> 
секунд
</form>
<form action="insystem/1.php" method="post">
<input type="hidden"
name="logint"
value="$logins">
</form>
</body>
</html>
HERE;
echo $formers;
print $logint;}
?>

qbasic, 05 Января 2011

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

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

+129

1
2
3
4
5
6
7

Exception Myex;
                int responseVal = HelperMethods.DoPost(url, postdata, "text/xml; charset=utf-8", ref xmlResponse, null, out Myex);
                XmlDocument xmlDoc = new XmlDocument();
                xmlDoc.Load(new StringReader(xmlResponse));

                string StatusID = xmlDoc.ChildNodes[1].ChildNodes[0].FirstChild.Value;
                if (StatusID.ToLower() != "0")

Последняя строка не может не радовать. Никогда не знаешь каким большим может быть ноль ...

Othello, 03 Января 2011

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

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