1. C# / Говнокод #1859

    +132.2

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    cboRole.DataSource = Controller.GetViewForRole();
cboRole.DataBind();

try {
   cboRole.SelectedValue = roleId.ToString();
} catch (ArgumentOutOfRangeException) {
   cboRole.Items.Add(new ListItem(roleName, roleId.ToString()));
   cboRole.SelectedValue = roleId.ToString();
}

    Нету значения - не проблема, создадим и присвоим. Не беда, что инициализация рядом...

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

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

  2. C++ / Говнокод #1858

    +59.9

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    void someMethod(const std::string& name,  std::autp_ptr<SomeClass> p);

{
    ...
    std::auto_ptr<SomeClass> p(new SomeClass());
    someMethod(p->GetName(), p);
    ...
}

    Самое забавное, что при определенных условиях оно может работать.

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

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

  3. Java / Говнокод #1857

    +71.9

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    if (!selectAndAdd(select, administrators,
                                    userUid)) //shouldn't be so but...
                            {
                                throw new IllegalStateException(
                                        "World is full of mystical things. That is one of them");
                            }

    Мир полон таинственных вещей! :-)

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

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

  4. PHP / Говнокод #1856

    +160.6

    1. 1
    2. 2
    3. 3
    $query=mysql_query("SELECT * FROM `PS_card_list` WHERE `id_categ`='".$_GET['point']."';");

while ($row=mysql_fetch_array($query)) $card_on_plan++;

    Наследие военных разработчиков

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

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

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

    +162.5

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    //было:

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

//стало:

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

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

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

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

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

    −91.4

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 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)

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

    +149.8

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    function getOffset(elem) {
    if (elem.getBoundingClientRect) {
        // "правильный" вариант
        return getOffsetRect(elem)
    } else {
        // пусть работает хоть как-то
        return getOffsetSum(elem)
    }
}

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

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

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

    +55.5

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 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)

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

    +52.9

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

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

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

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

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

    +52.3

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 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)