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

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

В номинации:

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

За время:

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

+70

1
2
3
4

if(url.getProtocol().equals("http") || true)
{
   // ...
}

Индийский код добрался и до Android...

Highlander, 29 Ноября 2010

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

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

+70

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

Map<Integer, ArrayList<?>> // filterId : filterParams
{
	//
	// Фильтр занятости квартир (не вписывается в API, поэтому здесь заведены отдельные методы)
	//
	
    // хранит: showReserved, DateFrom, DateTo
    private ArrayList<?> reservedFilterParams

    public void onDateFilterChanged(Date dateFrom, Date dateTo, boolean showReserved)
    {
	this.put(QuickFilters.FILTER_RESERVED_ID, reservedFilterParams)
		
	reservedFilterParams = ArrayUtils.getOrCreateArrayListWithElements(
			this, QuickFilters.FILTER_RESERVED_ID, new Boolean(showReserved), dateFrom, dateTo )
	
	if (dateFrom == null && dateTo == null)
        {
            reservedFilterParams.set(0,new Boolean(true))
            this.remove(QuickFilters.FILTER_RESERVED_ID)
        }
    }

    public boolean doShowReserved()
    {
        return reservedFilterParams ? reservedFilterParams.get(0).booleanValue() : true
    }

    public Date getDateFrom()
    {
        return reservedFilterParams?.get(1)
    }

    public Date getDateTo()
    {
        return reservedFilterParams?.get(2)
    }

}

Язык Groovy, стиль - моя переходная форма от Java. Это я сам написал. Вообще мне даже нравится. Есть фильтры для поиска. У каждого фильтра есть только ему ведомые параметры. Приведенный код - это мапка для хранения того, какие пользователь выбрал фильтры и что в них установил.

Есть API который я делаю, состоящий из типовых фильтров. Этот API автоматически юзает мапку через get/put. Но бывает, придумывается какой-то нетиповой фильтр, и его параметры приходится хардкодить в этой мапке. Короче, мне понравилась идея с мапкой, ведь даже если параметры захардкодены, то они всё равно пихаются в мапку. И читаемость для таких случаев навысоте.

Bugdog, 20 Октября 2010

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

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

+70

01
02
03
04
05
06
07
08
09
10

/**
 * Окно, которое реагирует на нажатие клавиши Esc
 * определенным образом (например, окно закрывается).
 * Для этого в потомках класса необходимо реализовать метод
 * doEscapeKeyAction(), который будет вызываться при нажатии
 * клавиши ESC.
 */
public abstract class JEscFrame extends ProfilableFrame {
...
}

borka, 22 Августа 2010

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

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

+70

1
2
3
4
5
6
7
8
9

public int someGetCountMethod() {
        try {
            ResultSet rs2 = db.select("call someGetCountMethod('')");
            rs2.first();
            return rs2.getInt(1);
        } catch (Exception ex) {
            return 0;
        }
    }

где-то откапал вот такое

zlob.jc, 25 Мая 2010

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

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

+70

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

for(int i = 0; i < s.length(); i++)
        {
            String c = s.substring(i, i + 1);
            if("_".equals(c) || " ".equals(c) || "-".equals(c))
            {
                capitalize = true;
                continue;
            }
            if(c.toUpperCase().equals(c))
            {
                if(lastDecapitalized && !lastCapital)
                    capitalize = true;
                lastCapital = true;
            } else
            {
                lastCapital = false;
            }
            if(capitalize)
            {
                if(p == null || !p.equals("_"))
                {
                    result.append(c.toUpperCase());
                    capitalize = false;
                    p = c;
                } else
                {
                    result.append(c.toLowerCase());
                    capitalize = false;
                    p = c;
                }
            } else
            {
                result.append(c.toLowerCase());
                lastDecapitalized = true;
                p = c;
            }
        }

Позор Jboss'овцам!
это код из HibernateTools, теперь понятно почему оно такое глючное Г

asdqwezx, 27 Августа 2009

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

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

+69.9

1
2
3

public void ejbPassivate() throws EJBException, RemoteException {
		return;
	}

Истинный китайский девелопер никогда не доверяет компилятору! Ато если ж в функции тело пустое, мало ли, вдруг она никогда не возвратиться. Какой выход? Написать об ентом явно.

cheap_chinese, 12 Августа 2009

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

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

+69.8

1
2
3
4
5
6
7
8

vector<int> :: iterator itBegins = array.begin(); 
vector<int> :: iterator itEnds = array.end();

while(itBegins != itEnds)
{
         pVec.pop_back();
         it++;
}

Сей артефакт остался в коде ввиду лености переписывать его в 100 файлах сразу после удаления того, ради чего собственно затевался while(!pVec.empty()) ... прямо скажем - отлов ЭТОГО занял немало времени и нервов... тк на первый взгляд оно должно... а что ты видишь в этом коде, %юзернейм% ?

(исправленная версия XD )

Zeks, 02 Ноября 2009

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

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

+69.6

1
2

$s = mysql_query('SELECT * FROM `nazva_stolbca` where `nazva_stolbca` = `$_GET['name']`'); 
if ($s == '' ) { echo 'такого значения нет'};

Я боюсь :(

guest, 13 Июня 2009

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

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

+69.5

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

#include <iostream>
using namespace std;
class MyException
{
public:
	MyException(char *msg) { m_msg = msg; }
	~MyException(){}
	char *GetError(){return m_msg;}
protected:
	char *m_msg;
};

int main()
{
	int *buffer;
	try
	{
		buffer = new int[255];
		if (buffer == NULL)
		{
			MyException *exception = 
					new MyException("Memory allocation failed!");
			throw exception;
		}
		else
			delete buffer;
	}
	catch (MyException *exception)
	{
		char *msg = exception->GetError();
		cout << msg << endl;
	}
return 0;

}

Книга "Использование Viusal C++" Кейт Грегори. Глава 24 о предотвращении утечек памяти и работе с исключениями.

ccoder, 20 Февраля 2010

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

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

+69.2

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

List<Measure> meaList = q.getResultList();
        try
        {
            if (meaList.size() == 0)
            {
                throw new Exception("Measure not found: "+_wsPrice.getMeasureIso());
            }
            else
            {
                p.setMeasure(meaList.get(0));
            }
        }
        finally
        {
            meaList.clear();
        }

А вот ещё порция паранойи от моего коллеги

konsoletyper, 06 Апреля 2010

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

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