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

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

+60.3

1
2
3
4
5

void AcquireSomething(..., boost::shared_ptr<Something> out)
{
  ...
  out.reset(new Something());
}

жесть

g26g, 23 Сентября 2009

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

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

+73.7

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

public Object save(Object entity) {
        final EntityManager em = getEntityManager();
        try {
            final EntityTransaction et = em.getTransaction();
            try {
                et.begin();
                em.persist(entity);
                et.commit();
            } finally {
                if (et != null && et.isActive()) {
                    entity = null;
                    et.rollback();
                }
            }
        } finally {
            if (em != null && em.isOpen()) {
                em.close();
            }
        }
        return entity;
    }

Простое и понятное управление транзакциями придумали трусы!
А rollback в секции finally, и вложенные try-catch секции -- вообще модная сейчас фишка.

rh.mbox, 23 Сентября 2009

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

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

+159.1

1

paramsObj.templateName = templateNewWindow.items.items[0].items.items[0].items.items[0].items.items[0].getValue();

Получение значения ExtJS'ного контрола, который спрятан глубоко в дереве.

stan, 22 Сентября 2009

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

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

−179

1
2
3
4
5
6

Выборка=Запрос.Выполнить().Выбрать();
Пока Выборка.Следующий() Цикл
	ФизЛицо=Выборка.Физлицо;
	Состояние("Обробатывается - "+ФизЛицо.Ссылка);
...
КонецЦикла;

Если новичок порой может забыть указать реквизит ".Ссылка", где это требуется. То тут всё наоборот. Видать, чел настолько поднаторел, что только он знает, нахера тут ссылка. (про грамматику молчу)

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

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

Perl / Говнокод #1882

−122

1
2
3

my $title=$self->{ditems}->get_field_value({SET=>'City', itemid=>$cityid, field=>'genitive_case', notitle=>1});
$title='в '.$self->{ditems}->get_field_value({SET=>'City', itemid=>$cityid, field=>'prepositional_case', notitle=>1}) unless length($title);
return '' if $title eq 'в ';

poltora, 22 Сентября 2009

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

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

+128.3

1

List<estExecPeriod> lst = TEDManager.getExecPeriodByIDExecWork(budget.IdBudget);

iley, 22 Сентября 2009

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

Pascal / Говнокод #1880

+91.1

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

s := TStringList.Create;
  LocalIniFile.ReadSection(sPad, s);
  for x := 0 to ActionListMenu.ActionCount - 1 do
  begin
    for y := 0 to 9 do
    begin
      for z := 0 to s.Count - 1 do
      begin
        if (Length(s[z]) <> Length('Tag0')) and (s[z] = 'Tag' + IntToStr(ActionListMenu[x].Tag + y)) then
        begin
          try
            ActionListMenu[x].Execute;
          except
          end;
        end;
      end;
    end;
  end;
  s.Free;

Эм, ну комментарии излишни?

Drone, 22 Сентября 2009

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

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)

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

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

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

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

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

Perl / Говнокод #1882

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

Pascal / Говнокод #1880

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

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

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