По колено в коде — Говнокод.ру

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

+19

1
2
3
4

Class1* c1 = (Class1*)malloc(sizeof(Class1)*N);
Class2* c2 = (Class2*)malloc(sizeof(Class2)*N);
for (long i = 0; i < N; i++) c1[i] = Class1();
for (long i = 0; i < N; i++) c2[i] = Class2();

Рассказать ему про new[] / delete[]?

runewalsh, 29 Ноября 2012

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

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

+140

1

if (maxWidth < 96) {maxWidth = 96 }

Большой Брат следит за тобой, переменная.

BYDLOCOD, 29 Ноября 2012

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

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

+102

001
002
003
004
005
006
007
008
009
010
011
012
013
014
015
016
017
018
019
020
021
022
023
024
025
026
027
028
029
030
031
032
033
034
035
036
037
038
039
040
041
042
043
044
045
046
047
048
049
050
051
052
053
054
055
056
057
058
059
060
061
062
063
064
065
066
067
068
069
070
071
072
073
074
075
076
077
078
079
080
081
082
083
084
085
086
087
088
089
090
091
092
093
094
095
096
097
098
099
100

switch (occupation.name)
{
	case "architecture and engineering occupations:":
		if (occupation.isNB)
		{
			if (occupation.maleId == ex_nb_id)
			{
				model.ArchitectureAndEngineeringOccupationsMalePercentageNB = occupation.percent;
			}
			if (occupation.femaleId == ex_nb_id)
			{
				model.ArchitectureAndEngineeringOccupationsFemalePercentageNB = occupation.percent;
			}
		}
		else
		{
			if (occupation.maleId == ex_city_id)
			{
				model.ArchitectureAndEngineeringOccupationsMalePercentageCity = occupation.percent;
			}
			if (occupation.femaleId == ex_city_id)
			{
				model.ArchitectureAndEngineeringOccupationsFemalePercentageCity = occupation.percent;
			}
		}
	break;

	case "arts, design, entertainment, sports, and media occupations:":
		if (occupation.isNB)
		{
			if (occupation.maleId == ex_nb_id)
			{
				model.ArtsDesignEntertainmentSportsAndMediaOccupationsMalePercentageNB = occupation.percent;
			}
			if (occupation.femaleId == ex_nb_id)
			{
				model.ArtsDesignEntertainmentSportsAndMediaOccupationsFemalePercentageNB = occupation.percent;
			}
		}
		else
		{
			if (occupation.maleId == ex_city_id)
			{
				model.ArtsDesignEntertainmentSportsAndMediaOccupationsMalePercentageCity = occupation.percent;
			}
			if (occupation.femaleId == ex_city_id)
			{
				model.ArtsDesignEntertainmentSportsAndMediaOccupationsFemalePercentageCity = occupation.percent;
			}
		}
	break;

	case "business and financial operations occupations:":
		if (occupation.isNB)
		{
			if (occupation.maleId == ex_nb_id)
			{
				model.BusinessAndFinancialOperationsOccupationsMalePercentageNB = occupation.percent;
			}
			if (occupation.femaleId == ex_nb_id)
			{
				model.BusinessAndFinancialOperationsOccupationsFemalePercentageNB = occupation.percent;
			}
		}
		else
		{
			if (occupation.maleId == ex_city_id)
			{
				model.BusinessAndFinancialOperationsOccupationsMalePercentageCity = occupation.percent;
			}
			if (occupation.femaleId == ex_city_id)
			{
				model.BusinessAndFinancialOperationsOccupationsFemalePercentageCity = occupation.percent;
			}
		}
	break;

	case "community and social services occupations:":
		if (occupation.isNB)
		{
			if (occupation.maleId == ex_nb_id)
			{
				model.CommunityAndSocialServicesOccupationsMalePercentageNB = occupation.percent;
			}
			if (occupation.femaleId == ex_nb_id)
			{
				model.CommunityAndSocialServicesOccupationsFemalePercentageNB = occupation.percent;
			}
		}
		else
		{
			if (occupation.maleId == ex_city_id)
			{
				model.CommunityAndSocialServicesOccupationsMalePercentageCity = occupation.percent;
			}
			if (occupation.femaleId == ex_city_id)
			{
				model.CommunityAndSocialServicesOccupationsFemalePercentageCity = occupation.percent;
			}
		}

эх жаль здесь ограничение на 100 строк всего лишь...не поместилось всего 200+ отборного ветвления. известный до боли мегапрограммер наваял очередной шедевр)))) по мотивам http://govnokod.ru/10448 и http://govnokod.ru/10313 и еще http://govnokod.ru/12136...

