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

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

В номинации:

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

За время:

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

+153.4

1
2
3
4
5
6
7
8

$db = new PDO("mysql:host=" . $config['dbhost'] . ";dbname=" . $config['dbname'], $config['dbuser'], $config['dbpassword']);
$db->exec("set names utf8");
$result = $db->query("call get_user_account('$session_id');");
foreach ($result as $row)
{
    die($row['account']);
}
die("err");

Готический output.

Bobby, 16 Апреля 2010

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

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

+156.2

1
2
3
4
5
6
7
8
9

switch ($result['state']) 
{
    case 'Oregon':
	$row->state = 'OR';
    break;
   case $row->state=='Washington':
	$row->state = 'WA';
   break;
}

нашел у своего потрудника, долго не мог понять что второй кейс делает и как это вообще работало

quard, 09 Апреля 2010

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

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

+115.4

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

public partial class AbstractPickerBrw : Form
{
       //...
        public AbstractPickerBrw ()
        {
              this.Shown += new System.EventHandler(this.AbstractPickerBrw_Shown);
        }
        //...

        private void AbstractPickerBrw_Shown(object sender, System.EventArgs e)
        {
            Width = Width % 2 == 0 ? Width + 1 : Width - 1;
        }
}

Это мой код, просто не мог добиться перерисовки диалога при смене данных в WinForms и решил не заморачиваться.

xaoc, 01 Апреля 2010

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

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

−116.2

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

# TODO: выбросить нахер метод with_company_and_state и правильно написать
# этот named scope
named_scope :with_company_and_state_non_uniq, lambda { |company, state|
  { :conditions => [ 'products.company_id = :company_id AND ' +
        'state_mask = :state_mask', { :company_id => company,
        :state_mask => OrderSet::STATES.index(state) || 0 } ],
    :order => 'order_sets.created_at DESC',
    :joins => 'INNER JOIN products' }
}

def self.with_company_and_state(company, state)
  with_company_and_state_non_uniq(company, state).uniq
end

Бывает, что SELECT DISTINCT сделать не всегда возможно. На самом деле, такая выборка сосёт.

eveel, 28 Марта 2010

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

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

+76.8

1
2
3
4
5
6
7
8
9

public void isChangenabledExc(){
        if(isNew){
            return;
        }
        if(true)return;//TODO
        if(!isChangenabled()){
            throw new ChangeEIsNotLockedException();
        }
    }

В этой функции находится единственный коментарий из всего файла.
Видимо в планах эта функция должна была делать нечто хорошее, но
планы так и не были осуществелны.
В результате получилась функция, которая ничего не делает, а только проверяет какие то условия.
поиском обнаружил 5 вызовов этой функции.

maxt, 26 Марта 2010

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

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

+118

1
2
3
4
5

<asp:Repeater runat="server" ID="rpAccountType">
	<ItemTemplate>
		<input type="checkbox" name="cbGroupAccountType" id="cbAccountType_<%#((KeyValuePair<string, object>)Container.DataItem).Value %>" value="<%#((KeyValuePair<string, object>)Container.DataItem).Value %>" <asp:Literal Id="lAccountTypeChecked" runat="server" />/><label for="cbAccountType_<%#((KeyValuePair<string, object>)Container.DataItem).Value %>"><asp:Literal runat="server" ID="lAccountTypeName" /></label><br />
	</ItemTemplate>
</asp:Repeater>

ASP.NET. Значения генерируемых чекбоксов, в задумке, еще и на code-behind сильно влияют.

terR0Q, 23 Марта 2010

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

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

−88

01
02
03
04
05
06
07
08
09
10
11

if (_root["currKeyState"] == 3 and _root["currKeyStateCaps"] == "off") {
			_root["currKeyState"] = 1;
			if ("titleNewRec" == _root["currentInput"] or "msgText" == _root["currentInput"] or "prvSearchStr" == _root["currentInput"] or "mdengiId" == _root["currentInput"] or "fio" == _root["currentInput"] or "alfabplace" == _root["currentInput"] or "commut" == _root["currentInput"]) {
			} else {
			}
		} else if (_root["currKeyState"] == 4 and _root["currKeyStateCaps"] == "off") {
			_root["currKeyState"] = 2;
			if ("titleNewRec" == _root["currentInput"] or "msgText" == _root["currentInput"] or "prvSearchStr" == _root["currentInput"] or "mdengiId" == _root["currentInput"] or "fio" == _root["currentInput"] or "alfabplace" == _root["currentInput"] or "commut" == _root["currentInput"]) {
			} else {
			}
		}

Хотел посмотреть, как реализована логика в старом коде и нашел вот это...

-=Deus=-, 22 Марта 2010

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

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

+152.9

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

procedure TForm1.Button2Click(Sender: TObject);
var i:dword;
begin
if (Checkbox1.Checked)or(Checkbox3.Checked)or(Checkbox2.Checked) then
begin
form1.Cursor:=crhourglass;
Sleep(500);
progressbar1.Position:=10;
Sleep(1000);
progressbar1.Position:=20;
Sleep(500);
progressbar1.Position:=50;
Sleep(1000);
progressbar1.Position:=100;
Sleep(500);
end;

Трудимся в поте лица. :D

Говногость, 13 Марта 2010

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

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