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

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

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

1. PHP (5711)
2. C++ (2742)
3. Куча (2397)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1120)
8. Pascal (648)
9. Python (591)
10. 1C (540)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (44)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

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

   За время:

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

2. C# / Говнокод #19521

   +2

   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

   if (t.IsFaulted)
   {
       try
       {
           throw t.Exception.InnerException;
       }
       catch (ObjectDisposedException)
       {
           Dispose();
           return;
       }
       ...
   }

   cykablyad, 24 Февраля 2016

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

3. 1C / Говнокод #19518

   −98

   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

   ВыборкаЗаказов = Запрос.Выполнить().Выбрать(ОбходРезультатаЗапроса.ПоГруппировкам);
   
   	Пока ВыборкаЗаказов.Следующий() Цикл
   
   		ВыборкаГрузополучателей = ВыборкаЗаказов.Выбрать(ОбходРезультатаЗапроса.ПоГруппировкам);
   		Пока ВыборкаГрузополучателей.Следующий() Цикл
   
   			ВыборкаАдресовДоставки = ВыборкаГрузополучателей.Выбрать(ОбходРезультатаЗапроса.ПоГруппировкам);
   			Пока ВыборкаАдресовДоставки.Следующий() Цикл
   
   				РезультатЗапроса = ВыборкаАдресовДоставки.Выбрать();
   				СтавкаНДС = ПолучитьСтавкуНДСдляДокументов(Переменные.ОргПроизводитель, РезультатЗапроса[0].Контрагент);
   				Если РезультатЗапроса.Количество() <> 0 Тогда
    
   					ТАблицаДАнных = ПреобразоватьВыборкуВТаблицуЗначение(РезультатЗапроса);
   
   Функция ПреобразоватьВыборкуВТаблицуЗначение(ИсходнаяВыборка)
   	
   	ТаблицаРЕзультат = Новый ТаблицаЗначений;
   	
   	Для каждого Колонка Из ИсходнаяВыборка.Колонки Цикл
   		ТаблицаРезультат.Колонки.Добавить(Колонка.Имя);	
   	КОнецЦикла;
   	
   	
   	Для каждого СтрокаВыборки из ИсходнаяВыборка Цикл
   		СтрокаРезультата = ТаблицаРЕзультат.Добавить();
   		ЗаполнитьЗначенияСвойств(СтрокаВыборки, СтрокаРезультата);
   	КонецЦикла;
   	
   	Возврат ТаблицаРЕзультат;
   КонецФункции

   Я допускаю, что это было написано в староглиняные времена, когда у РезультатЗапроса не было метода Выгрузить() но блин! Всей истории 1С я не знаю, и сомневаюсь, что такие времена когда-то были. А вообще что-то подсказывает что песатель просто не знал...

   levxxx, 24 Февраля 2016

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

4. PHP / Говнокод #19457

   +2

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6
   7. 7
   8. 8
   9. 9

   $host = Doctrine_Core::getTable("TransfertConfij")->find(1)->emailserver;
   $emailto = $form->getValue('addresstestemail');
   $port = Doctrine_Core::getTable("TransfertConfij")->find(1)->port;
   $encriptyon = Doctrine_Core::getTable("TransfertConfij")->find(1)->encriptyon;
   $login = Doctrine_Core::getTable("TransfertConfij")->find(1)->loginemailserver;
   $pass = Doctrine_Core::getTable("TransfertConfij")->find(1)->passemailserver;
   $content = $form->getValue('contenttestemail');
   
   // Далее код отправки письма.

   5 запросов? Какая разница? работает ведь!

   rkgrep, 14 Февраля 2016

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

