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

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

В номинации:

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

За время:

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

+144.5

1
2
3
4
5
6

string str=...;

if(str!=null && str.Length>0)
{
....
}

Без комментариев

AndrewK, 20 Октября 2009

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

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

+63.4

1
2
3
4
5
6
7
8

void RoomFactory::roomCreated(Siis::Room* room,bool created)
{
 if(!created)
 {
 rooms_.remove(room->peer()->jid().bare());
 delete room;
 }
}

Не индус...

nesquikm, 19 Октября 2009

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

Ruby / Говнокод #1991

−119.8

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

case @demand.status #зависит также от временных групп
	when 1,4
		@status_list[2]=@@statuses[2]	
	when 2,7,8	
		@status_list[2]=@@statuses[2]
		@status_list[3]=@@statuses[3]
		@status_list[7]=@@statuses[7]
		@status_list[8]=@@statuses[8]
	when 3
		@status_list[3]=@@statuses[3]
		if @is_admin || @demand.watchers.include?(session[:user]) || @demand.heads.include?(session[:user]) ||@see_later_demands_watchers || @see_later_demands_heads
			@status_list[4]=@@statuses[4]
		end
		if @is_admin || @demand.heads.include?(session[:user]) || @see_later_demands_heads
			@status_list[6]=@@statuses[6]
		end
	when 6
		if @is_admin || @demand.watchers.include?(session[:user]) || @demand.heads.include?(session[:user]) ||@see_later_demands_watchers || @see_later_demands_heads
			@status_list[4]=@@statuses[4]
		end	
	else
	end

магия

Rommel, 18 Октября 2009

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

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

+151

01
02
03
04
05
06
07
08
09
10

public function __construct()
  {    
    $this->post = $_POST;
    unset($_POST);
  }
  
  public function get($var_name)
  {
    return $this->post[$var_name];
  }

ещё бы сделали ф-ю
public function post($var_name)
{
return $this->get[$var_name];
}

paranoid, 16 Октября 2009

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

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

+134.7

1
2
3
4

Настоящая шиза - это когда ты сидишь часами и придумываешь говнокод по извращённей, 
только для того что бы выложить его на govnokod.ru
:)
переделка одной цитаты с баша (с)

nico-izo, 11 Октября 2009

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

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)

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

+136.4

1
2
3
4

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

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

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

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