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

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

В номинации:

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

За время:

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

+58

1
2
3

$attr_id = json_decode($_GET['attr_id']);
$obj_id = json_decode($_GET['obj_id']);
$a_href = $_GET['file_id'];

Разгребаю что осталось от недавно уволенного джуниора.
2 json_decoda толкают в уныние. При том что джсон туда вообще не приходит =\

Жаль вот уже, не сохранился код, в к-м он пришедший на _клиент в браузер_ json парсил руками(JSON.parse() - для слабых)

vitaly, 29 Июня 2012

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

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

+61

1
2
3
4
5
6
7
8

if (is_page('orders_item')) {
	die ("<meta http-equiv='refresh' content='0; url=".$base_url."/orders_item.php?pid=".$_GET['pid']."'>");
} elseif (is_page('orders_garant')) {
	die ("<meta http-equiv='refresh' content='0; url=".$base_url."/orders_garant.php?pid=".$_GET['pid']."'>");
} else {
	die ("<meta http-equiv='refresh' content='0; url=".$base_url."/orders_item.php?pid=".$_GET['pid']."'>");
}
exit();

Хоть опросник пиши:
- чем первый die() отличается от последнего?
- почему их 3?
- кто будет экранировать $_GET?
- зачем в конце exit() подпирает die()?

domaster, 25 Июня 2012

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

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

+159

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

function group_sorting_ABC(array, mode)
{
    var arr = [],
        resArray = [],
        add;
    for (var i = 0; i < array.length; i++)
    {
        var key;
        if (array[i].index && mode == 'index')
        {
            switch (array[i].index.toString().length)
            {
            case 1:
                add = '00';
                break;
            case 2:
                add = '0';
                break;
            case 3:
                add = '';
                break;
            }
            key = add + array[i].index;
        }
        else
        {
            key = array[i].name;
        }
        arr.push(key + '@' + i);
    }
    arr = arr.sort();
    for (var i = 0; i < arr.length; i++)
    {
        var a = arr[i].split('@');
        var item = array[a[1]];
        resArray.push(item);
    }
    return resArray;
}

Наковырял в middleware set top box-а MAG-250.

vovams, 23 Июня 2012

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

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

+56

1
2
3
4
5
6
7

if ('cardiscon' == $key) {
	echo '<td class="even cardiscon">'.$row['name'].':</td>';
} elseif ('cardprice' == $key) {
	echo '<td class="even cardiscon">'.$row['name'].':</td>';
} else {
	echo '<td class="even">'.$row['name'].':</td>';
}

Странная конструкция

domaster, 21 Июня 2012

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

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

+135

1
2
3
4
5
6

// заполнить кадр на пересылку
for (i = 1; i<frame[0]; i++) 
{	sciOutFrame[i] = frame[i];}

// Установим длину кадра ( в первый байт кадра )
sciOutFrame[0] = i;

boolivar, 07 Июня 2012

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

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

+75

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

try {
    try{
        BasicFormatKeywordsReader keyRep = new BasicFormatKeywordsReader(new ExcelBook(fileName),keywordName,sheetName);
        keyRep.readKeywords(keyword);
    } catch (Throwable ex){
        throw new TestCaseException("Can't initialize flow",ex);
    }
    for(Executable executable:keyword.getChildren()) {
        executable.execute(getRequest());
    }
} catch (TestCaseException ex) {
    throw ex;
} catch (Throwable ex){
    throw new TestCaseException(ex);
}

Талантливо!

nafania217518, 05 Июня 2012

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

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

+92

1

double e1 = (((-Data[65021] + 1.25) / 2) > 0) && (((-Data[65021] + 1.25) / 2) < 6) ? 180 * Math.Acos(((-Data[65021] + 1.25) / 2) / 6.07) / Math.PI : 90;

Просто вычисляется в одну строчку угол. В алгоритме расчета параметров тех.процесса.

maitredesir, 05 Июня 2012

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

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

+155

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

ISQ.Data.checkString=function(e,f){
    var d=true;
    switch(f){
    case"email":
	var a=/^[\w-_\&.]+\@([\w-_]+\.[\w-_]+){1,3}$/;
	var g=new RegExp("[~!#$%^&*()=''\"\\/><?[]{}|]","g");
	return(e.search(g)===-1)&&(e.match(a))&&(e!=="");
	break;
    case"number":
	var b=/^[-]?[0-9]+\.?[0-9]?$/;
	return b.test(e);
	break;
    case"username":
	var c=new RegExp(".+@.+");
	return(c.test(e));
	break;
    case"url":
	var c=new RegExp("^https?://([a-zA-Z0-9]+.){1,}[a-zA-Z0-9]+");
	return(c.test(e));
	break
    }
    return d
}

// Функция универсальной валидации.
// (Но незадолго до того было еще и)

ISQ.Data.getEmailAddress=function(e){
    var a=/[\w-_\&.]+\@[\w-_]+\.+[\w-_.]+/g;
    var d=e.match(a);
    if(d===null||d.length===0){
	return null
    }
    var f=new RegExp();
    f.compile("[~!#$%^&*()=`'\"\\/><?[]{}|]","g");
    var b=new Array();
    for(var c=0;c<d.length;++c){
	if(d[c].search(f)===-1){
	    b.push(d[c])
	}
    }
    return b
}

Еще одна творческая работа.

wvxvw, 03 Июня 2012

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

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

+141

1

foreach ($collector as $valuev)  {

Очепятка.

T_ENGINE, 02 Июня 2012

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

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

+101

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
66
67
68
69
70

private void FillLocalService(int SelectedPageIndex)
        {
            //Set category by search
            string search = inputSearch.Value == inputSearchValue ? string.Empty : inputSearch.Value;
            if (inputSearch.Value != inputSearchValue && !string.IsNullOrWhiteSpace(inputSearch.Value))
            {
                if (inputSearch.Value.StartsWith("landscap", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Landscapers";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("baby sitter", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Baby sitters";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("doctor", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Doctor";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("cleaning service", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Cleaning services";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("pet sitter", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Pet sitters";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("handyman", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Handyman";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("electrical service", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Electrical services";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("tutor", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Tutor";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("dog walker", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Dog walker";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("plumb", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Plumbing";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("Caterer", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Caterers";
                    search = string.Empty;
                }
                else if (inputSearch.Value.Equals("misc", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Misc";
                    search = string.Empty;
                }
            }

200+ строк кода
}

тот же девелопер, та же .aspx страница)))))

mangyst, 30 Мая 2012

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

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