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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

1. PHP (5712)
2. C++ (2746)
3. Куча (2411)
4. JavaScript (2035)
5. C# (1930)
6. Java (1503)
7. Си (1122)
8. Pascal (648)
9. Python (592)
10. 1C (541)
11. SQL (433)
12. ActionScript (292)
13. Objective C (202)
14. bash (201)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (46)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

   • за сегодня
   • за неделю
   • за месяц
   • за всё время

2. PHP / Говнокод #8411

   +165

   1. 1
   2. 2

   mysql_query("DELETE FROM tblTokens WHERE intRestaurantID = $id AND cToken = '$token';");
   mysql_query("INSERT INTO tblTokens (intRestaurantID, cToken) VALUES ($id, '$token');");

   Увеличиваем id, наверное.

   1101_debian, 03 Ноября 2011

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

3. ActionScript / Говнокод #8395

   −112

   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
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19
   20. 20
   21. 21
   22. 22
   23. 23
   24. 24
   25. 25
   26. 26
   27. 27
   28. 28
   29. 29
   30. 30
   31. 31
   32. 32
   33. 33
   34. 34
   35. 35
   36. 36
   37. 37
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47
   48. 48
   49. 49
   50. 50

   _friend_.D2.mc_LOJKA.gotoAndStop(_friend_.D2.mc_LOJKA._totalframes) 
     if (FR_GETKEY(64) == "A") {_friend_.D2.mc_LOJKA.gotoAndStop(1)}
     if (FR_GETKEY(352) == "A") {_friend_.D2.mc_LOJKA.gotoAndStop(2)}
     if (FR_GETKEY(356) == "A") {_friend_.D2.mc_LOJKA.gotoAndStop(3)}
     if (FR_GETKEY(363) == "A") { _friend_.D2.mc_LOJKA.gotoAndStop(4) }
     if (FR_GETKEY(353) == "A") {_friend_.D2.mc_LOJKA.gotoAndStop(4)}
    _friend_.D2.mc_TARELKI.gotoAndStop(_friend_.D2.mc_TARELKI._totalframes) 
     if (FR_GETKEY(5) == "A") {_friend_.D2.mc_TARELKI.gotoAndStop(1)}
     if (FR_GETKEY(4) == "A") {_friend_.D2.mc_TARELKI.gotoAndStop(2)}
     if (FR_GETKEY(3) == "A") {_friend_.D2.mc_TARELKI.gotoAndStop(3)}
    _friend_.D4.mc_BUTILKA.gotoAndStop(_friend_.D4.mc_BUTILKA._totalframes) 
     if (FR_GETKEY(42) == "A") {_friend_.D4.mc_BUTILKA.gotoAndStop(1)}
     if (FR_GETKEY(43) == "A") {_friend_.D4.mc_BUTILKA.gotoAndStop(2)}
     if (FR_GETKEY(44) == "A") {_friend_.D4.mc_BUTILKA.gotoAndStop(3)}
    _friend_.D4.mc_NEPROLIVAIKA.gotoAndStop(_friend_.D4.mc_NEPROLIVAIKA._totalframes) 
     if (FR_GETKEY(357) == "A") {_friend_.D4.mc_NEPROLIVAIKA.gotoAndStop(1)}
     if (FR_GETKEY(26) == "A") {_friend_.D4.mc_NEPROLIVAIKA.gotoAndStop(2)}
    _friend_.D4.mc_SOSKA.gotoAndStop(_friend_.D4.mc_SOSKA._totalframes) 
     if (FR_GETKEY(40) == "A") {_friend_.D4.mc_SOSKA.gotoAndStop(1)}
     if (FR_GETKEY(38) == "A") {_friend_.D4.mc_SOSKA.gotoAndStop(2)}
     if (FR_GETKEY(37) == "A") {_friend_.D4.mc_SOSKA.gotoAndStop(3)}
    _friend_.D4.mc_ERSHIK.gotoAndStop(_friend_.D4.mc_ERSHIK._totalframes) 
     if (FR_GETKEY(121) == "A") {_friend_.D4.mc_ERSHIK.gotoAndStop(1)}
    _friend_.D5.mc_NAGRUDNIK.gotoAndStop(_friend_.D5.mc_NAGRUDNIK._totalframes) 
     if (FR_GETKEY(362) == "A") {_friend_.D5.mc_NAGRUDNIK.gotoAndStop(1)}
    _friend_.D5.mc_TERMOS.gotoAndStop(_friend_.D5.mc_TERMOS._totalframes) 
     if (FR_GETKEY(46) == "A") {_friend_.D5.mc_TERMOS.gotoAndStop(1)}
    _friend_.D7.F47.gotoAndStop(_friend_.D7.F47._totalframes) 
     if (FR_GETKEY(47) * 1 > 0) { _friend_.D7.F47.gotoAndStop(1); _friend_.D7.F47.NUM.gotoAndStop(FR_GETKEY(47)*1)}
    _friend_.D7.F50.gotoAndStop(_friend_.D7.F50._totalframes) 
     if (FR_GETKEY(50) * 1 > 0) { _friend_.D7.F50.gotoAndStop(1);_friend_.D7.F50.NUM.gotoAndStop(FR_GETKEY(50)*1)}
    _friend_.D7.F49.gotoAndStop(_friend_.D7.F49._totalframes) 
     if (FR_GETKEY(49) * 1 > 0) { _friend_.D7.F49.gotoAndStop(1);_friend_.D7.F49.NUM.gotoAndStop(FR_GETKEY(49)*1)}
    _friend_.D7.F48.gotoAndStop(_friend_.D7.F48._totalframes) 
     if (FR_GETKEY(48) * 1 > 0) { _friend_.D7.F48.gotoAndStop(1); _friend_.D7.F48.NUM.gotoAndStop(FR_GETKEY(48)*1)}
    _friend_.D8.F17.gotoAndStop(_friend_.D8.F17._totalframes) 
     if (FR_GETKEY(17) * 1 > 0) { _friend_.D8.F17.gotoAndStop(1);_friend_.D8.F17.NUM.gotoAndStop(FR_GETKEY(17) * 1)}
    _friend_.D8.F61.gotoAndStop(_friend_.D8.F61._totalframes) 
     if (FR_GETKEY(61) * 1 > 0) { _friend_.D8.F61.gotoAndStop(1);_friend_.D8.F61.NUM.gotoAndStop(FR_GETKEY(61)*1)}
    _friend_.D8.F62.gotoAndStop(_friend_.D8.F62._totalframes) 
     if (FR_GETKEY(62) * 1 > 0) { _friend_.D8.F62.gotoAndStop(1);_friend_.D8.F62.NUM.gotoAndStop(FR_GETKEY(62)*1)}
    _friend_.D8.F20.gotoAndStop(_friend_.D8.F20._totalframes) 
     if (FR_GETKEY(20) * 1 > 0) { _friend_.D8.F20.gotoAndStop(1);_friend_.D8.F20.NUM.gotoAndStop(FR_GETKEY(20)*1)}
    _friend_.D8.F19.gotoAndStop(_friend_.D8.F19._totalframes) 
     if (FR_GETKEY(19) * 1 > 0) { _friend_.D8.F19.gotoAndStop(1);_friend_.D8.F19.NUM.gotoAndStop(FR_GETKEY(19)*1)}
    _friend_.D8.F18.gotoAndStop(_friend_.D8.F18._totalframes) 
     if (FR_GETKEY(18) * 1 > 0) { _friend_.D8.F18.gotoAndStop(1);_friend_.D8.F18.NUM.gotoAndStop(FR_GETKEY(18)*1)}
    _friend_.D9.F25.gotoAndStop(_friend_.D9.F25._totalframes) 
     if (FR_GETKEY(25) * 1 > 0) { _friend_.D9.F25.gotoAndStop(1);_friend_.D9.F25.NUM.gotoAndStop(FR_GETKEY(25)*1)}
    _friend_.D10.F24.gotoAndStop(_friend_.D10.F24._totalframes)

   И еще 200 строк продолжения за кадром.

   kyzi007, 02 Ноября 2011

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

