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

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

В номинации:

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

За время:

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

+146

1
2
3
4
5
6
7

<table width="100%" border="0" cellspacing="0" cellpadding="0">
                <tr>
                                <td id="menu"><a id="1" href="javascript:void(0);">Link 1</a></td>
                                <td id="menu"><a id="2" href="javascript:void(0);">Link 2</a></td>
                                <td id="menu"><a id="3" href="javascript:void(0);">Link 3</a></td>
                </tr>                
</table>

ну вкладки html тут нет

nepster, 09 Марта 2011

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

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

+98

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

Procedure TreeNodeBuilding(Tree: TTreeView; Query: TZQuery; DestinationNode: TTreeNode;
                          IdColumn, NameColumn, FilterColumn, TableType: String; Filtering:Boolean; ImageIndex:Integer);
  //-------------------------------------------------//
  Function AddOrNot:Boolean; //Фильтрация по ИД
  var
    Add: Boolean;
  begin
  Add:=False;
  If Filtering=False then Add:=True else
    begin
    if DestinationNode=nil then Add:=True else
      begin
      MyNodeParamPtr:=DestinationNode.Data;
      if MyNodeParamPtr^.id=Query.FieldValues[FilterColumn] then Add:=True;
      end;
    end;
  Result:=Add;
  end;
  //-------------------------------------------------//
  Procedure AddTreeNodeToViewer; //Добавление ветки
  var
    MyNode: TTreeNode;
  begin
  //Добавляем новую запись в массив и заполняем её
//  Inc(ArrMyNodeParamLength);
//  SetLength(ArrMyNodeParam, ArrMyNodeParamLength);
  New(MyNodeParamPtr);
  MyNodeParamPtr^.id:=Query.FieldValues[IdColumn];
  MyNodeParamPtr^.table:=TableType;
  //Добавляем новый узел дерева и указатель на соотв запись
  if DestinationNode=nil then
  MyNode:=Tree.Items.AddObject(DestinationNode, VarToStr(Query.FieldValues[NameColumn]), MyNodeParamPtr)
    else MyNode:=Tree.Items.AddChildObject(DestinationNode, VarToStr(Query.FieldValues[NameColumn]), MyNodeParamPtr);
  MyNode.ImageIndex:=ImageIndex;
  MyNode.SelectedIndex:=ImageIndex;
  end;
  //-------------------------------------------------//
begin
//
Query.First;
if AddOrNot=True then AddTreeNodeToViewer;
Query.Next;
while not Query.Eof do
  begin
  if AddOrNot=True then AddTreeNodeToViewer;
  Query.Next;
  end;
end;

Добавление ветки в TreeView. Я тут явно перемудрил

nikitasnv, 02 Марта 2011

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

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

+115

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

char[] splitter = { ',' };
string types = hashtable[FlagsEnumValue].ToString();
string[] typesStringArray = types.Split(splitter, StringSplitOptions.RemoveEmptyEntries);
ArrayList typesArray = new ArrayList();

foreach (string str in typesStringArray)
{
   foreach (string type in Enum.GetNames(typeof(FlagsEnum)))
   {
      if (type == str.Trim())
      {
         typesArray.Add((FlagsEnum)Enum.Parse(typeof(FlagsEnum), str, true));
         break;
      }
   }
}


foreach (FlagsEnum type in typesArray)
{
   if ((someObject.field & type) > 0)
   {
      typeFound = true;
   }
   else
   {
      typeFound = false;
      break;
   }
}

Автор хотел чтобы его любили. Точнее он хотел сконвертировать строковое представление битового енама в инам и сравнить по маске с проперти обьекта. Если бы автор прочел документацию то написал бы так:
string types = hashtable[FlagsEnumValue].ToString();
if (types != "")
{
FlagsEnum enum = (FlagsEnum)Enum.Parse(typeof(FlagsEnum), types, ignoreCase: true);
if ((enum & someObject.field) == enum)
typeFound = true;
}

piocsic, 01 Марта 2011

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

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

+157

1
2
3
4
5

if($_ENV["COMPUTERNAME"]!='BX') 
{
      CopyDirFiles($_SERVER["DOCUMENT_ROOT"]."/bitrix/modules/subscribe/install/admin", $_SERVER["DOCUMENT_ROOT"]."/bitrix/admin"); 
// и еще куча аналогичного
}

шедевральная проверка
битрикс, да :)

elw00d, 28 Февраля 2011

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

ActionScript / Говнокод #5813

−107

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

public function receive(streamName:String):void
{
	stopReceive()
	
	if(!netConnection.connected)
	{
		receiveStreamName = streamName;
		LogAppender.getInstance().appendLog(LogType.STREAM_LOG,"FMSStream.receive?not connected, set streamName: " + receiveStreamName);
		return;
	}
	
	nsReceive = new NetStream(netConnection);
	nsReceive.addEventListener(NetStatusEvent.NET_STATUS, handleReceiveNetStreamStatus);
	nsReceive.play(streamName, -2, -1, true);
	dispatchEvent(new VRDataEvent(VRDataEvent.OPPONENT_STREAM_UPDATED));
	// default audio sound
	var st:SoundTransform = nsReceive.soundTransform;
	st.volume = 0.7;
	nsReceive.soundTransform = st;			
	examineCallTimer.start();
	LogAppender.getInstance().appendLog(LogType.STREAM_LOG,"FMSStream.receive?play=" + streamName);
}

Уже за новыми наушниками собирался идти...

wvxvw, 27 Февраля 2011

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

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