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

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

В номинации:

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

За время:

Си / Говнокод #5997

+105

1

int c = ((i-(i%(int)pow(10,p)))/(int)pow(10,p))%10;

Выделение из числа I цифры, стоящей на месте P с конца.
=>
i = 1234, p = 2, c => 2

danilissimus, 16 Марта 2011

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

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

+120

1
2
3
4

string tmpS;
//идут всякие монипуляции с этой строкой
//. . .
tmpS.Remove(0, tmpS.Length);

Реальный код из коммерческого продукта.

inser, 04 Марта 2011

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

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)

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