5. 1C / Говнокод #19392

   −99

   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

   Если Сальдо = Перечисления.ДаНет.Да И Кредит = Перечисления.ДаНет.Да Тогда
               
   	СуммаКОплате   =  Строка.Сальдо +  Строка.Начислено +  Строка.Разовые - Строка.Оплата - Льгота;
   	Строка.Коплате = СуммаКОплате;
               
   ИначеЕсли Сальдо = Перечисления.ДаНет.Нет И Кредит = Перечисления.ДаНет.Да Тогда
               
   	СуммаКОплате   = Строка.Начислено + Строка.Разовые - Строка.Оплата - Льгота - ?(Строка.Сальдо < 0, Строка.Сальдо * -1, 0);
   	Строка.Коплате = СуммаКОплате;
               
   ИначеЕсли Сальдо = Перечисления.ДаНет.Да И Кредит = Перечисления.ДаНет.Нет Тогда
               
   	СуммаКОплате = ?(Строка.Сальдо > 0, Строка.Сальдо , 0) +  Строка.Начислено + Строка.Разовые - Строка.Оплата  - Льгота;
   	Строка.Коплате = СуммаКОплате;
               
   ИначеЕсли Сальдо = Перечисления.ДаНет.Нет И Кредит = Перечисления.ДаНет.Нет Тогда
               
   	СуммаКОплате = Строка.Начислено + Строка.Разовые - Льгота;
   	Строка.Коплате = СуммаКОплате;
               
   КонецЕсли;

   Проняло до слез. Ребята не знают простой математики, зато им известна условная тернарная операция. Привел самый эпичный отрывок, но там впору публиковать всю конфигурацию.

   ivan_0110, 03 Февраля 2016

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

