По колено в коде — Говнокод.ру

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

+141.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
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
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89

switch (Step)
{
    case 6:
        {
            goto case 5;
        }
    case 5:
        {
            Step = 2;
            goto case 2;
        }
    case 4:
        {
            if (ReadOnly)
            {
                Step = 2;
                goto case 2;
            }
            pollingMatrix.Fill(Id, pollingRow.RiskObjectTypeId, languageId, connection);
            Scenario.Fill(Id, pollingRow.RiskObjectTypeId, languageId, connection);
            tbMain.SetCurrentTab(tcEnding);
            break;
        }
    case 3:
        {
            if (ReadOnly)
            {
                btnPrevPg4.Visible = false;
                btnNextPg4.Visible = false;
            }
            CleanScenarios();
            Scenario.Fill(Id, pollingRow.RiskObjectTypeId, languageId, connection);
            tbMain.SetCurrentTab(tcScenarios);
            break;
        }
    case 2:
        {
            if (ReadOnly)
            {
                btnPrevPg3.Visible = false;
                btnNextPg3.Visible = false;
            }
            pollingMatrix.Fill(Id, pollingRow.RiskObjectTypeId, languageId, connection);
            tbMain.SetCurrentTab(tcPollingMatrix);
            break;
        }
    case 1:
        {
            if (ReadOnly)
            {
                btnPrevPg2.Visible = false;
                btnNextPg2.Visible = false;
				//tvRiskFactors.Enabled = false;
				//tvRiskObjects.Enabled = false;							

				//Раскрывает ноду если есть отмеченные дочерние элементы							
				tvRiskObjects.ExpandAllNodes();
            }
            pollingMatrix.Fill(Id, pollingRow.RiskObjectTypeId, languageId, connection);
            doRiskFactorTree();
            doRiskObjectTree();
            hcpRiskObjects.PanelName = pollingRow.QSARiskObjectTypeNameDictionaryName;
            tbMain.SetCurrentTab(tcRiskFactors);
            break; //goto case 0;
        }
    case 0:
        {
            if (ReadOnly)
            {
                Name.ReadOnly = true;
                Comment.ReadOnly = true;
                Hypothesis.ReadOnly = true;
                calDateTo.Enabled = true;
                RiskObjectTypeId.ReadOnly = true;
                btnNextPg1.Visible = false;
            	Instructions.ReadOnly = true;
            }
            tbMain.SetCurrentTab(tcPolling);
            break;
        }
    default:
        {
#if (DEBUG)
            exc_viewer.ViewException(@"Неверный Step, провалились в default");
#endif
            Step = 0;
            goto case 0;
        }
}

Часть метода из 200 строк в кодбехайнде ASP.NET странички...
Красота!!! =)

naumenkopa, 21 Сентября 2009

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

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

−863.7

1
2
3

Нашёл в проекте
SELECT DISTINCT sale_date, SUM(sale_price), buyer FROM ... 
GROUP BY sale_date, buyer;

DISTINCT здесь лишний

Oleg_quadro, 21 Сентября 2009

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

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

+157.3

01
02
03
04
05
06
07
08
09
10
11

function SetStyleForFilterOver(this2) {
    this2.style.border = "1px solid #ea0000";
    this2.style.color = "white";
    this2.style.background = "#404040";
}
function SetStyleForFilterOut(this2) {
    this2.style.border = "1px solid #404040";
    this2.style.color = "#ea0000";
    this2.style.color = "#ea0000";
    this2.style.background = "#eeeeee";
}

Тс-с-с-с... Только не говорите автору про css и селектор ":hover" :о)

Ordos, 21 Сентября 2009

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

1C / Говнокод #1876

−92.1

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

