Лучший говнокод за всё время

Лучший говнокод

В номинации:

Лучший рейтинг
Самый комментируемый

За время:

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

+126

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

public static bool GetSafeBool(object val, bool defaultVal)
        {
            //TODO: check functionality
            bool result = defaultVal;
            try
            {
                if (val != null)
                {
                    string str = val.ToString().Trim();
                    // compare ignore case, for performance
                    result = (0 == string.Compare(str, true.ToString(), true) || str == "1" || str == "-1");
                }
            }
            catch { }
            return result;
        }

Продолжая тему расовых индусов...

fade, 27 Сентября 2011

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

ActionScript / Говнокод #7966

−115

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

package aerys.minko.scene.node.group
{
	...
	public class LoaderGroup extends Group implements IEventDispatcher
	{
		...
		public static function loadBytes(bytes : ByteArray, parserOptions : ParserOptions = null) : LoaderGroup
		{
			return new LoaderGroup().loadBytes(bytes, parserOptions);
		}
		...
		public function loadBytes(bytes : ByteArray, parserOptions : ParserOptions = null) : LoaderGroup
		{
			...

minko, конечно, интересный 3д движок с нестандартными решениями, но вот такие выебоны вгоняют в ступор. я даже не знал, что такое компилится.

makc3d, 24 Сентября 2011

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

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

+119

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
54
55
56
57
58
59
60
61
62
63
64

// создаём источник для репитера

private DataTable EventsDataTable
        {
            get
            {
                DataTable dt = new DataTable();
                dt.Columns.Add(
                    new DataColumn("ID", typeof(string)));
                dt.Columns.Add(
                    new DataColumn("day", typeof(string)));
                dt.Columns.Add(
                    new DataColumn("date", typeof(DateTime)));
                dt.Columns.Add(
                    new DataColumn("title", typeof(string)));
                dt.Columns.Add(
                    new DataColumn("url", typeof(string)));
                dt.Columns.Add(
                    new DataColumn("description", typeof(string)));
                dt.Columns.Add(
                    new DataColumn("location", typeof(string)));
                dt.Columns.Add(
                    new DataColumn("place", typeof(string)));
                dt.Columns.Add(
                    new DataColumn("FileDirRef", typeof(string)));
                // Добавляем строчки
                foreach (EventInfo ei in CalendarEvents)
                {
                    DataRow dr = dt.NewRow();
                    dr["day"] = ei.EventDate.Date.ToString("ddMMyyyy");
                    dr["date"] = ei.EventDate;
                    dr["title"] = ei.Title;
                    dr["location"] = ei.Location;
                    dr["ID"] = ei.ID;
                    dr["FileDirRef"] = ei.FileDirRef;
                    dt.Rows.Add(dr);
                }
                return dt;
            }
        }


protected void repeaterItemDataBound(object sender, RepeaterItemEventArgs e)
{
if (e != null
                    && e.Item != null
                    && e.Item.DataItem != null
                    && e.Item.DataItem is DataRow)
                {
                    DataRow dataItem = (DataRow)e.Item.DataItem;

                    Label date = (Label)(e.Item.FindControl("date"));
                    date.Text = 
                        dataItem["date"] != null
                        ? Convert.ToDateTime(dataItem["date"].ToString()).ToString()
                        : Convert.ToDateTime(dataItem["Created"].ToString()).ToString();
                    date.Text = date.Text.Substring(0, date.Text.Length - 3);

                    HyperLink title = (HyperLink)(e.Item.FindControl("title"));
                    title.Text = dataItem["title"].ToString();
                    Label location = (Label)(e.Item.FindControl("location"));
                    location.Text = "Расположение: " + dataItem["location"].ToString();
                }
}

Современный способ привязки данных в asp.net Repeater

Gnet, 22 Июня 2011

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

Си / Говнокод #6973

+142

1
2
3
4
5
6

case 43: /* + */
    sum_up += sum_up > 1 ? 0 : 1;
    return 1;
case 45: /* - */
    sum_up -= sum_up < 1 ? 0 : 1;
    return 1;

Если выше/ниже лимита то добавляем/отнимаем ничего.

ЗЫ sum_up это переменная контролирующая уровень вербозности (0,1,2) специальной мониторной проги. кнопками +/- можно интерактивно контролировать. К слову, сверху это уже правленый код. Оригинальный оригинал:

if (c == 43) { sum_up = sum_up > 1 ? sum_up : ++sum_up; return 1; }

if (c == 45) { sum_up = sum_up < 1 ? sum_up : --sum_up; return 1; }

Dummy00001, 16 Июня 2011

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

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

+83

01
02
03
04
05
06
07
08
09
10

try
		{
			nIndex = Integer.parseInt(sRowIndex.trim());
		}
		catch(NumberFormatException e)
		{
			//异常处理
			nIndex = -2;
		}
		if(nIndex < 0) return;

Китайцы продолжают отжигать.

lucidfox, 14 Июня 2011

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

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

+167

1

$r = $this->client->getBerechneteGrundversorgungsTarifebyPLZundVerbrauchKundenart($this->params);

По сути не говнокод, но нечитабельность налицо...

vov4ik, 13 Июня 2011

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

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

+158

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
54
55
56
57
58
59

public function getRelatedDataProvider($relationName,$config=array(),$params=array()){
		
		$md = $this->getMetaData();
		if(!isset($md->relations[$relationName])){
			throw new CDbException(Yii::t('yii','{class} does not have relation "{name}".',
				array('{class}'=>get_class($this), '{name}'=>$relationName)));
		}
		
		$relation = $md->relations[$relationName];
		if(!($relation instanceof CHasManyRelation || $relation instanceof CManyManyRelation)){
			throw new CDbException(Yii::t('yii','Связь {class}.{name} имеет тип, отличный от Has_Many и Many_Many.',
				array('{class}'=>get_class($this), '{name}'=>$relationName)));
		}
		
		// return new CArrayDataProvider($this->getRelated($relationName,true,$params), $config);
		
		var_dump($relation);
		
		// копируем критерии из связи
		$criteria = new CDbCriteria;
		foreach(array('select', 'condition', 'params', 'limit', 'offset', 'order', 'group', 'join', 'having', 'with', 'alias', 'index', 'together') as $param){
			$criteria->$param = $relation->$param;
		}
		
		// склеиваем параметры с критерией
		$criteria->mergeWith(
			is_string($params)
				? array('condition' => $params)
				: $params
		);
		
		// добавляем связь через fk
		// ВОТ ТУТ Я НИЧЕГО НЕ ПОНИМАЮ :)
		$thisTable = $this->getTableAlias();
		$childModel = CActiveRecord::model($relation->className);
		$childMd = $childModel->getMetaData();
		$childTable = empty($relation->alias) ? $childModel->getTableAlias() : $relation->alias;
		//$criteria->join .= " {$relation->joinType} {$thisTable} ON {$thisTable}.{$relation->foreignKey}={$childTable}.{$childMd->tableSchema->primaryKey}";
		$criteria->params[':fkrel'] = $this->{$relation->foreignKey};
		$criteria->addCondition("{$childTable}.{$relation->foreignKey}=:fkrel");
		
		
		// собираем кофиг для CActiveDataProvider
		$dpConfig = array(
			'criteria' => $criteria,
		) + $config;
		
		echo "CActiveDataProvider( {$relation->className} , criteria: ".var_export($dpConfig['criteria']->toArray(),true)." )";
		
		// 
		$dataProvider = new CActiveDataProvider(
			$relation->className,
			$dpConfig
		);
		
		return $dataProvider;
	}
	
}

res-nullis, 26 Мая 2011

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

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

+128

1

long.Parse(Convert.ToString(Convert.ToSingle(Item.Value.ToString())))

wiz, 17 Мая 2011

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

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

+162

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

public static function create_folder ( $path, $rights = self::default_rights )
	{
		$result = true;
		// Получаем список папок
		$folders = explode( DIRECTORY_SEPARATOR, trim( self::get_absolute_path( $path ), DIRECTORY_SEPARATOR ) );
		$exists = true;
		// Ищем существующую папку
		while ( $exists && sizeof( $folders ) )
		{
			$exists_folder[] = array_shift( $folders );
			$exists = realpath( implode( DIRECTORY_SEPARATOR, $exists_folder ) );
		}
		if ( !file_exists( implode( DIRECTORY_SEPARATOR, $exists_folder ) ) )
		{
			array_unshift( $folders, array_pop( $exists_folder ) );
		}
		$parent = implode( DIRECTORY_SEPARATOR, $exists_folder );
		unset( $exists_folder );
		// поочередно создаем папки
		foreach ( $folders as & $folder )
		{
			$parent .= DIRECTORY_SEPARATOR . $folder;
			$result = mkdir( $parent, $rights );
		}
		clearstatcache( );
		return $result;
	}

Вот что значит не знать про необязательный аргумент recursive у mkdir...

GhOsTMZ, 14 Февраля 2011

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

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