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

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

+173

1
2
3
4
5

if(isset($_GET['catId'])){
...
} else if($_GET['catId']>0){
...
}

CubeCart v3.0.15

Ghost, 24 Сентября 2009

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

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

+51.5

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

void CNewAnketaOrg::OnSeachOrg() 
{
	if(m_SeachOrg.GetCheck())
	{
		m_Create.SetCheck(FALSE);
		m_SeachOur.SetCheck(FALSE);
		m_SeachFZ.SetCheck(FALSE);
		CFindOrganization find(this);
		if (find.DoModal() == IDOK)
		{
			...
		}
		...
	}
}

Угадайте какое сообщение привязано к методу с вызовом формы.

Criperum, 24 Сентября 2009

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

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

+161.3

01
02
03
04
05
06
07
08
09
10

$proc = new XsltProcessor();
$proc->importStylesheet($xsl);
$html = $proc->transformToXML($xmlOut);

// Добро пожаловать в реальный мир!
// Мы не знаем XSLT, поэтому кое-что поправим прямо тут.
// Да здравствуют регекспы и грязные хаки!

$html = preg_replace("/(<a href=\"\/futures[^>]+>)(.+?)<\/a>/ims","\$2",$html);
return $html;

Кусок класса XSLTTransformer.
Оригинальные комменты и час потраченого времени на поиски неработающих ссылок в xsl-шаблонах.

VermiVermi, 24 Сентября 2009

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

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

+66.8

1

bind "attack" kill

Классика консоли CS

Сан Саныч, 24 Сентября 2009

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

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

+159.6

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

class SomeClass {
...
	public function get_stop_words(){
	    $stem_stop_words = array();
		....
	    return $stem_stop_words;
	}
...
}

//далее в коде:

$someClass = new SomeClass();
if($someClass->connect()){
    $someClass->stem_stop_words = $someClass->get_stop_words();
    ....
}

ООП не для нас. Причем $this->stem_stop_words не объявлена в классе.

VermiVermi, 23 Сентября 2009

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

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

+65

1
2
3
4
5
6

if (s.length() < 2)
            return false;
if (s.charAt(1) == ':') {
            return true;
}
return false;

lian, 23 Сентября 2009

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

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

+155.4

1
2
3
4
5

function strtolowerex($str)
{
	$str = strtr($str, "АБВГДЕЁЖЗИЙКЛМНОПРСТУФХЦЧШЩЪЫЬЭЮЯ", "абвгдеёжзийклмнопрстуфхцчшщъыьэюя");
	return strtolower($str);
}

я может чего-то не понимаю? 0_о

gorky, 23 Сентября 2009

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

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

+158.8

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

$fStr = fgets($fp);
if ($k==0 && $IsFirst)
continue;
$arLine = explode(";", $fStr);
$type = intval($arLine[0]);
$articul = trim($arLine[1]);
$section_id = intval($arLine[2]);
$model = trim($arLine[3]);
$name = trim($arLine[4]);
$brand = trim($arLine[5]);
$width = trim($arLine[6]);
$radius = trim($arLine[7]);
$pcd = trim($arLine[8]);
$et = trim($arLine[9]);
$dia = trim($arLine[10]);
$dop = trim($arLine[11]);
$color = trim($arLine[12]);

//и чуть позже

$query = "INSERT INTO ".$dbCatalogElProp." VALUES (null, ".$element_id.", 38, 'S', '".$articul."', null, '')";
$res = SQLResult($query);
$query = "INSERT INTO ".$dbCatalogElProp." VALUES (null, ".$element_id.", 25, 'L', '', ".$type.", '')";
$res = SQLResult($query);
$query = "INSERT INTO ".$dbCatalogElProp." VALUES (null, ".$element_id.", 17, 'S', '".$width."', null, '')";
$res = SQLResult($query);
$query = "INSERT INTO ".$dbCatalogElProp." VALUES (null, ".$element_id.", 18, 'S', '".$radius."', null, '')";
$res = SQLResult($query);
$query = "INSERT INTO ".$dbCatalogElProp." VALUES (null, ".$element_id.", 19, 'S', '".$pcd."', null, '')";
$res = SQLResult($query);
$query = "INSERT INTO ".$dbCatalogElProp." VALUES (null, ".$element_id.", 20, 'S', '".$et."', null, '')";
$res = SQLResult($query);
$query = "INSERT INTO ".$dbCatalogElProp." VALUES (null, ".$element_id.", 45, 'S', '".$dia."', null, '')";
$res = SQLResult($query);
$query = "INSERT INTO ".$dbCatalogElProp." VALUES (null, ".$element_id.", 21, 'S', '".$color."', null, '')";
$res = SQLResult($query);
$query = "INSERT INTO ".$dbCatalogElProp." VALUES (null, ".$element_id.", 22, 'S', '0', null, '')";
$res = SQLResult($query);
$query = "INSERT INTO ".$dbCatalogElProp." VALUES (null, ".$element_id.", 23, 'S', '', null, '')";
$res = SQLResult($query);

gorky, 23 Сентября 2009

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

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

+60.3

1
2
3
4
5

void AcquireSomething(..., boost::shared_ptr<Something> out)
{
  ...
  out.reset(new Something());
}

жесть

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

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

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

+73.7

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

public Object save(Object entity) {
        final EntityManager em = getEntityManager();
        try {
            final EntityTransaction et = em.getTransaction();
            try {
                et.begin();
                em.persist(entity);
                et.commit();
            } finally {
                if (et != null && et.isActive()) {
                    entity = null;
                    et.rollback();
                }
            }
        } finally {
            if (em != null && em.isOpen()) {
                em.close();
            }
        }
        return entity;
    }

Простое и понятное управление транзакциями придумали трусы!
А rollback в секции finally, и вложенные try-catch секции -- вообще модная сейчас фишка.

rh.mbox, 23 Сентября 2009

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

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

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

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

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

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

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

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

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

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

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

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