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

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

В номинации:

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

За время:

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

+150

1
2
3

if (is_null($a)) return 'null';
if ($a === false) return 'false';
if ($a === true) return 'true';

Не делайте так. Есть is_bool(), is_null(), or в конце концов.

guest, 02 Марта 2009

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

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

+142

1
2
3

<?php
eval("echo 'Hello, world!'");
?>

гомно =)

guest, 01 Марта 2009

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

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

+156.1

1
2
3
4
5

<?php
define('FLAG', true);
/*немного другого кода*/
while(FLAG){ if(FLAG===true && FLAG!==false) break; }
?>

Ну это надо же!!!
Зачем так писать?

guest, 01 Марта 2009

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

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

+149

1
2
3
4
5
6
7
8
9

//Iterate through temp array to complete the catpath of all categories
    $ready = false;
    while ($ready == false) {
      $ready = true;
      .....
          } else {
            $ready = false;
          }
        }

А как вам такое?

guest, 28 Февраля 2009

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

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

+154

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

function query_safe()
	{
		$time_before = $this->get_real_time();

		if(!$this->connected) $this->connect(DBUSER, DBPASS, DBNAME, DBHOST);
		
		$args = func_get_args();
        $tmpl =& $args[0];
        $tmpl = str_replace("%", "%%", $tmpl);
        $tmpl = str_replace("?", "%s", $tmpl);
        foreach ($args as $i=>$v) {
            if(!$i) continue;
            if(is_int($v)) continue;
            $args[$i] = "'".mysql_escape_string($v)."'";
        }
        for($i=$c=count($args)-1; $i<$c+20; $i++)
            $args[$i+1] = "UNKNOWN_SQL_COMMAND_$i";
        
        $sql = call_user_func_array("sprintf", $args);
        //echo $sql;
        exit;
        
		if(!($this->query_id = mysql_query($sql, $this->db_id) )) {

			$this->mysql_error = mysql_error();
			$this->mysql_error_num = mysql_errno();
			if($show_error) {
				$this->display_error($this->mysql_error, $this->mysql_error_num, $query);
			}
		}
		
		$this->MySQL_time_taken += $this->get_real_time() - $time_before;
		$this->query_num ++;
		return $this->query_id;
	}

Что же тут творится?? :)))

guest, 26 Февраля 2009

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

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

+151.5

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
65
66
67
68
69
70
71
72
73
74

function custom_print ($custom_category, $custom_template, $aviable, $custom_from, $custom_limit, $custom_cache, $do){
  global $db, $is_logged, $member_id, $xf_inited, $cat_info, $config, $user_group, $category_id, $_TIME, $lang;


  $do = $do ? $do : "main";
  $aviable = explode ('|', $aviable);

  if(!(in_array($do, $aviable)) AND ($aviable[0] != "global")) return "";

  $custom_category  = $db->safesql(str_replace(',', '|', $custom_category));
  $custom_from = intval($custom_from);
  $custom_limit = intval($custom_limit);
  $thisdate = date ("Y-m-d H:i:s", (time()+ $config['date_adjust']*60));

  if (intval($config['no_date'])) $where_date = " AND date < '".$thisdate."'"; else $where_date = "";

  $tpl = new dle_template;
  $tpl->dir = TEMPLATE_DIR;

  //if ($custom_cache == "yes") $config['allow_cache'] = "yes"; else $config['allow_cache'] = false;
  if ($is_logged AND ($user_group[$member_id['user_group']]['allow_edit'] AND !$user_group[$member_id['user_group']]['allow_all_edit'])) $config['allow_cache'] = false;

  $content = dle_cache("custom", "cat_".$custom_category."template_".$custom_template."from_".$custom_from."limit_".$custom_limit, true);

  if ($content) { return $content; }
  else {

  $allow_list = explode (',', $user_group[$member_id['user_group']]['allow_cats']);

  if ($allow_list[0] != "all") {

    if ($config['allow_multi_category']) {

      $stop_list = "category regexp '[[:<:]](".implode ('|', $allow_list).")[[:>:]]' AND ";

    } else {

      $stop_list = "category IN ('".implode ("','", $allow_list)."') AND ";

    }

  } else $stop_list = "";

  if ($user_group[$member_id['user_group']]['allow_short']) $stop_list = "";

  if ($cat_info[$custom_category]['news_sort'] != "") $config['news_sort'] = $cat_info[$custom_category]['news_sort'];
  if ($cat_info[$custom_category]['news_msort'] != "") $config['news_msort'] = $cat_info[$custom_category]['news_msort'];

    if ($config['allow_multi_category']) {

      $where_category = "category regexp '[[:<:]](".$custom_category.")[[:>:]]'";

    } else {

      $custom_category = str_replace ("|", "','", $custom_category);
      $where_category = "category IN ('".$custom_category."')";

    }

    $sql_select = "SELECT " . PREFIX . "_post.id, gallery, autor, date," . PREFIX . "_post.image," . PREFIX . "_post.imgtype, short_story, full_story, " . PREFIX . "_post.xfields, title, category, alt_name, " . PREFIX . "_post.comm_num, " . PREFIX . "_post.allow_comm, allow_rate, " . PREFIX . "_post.rating, " . PREFIX . "_post.vote_num, news_read, " . PREFIX . "_post.flag, " . PREFIX . "_users.fullname FROM " . PREFIX . "_post , " . PREFIX . "_users WHERE " . PREFIX . "_post.autor=" . PREFIX . "_users.name and ".$stop_list.$where_category." AND approve = '1'".$where_date." ORDER BY ".$config['news_sort']." ".$config['news_msort']." LIMIT ".$custom_from.",".$custom_limit;
//echo $sql_select;
  include (ENGINE_DIR.'/modules/show.custom.php');

    if ($config['files_allow'] == "yes")
      if ( strpos( $tpl->result['content'], "[attachment=" ) !== false)
      {
        $tpl->result['content'] = show_attach($tpl->result['content'], $attachments);
      }

    create_cache("custom", $tpl->result['content'], "cat_".$custom_category."template_".$custom_template."from_".$custom_from."limit_".$custom_limit, true);

  }
  return $tpl->result['content'];
}

