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

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

+135

1
2
3

echo "  <h2><a href='index.php'>Выставок нет.</a></h2>";        
echo "<br><p>К сожалению выставок нет.</p>";
echo "<br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br><br>";

пыхапист не осилил CSS

Sulik78, 12 Октября 2011

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

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

+80

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

public class Context {
    public int countSuccess;
    public int countFailed;
    // ....
    public void markSuccess() {
        countSuccess++;
        countFailed--;
    }

    public void markSuccessAll() {
        countSuccess += countFailed;
        countFailed = 0;
    }   
    // ....
}

Вот такая вот супер-абстракция. Пример клиентского кода:

public void processRequest(Context ctx) {

    // ...

    ctx.countFailed = elems.size();

    for (String elem : elems) {

        boolean success = doSomething(elem);

        if (success) {

            ctx.markSuccess();

        }

    }

}

roman-kashitsyn, 12 Октября 2011

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

1C / Говнокод #8171

−134

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

Для Каждого ТекущаяСтрока Из ОтработанноеВремя Цикл
        ТекущаяСтрока.ИтогоЧасов=0;
		ТекущаяСтрока.ИтогоЧасов=ТекущаяСтрока.ПервыйЧасов1+ТекущаяСтрока.ПервыйЧасов2+ТекущаяСтрока.ПервыйЧасов3+
		ТекущаяСтрока.ПервыйЧасов4+ТекущаяСтрока.ПервыйЧасов5+ТекущаяСтрока.ПервыйЧасов6+ТекущаяСтрока.ПервыйЧасов7+
		ТекущаяСтрока.ПервыйЧасов8+ТекущаяСтрока.ПервыйЧасов9+ТекущаяСтрока.ПервыйЧасов10+ТекущаяСтрока.ПервыйЧасов11+
		ТекущаяСтрока.ПервыйЧасов12+ТекущаяСтрока.ПервыйЧасов13+ТекущаяСтрока.ПервыйЧасов14+ТекущаяСтрока.ПервыйЧасов15+
		ТекущаяСтрока.ПервыйЧасов16+ТекущаяСтрока.ПервыйЧасов17+ТекущаяСтрока.ПервыйЧасов18+ТекущаяСтрока.ПервыйЧасов19+
		ТекущаяСтрока.ПервыйЧасов20+ТекущаяСтрока.ПервыйЧасов21+ТекущаяСтрока.ПервыйЧасов22+ТекущаяСтрока.ПервыйЧасов23+
		ТекущаяСтрока.ПервыйЧасов24+ТекущаяСтрока.ПервыйЧасов25+ТекущаяСтрока.ПервыйЧасов26+ТекущаяСтрока.ПервыйЧасов27+
		ТекущаяСтрока.ПервыйЧасов28+ТекущаяСтрока.ПервыйЧасов29+ТекущаяСтрока.ПервыйЧасов30+ТекущаяСтрока.ПервыйЧасов31;	
		
		ТекущаяСтрока.ИтогоОклад=0;
		ТекущаяСтрока.ИтогоОклад= ОКр(ТекущаяСтрока.ИтогоЧасов/ТекущаяСтрока.НормаЧасов*ТекущаяСтрока.Оклад,2);
		
		ТекущаяСтрока.Выручка=0;
		
		Если ТекущаяСтрока.ПервыйЧасов1>0 тогда
			НайденнаяСтрока = РабочаяТаблица.Найти(1, "День");
			Если НайденнаяСтрока = Неопределено Тогда			
			Иначе
				ТекущаяСтрока.Выручка = ТекущаяСтрока.Выручка+НайденнаяСтрока.Продано;
			КонецЕсли;	
		КонецЕсли;	
		Если ТекущаяСтрока.ПервыйЧасов2>0 тогда
			НайденнаяСтрока = РабочаяТаблица.Найти(2, "День");
			Если НайденнаяСтрока = Неопределено Тогда			
			Иначе	
				ТекущаяСтрока.Выручка = ТекущаяСтрока.Выручка+НайденнаяСтрока.Продано;
			КонецЕсли;	
		КонецЕсли;	
		Если ТекущаяСтрока.ПервыйЧасов3>0 тогда
			НайденнаяСтрока = РабочаяТаблица.Найти(3, "День");
			Если НайденнаяСтрока = Неопределено Тогда			
			Иначе
				ТекущаяСтрока.Выручка = ТекущаяСтрока.Выручка+НайденнаяСтрока.Продано;
			КонецЕсли;	
		КонецЕсли;	
		Если ТекущаяСтрока.ПервыйЧасов4>0 тогда
			НайденнаяСтрока = РабочаяТаблица.Найти(4, "День");
			Если НайденнаяСтрока = Неопределено Тогда			
			Иначе
				ТекущаяСтрока.Выручка = ТекущаяСтрока.Выручка+НайденнаяСтрока.Продано;
			КонецЕсли;	
		КонецЕсли;
		Если ТекущаяСтрока.ПервыйЧасов5>0 тогда
			НайденнаяСтрока = РабочаяТаблица.Найти(5, "День");
			Если НайденнаяСтрока = Неопределено Тогда			
			Иначе
				ТекущаяСтрока.Выручка = ТекущаяСтрока.Выручка+НайденнаяСтрока.Продано;
			КонецЕсли;	
		КонецЕсли;	

