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

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

В номинации:

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

За время:

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

+73.1

1

if (e.getClickCount() >= 2 && e.getClickCount() < 8) {

При том количество кликов больше восьми не обрабатывается))

lian, 05 Октября 2009

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

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)

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)

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

+98

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

BOOL NewStartUsed()
{
  if ((BOOL) -1 == bNewStartUsed) {
    bNewStartUsed = FALSE;
    bNewStartUsed = TRUE;
    int iVal = bNewStartUsed;

    if (GetConfigInt(&iVal, "NewStart", "Common", ConfigFile)) {
      bNewStartUsed = (0 != iVal);
    }
  }
  return bNewStartUsed;
}

просто но комментс...

kubrinsky, 18 Сентября 2009

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

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

+71.9

1
2
3
4
5
6

if (!selectAndAdd(select, administrators,
                                    userUid)) //shouldn't be so but...
                            {
                                throw new IllegalStateException(
                                        "World is full of mystical things. That is one of them");
                            }

Мир полон таинственных вещей! :-)

sweetshadow, 17 Сентября 2009

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

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

+100.1

1

ping -n 5 localhost > nul

К сожалению в Windows не предусмотрено никаких штатных команд для организации задержки, поэтому для того, чтобы в скрипте реализовать задержку в n секунд, например чтобы дождаться пока процесс завершится и освободит ресурс, приходится или качать/писать сторонние утилиты или прибегать к подобной чёрной магии.

Sauron, 09 Сентября 2009

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

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

+144

1
2
3
4
5
6
7

public void onApplicationStateChanged(int state)
{
            if (state == Application.APPSTATE_UNINITIALIZED)
            {
                       // TODO: what to do?
            }
}

Чувак начал писать и забыл о чем...

vassabi, 08 Сентября 2009

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

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

+160.2

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

if($showmode == 0 || $showmode == 2):
	if($count['user'] != 0):
		if($count['user'] == 1):
		    echo '<p class="tpwhosonline">Members Online: <span>1</span></p>';
		else:
		    if($count['user'] > 1):
			    echo '<p class="tpwhosonline">Members Online: <span>' . $count['user'] . '</span></p>';
			endif;
		endif;
	else:
		echo '<p class="tpwhosonline">Members Online: <span>0</span></p>';
	endif;
	
    if($count['guest'] != 0):
		if($count['guest'] == 1):
		    echo '<p class="tpwhosonline">Guests Online: <span>1</span></p>';
		else:
		    if($count['guest'] > 1):
			    echo '<p class="tpwhosonline">Guests Online: <span>' . $count['guest'] . '</span></p>';
			endif;
		endif;
	else:
	    echo '<p class="tpwhosonline">Guests Online: <span>0</span></p>';
    endif;
endif;

и такая вот хрень еще и за бабки продается (Joomla template)

endi, 04 Сентября 2009

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

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