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

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

В номинации:

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

За время:

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

+147

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
90
91
92
93
94
95
96
97

int offset_800=0;
	if(SCREEN_WIDTH == 800) offset_800 = 20;

	addItem(TNM_GAMELOFT_LOG_GREEN_ITEM, done, complete_log[lang][0], TNM_ItemCoords[TNM_GAMELOFT_LOG_GREEN_ITEM][0]*((float)1200/(float)SCREEN_WIDTH)-35-offset_800/*-offset_shtamp*3*/, TNM_ItemCoords[TNM_GAMELOFT_LOG_GREEN_ITEM][1]*((float)720/(float)SCREEN_HEIGHT)/*-offset_shtamp*3*/, -1, ITEM_INVISIBLE)
		->attachTo2dItem(pItemListBG);
	addItem(TNM_GAMELOFT_LOG_RED_ITEM, done, complete_log[lang][1], TNM_ItemCoords[TNM_GAMELOFT_LOG_RED_ITEM][0]*((float)1200/(float)SCREEN_WIDTH)-35-offset_800/*-offset_shtamp*3*/, TNM_ItemCoords[TNM_GAMELOFT_LOG_RED_ITEM][1]*((float)720/(float)SCREEN_HEIGHT)/*-offset_shtamp*3*/, -1, ITEM_INVISIBLE)
		->attachTo2dItem(pItemListBG);
	
//#ifdef WIN32

	addItem(TNM_CHEAT_WIN, main_all, 30, TNM_ItemCoords[TNM_CHEAT_WIN][0],TNM_ItemCoords[TNM_CHEAT_WIN][1], 31, ITEM_INVISIBLE);
	addItem(TNM_CHEAT_LOSE, main_all, 30, TNM_ItemCoords[TNM_CHEAT_LOSE][0],TNM_ItemCoords[TNM_CHEAT_LOSE][1], 31, ITEM_INVISIBLE);
//#endif
     int offY=0;
	int offset=0,
		offsetmailX=0;
	
	if(SCREEN_WIDTH ==800||SCREEN_WIDTH == 1024)
		offset=2;
	
	if(SCREEN_WIDTH==1366||SCREEN_WIDTH == 1440){
		offY=4;
		offset=2;
	}
	if( SCREEN_WIDTH==1680)
	{ 
		offY=7;
		offset=2;
		offsetmailX=60;
	}
		if( SCREEN_WIDTH==1920)
	{ 
		offY=7;
		offset=5;
		offsetmailX=60;
	}

		int y_800=0;

		if(SCREEN_WIDTH == 1440) offsetmailX = 20;
		if(SCREEN_HEIGHT == 800) {
			offset = 2;
			offY = 1;
			
		}
		if(SCREEN_WIDTH == 2560) offset=2;
		if(SCREEN_WIDTH == 1024) offY+=1;
		if(SCREEN_WIDTH == 800&&(Text::getInstance().getLanguage()== Text::LANG_SC||Text::getInstance().getLanguage()== Text::LANG_KR)) {
			offY-=1;
			y_800 = 3;
		}

		

		if((SCREEN_WIDTH == 1680||SCREEN_HEIGHT == 1080)&&(Text::getInstance().getLanguage()== Text::LANG_SC||Text::getInstance().getLanguage()== Text::LANG_KR)){
			//offset-=2;
			offY-=2;
		}
		if((SCREEN_HEIGHT == 1200)&&(Text::getInstance().getLanguage()== Text::LANG_SC||Text::getInstance().getLanguage()== Text::LANG_KR))
			offY-=2;

	addText(TNM_MESSAGE_SENDER_ITEM, STR_SENDER, SPRITE_FONT_S2_EG, TNM_ItemCoords[TNM_MESSAGE_SENDER_ITEM][0], TNM_ItemCoords[TNM_MESSAGE_SENDER_ITEM][1]/*-offY*/, ITEM_INVISIBLE, LEFT)
		->attachTo2dItem(pItemListBG);
	
	

	addText(TNM_MESSAGE_SUBJECT_ITEM, STR_SUBJECT, SPRITE_FONT_S2_EG, TNM_ItemCoords[TNM_MESSAGE_SUBJECT_ITEM][0], TNM_ItemCoords[TNM_MESSAGE_SUBJECT_ITEM][1]-offset-offY*1.5f+y_800, ITEM_INVISIBLE, LEFT)
		->attachTo2dItem(pItemListBG);
	addText(TNM_MESSAGE_TIME_ITEM, STR_RECEIVED, SPRITE_FONT_S2_EG, TNM_ItemCoords[TNM_MESSAGE_TIME_ITEM][0], TNM_ItemCoords[TNM_MESSAGE_TIME_ITEM][1]-offY*1.07f,  ITEM_INVISIBLE, LEFT)
		->attachTo2dItem(pItemListBG);
	//addText(TNM_MAIL_TITLE_ITEM, "Sender: \nRecieved: \nSubject: ", fontS, TNM_ItemCoords[TNM_MAIL_TITLE_ITEM][0], TNM_ItemCoords[TNM_MAIL_TITLE_ITEM][1], ITEM_INVISIBLE, LEFT|TOP);
	int space = lang == Text::LANG_IT ? 10 : 0;
	if(lang == Text::LANG_KR)
		space = 30;
	int spaceY=0;

	
	if(SCREEN_WIDTH!=1680&&SCREEN_WIDTH!=1920)
		switch( pGame->getCurrentLanguage() )
	{
	case Text::LANG_KR:
		spaceY = 2;
		break;
	case Text::LANG_SC:
		spaceY = 2;
		break;
	case Text::LANG_JP:
		spaceY = 2;
		break;
	}

	addText(TNM_MAIL_TITLE1_ITEM, m_mailTitle, SPRITE_FONT_S2_EG, TNM_ItemCoords[TNM_MAIL_TITLE1_ITEM][0] + space, TNM_ItemCoords[TNM_MAIL_TITLE1_ITEM][1], ITEM_INVISIBLE, LEFT|TOP)
		->attachTo2dItem(pItemListBG);
	addText(TNM_MAIL_TITLE2_ITEM, m_mailTitle, SPRITE_FONT_S2_EG, TNM_ItemCoords[TNM_MAIL_TITLE2_ITEM][0] + space, TNM_ItemCoords[TNM_MAIL_TITLE2_ITEM][1]-offY*1.07f, ITEM_INVISIBLE, LEFT|TOP)
		->attachTo2dItem(pItemListBG);
	addText(TNM_MAIL_TITLE3_ITEM, m_mailTitle, SPRITE_FONT_S2_EG, TNM_ItemCoords[TNM_MAIL_TITLE3_ITEM][0] + space, TNM_ItemCoords[TNM_MAIL_TITLE3_ITEM][1]-offset-offY*1.5f+spaceY, ITEM_INVISIBLE, LEFT|TOP)
		->attachTo2dItem(pItemListBG);

