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

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

В номинации:

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

За время:

JavaScript / Говнокод #12817

+151

1

if (Scene.generatedFast || (typeof generatedFast != "undefined" && generatedFast) )

Lure Of Chaos, 28 Марта 2013

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

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

+146

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

getInsideText($part1[0],'<string>','</string>',1,true);

function getInsideText($str,$fstr,$lstr,$limit=0,$trim=true){
    $temp_arr=array();
    $lcnt=0;
    while(strpos($str,$fstr)!==false && ($limit ? $lcnt<$limit : true)){
        $fpos=($fstr ? strpos($str,$fstr)+strlen($fstr) : 0);
        $str=substr($str,$fpos);
        $lpos=strpos($str,$lstr);
        $val=($lpos!==false ? substr($str,0,$lpos) : $str);
        $temp_arr[]=($trim ? trim($val) : $val);
        $str=substr($str,$lpos+strlen($lstr));
        $lcnt++;
    }
    return ($limit==1 ? (isset($temp_arr[0]) ? $temp_arr[0] : '') : $temp_arr);
}

Конечный автомат своими силами.

alexx, 21 Марта 2013

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

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

+16

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
52
53
54
55
56
57
58
59
60
61
62
63
64
65

class Context;

class AbstractState
{
	Context * m_context;

protected:
	Context * context() const { return m_context; }

public:
	AbstractState(Context * context) : m_context(context) { };
	virtual ~AbstractState() { }
	virtual void doSomething() = 0;
};

class Context
{
	std::unique_ptr<AbstractState> m_state;

public:
	enum State
	{
		State1,
		State2,
	};
	Context() { switchToState(State1); }
	void switchToState(State newState);
	void doSomething() { m_state->doSomething(); }
	void someCleanup() { }
};

class ConcreteState1 : public AbstractState
{
public:
	ConcreteState1(Context * context) : AbstractState(context) { }
	virtual void doSomething()
	{
		context()->switchToState(Context::State2);
		context()->someCleanup();
	}
};

class ConcreteState2 : public AbstractState
{
public:
	ConcreteState2(Context * context) : AbstractState(context) { }
	virtual void doSomething()
	{
		context()->switchToState(Context::State1);
		context()->someCleanup();
	}
};

void Context::switchToState(State newState)
{
	switch(newState)
	{
	case State1:
		m_state.reset(new ConcreteState1(this));
		return;
	case State2:
		m_state.reset(new ConcreteState2(this));
		return;
	}
}

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

Xom94ok, 10 Марта 2013

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

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

−169

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

DECLARE @IsPerson BIT  
 ,@ContactTypeID INT  

 SET @ContactTypeID = CASE WHEN @IsPerson = 0 THEN 3 ELSE 4 END  

 IF (LEN(ISNULL(@FirstName,'')) <= 0 AND LEN(ISNULL(@LastName,'')) <= 0 AND (LEN(@EntityName) > 0 OR LEN(@TradingAs) > 0))  
 BEGIN  
  SET @IsPerson = 0  
 END   
 ELSE  
 BEGIN  
  SET @IsPerson = 1  
 END

Такая себе Stored Procedure...
Magic numbers - это всё фигня. Меня всегда интересовало, почему тип контакта всегда 4?
И, да, строки идут именно в такой очерёдности.

kore_sar, 28 Февраля 2013

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

JavaScript / Говнокод #12660

+155

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

//...
                else if (data.indexOf('none') == -1)
                {
                    // Экспорт в работе
                    // Повторять запрос раз в nSec секунд.
                    nSec = 3;
                    document.getElementById("export_loading").show();
                    setTimeout("loadFile({{subject_id}}, '')", nSec*1000);
                }
//...
<div class='loader' id="export_loading" style="visibility: hidden; display: inline-block; text-align: center; margin-right: 13px;"><img src='/app/images/loading.gif'/> идёт формирование списка авторов</div>
<div id="messages_container" style="visibility: hidden; padding: 0px 30px 0 35px;">
</div>

Чувак вообще прикалывается)

greevex, 27 Февраля 2013

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

JavaScript / Говнокод #12641

+152

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

function fixMootoolsJSON(thing) {
    var i, member, pattern = /^"\[.*\]"$/, copy;
    if (thing instanceof Array) {
        for (i = 0; i < thing.length; i++) {
            member = thing[i];
            if (typeof member == "string" && pattern.test(member)) {
                thing[i] = fixMootoolsJSON(JSON.decode(member));
            }
        }
    } else if (typeof thing == "object") {
        copy = { };
        for (i in thing) {
            if (thing.hasOwnProperty(i)) {
                copy[i] = fixMootoolsJSON(JSON.decode(thing[i]));
            }
        }
        for (i in copy) {
            if (copy.hasOwnProperty(i)) {
                thing[i] = copy[i];
            }
        }
    }
    return thing;
}

http://outsourceror.blogspot.co.il/2011/04/mootools-intrudes-on-native-json-and.html

Но вы не подумайте, оказалось, что я был первый (в нашем небольшом коллективе), кто это заметил, а (в нашем небольшом коллективе) было принято JSON прям как есть в базу сохранять...

wvxvw, 24 Февраля 2013

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

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

+142

1
2
3
4

$arr = date("w", strtotime($date_bron));
$date = $arr;
if ($arr == 0)
     $date = 0;

Система бронирования столика для бара...

unst, 17 Февраля 2013

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

Куча / Говнокод #12603

+141

1

http://rosettacode.org/wiki/Category:Programming_Tasks

Сегодня это реально сразу куча.

LispGovno, 16 Февраля 2013

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

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

+77

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

byte[] buffer = new byte[BUFFER_SIZE];
ReadState readState = ReadState.BOUNDARY;

InputStream input = request.getInputStream();
int read = input.read(buffer);
int pos = 0;

// This is a fail-safe to prevent infinite loops from occurring in some environments
int loopCounter = 20;

while (read > 0 && loopCounter > 0) {
    for (int i = 0; i < read; i++) {
        switch (readState) {
            // Pos is calculated...
            case BOUNDARY: 
            case HEADERS: 
            case DATA: 
        }
    }

    if (pos < read) {
        // move the bytes that weren't read to the start of the buffer
        int bytesNotRead = read - pos;
        System.arraycopy(buffer, pos, buffer, 0, bytesNotRead);
        read = input.read(buffer, bytesNotRead, buffer.length - bytesNotRead);

        // Decrement loopCounter if no data was readable
        if (read == 0) {
            loopCounter--;
        }

        read += bytesNotRead;
    } else {
        read = input.read(buffer);
    }
}

Кусок исходников из недров JBoss Seam(наткнулся профайлером).
Пацаны пофиксили багу с бесконечным циклом и 100 утилизацией CPU.
Весь класс:
https://www.java2s.com/Open-Source/Java/JBoss/jboss-seam-2.2.0/org/jboss/seam/web/MultipartRequestImpl.java.htm
Версия с бесконечным циклом:
http://www.docjar.com/html/api/org/jboss/seam/web/MultipartRequestImpl.java.html

psvm, 16 Февраля 2013

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

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