По колено в коде — Говнокод.ру

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

+999

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

#include <iostream>
#include <algorithm>
#include <stdlib.h>

const size_t MB = 1024*1024;
size_t MOD = 0;

unsigned char uniqueNumber () {
  static unsigned char number = 0;
  return ++number % MOD;
}

int main(int argc, char** argv) {
  if (argc < 3) {
    return 1;
  }

  size_t BLOCK_SIZE = atoi(argv[1]) * MB;
  MOD = atoi(argv[2]);

  unsigned char* garbage = (unsigned char *) malloc(BLOCK_SIZE);

  std::generate_n(garbage, BLOCK_SIZE, uniqueNumber);
  std::sort(garbage, garbage + BLOCK_SIZE);

  free(garbage);

  return 0;
}

http://habrahabr.ru/blogs/cpp/138132/

It makes me cry. Понятно, что это всего лишь демонстрационный пример. Но все таки это не оправдание. Итак, начнем по порядку с самого худшего:
1. Сишные malloc/free вперемешку с STL-алгоритмами. WTF? Зачем?
2. Глобальная переменная? Автор не осилил хотя бы bind? Который, к тому же, уже давно std::bind.
3. Uppercase для локальной переменной.
4. Отступ в джва пробела.

invi, 15 Февраля 2012

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

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

+997

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

void TexQ::Move()
{
	mx = quad.v[0].x;
	my = quad.v[0].y;
	if(hge->Input_GetKeyState(HGEK_RBUTTON) || hge->Input_KeyDown(HGEK_LBUTTON))
	{
		hge->Input_GetMousePos(&nx, &ny);
		move = true;
		sx = GetPositionX();
		sy = GetPositionY();
		subx = nx - sx;
		suby = ny - sy;

		if ( abs(subx) > abs(suby) ) 
		{
			iter = abs(subx);
			ms = subx;
			subx = (subx < 0) ? -1.0f : 1.0f;
			suby = (suby < 0 ) ? suby/abs(ms) : abs(suby/ms);
		}
		else 
		{
			iter = abs(suby);
			ms = suby;
			suby = (suby < 0) ? -1.0f : 1.0f;
			subx = (subx < 0 ) ? subx/abs(ms) : abs(subx/ms);
		}
	}

	if (move) 
	{	
		if (iter >= shift)
		{
			iter-=shift;
			this->SetPosition(
				GetPositionX() + subx * shift, 
				GetPositionY() + suby * shift);
		}
		else
		{
			this->SetPosition(nx , ny);
			move = false;
		}

	}

}

Хреновый из меня математик. Этот метод втыкается в функцию фрейминга. И текстура едет по прямой туда, куда мышью тыкнешь.
Позже узнал, как можно тригонометрией в 5 строк написать такую же )

idec, 14 Февраля 2012

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

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

+153

1
2
3
4
5

dData->RowCount++;

         for (int i = 0; i < dData->RowCount - 1; i++)
               for (int j = 0; j < dData->ColumnCount; j++)
                    dData->Rows[i]->Cells[j]->Value = dData->Rows[i + 1]->Cells[j]->Value;

Мучил седня DataGridView, при dData->RowCount++ создавалась новая строчка в таблице, но только сверху. Остальное съезжало вниз на 1 ячейку. Я боролся полчаса с этим, пока не объяснил проблему товарищу. он предложил свой вариант всего кода:
dData->Rows->Add();
:DDDD

idec, 14 Февраля 2012

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

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

+152

1
2
3
4

if ($current['username'] == 'admin')
{
        $admin = true;
}

kafeman, 14 Февраля 2012

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

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

+130

1
2
3