************************************************************  и так далее :)

Почти 300 строк говнокода превращаются в 10 :D

Для Инд = 1 По 31 Цикл
Если ТекущаяСтрока["ПервыйЧасов" + Инд] > 0 Тогда
ТекущаяСтрока.ИтогоЧасов = ТекущаяСтрока.ИтогоЧасов + ТекущаяСтрока["ПервыйЧасов" + Инд];

НайденнаяСтрока = РабочаяТаблица.Найти(Инд, "День");
Если НайденнаяСтрока <> Неопределено Тогда
ТекущаяСтрока.Выручка = ТекущаяСтрока.Выручка+НайденнаяСтрока.Пр одано;
КонецЕсли;
КонецЕсли;
КонецЦикла;

VikUp, 12 Октября 2011

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

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

+168

01
02
03
04
05
06
07
08
09
10
11

if (!is_array($arFilter)) 
			$filter_keys = Array();
		else
			$filter_keys = array_keys($arFilter);

		for ($i=0; $i<count($filter_keys); $i++)
		{
			$key = strtoupper($filter_keys[$i]);
			$val = $arFilter[$filter_keys[$i]];
// ....
}

по всей видимости, в битриксе программистам выдают тяжелые наркотики при входе.. до такой реализации foreach я бы никогда не додумался!

belukov, 12 Октября 2011

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

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

+67

1
2
3
4

for (char c = '0'; c <= '9'; c++) {
	// personally, I like java better than c or c++
	RANDOM_PASSWORD_CHARS[i++] = c;
}

lucidfox, 12 Октября 2011

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

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

+160

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

<?php
class Html {
	static private $ShowSiblings; // Управляет отображением похожих категорий
	static function Initialize($showSiblings) {
		self::$ShowSiblings = $showSiblings;
	}
	static function SimpleLink($target, $text) { // Простая ссылка
		return '<a href="'.$target.'">'.$text.'</a>';
	}
	static function CategorySiblingLink($targetCategory, $text) { // Формирует ссылку на похожую категорию
		return '<a href="products.php?cat='.$targetCategory.'&page=1">'.$text.'</a>';
	}
	static function CategoryLink($targetCategory, $text) { // Формирует ссылку на указанную категорию
		return '<a href="products.php?cat='.$targetCategory.'&page=1">'.$text.'</a>';
	}
}
?>

Один из классов каталога продукции.

rasufa, 12 Октября 2011

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

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

+113

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

public List<ReducedPayer> red_pay_list
        {
            get
            {
                if (Session["red_pay_list"] == null)
                    Session["red_pay_list"] = new List<ReducedPayer>();

                return (List<ReducedPayer>)Session["red_pay_list"];
            }

            set { Session["red_pay_list"] = value; }
        }

без коментариев

bercerker, 12 Октября 2011

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

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

+162

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

function unzip($location,$newLocation){ 
    if(exec("unzip $location",$arr)) { 
        mkdir($newLocation); 
        for($i = 1;$i< count($arr);$i++) { 
            $file = trim(preg_replace("~inflating: ~","",$arr[$i])); 
            copy($location.'/'.$file,$newLocation.'/'.$file); 
            unlink($location.'/'.$file); 
        } 
        return TRUE; 
    } else { 
        return FALSE; 
    } 
}

оттуда же, http://habrahabr.ru/blogs/php/130196/

тоже непонятно, нафига такие сложности потеха, когда есть стандартное расширение

http://www.php.net/manual/en/function.ziparchive-extractto.php

Lure Of Chaos, 11 Октября 2011

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

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

+163

1
2
3
4
5
6
7

$image = 'image.jpg';  
$ext = getext($image);   
if($ext == ".jpg") {   
    $img = ImageCreateFromJpeg($image);   
} else {   
    echo 'Необходимо использовать JPG';   
}

http://habrahabr.ru/blogs/php/130196/

+12 theaqua11 октября 2011, 22:23#

Без обид, но такое ощущение, что некоторые листинги ошибочно появились тут, хотя им место на govnokod.ru.

окей, берем самые веселые кусочки. но честно, в сабже все функции -- потеха девятиклассника.

в данном случае, непонятно, почему только жпег и зачем столько телодвижений для файла, имя которого известно.

Lure Of Chaos, 11 Октября 2011

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

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