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

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

В номинации:

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

За время:

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

+160

01
02
03
04
05
06
07
08
09
10

//Я знаю что такое closure, поэтому добавляем в начале и конце:
(function(){

$(function(){

//....код мы будем писать здесь...

});

})();

А ты знаешь, что такое closure, %govnouser%? Найдено здесь http://habrahabr.ru/blogs/webdev/93331/

HyperGeek, 12 Мая 2010

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

VisualBasic / Говнокод #3142

−114

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

Public Function Cost(ByVal Vip1 As Boolean, ByVal Dat1 As Date, ByVal Kol1 As Integer, ByVal Cost1 As Decimal) As Decimal

        If Vip1 Then
            Return Bas(Cost1, Kol1) * 0.9 + IIf(Cost1 * Kol1 < 50, 50, Bas(Cost1, Kol1) * 0.11)
        Else
            Dim Sk1 As Decimal
            If Leto(Dat1) Then
                Sk1 = IIf(Bas(Cost1, Kol1) < 1000, 2, 3)
            Else
                Sk1 = IIf(Bas(Cost1, Kol1) < 1100, 3, 0)
            End If
            Dim Sk2 As Decimal = IIf(Kol1 > 500, 5, 0)
            Dim Sk3 As Decimal
            If Leto(Dat1) Then
                Sk3 = IIf(Bas(Cost1, Kol1) < 100, 100, Bas(Cost1, Kol1) * 0.1)
            Else
                Sk3 = IIf(Bas(Cost1, Kol1) < 110, 110, Bas(Cost1, Kol1) * 0.11)
            End If
            Return (Bas(Cost1, Kol1) - Bas(Cost1, Kol1) * Sk1 * 100 - Bas(Cost1, Kol1) * Sk2 * 100 + Sk3)
        End If
    End Function

Стандартный ынтырпрайз-говнокод, но изюминка, собсно, не в нём, а в нём:

http://www.sql.ru/forum/actualthread.aspx?bid=9&tid=467673&hl=

2striker. теперь предпросмотр показывает всегда не php, а си. Спасибо за фикс.

cfdev, 01 Мая 2010

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

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

+161.4

1
2
3
4
5
6

$taskName=$_GET[""];
$taskId=$_GET[""];
$objId=$_GET[""];
$catId=$_GET[""];

$this->DoInsertQuery("crm_task","TaskID,RefObjId,ObjectID",array("TaskID"=>$taskId,"RefObjId"=>$catId,"ObjectID"=>$objId));

а вам слабо?

nur, 19 Апреля 2010

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

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

+162.6

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

if(	!empty($argv[1]) &&
	is_array($REQUEST = unserialize(urldecode($argv[1]))))
{
	if(!empty($REQUEST['torrent']))
		$torrent_id=$REQUEST['torrent'];
	if(!empty($REQUEST['seed']))
		$seed=$REQUEST['seed'];
	if(!empty($REQUEST['submit_id']))
		$submit_id=$REQUEST['submit_id'];
	if(!empty($REQUEST['account_id']))
		$account_id=$REQUEST['account_id'];
	if(!empty($REQUEST['tracker_id']))
		$tracker_id=$REQUEST['tracker_id'];
}
if(empty($torrent_id))
{
	if(empty($_REQUEST['torrent']))
		exit("!torrent");
	else $torrent_id=$_REQUEST['torrent'];
}
if(empty($seed))
	$seed=(!empty($_REQUEST['seed']))?$_REQUEST['seed']:0;
if(empty($submit_id))
{
	if(empty($_REQUEST['submit_id']))
		exit("!submit_id");
	else $submit_id=$_REQUEST['submit_id'];
}
if(empty($account_id))
{
	if(empty($_REQUEST['account_id']))
		exit("!account_id");
	else $account_id=$_REQUEST['account_id'];
}
if(empty($tracker_id))
{
	if(empty($_REQUEST['tracker_id']))
		exit("!tracker_id");
	else $tracker_id=$_REQUEST['tracker_id'];
}

Продолжаем бороться с сотрудником.
Мальчик на полном серьезе не понимает, почему вот этот фрагмен - кромешный ад и его надо переписать.

dzugaru, 19 Апреля 2010

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

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

+158.8

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

// PRIVACY LEVEL OPTIONS
function winks_privacy_levels($privacy_level) {
	global $database, $functions_general, $functions_winks;
  
  	$lang_query = $database->database_query("SELECT languagevar_value FROM se_languagevars WHERE languagevar_id = 14000105");
	$lang_array = Array();
	while($item = $database->database_fetch_assoc($lang_query)) {$case0 = $item[languagevar_value];}
  	$lang_query = $database->database_query("SELECT languagevar_value FROM se_languagevars WHERE languagevar_id = 14000106");
	$lang_array = Array();
	while($item = $database->database_fetch_assoc($lang_query)) {$case1 = $item[languagevar_value];}
  	$lang_query = $database->database_query("SELECT languagevar_value FROM se_languagevars WHERE languagevar_id = 14000107");
	$lang_array = Array();
	while($item = $database->database_fetch_assoc($lang_query)) {$case2 = $item[languagevar_value];}

	
	switch($privacy_level) {
	  case 0: $privacy = $case0; break;
	  case 1: $privacy = $case1; break;
	  case 2: $privacy = $case2; break;
	  default: $privacy = ""; break;
	}
	return $privacy;
}

