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

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

В номинации:

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

За время:

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

+164

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

for(i = 0;;i++) {
		tmpS[i]=cin.get();
		if(tmpS[i] == '\n') {
			i--;
			break;
		}
		if(!isdigit(tmpS[i]))
			i--;
	}
...
for(k = 0; k < i; k++)
	for(l =0; l < i; l++) {
		tmp=t[l];
		t[l]=t[l+1];
		t[l+1]=tmp;
	}

циклический сдвиг строки. Найдено у себя же.

ReallyBugMeNot, 02 Ноября 2010

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

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

+147

1
2
3
4

for($i = 1; $i <= $num_page; $i++){
      if($Page != $i) echo"<a href=\"/news/p:$i/\">"; else echo"<b>["; echo"$i"; if($Page != $i) echo"</a>"; else echo"]</b>";
      if($i != $num_page) echo"&nbsp;| ";
    }

hardstyler, 26 Октября 2010

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

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

+70

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

Map<Integer, ArrayList<?>> // filterId : filterParams
{
	//
	// Фильтр занятости квартир (не вписывается в API, поэтому здесь заведены отдельные методы)
	//
	
    // хранит: showReserved, DateFrom, DateTo
    private ArrayList<?> reservedFilterParams

    public void onDateFilterChanged(Date dateFrom, Date dateTo, boolean showReserved)
    {
	this.put(QuickFilters.FILTER_RESERVED_ID, reservedFilterParams)
		
	reservedFilterParams = ArrayUtils.getOrCreateArrayListWithElements(
			this, QuickFilters.FILTER_RESERVED_ID, new Boolean(showReserved), dateFrom, dateTo )
	
	if (dateFrom == null && dateTo == null)
        {
            reservedFilterParams.set(0,new Boolean(true))
            this.remove(QuickFilters.FILTER_RESERVED_ID)
        }
    }

    public boolean doShowReserved()
    {
        return reservedFilterParams ? reservedFilterParams.get(0).booleanValue() : true
    }

    public Date getDateFrom()
    {
        return reservedFilterParams?.get(1)
    }

    public Date getDateTo()
    {
        return reservedFilterParams?.get(2)
    }

}

Язык Groovy, стиль - моя переходная форма от Java. Это я сам написал. Вообще мне даже нравится. Есть фильтры для поиска. У каждого фильтра есть только ему ведомые параметры. Приведенный код - это мапка для хранения того, какие пользователь выбрал фильтры и что в них установил.

Есть API который я делаю, состоящий из типовых фильтров. Этот API автоматически юзает мапку через get/put. Но бывает, придумывается какой-то нетиповой фильтр, и его параметры приходится хардкодить в этой мапке. Короче, мне понравилась идея с мапкой, ведь даже если параметры захардкодены, то они всё равно пихаются в мапку. И читаемость для таких случаев навысоте.

Bugdog, 20 Октября 2010

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

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

−85

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

package {
	import flash.utils.setInterval;
	import flash.utils.clearInterval;
	import flash.display.Sprite;
	import flash.display.StageScaleMode;
	import flash.display.StageAlign;

	import pages.loadPage;
	import pages.rokerPage;

	[SWF(width='600', height='690', backgroundColor='#151515', frameRate='30')]

	public class pokerLegalize extends Sprite {
		public var uid:String;
		public var page1:loadPage = new loadPage();
		public var page2:rokerPage = new rokerPage();
		private var loadInterval:uint;
		private var flashVars:Object;

		public function pokerLegalize()
		{
			stage.scaleMode = StageScaleMode.NO_SCALE;
			stage.align = StageAlign.TOP_LEFT;
			addChild(page1);
			loadInterval = setInterval(loadComplite, 100);
		}

		private function loadComplite():void
		{
			if(page1.getLoadStatus()) {
				clearInterval(loadInterval);
				flashVars = stage.loaderInfo.parameters as Object;
							
				removeChild(page1);
				addChild(page2);
				pageResezer(2);
			}
		}

		private function pageResezer(page:uint):void
		{
			if(page == 2) {
				page2.wall.wall.wallArray = page1.getVars('wallArray');
				page2.friendsWall.friendsWall.friendsFull = page1.getVars('friendsArray');
				
				page2.friendsWall.friendsWall.createWall();
				page2.friendsWall.resize();
				
				page2.wall.wall.lengthXML = page1.getterBrick.getLenght();
				page2.wall.wall.setBrokenMode(true, false, 1);
				page2.wall.setFlashVars(flashVars);
				page2.wall.addAllEventListeners();		
				page2.flashVars = flashVars;
			}
		}
	}
}

По ТЗ, мне надо разобраться в этом коде... Внимание на функцию pageResezer... хотя интервал тоже не плох @_@

KirAmp, 20 Октября 2010

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

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

+124

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

namespace Test.Data
{
    public class Connection
    {
        public static SqlConnection getConnection()
        {
            string server = "";

            if (!File.Exists(@"C:\ConnectionParameters.ini"))
                LogFileAdministration.WriteMessage("The ConnectionParameters.ini file does not exist.");

            else
            {
                StreamReader reader = new StreamReader(@"C:\ConnectionParameters.ini");
                server = reader.ReadLine();
                reader.Close();
            }

            return new SqlConnection(@"Data Source= " + server + "; Integrated Security=SSPI; Initial Catalog=Test");
        }
    }
}

Итальянский программист. Из Италии. Внезапно попросился на работу. Отослал резюме. Дали ему тестовое задание - намутить поиск по табличке, по ключевой фразе с пейджингом.
Что из этого вышло - смотрим улыбаемся.

ursus, 18 Октября 2010

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

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

+165

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

function FormatCurrency($fSum, $strCurrency)
{
	return CurrencyFormat($fSum, $strCurrency);
	/*
	if (!isset($fSum) || strlen($fSum)<=0) return "";

	$arCurFormat = CCurrencyLang::GetCurrencyFormat($strCurrency);

	if (!isset($arCurFormat["DECIMALS"])) $arCurFormat["DECIMALS"] = 2;
	$arCurFormat["DECIMALS"] = IntVal($arCurFormat["DECIMALS"]);
	if (!isset($arCurFormat["DEC_POINT"])) $arCurFormat["DEC_POINT"] = ".";
	if (!isset($arCurFormat["THOUSANDS_SEP"])) $arCurFormat["THOUSANDS_SEP"] = "\\"."xA0";
	$tmpTHOUSANDS_SEP = $arCurFormat["THOUSANDS_SEP"];
	eval("\$tmpTHOUSANDS_SEP = \"$tmpTHOUSANDS_SEP\";");
	$arCurFormat["THOUSANDS_SEP"] = $tmpTHOUSANDS_SEP;
	if (!isset($arCurFormat["FORMAT_STRING"])) $arCurFormat["FORMAT_STRING"] = "#";

	$num = number_format($fSum, $arCurFormat["DECIMALS"], $arCurFormat["DEC_POINT"], $arCurFormat["THOUSANDS_SEP"]);

	return str_replace("#", $num, $arCurFormat["FORMAT_STRING"]);
	*/
}

1C-Bitrix,
/bitrix/modules/catalog/include.php

Under, 12 Октября 2010

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

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