6. C# / Говнокод #19364

   +2

   1. 1
   2. 2
   3. 3
   4. 4

   public ActionResult Exception()
   {
     throw new Exception("You better not to execute this!");
   }

   нашел говнокод, оставленный прошлым архитектором)

   govnokoder1488, 28 Января 2016

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

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

   +2

   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

   switch (button.Tag.ToString()) {
                   case "Ezp":
                   {
                       if (nowLayer.OpenedFiles.Count == 0)
                       {
                           nowLayer.OpenedFiles = new List<ExemplarFile> { new ExemplarFile("") };
                       }
                       if (nowLayer.OpenedFiles[0].EdinZemleps == null)
                       {
                           nowLayer.OpenedFiles[0].EdinZemleps = new List<EdinZemlep>();
                       }
                       var strName = ":ЕЗ" + (nowLayer.OpenedFiles[0].EdinZemleps.Count + 1);
                       nowLayer.OpenedFiles[0].EdinZemleps.Add(new EdinZemlep(strName) { Status = StatusEnum.Образуемый });
                       _ezpListTreeView.Add(new EzpTreeView { NameEzp = strName, ContourList = new List<ContourTreeView>(), ParcelList = new List<PartZuTreeView>(), IsEzpVisible = true, IsSelectedEzp = false } );
                       _dictionaryEzpTreeViewFile.Add(_ezpListTreeView[_ezpListTreeView.Count - 1], nowLayer.OpenedFiles[0]);
                       _dictionaryEzpTreeViewEdinZemlep.Add(_ezpListTreeView[_ezpListTreeView.Count - 1], nowLayer.OpenedFiles[0].EdinZemleps[nowLayer.OpenedFiles[0].EdinZemleps.Count - 1]);
                       break;
                   }
                   case "ContourEzp":
                   {
                       indexEz = TreeViewStackPanelMain.Children.IndexOf(((button.Parent as Grid).Parent as StackPanel).Parent as Grid) - _partZuListTreeView.Count;
                       var strName = (nowLayer.OpenedFiles[0].EdinZemleps[indexEz].CadNum != "" ? nowLayer.OpenedFiles[0].EdinZemleps[indexEz].CadNum : nowLayer.OpenedFiles[0].EdinZemleps[indexEz].LastName) + "(" + (nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Contours.Count + 1) + ")";
                       nowLayer.OpenedFiles[0].EdinZemleps[indexEz].AddContour(new Contour(strName) { Status = StatusEnum.Образуемый });
                       if (_ezpListTreeView[indexEz].ContourList == null)
                       {
                           _ezpListTreeView[indexEz].ContourList = new List<ContourTreeView>();
                       }
                       strName = nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Contours.Last().Name;
                       _ezpListTreeView[indexEz].ContourList.Add(new ContourTreeView { NameContour = strName, IsContourVisible = true, IsSelectedContour = false } );
                       for (var i = 0; i < _ezpListTreeView[indexEz].ContourList.Count; i++)
                       {
                           _ezpListTreeView[indexEz].ContourList[i].NameContour = nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Contours[i].Name;
                       }
                       break;
                   }
                   case "ZuEzp":
                   {
                       indexEz = TreeViewStackPanelMain.Children.IndexOf(((button.Parent as Grid).Parent as StackPanel).Parent as Grid) - _partZuListTreeView.Count;
                       var strName = ":ЗУ" + (nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels.Count + 1);
                       if (_ezpListTreeView[indexEz].ParcelList == null)
                       {
                           _ezpListTreeView[indexEz].ParcelList = new List<PartZuTreeView>();
                       }
                       nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels.Add(new Parcel(strName) { Status = StatusEnum.Образуемый });
                       _ezpListTreeView[indexEz].ParcelList.Add(new PartZuTreeView { NamePartZu = strName, ContourList = new List<ContourTreeView>(), PartList = new List<PartTreeView>(), IsPartZuVisible = true, IsSelectedPartZu = false });
                       _dictionaryPartZuTreeViewFile.Add(_ezpListTreeView[indexEz].ParcelList[_ezpListTreeView[indexEz].ParcelList.Count - 1], nowLayer.OpenedFiles[0]);
                       _dictionaryPartZuTreeViewParcel.Add(_ezpListTreeView[indexEz].ParcelList[_ezpListTreeView[indexEz].ParcelList.Count - 1], nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels[nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels.Count - 1]);
                       break;
                   }              
                   case "PartZuEzp":
                   {
                       indexEz = TreeViewStackPanelMain.Children.IndexOf(((((((button.Parent as Grid).Parent as StackPanel).Parent as Grid).Parent as StackPanel).Parent as Grid).Parent as StackPanel).Parent as Grid) - _partZuListTreeView.Count;
                       indexZu = ((((((button.Parent as Grid).Parent as StackPanel).Parent as Grid).Parent as StackPanel).Parent as Grid).Parent as StackPanel).Children.IndexOf((((((button.Parent as Grid).Parent as StackPanel).Parent as Grid).Parent as StackPanel).Parent as Grid));
                       var sbPrclName = nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels[indexZu].LastName + "/чзу" + (nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels[indexZu].Subparcels.Count + 1);
                       nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels[indexZu].Subparcels.Add(new Subparcel(sbPrclName) { Status = StatusEnum.Образуемый });
                       if (_ezpListTreeView[indexEz].ParcelList[indexZu].PartList == null)
                       {
                           _ezpListTreeView[indexEz].ParcelList[indexZu].PartList = new List<PartTreeView>();
                       }
                       _ezpListTreeView[indexEz].ParcelList[indexZu].PartList.Add(new PartTreeView { NamePart = sbPrclName, ContourList = new List<ContourTreeView>(), IsPartVisible = true, IsSelectedPart = _ezpListTreeView[indexEz].ParcelList[indexZu].IsSelectedPartZu });
                       _dictionaryPartTreeViewSubparcel.Add(_ezpListTreeView[indexEz].ParcelList[indexZu].PartList[_ezpListTreeView[indexEz].ParcelList[indexZu].PartList.Count - 1], nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels[indexZu].Subparcels[nowLayer.OpenedFiles[0].EdinZemleps[indexEz].Parcels[indexZu].Subparcels.Count - 1]);
                       break;
                   }

   и это продолжается, продолжается и продолжается....

   deathguard771, 27 Января 2016

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

8. C# / Говнокод #19306

   +1

   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
   65. 65
   66. 66
   67. 67
   68. 68
   69. 69
   70. 70
   71. 71
   72. 72
   73. 73
   74. 74
   75. 75
   76. 76
   77. 77
   78. 78
   79. 79
   80. 80
   81. 81
   82. 82
   83. 83
   84. 84
   85. 85
   86. 86
   87. 87
   88. 88
   89. 89
   90. 90
   91. 91
   92. 92
   93. 93
   94. 94
   95. 95
   96. 96
   97. 97
   98. 98
   99. 99

   namespace SmalltalkPHP
   {
   	class Model
   	{
   		internal static void CreateClass(string className)
   		{
   			ClassObject = new Class(className);
   		}
   
   		public static Class ClassObject { get; set; }
   		public class Message
   		{
   			public class Arguments
   			{
   				private SortedList arguments = new SortedList();
   				public void Add(String key, String name)
   				{
   					this.arguments.Add(key, name);
   				}
   				public String AsPhp()
   				{
   					String[] sb = new String[arguments.Count];
   					int i = 0;
   					foreach (DictionaryEntry arg in arguments)
   					{
   						if ((String)(arg.Value) != "") sb[i] = "$" + arg.Value;
   						i++;
   					}
   					return String.Join(", ", sb);
   				}
   				public String MakeFunctionName()
   				{
   					String[] sb = new String[arguments.Count];
   					int i = 0;
   					foreach (DictionaryEntry arg in arguments)
   					{
   						if ((String)(arg.Key) != "") sb[i] = (String)arg.Key;
   						i++;
   					}
   					return String.Join("_", sb);
   				}
   			}
   			public class Generic
   			{
   				public virtual string AsPhp()
   				{
   					return "nya";
   				}
   			}
   			public class Unary : Generic
   			{
   				public Unary(String name, Boolean isStatic = false)
   				{
   					this.Name = name;
   					this.Arguments = new Arguments();
   					this.IsStatic = isStatic;
   				}
   				public string Name { get; set; }
   				public Arguments Arguments { get; set; }
   				public Boolean IsStatic { get; set; }
   				public string PhpHeader
   				{
   					get
   					{
   						return String.Format("public {0}function {1}()", IsStatic ? "static " : "", Name);
   					}
   				}
   
   				public override string AsPhp()
   				{
   					return PhpHeader;
   				}
   			}
   			public class Keyword : Generic
   			{
   				public Keyword(Boolean isStatic = false)
   				{
   					this.Arguments = new Arguments();
   					this.IsStatic = isStatic;
   				}
   				public string Name { get { return Arguments.MakeFunctionName(); } }
   				public Arguments Arguments { get; set; }
   				public Boolean IsStatic { get; set; }
   				public string PhpHeader
   				{
   					get
   					{
   						return String.Format("public {0}function {1}({2})", IsStatic ? "static " : "", Name, Arguments.AsPhp());
   					}
   				}
   
   				public override string AsPhp()
   				{
   					return PhpHeader;
   				}
   			}
   		}
   	}
   }

   Писал конвертер Smalltalk -> PHP
   Где-то еще валяется Smalltalk -> Erlang

   kgm-rj, 14 Января 2016

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

9. C# / Говнокод #19304

   −2

   1. 1
   2. 2
   3. 3

   var indexes = T.Select((item, index) => new { Item = item, Index = index })
                   .Where(nitem => n.Item == 0).Select(nindex => n.Index).ToArray();
   Console.WriteLine("Первый: {0}, последний: {1}", indexes.First(), indexes.Last());

   Не слишком много item index? index item?

   Govno_Coder, 14 Января 2016

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

10. JavaScript / Говнокод #19181

    +8

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5

    if(user.pass = pass_to_hash(password)){
           return done(null, user);
    }else{
           return done('incorrect password');
    }

    Факир был пьян, и запер дверь, забыв ее закрыть)

    Darth, 15 Декабря 2015

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

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

    +1

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

    var cloths = {
      <?php foreach ($clothList as $cloth): ?>
        "<?=$cloth->id()?>": {
          "id": <?=$cloth->id()?>,
          "name": "<?=$cloth->name()?>",
          "public_name": "<?=str_replace('"', '\"', $cloth->public_name())?>"
        },
      <?php endforeach; ?>
      "dummy": {}
    };

    kgm-rj, 14 Декабря 2015

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

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

• 1976
• …
• 310
• 309
• 308
• 307
• 306
• 305
• 304
• 303
• 302
• …
• 1