4. C++ / Говнокод #8132

   +146

   1. 1
   2. 2
   3. 3

   const static int m=5, n=5;
   int matrix[m][n];
   cout << 1[2[matrix]];

   Пруй:

   http://ideone.com/bO5Gn

   RS-232, 09 Октября 2011

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

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

   +126

   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
   15. 15
   16. 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)

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

   −115

   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

   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)

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

   +119

   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
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19
   20. 20
   21. 21
   22. 22
   23. 23
   24. 24
   25. 25
   26. 26
   27. 27
   28. 28
   29. 29
   30. 30
   31. 31
   32. 32
   33. 33
   34. 34
   35. 35
   36. 36
   37. 37
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47
   48. 48
   49. 49
   50. 50
   51. 51
   52. 52
   53. 53
   54. 54
   55. 55
   56. 56
   57. 57
   58. 58
   59. 59
   60. 60
   61. 61
   62. 62
   63. 63
   64. 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)

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

   +142

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 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)

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

   +83

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10

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

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

   lucidfox, 14 Июня 2011

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

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

    +167

    1. 1

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

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

    vov4ik, 13 Июня 2011

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

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

    +158

    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
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 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)

• ← влево
• вправо →

• 1979
• …
• 1673
• 1672
• 1671
• 1670
• 1669
• 1668
• 1667
• 1666
• 1665
• …
• 1