mangyst, 29 Ноября 2012

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

1C / Говнокод #12222

−119

1
2
3
4
5
6

ВЫБРАТЬ
	ЕСТЬNULL(СрокПолезногоИспользования, 0) КАК СрокПолезногоИспользования
ИЗ
	РегистрСведений.РасчетАмортизацииОС.СрезПоследних
ГДЕ
	ОС = &ОсновноеСредство

Типовая конфигурация. Там такой запрос используют в печатной форме Основных средств... Думает по 10-15 минут
2 глупых ошибки:
1. Проверка на NULL, в данном случае его быть не может (хотя это особо ни на что не влияет)
2. Условие по законам 1С должно применяться в РегистрСведений.РасчетАмортизацииОС.Срез Последних(,ОС = &ОсновноеСредство), иначе сильно падает скорость выполнения.

ok0504, 29 Ноября 2012

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

1C / Говнокод #12221

−120

1
2
3

Пока ТЗРаспределения.ТаблицаРеализацийИВозвратовНеСвернутая.ТаблицаНоменклатур.ТаблицаРеализацийНеСв.ТаблицаВозвратовНеСв.ПолучитьСтроку()=1 Цикл
     СтрокаВозвратов = СтрокаВозвратов + ТЗРаспределения.ТаблицаРеализацийИВозвратовНеСвернутая.ТаблицаНоменклатур.ТаблицаРеализацийНеСв.ТаблицаВозвратовНеСв.Возврат  	+ ";" ;
КонецЦикла;

Очередной шедевр... Живем в 6(!!!)-мерном мире. У меня чуть мозг не взорвался.

jerokee, 29 Ноября 2012

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

1C / Говнокод #12220

−122

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

Пока Выборка1...
...Выборка2...
...
Выборка7 = ПрименяемостьНаПрибор_ПоискВхождений(Выборка6.НомерЗаписиРСДСЕГл, НаДату).Выбрать();
Пока Выборка7.Следующий() Цикл
	ПрименяемостьПоВхождению = ПрименяемостьПоВхождению * Выборка7.Применяемость;
	
	
	Если Выборка7.НомерЗаписиРСДСЕГл = 0 Тогда
		// Достигнут прибор
		Применяемость = Применяемость + ПрименяемостьПоВхождению;
	Иначе
		Выборка8 = ПрименяемостьНаПрибор_ПоискВхождений(Выборка7.НомерЗаписиРСДСЕГл, НаДату).Выбрать();
		Пока Выборка8.Следующий() Цикл
			ПрименяемостьПоВхождению = ПрименяемостьПоВхождению * Выборка8.Применяемость;
			
			Если Выборка8.НомерЗаписиРСДСЕГл = 0 Тогда
				// Достигнут прибор
				Применяемость = Применяемость + ПрименяемостьПоВхождению;
				
			Иначе
				Сообщить("Превышение значения выборки... Странная ситуация, вообще не должна возникать!!!");																			
				Сообщить("Срочно обратитесь в Отдел ВТ. Постарайтесь записать ситуацию в которой возникла эта ошибка... !!!");																			
			КонецЕсли;
		КонецЦикла;
	КонецЕсли;
КонецЦикла;
...
...

По мотивам предыдущих постов. Все из одной конфигурации.

Ykov89, 29 Ноября 2012

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

1C / Говнокод #12219

−125

1
2
3
4
5
6

Если СтрНВрОбщ.Действия = Перечисления.Действия.Удалить Тогда
	// Такого быть не должно... 
	Сообщить("Это сообщение не должно появиться... Обратитесь в отдел ИТ! ");
ИначеЕсли СтрНВрОбщ.Действия = Перечисления.Действия.Добавить Тогда
	//Тру-ля-ля... другой говнокод
КонецЕсли;

При чем такого действительно "быть не должно", т.к. проверка есть и в другом модуле.

Ykov89, 29 Ноября 2012

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

1C / Говнокод #12218

−121

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
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94