// WINKS ENABLED
function winks_enabled(){
	global $database;
	
	$winks_query = $database->database_query("SELECT * FROM sp_winks_settings");
	$winks_array = Array();
		while($item = $database->database_fetch_assoc($winks_query)) {
			$winks_enabled = $item[enabled];
		}
	return $winks_enabled;
}

С первой функцией я так и не понял что автор хотел сделать
Ну а про вторую мне вообще нечего сказать :)

Rain, 14 Апреля 2010

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

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

+117.4

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

string GetTextDiv2(string text)
    {
        int mid = text.Length / 2;
        int r = text.IndexOf(" ", mid); if (r < 0) r = 5000;
        int l = text.IndexOf(" ", 0, mid); if (l < 0) l = 5000;
        if (r - mid > mid - l) // to left is closer
            mid = l;
        else mid = r;

        if (mid == 5000) return "&nbsp" + text;
        return "&nbsp" + text.Substring(0, mid) + " <br/>&nbsp" + text.Substring(mid, text.Length - mid);
    }

ASP.NET
// это красота просто :) делим текст пополам там где пробел, а дальше добавляем между частями перенос на новую строку :)

otvet_popravkodon, 13 Апреля 2010

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

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

+143.6

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

using System;


struct MyStruct : IDisposable
{
	int x;
	
	private void Show()
	{
		unsafe
		{
			MyStruct obj1 = new MyStruct();
			MyStruct* p;
			
			p = &obj1;
			p->x = 10;
			Console.WriteLine(x);
		}
	}
	
	public static void Run()
	{
		using(MyStruct obj1 = new MyStruct())
		{
			obj1.Show();
		}
	}
	
	public void Dispose() { }
}

class program
{
	static void Main()
	{
		MyStruct.Run();
	}
}

сабж

sergylens, 07 Апреля 2010

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

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

+72.4

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

interface Editor extends Window {

    /** Get edited entity  */
    Entity getItem();

    /**
     * Set parent datasource to commit into this datasource instead of database.
     * This method must be followed by {@link #setItem(com.contora.package.core.entity.Entity)}
     */
    void setParentDs(Datasource parentDs);

    /** Set edited entity. Invoked by the framework on opening the window. */
    void setItem(Entity item);

    /** Check validity by invoking validators on all components which support them */
    boolean isValid();

    /** Check validity by invoking validators on all components which support them */
    void validate() throws ValidationException;

    /** Validate and commit changes */
    boolean commit();

    /** Commit changes with optional validating */
    boolean commit(boolean validate);

    /**
     * Validate, commit and close if commit was successful.
     * Passes {@link #COMMIT_ACTION_ID} to associated {@link CloseListener}s
     */
    void commitAndClose();
}

public class PickupRegionLookuper extends AbstractLookup implements Editor{
    public PickupRegionLookuper(IFrame frame) {
        super(frame);
    }

    @Override
    protected void init(Map<String, Object> params) {
       //Здесь была какая то логика
    }

    @Override
    public Entity getItem() {
        return null;  
    }

    @Override
    public void setParentDs(Datasource parentDs) {
    }

    @Override
    public void setItem(Entity item) {
    }

    @Override
    public boolean isValid() {
        return false;
    }

    @Override
    public void validate() throws ValidationException {
    }

    @Override
    public boolean commit() {
        return false;
    }

    @Override
    public boolean commit(boolean validate) {
        return false;
    }

    @Override
    public void commitAndClose() {
    }
}

Встретилось в одном java-файле. Класс реализует интерфейс, определенный в том же файле, причем интерфейс package-private (!). При этом ни одного метода оттуда реально не используется. К тому же этот интерфейс человек перенес из другого класса, где тот был как раз public.

tinhol, 06 Апреля 2010

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

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

−857

1
2

SELECT * FROM [dbo].[Publications]												
where [UserId] not like 'NULL'

UserId, естественно, int

synapse, 01 Апреля 2010

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

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

+131.6

1
2
3
4
5
6
7

<div style="width:100%; text-align:center; height:95%; position: absolute ; left:0; top:0;  text-align:center;  "  >
      
      
            <table id="TableHead" cellpadding="0" cellspacing="0" style="text-align:center;width: 400px; margin-top:-150px; margin-left:-200px;    position:absolute; top: 50%; " align="center">

...
</div>

ну не красотень

alex_donetsk, 01 Апреля 2010

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

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