li.sexy-diigo, li.sexy-diigo:hover, li.sexy-digg, li.sexy-digg:hover, li.sexy-reddit, li.sexy-reddit:hover, li.sexy-stumbleupon, li.sexy-stumbleupon:hover, li.sexy-delicious, li.sexy-delicious:hover, li.sexy-yahoobuzz, li.sexy-yahoobuzz:hover, li.sexy-blinklist, li.sexy-blinklist:hover, li.sexy-technorati, li.sexy-technorati:hover, li.sexy-facebook, li.sexy-facebook:hover, li.sexy-twitter, li.sexy-twitter:hover, li.sexy-myspace, li.sexy-myspace:hover, li.sexy-mixx, li.sexy-mixx:hover, li.sexy-scriptstyle, li.sexy-scriptstyle:hover, li.sexy-designfloat, li.sexy-designfloat:hover, li.sexy-mail, li.sexy-mail:hover, li.sexy-comfeed, li.sexy-comfeed:hover, li.sexy-newsvine, li.sexy-newsvine:hover, li.sexy-devmarks, li.sexy-devmarks:hover, li.sexy-linkedin, li.sexy-linkedin:hover, li.sexy-google, li.sexy-google:hover, li.sexy-misterwong, li.sexy-misterwong:hover, li.sexy-izeby, li.sexy-izeby:hover, li.sexy-twittley, li.sexy-twittley:hover, li.sexy-tipd, li.sexy-tipd:hover, li.sexy-pfbuzz, li.sexy-pfbuzz:hover, li.sexy-friendfeed, li.sexy-friendfeed:hover, li.sexy-blogmarks, li.sexy-blogmarks:hover, li.sexy-fwisp, li.sexy-fwisp:hover, li.sexy-designmoo, li.sexy-designmoo:hover, li.sexy-bobrdobr, li.sexy-bobrdobr:hover, li.sexy-yandex, li.sexy-yandex:hover, li.sexy-memoryru, li.sexy-memoryru:hover, li.sexy-100zakladok, li.sexy-100zakladok:hover, li.sexy-moemesto, li.sexy-moemesto:hover, li.sexy-ning, li.sexy-ning:hover, li.sexy-hackernews, li.sexy-hackernews:hover, li.sexy-identica, li.sexy-identica:hover, li.sexy-printfriendly, li.sexy-printfriendly:hover, li.sexy-designbump, li.sexy-designbump:hover, li.sexy-fleck, li.sexy-fleck:hover, li.sexy-netvibes, li.sexy-netvibes:hover, li.sexy-netvouz, li.sexy-netvouz:hover, li.sexy-nujij, li.sexy-nujij:hover, li.sexy-globalgrind, li.sexy-globalgrind:hover, li.sexy-wikio, li.sexy-wikio:hover, li.sexy-xerpi, li.sexy-xerpi:hover, li.sexy-sphinn, li.sexy-sphinn:hover, li.sexy-blogospherenews, li.sexy-blogospherenews:hover, li.sexy-posterous, li.sexy-posterous:hover, li.sexy-techmeme, li.sexy-techmeme:hover, li.sexy-ekudos, li.sexy-ekudos:hover, li.sexy-pingfm, li.sexy-pingfm:hover, li.sexy-tomuse, li.sexy-tomuse:hover {
	background:url('sexy-sprite.png') no-repeat !important;
}

Красивая реализация знатных социальных кнопок... Хотя можно и
ul.socials li {
background:url('sexy-sprite.png') no-repeat;
}

nonamez, 14 Февраля 2012

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

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

+1013

1

if (!done && (done = true)) setlocale(LC_CTYPE, "");

из свежих ворнингов компилера.

Dummy00001, 14 Февраля 2012

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

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

+115

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

try
{
	foreach (MapObjectBase item in objects)
	{
		GisTrack track = item as GisTrack;

		if (track != null)
		{
			//ToolTipContent - наследник TextContent
			ToolTipContent oToolTip = (ToolTipContent)track.ContentList.FirstOrDefault<IContent>(a => a is TextContent);
			if (oToolTip != null)
			{
				track.ContentList.Remove(oToolTip);
			}
		}
	}

	this.map.Redraw(false);
}
catch (Exception ex)
{
	MonitoringManager.Instance.WriteLog(ex.ToString());
}

Linq - как граната. Смотря в чьих руках.

ICELedyanoj, 14 Февраля 2012

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

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

+148

1
2
3
4
5
6
7
8
9

vector<long long> fib;
	for
	(
		long long a=1,b=1;
		b=a+b,a=a+b,a>0&&b>0;
		cout<<b<<" "<<a<<endl,
		fib.push_back(b),
		fib.push_back(a)
	);

Just outputting fibonacci numbers as pairs and storing them in a vector...

wyand, 14 Февраля 2012

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

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

+120

1
2
3
4
5
6
7

public class BillingService : Handles<NewDayHasComeInMoscow>
{
        public void When(NewDayHasComeInMoscow ev)
        {
            // ...
        }
}

pavelhritonenko, 14 Февраля 2012

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

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

+66

1

count = indexFactory.getIndexHelper(section).getSearcher().docFreq(new Term("firm", Long.toString(firm)));

матрешка :)

zlob.jc, 14 Февраля 2012

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

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