Для каждого СтрГотИзд Из ГотовыеИзделия Цикл
	
	СсылкаДСЕ 		= ОпределитьДСЕГотовоеИзделиеПоНаименованию(СтрГотИзд.Значение);
	СсылкаЗаписьРС 	= НайтиПервуюЗаписьРС_ПоДСЕ(СсылкаДСЕ, ЭтотОбъект.НаДату);
	
	Если СсылкаЗаписьРС = Неопределено Тогда
		Предупреждение("На эту дату данных не существует!");
	Иначе
		
		
		ФормаИндикатора.КомментарийОбработкиДанных  = "Построение отчета: " + СсылкаДСЕ.ВидПредмета + " - " + СсылкаДСЕ.Наименование + ""; 
		
		ОбластьДанныеДобавить(ПолеТабДок, ОбластьДанные1, 1, СсылкаДСЕ, СсылкаЗаписьРС.Применяемость, Неопределено, ФормаИндикатора);
		Выборка2Ур = НайтиЗаписиРС_ПоКодуДСЕГл(СсылкаЗаписьРС.НомерЗаписиРСДСЕВх, НаДату).Выбрать();
		Пока Выборка2Ур.Следующий() цикл
			ОбластьДанныеДобавить(ПолеТабДок, ОбластьДанные2, 2, Выборка2Ур.ДСЕВходящий, Выборка2Ур.Применяемость, Выборка2Ур, ФормаИндикатора);
			
			Выборка3Ур = НайтиЗаписиРС_ПоКодуДСЕГл(Выборка2Ур.НомерЗаписиРСДСЕВх, НаДату).Выбрать();
			Пока Выборка3Ур.Следующий() цикл
				ОбластьДанныеДобавить(ПолеТабДок, ОбластьДанные3, 3, Выборка3Ур.ДСЕВходящий, Выборка3Ур.Применяемость, Выборка3Ур, ФормаИндикатора);
				
				Выборка4Ур = НайтиЗаписиРС_ПоКодуДСЕГл(Выборка3Ур.НомерЗаписиРСДСЕВх, НаДату).Выбрать();
				Пока Выборка4Ур.Следующий() цикл
					ОбластьДанныеДобавить(ПолеТабДок, ОбластьДанные4, 4, Выборка4Ур.ДСЕВходящий, Выборка4Ур.Применяемость, Выборка4Ур, ФормаИндикатора);
					
					Выборка5Ур = НайтиЗаписиРС_ПоКодуДСЕГл(Выборка4Ур.НомерЗаписиРСДСЕВх, НаДату).Выбрать();
					Пока Выборка5Ур.Следующий() цикл
						ОбластьДанныеДобавить(ПолеТабДок, ОбластьДанные5, 5, Выборка5Ур.ДСЕВходящий, Выборка5Ур.Применяемость, Выборка5Ур, ФормаИндикатора);
						
						Выборка6Ур = НайтиЗаписиРС_ПоКодуДСЕГл(Выборка5Ур.НомерЗаписиРСДСЕВх, НаДату).Выбрать();
						Пока Выборка6Ур.Следующий() цикл
							ОбластьДанныеДобавить(ПолеТабДок, ОбластьДанные6, 6, Выборка6Ур.ДСЕВходящий, Выборка6Ур.Применяемость, Выборка6Ур, ФормаИндикатора);
							
							Выборка7Ур = НайтиЗаписиРС_ПоКодуДСЕГл(Выборка6Ур.НомерЗаписиРСДСЕВх, НаДату).Выбрать();
							Пока Выборка7Ур.Следующий() цикл
								ОбластьДанныеДобавить(ПолеТабДок, ОбластьДанные7, 7, Выборка7Ур.ДСЕВходящий, Выборка7Ур.Применяемость, Выборка7Ур, ФормаИндикатора);
								
								Выборка8Ур = НайтиЗаписиРС_ПоКодуДСЕГл(Выборка7Ур.НомерЗаписиРСДСЕВх, НаДату).Выбрать();
								Пока Выборка8Ур.Следующий() цикл
									ОбластьДанныеДобавить(ПолеТабДок, ОбластьДанные8, 8, Выборка8Ур.ДСЕВходящий, Выборка8Ур.Применяемость, Выборка8Ур, ФормаИндикатора);
									
									Выборка9Ур = НайтиЗаписиРС_ПоКодуДСЕГл(Выборка8Ур.НомерЗаписиРСДСЕВх, НаДату).Выбрать();
									Пока Выборка9Ур.Следующий() цикл
										ОбластьДанныеДобавить(ПолеТабДок, ОбластьДанные9, 9, Выборка9Ур.ДСЕВходящий, Выборка9Ур.Применяемость, Выборка9Ур, ФормаИндикатора);
									КонецЦикла;
								КонецЦикла;
							КонецЦикла;
						КонецЦикла;
					КонецЦикла;
				КонецЦикла;
			КонецЦикла;
		КонецЦикла;
	КонецЕсли;
КонецЦикла;