Запрос.Текст = 
"ВЫБРАТЬ
|	СостояниеРаботниковОрганизаций.Регистратор,
|	СостояниеРаботниковОрганизаций.ФизЛицо,
|	СостояниеРаботниковОрганизаций.Организация,
|	СостояниеРаботниковОрганизаций.Приказ,
|	СостояниеРаботниковОрганизаций.Состояние,
|	СостояниеРаботниковОрганизаций.Период
|ИЗ
|	(ВЫБРАТЬ
|		СостояниеРаботниковОрганизаций.Регистратор КАК ДокументРегистратор
|	ИЗ
|		РегистрСведений.СостояниеРаботниковОрганизаций КАК СостояниеРаботниковОрганизаций
|	ГДЕ
|		СостояниеРаботниковОрганизаций.Период МЕЖДУ &ДатаНачала И &ДатаКонца
|		И СостояниеРаботниковОрганизаций.Организация = &Организация
|		И СостояниеРаботниковОрганизаций.ФизЛицо = &ФизЛицо
|		И СостояниеРаботниковОрганизаций.Приказ = &Приказ) КАК ДокРегистратор
|		ВНУТРЕННЕЕ СОЕДИНЕНИЕ РегистрСведений.СостояниеРаботниковОрганизаций КАК СостояниеРаботниковОрганизаций
|		ПО ДокРегистратор.ДокументРегистратор = СостояниеРаботниковОрганизаций.Регистратор";
.

Просто выбрать движения из регистра - это не круто.

Kopchuga, 21 Сентября 2009

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

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

+136.4

1
2
3
4

// Работаю asp.net программистом.
// Показал главному программисту, как с помощью System.Web.Routing можно легко делать человекопонятные урлы.
// Итог: Теперь у сайта урлы вида http://domain/filterDisks/42/0/14/1.
// Вот сижу и думаю, что значат все эти циферки :'о(

Ordos, 21 Сентября 2009

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

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

+55.9

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

while (true)
  {
    m_CurrentDevice += delta;

    if (m_CurrentDevice < 0)
      m_CurrentDevice = devices[DEV_PLATE]-1;

    if (m_CurrentDevice > devices[DEV_PLATE]-1)
      m_CurrentDevice = 0;

    break;
  }

Это конечно написано не специально - в результат целой кучи итераций и переделок. но результата это не отменяет

TObject, 21 Сентября 2009

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

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

+158.1

1
2
3
4
5
6
7
8
9

case 'down':
	if (parseInt(pageTop) + parseInt(pageHeight) - parseInt(elementTop) - parseInt(elementHeight) - parseInt(elementBorderTop) - parseInt(elementBorderBottom) >= parseInt(stepOfMove)) {
		var topParam = parseInt(elementTop) - 0 + parseInt(stepOfMove) + 'px';
		elementToMove.style.top = topParam;
		this.updateElementsParams(id,'top',topParam);
	} else {
		stepOfMoveElement.value = parseInt(pageTop) - 0 + parseInt(pageHeight) - parseInt(elementTop) - parseInt(elementHeight) - parseInt(elementBorderTop) - parseInt(elementBorderBottom);
	}
break;

и так миллион тыщ раз...

Sadie, 21 Сентября 2009

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

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

+60.6

1
2
3

static char *szClassName = new char[14];
static char *szCurrentDirectory = new char[MAX_BUFFER];
static char *szNewFolder = new char[MAX_BUFFER];

глобальные указатели рулят, delete нигде не вызывается

shomeser, 20 Сентября 2009

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

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

+158.3

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

<?php

function secure2() {
    secure();
}

function secure() {
    secure3();
}

function secure3() {
    if (!check_session()) {
        $cookie = try_cookie();
        tiny_login($cookie['login'], $cookie['pass'], 1);
        if (check_session()) {
            $ret = true;
        } else {
            $ret = false;
        }
    } else {
        $ret = true;
    }

    if (!$ret) {
        $_SESSION['loginerror'] = 'Ошибка авторизации';
        header('location:(ссылка)');
    }
}

?>

В догонку к http://govnokod.ru/1820 по просьбе в комментах
Реальный код из реального проекта. Что самое интересное — используется secure2
Как оно работает для меня — загадка.

Shock, 19 Сентября 2009

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

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

+163.8

1
2
3
4
5
6

//Я ТИПА ЗДЕСЬ СДЕЛАЛ НА ГАЛЕРЕЕ 4 КАРТИНКИ
               if(this.id=='gallery') {
                    this.previewCount = 4;
               } else {
                    this.previewCount = 4;
               }

Недавно код пересматривали. Авторы долго отнекивались

hagnar, 19 Сентября 2009

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

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