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

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

+162.5

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

//было:

  if(this.focusCount == this.blurCount + 1)
   {
     this.counter++;
   }

//стало:

   if(this.focusCount == this.blurCount + 1)
   {
     this.counter++;
   }
   else this.counter++;

Надо было из функции убрать проверку условия.

psychos, 17 Сентября 2009

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

Perl / Говнокод #1854

−91.4

01
02
03
04
05
06
07
08
09
10

sub inet_aton{
  my $n_ip;
  $sth1 = $dbh->prepare("SELECT inet_aton(\"$_[0]\") as inet_aton");
  $sth1->execute();
  while (my $ips = $sth1->fetchrow_hashref()) {
    $n_ip = "$ips->{'inet_aton'}";
  };
  $sth1->finish();
  return $n_ip;
};

вот так в одном коммерческом продукте получают ip-адрес в виде int'а

Mihard, 17 Сентября 2009

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

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

+149.8

1
2
3
4
5
6
7
8
9

function getOffset(elem) {
    if (elem.getBoundingClientRect) {
        // "правильный" вариант
        return getOffsetRect(elem)
    } else {
        // пусть работает хоть как-то
        return getOffsetSum(elem)
    }
}

code_master, 16 Сентября 2009

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

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

+55.5

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

class cCar: public GAMEPLAYER::cPlayerRef, public cScrObject,
	public IspSetOnParkingPlace<GAMEPLAYER::cCar>, 
	public IspCheckLoadedState<GAMEPLAYER::cCar>,
//	public IspUseInitialPositioning<GAMEPLAYER::cCar>,
	public IspSetVelocity<GAMEPLAYER::cCar>,
	public IspTracetoPosition<GAMEPLAYER::cCar>,
	public IspChangeVehicle<GAMEPLAYER::cCar>,
	public IspChasePlayer<GAMEPLAYER::cCar>,
	public IspRaceParamsPlayer<GAMEPLAYER::cCar>,
	public IcallbackParked<GAMEPLAYER::cCar>,
	public ALTERNATIVEK::cstore_this<GAMEPLAYER::cCar>
{ //... дальше не интересно

Вот такой суровый cCar :)

generalgda, 16 Сентября 2009

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

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

+52.9

1
2
3
4
5

void SomeFunction(SomeClass* cls)
{
    std::auto_ptr<SomeClass> tmp(cls);
    SomeObject.SomeMethod(tmp.release());
}

несколько раз видел такое в разных вариациях (поэтому вместо копи-пасты - абстракция).
смысл сей конструкции упорно ускользает от меня :)

g26g, 16 Сентября 2009

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

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

+52.3

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

void GAMEPLAYER::cCar::abandonCar(GAMEPLAYER::cPlayer* player){
	for(unsigned int i=0;i<size();++i){
		cBaseAnimManager * bam = get(i);
		if(!bam->isCarRider())
			continue;
		cCarRider* rdr = (cCarRider*)bam;
		if(rdr->gScript()==player->gScript()){
			delete rdr;
			storage.erase(i);
			--i;
		}
	}
}

НЕ time-critical код из некоторой игры
Наблюдаем:
1) непониманием того, зачем нужно наследование
2) плохо пахнущие имена переменных

generalgda, 16 Сентября 2009

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

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

+129.3

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

// IPrincipal Implementation
public bool IsInRole(string role)
{
  return Array.BinarySearch( _roles, role ) >=0 ? true : false;
}
public IIdentity Identity
{
  get
  {
    return _identity;
  }
}

Взято из Building Secure ASP.NET Applications: Authentication, Authorization, and Secure Communication (http://msdn.microsoft.com/en-us/library/aa302401.aspx)

Step 4. Create a Class that Implements and Extends IPrincipal
6. Implement the IsInRole method and Identity property defined by the IPrincipal interface as shown below.

mikle.smirnov, 16 Сентября 2009

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

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

+147.7

1

$this->view->isAdmin = ($auth->getIdentity()->role_id == 1) ? true : false;

Называется: "заработался".
Написал и только потом дошло.

oldfornit, 16 Сентября 2009

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

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

+185.3

1

echo "Время генерации: 0.0".rand(10,99)." сек.";

Без комментариев))) Я под столом))

greevex, 16 Сентября 2009

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

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

+60.3

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

void CExeTuner::ResFill2Tree(uchar *baddr, uchar *addr, TRes *now, int level)
{
	int len;
	now->size1=*(uword *)(addr+0x0C); now->size2=*(uword *)(addr+0x0E);
	if (now->size1+now->size2) now->mas=new TRes * [(now->size1+now->size2)];
	ulong long1,long2;
	for (int i=0; i<(now->size1+now->size2); i++)
	{
		now->mas[i]=new TRes;
		now->mas[i]->back=now;
		now->mas[i]->name=new char [32];
		now->mas[i]->mas=NULL; now->mas[i]->data=NULL;
		long1=*(ulong *)(addr+0x10+(i<<3));
		long2=*(ulong *)(addr+0x14+(i<<3));
		if (long1&0x80000000) {
			long1&=0x7FFFFFFF;
			now->mas[i]->isname=true;
			len=*(uword *)(baddr+long1);
			now->mas[i]->rname1=new uchar [len*2+2+1];
				now->mas[i]->rname1[len*2+2]=0;
				memcpy(now->mas[i]->rname1,baddr+long1,len*2+2);
			now->mas[i]->name=new char [len+1];
				now->mas[i]->name[len]=0;
				for (int j=0; j<=len; j++)
				now->mas[i]->name[j]=now->mas[i]->rname1[(j+1)*2];
		} else {
			now->mas[i]->isname=false;
			now->mas[i]->rname2=long1;
			if (!ResNumToStr(long1,now->mas[i]->name,level)) _itoa(long1,now->mas[i]->name,10);
		}
		if (long2&0x80000000) {
			long2&=0x7FFFFFFF;
			now->mas[i]->isdir=true;
			ResFill2Tree(baddr, baddr+long2, now->mas[i], level+1);
		} else {
			now->mas[i]->isdir=false;
			now->mas[i]->size1=*(ulong *)(baddr+long2+4);
			now->mas[i]->data=new uchar [now->mas[i]->size1];
			if ((*(ulong *)(baddr+long2)+now->mas[i]->size1)>(*ExeMain.ResDirAddrOBJ+*ObjMas[ExeMain.ResPosInObj].size))
			{ now->mas[i]->data=NULL; now->mas[i]->size1=0;
			} else memcpy(now->mas[i]->data,baddr-*ExeMain.ResDirAddrOBJ+*(ulong *)(baddr+long2),now->mas[i]->size1);
		}
	}
}

Древнющий проект откопал))
Метод извлекает все ресурсы из PE файла в структуру типа дерево))

k06a, 15 Сентября 2009

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

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