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

    В номинации:
    За время:

  2. 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)

  3. 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)

  4. 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)

  5. 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)

  6. 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)

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

  8. 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)

  9. 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)

  10. 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)

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

    +2

    1. 1
    2. 2
    3. 3
    4. 4
    if (!$subscription) {
    return $this->redirect(['action' => 'cancel_success']);
    throw new BadRequestException();
}

    epadqwbd, 11 Декабря 2015

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