Не пишите так функции, пожалейте того, кто будет ее поддерживать после вас.

guest, 26 Февраля 2009

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

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

+147.9

01
02
03
04
05
06
07
08
09
10

typedef enSceneObjectsManager::tObjects::const_iterator It;
if (objects.size() != 0)
    for (It it1(objects.begin()); it1 != objects.end() - 1; ++it1)
        if ((*it1)->GetObjectType() == enSceneObject3D::GetClassType())
            for (It it2(it1 + 1); it2 != objects.end(); ++it2)
                if (((enSceneObject3D*)(enSceneObject*)(*it1))->GetResource().asInt() == 
                    ((enSceneObject3D*)(enSceneObject*)(*it2))->GetResource().asInt())
                    if ((*it1)->GetTransform() == (*it2)->GetTransform())
                        if (FindObject(theList, (*it2)) == -1)
                            InsertObject(theList, (*it2));

Ответ на http://www.govnokod.ru/613
Сурова Java, но C++ суровее.

guest, 26 Февраля 2009

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

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

+132.2

01
02
03
04
05
06
07
08
09
10

(function($) {
	$(document).ready(function(){
		//Love jQuery :)
		$("#noscript_testimonials").parent().append($('#noscript_testimonials > *').clone().css({"overflow": "hidden", "position": "relative", "width": "250px", "margin-left": "15px"}).find(".noscript_testimonials").each(function (){ $(this).remove();	}).end()).find("#noscript_testimonials").remove().end().find('#testimonials').cycle({
		  fx: 'scrollDown',
		  cleartype: true,
		  pause: true
		});
	});
})(jQuery);

И обфускация не нужна)

guest, 25 Февраля 2009

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

VisualBasic / Говнокод #605

−281.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

Public Function GetFormatedValue(ByVal columnName As String, ByVal value As Object) As String
	Dim result As String = value.ToString()
	Dim vint As Integer
	If (TypeOf value Is Decimal) Then
		vint = CType(CType(value, Decimal), Integer)
	ElseIf (TypeOf value Is Integer) Then
		vint = CType(value, Integer)
	Else
		vint = CType(Decimal.Parse(value.ToString(), System.Globalization.CultureInfo.InvariantCulture), Integer)
	End If

	Dim stringValue = ConvertStringToInt(value)

	Select Case foundedColumn.Format
		Case "3"
			result = String.Format("${0}M", GetSplitValue(CType(CType(stringValue, Integer) / 1000000, Integer)))
		Case "2"
			result = String.Format("${0}K", GetSplitValue(CType(CType(stringValue, Integer) / 1000, Integer)))
		Case "1"
			result = String.Format("${0}", GetSplitValue(vint))
	End Select
	Return result
End Function

Private Function ConvertStringToInt(ByVal value As Object) As String
	Return CType(value, String).Replace(",", String.Empty).Replace(".", String.Empty).Replace(" ", String.Empty)
End Function

Сначало падал exception на 9 строчке, т.к. не была указана культура, вообще.
После того как указали культуру, все, через кейс на 20 строчке, проходило успешно
Далее, когда изменился формат, для правильной конверсии закодили функцию ConvertStringToInt (line 25)

guest, 24 Февраля 2009

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

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