Функция НайтиЗаписиРС_ПоКодуДСЕГл(КодДСЕГл, НаДату, ДопустимаяЗамена = 0) Экспорт
	Запрос = Новый Запрос;
	Запрос.Текст = 
	"ВЫБРАТЬ
	|	СтруктураПриборов1СрезПоследних.ДСЕГлавный,
	|	СтруктураПриборов1СрезПоследних.ДСЕВходящий КАК ДСЕВходящий,
	|	СтруктураПриборов1СрезПоследних.Прибор,
	|	СтруктураПриборов1СрезПоследних.НомерЗаписиРСДСЕВх,
	|	СтруктураПриборов1СрезПоследних.НомерЗаписиРСДСЕГл,
	|	СтруктураПриборов1СрезПоследних.ДопустимаяЗамена,
	|	СтруктураПриборов1СрезПоследних.Применяемость,
	|	СтруктураПриборов1СрезПоследних.ДСЕВходящий.Ссылка,
	|	СтруктураПриборов1СрезПоследних.ДСЕВходящий.ВидПредмета КАК ВидПредмета,
	|	СтруктураПриборов1СрезПоследних.ДСЕВходящий.ВидПредмета.Порядок КАК ДСЕВходящийВидПредметаПорядок,
	|	СтруктураПриборов1СрезПоследних.ДСЕВходящий.Код КАК Код,
	|	СтруктураПриборов1СрезПоследних.Заимствованные,
	|	СтруктураПриборов1СрезПоследних.Период,
	|	СтруктураПриборов1СрезПоследних.ДатаОкончанияИспользования,
	|	СтруктураПриборов1СрезПоследних.ВводНового,
	|	СтруктураПриборов1СрезПоследних.ИзменениеСоставаИзделия
	|ИЗ
	|	РегистрСведений.СтруктураПриборов1.СрезПоследних(
	|		&НаДату,
	|		НомерЗаписиРСДСЕГл = &КодДСЕГл
	|		    И (ДатаОкончанияИспользования = &ПустаяДата
	|		        ИЛИ ДатаОкончанияИспользования > &НаДату)
	|		    И ДопустимаяЗамена = &ДопустимаяЗамена) КАК СтруктураПриборов1СрезПоследних
	|
	|УПОРЯДОЧИТЬ ПО
	|	ДСЕВходящийВидПредметаПорядок,
	|	Код";
	Запрос.УстановитьПараметр("НаДату", 	НаДату);
	Запрос.УстановитьПараметр("КодДСЕГл", 	КодДСЕГл);
	Запрос.УстановитьПараметр("ПустаяДата", '00010101');
	Запрос.УстановитьПараметр("ДопустимаяЗамена", ДопустимаяЗамена);
	Возврат(Запрос.Выполнить());
КонецФункции

Прелесть! Выполняется около минуты. При чем этот код дублируется почти во всех отчетах.
После оптимизации объем кода сократился в 3 раза, а время выполнения составило 1-2 сек.

Ykov89, 29 Ноября 2012

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

1C / Говнокод #12217

−122

1
2
3
4

Результат = Запрос.Выполнить().Выбрать();
Пока Результат.Следующий() > 0 Цикл
       // что-то делаем
КонецЦикла;

Argonavt, 29 Ноября 2012

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

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

+139

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

/* В комментариях то что хотел Автор от своей программы 	*/
/* Радует однако, что она компилируется и во время		*/
/* работы не выдает ни каких ошибок				*/
/* Порядок операторов сохранен.					*/
/* Тот же код только вырезанно "лишнее".			*/

/* Эта ф-ия находится в другом cpp файле */
float f(float x, float y)
{
	/* Тут что-то происходит с x и y */
	return x, y;
}

void main(void)
{
	float **m;

	/* ... */
	float f(float x, float y);
	/* ... */

	/* Задается N					*/
	float N;
	/* ... */

	/* Выделяется память под массив m[N][2] 	*/
	m = (float **)malloc(sizeof(float*));
	for(i = 0; i < N; i++)
	{
		m[i] = (float*)malloc(sizeof(float*) * 2);
	}

	/* ... */
	for(i = 0; i < N; i++)
	{
		/* Массив заполняется числами						*/
		/* В m[i][] должны быть записанны числа x, y измененый ф-ей f 		*/
		/* т. е.	m[i][0] = xf						*/
		/*		m[i][1] = yf						*/
		for(j = 0; j < 2; j++)
		{
			m[i][j] = f(x, y);
			/* ... */
		}
	}

	/* ... */
}

dreesto, 28 Ноября 2012

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

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

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

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

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

1C / Говнокод #12222

1C / Говнокод #12221

1C / Говнокод #12220

1C / Говнокод #12219

1C / Говнокод #12218

1C / Говнокод #12217

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