Еще один малый кусок. По аналогии с выложеным ранее.

BlaZe, 27 Июля 2011

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

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

+147

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

private static Random rnd = new Random();

  public static int getRand(int start, int end) {
    boolean check = false;
    int number = 0;
    if (start >= end) {
      final int temp = end;
      end = start;
      start = temp;
    }
    while (!check) {
      number = Math.abs(rnd.nextInt() % end);
      if ((number >= start) && (number < end)) {
        check = true;
      }
    }
    return number;
  }

эпический метод, найден в исходниках игрушки

Lure Of Chaos, 27 Июля 2011

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

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

+149

1

var save = if($('input#remmepls').attr('checked="checked"')){1}else{0};

Думаю и так всё понятно

TRANE73, 26 Июля 2011

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

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

+976

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

public void OpredelenieKletki(TipiReljefaMiraVelikogo tipKletkiEtt)
{
tipKletki = tipKletkiEtt;
 
#region огромное назначение координат ректанглам вырезания из разнобоя
 
if (kameraEtojKletki.sasUvelicenie)
{
    switch (tipKletki)
    {
        case TipiReljefaMiraVelikogo.zelenaRovnaAA:
            malOcered = 1; kadrSasVirezanija = new Rectangle(0, kameraEtojKletki.sasKudaSmotrit * 79, 162, 79);
            smewenieSasKadra = new Vector2(18f, 98f); animacijaLi = false; nomRaznoboja = 19; break;
        case TipiReljefaMiraVelikogo.zelenaRovnaBB:
            malOcered = 1; kadrSasVirezanija = new Rectangle(0, 316 + kameraEtojKletki.sasKudaSmotrit * 79, 162, 79);
            smewenieSasKadra = new Vector2(18f, 98f); animacijaLi = false; nomRaznoboja = 19; break;
        case TipiReljefaMiraVelikogo.zelenaRovnaCC:
            malOcered = 1; kadrSasVirezanija = new Rectangle(0, 632 + kameraEtojKletki.sasKudaSmotrit * 79, 162, 79);
            smewenieSasKadra = new Vector2(18f, 98f); animacijaLi = false; nomRaznoboja = 19; break;
        case TipiReljefaMiraVelikogo.zelenaRovnaDD:
            malOcered = 1; kadrSasVirezanija = new Rectangle(0, 948 + kameraEtojKletki.sasKudaSmotrit * 79, 162, 79);
            smewenieSasKadra = new Vector2(18f, 98f); animacijaLi = false; nomRaznoboja = 19; break;
 
        case TipiReljefaMiraVelikogo.zelenaRovnaVertPalkaAA:
            malOcered = 1; kadrSasVirezanija = new Rectangle(0, 1264 + kameraEtojKletki.sasKudaSmotrit * 79, 162, 79);
            smewenieSasKadra = new Vector2(18f, 98f); animacijaLi = false; nomRaznoboja = 19; break;
        case TipiReljefaMiraVelikogo.zelenaRovnaVertPalkaAb:
            malOcered = 1; kadrSasVirezanija = new Rectangle(0, 1580 + kameraEtojKletki.sasKudaSmotrit * 79, 162, 79);
            smewenieSasKadra = new Vector2(18f, 98f); animacijaLi = false; nomRaznoboja = 19; break;
 
        case TipiReljefaMiraVelikogo.zelenaRovnaVertPalkaBB:
            malOcered = 1; kadrSasVirezanija = new Rectangle(162, 1264 + kameraEtojKletki.sasKudaSmotrit * 79, 162, 79);
            smewenieSasKadra = new Vector2(18f, 98f); animacijaLi = false; nomRaznoboja = 19; break;
        case TipiReljefaMiraVelikogo.zelenaRovnaVertPalkaBc:
            malOcered = 1; kadrSasVirezanija = new Rectangle(162, 1580 + kameraEtojKletki.sasKudaSmotrit * 79, 162, 79);
            smewenieSasKadra = new Vector2(18f, 98f); animacijaLi = false; nomRaznoboja = 19; break;

     ... ... ...

Класс который достает из спрайтшита нужную текстуру и передает её на отрисовку.

ArtFeel, 21 Июля 2011

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

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

+164

1
2
3
4
5
6
7
8

function calculateBaggageFields(selectedNormalOutboundOption, selectedNormalInboundOption,
				numOriginalNormalBagsOutboundField, numOriginalNormalBagsInboundField,
				numAdditionalBagsField, numAdditionalBagsOutboundField, numAdditionalBagsInboundField,
				selectedOutsizeOutboundOption, selectedOutsizeInboundOption,
				totalForThisPassengerField,
                                selectedExcessKilosOutboundOption,selectedExcessKilosInboundOption,
                                numOriginalExcessKilosOutboundField,numOriginalExcessKilosInboundField,
                                numAdditionalExcessKilos,numAdditionalExcessKilosOutbound,numAdditionalExcessKilosInbound)

продакшн-говно за работой!

hrls, 14 Июля 2011

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

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

+147

1
2
3
4
5
6
7
8

if(!xmlDate.equals(null))
{
...
}
else
{
        return null;
}

Для полноты картинки смотрим метод equals в XMLGregorianCalendar.java. Стажеры такие стажеры...

Art, 14 Июля 2011

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

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

−851

1

facepalm.sql

В таблице платежей несколько полей с цифрами - _amount и _value.
В связаной таблице есть поле PaymentAmount.
Угадайте, какому полю платежа оно соответствует 7:

wiz, 13 Июля 2011

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

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

+185

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

if ($_GET['code1c_Connected']=='') {
				if ($_GET['type']!="shop") {
					if ($_GET['type']!="lk") {
						if ($_GET['type']!="home") {
						if ($_GET['type']!="login_reg") {
							if ($_GET['type']!="reg_quick") {
								if ($_GET['type']!="order") {
									if ($_GET['type']!="order_g") {
								       if ($GET['type']!="payment") {
										if ($_GET['type']!='contakts') {
											if ($_GET['type']!='contact_kiev_volinskaya47') {
												if ($_GET['type']!='contact_kiev_lesi_ukrainki15') {
													if ($_GET['type']!='contact_kiev_vozduhoflotskiy17') {
														if ($_GET['type']!='contact_kiev_malinovskogo12') {
															if ($_GET['type']!='contact_kiev_solomenskaya16') {
																if ($_GET['type']!='contact_kiev_yakuba2') {
																	if ($_GET['type']!='contact_odessa_tiraspolskaya35') {
																		if ($_GET['type']!='contact_odessa') {
																			if ($_GET['type']!='contact_kharkov') {
																				if ($_GET['type']!='contact_kharkov_fonvizena18') {
																					if ($_GET['type']!='contact_dnepropetrovsk') {
																						if ($_GET['type']!='contact_dnepropetrovsk_suvorova14') {
																							if ($_GET['type']!='contact_donetsk') {
																								if ($_GET['type']!='contact_donetsk_shorsa24') {
																									if ($_GET['type']!='contact_zaporozhye') {
																										if ($_GET['type']!='contact_zaporozhye_borodinskaya52') {
																											if ($_GET['type']!='contact_sevastopol') {
																												if ($_GET['type']!='contact_sevastopol_vosstavshuh4') {

Сегодня работал с одним проектом, вот на что наткнулся.

bondar, 05 Июля 2011

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

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

+137

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

int    bufImin[32]={ 4000,  4000,  4000,  4000,  4000,  4000,  4000,  4000,  4000,  4000,  4000,  4000,  4000,  4000,  4000,  4000};
int    bufImax[32]={20000, 20000, 20000, 20000, 20000, 20000, 20000, 20000, 20000, 20000, 20000, 20000, 20000, 20000, 20000, 20000};
double bufPmin[32]={    0,     0,     0,     0,     0,     0,     0,     0,     0,   315,     0,     0,     0,     0,     0,     0};
double bufPmax[32]={    6,    10,     1,    10,    40,    10 ,   10,    10,    10,  -315,     0,     0,   100,   100,    10,  1000};

/*...*/
bufPmin[9]=(double)CR1_Lb1_4mA*5-500.0;
bufPmax[9]=(double)CR1_Lb1_20mA*5-500.0;
Presh[9]=CurrentToPresh(I_9, bufImax[9],bufImin[9],bufPmax[9],bufPmin[9]);

bufPmin[10]=(double)CR1_Lb2_4mA*5-500.0;
bufPmax[10]=(double)CR1_Lb2_20mA*5-500.0;
Presh[10]=CurrentToPresh(I_10, bufImax[10],bufImin[10],bufPmax[10],bufPmin[10]);

/*...*/

bufPmax[10]=(double)CR5_Tg_max;
Presh[15]==CurrentToPresh(I_15, bufImax[15],bufImin[15],bufPmax[15],bufPmin[15]);

Разгребаю г*вно от предыдущего программиста...
- зачем массивы на 32 элемента, если их всегда 16.
- зачем массивы с одинаковыми данными.
- зачем инициализировать массив числами, которые никогда не используются.
- массивы используются только для передачи значений в функцию.
- 17 строка. Это ж надо спутать индекс; как незаметно.
- 18 строка. :) ==3
- "Presh" отнюдь не значит "precious", как подумали б американцы. Это Pressure по своей сути.

MereNonsense, 25 Июня 2011

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

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

+160

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

public function getList($sort) {
        if ($sort == 'id')
            $select = $this->select()->order('id ASC');            
        if ($sort == 'id2')
            $select = $this->select()->order('id DESC');
        if ($sort == 'username')
            $select = $this->select()->order('username ASC');
        if ($sort == 'username2')
            $select = $this->select()->order('username DESC');
        if ($sort == 'firstname')
            $select = $this->select()->order('firstname ASC');
        if ($sort == 'firstname2')
            $select = $this->select()->order('firstname DESC');
        if ($sort == 'lastname')
            $select = $this->select()->order('lastname ASC');
        if ($sort == 'lastname2')
            $select = $this->select()->order('lastname DESC');
        if ($sort == 'lastdate')
            $select = $this->select()->order('lastdate ASC');
        if ($sort == 'lastdate2')
            $select = $this->select()->order('lastdate DESC');
        if ($sort == 'regdate')
            $select = $this->select()->order('regdate ASC');
        if ($sort == 'regdate2')
            $select = $this->select()->order('regdate DESC');
        if ($sort == 'city')
            $select = $this->select()->order('city ASC');
        if ($sort == 'city2')
            $select = $this->select()->order('city DESC');
        if ($sort == 'vote')
            $select = $this->select()->order('vote ASC');
        if ($sort == 'vote2')
            $select = $this->select()->order('vote DESC');
        $result = $this->fetchAll($select);
        return $result;
    }

Кусочек кода, написанный в классической манере "if-else" для Zend Framework

coderov, 20 Июня 2011

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

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