1. Список говнокодов пользователя VladC

    Всего: 9

  2. 1C / Говнокод #28173

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

    Нашёл сие поделие в базе одного клиента, придраться хочется ко всему, начиная с лютого нэйминга - имя функции ПолучитьСеб(), не намного лучше чем Функция1(), имя переменной в запросе бухСС тоже так себе.
    Задача функции вернуть себестоимость единицы товара, при этом число она возвращает только если в запросе что-то есть, понятно что в остальных случаях вернет Неопределено, но себестоимость это всегда число, никаких Неопределено быть не должно.
    Если посмотреть текст запроса, то видно, что поля выборки на 100% беруться из одной таблицы, зачем там ещё одну прилепили, да ещё и с полным соединением - теряюсь в догадках.
    Упорядочивание по полю Партия вообще смысла не имеет, если уж речь идёт о получении себестоимости, то скорей автор хотел получить её из первой партии, что косвенно подвтерждается выражением "ПЕРВЫЕ 1", но дело в том, что упорядочивание по ссылке (Партия - это элемент справочника Партии) не имеет никакой привязки к дате, т.е. нужно было писать так: "Упорядочить По Партия.ДатаПартии".
    Здесь не видно, но функция вызывается только в двух местах модуля и всегда с конкретным значением номенклатуры, т.е. условие виртуальной таблицы может быть оптимизировано до "Номенклатура = &Номенклатура".
    Ну и на последок - в данном случае полное соединение конечно нужно ликвидировать, но если бы оно действительно было нужно, то виртуальные таблицы не стоит связывать между собой, предварительно их нужно помещать во временные таблицы и индексировать по полям условий соединения.

    VladC, 17 Мая 2022

    Комментарии (1)
  3. 1C / Говнокод #25592

    0

    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
    Запрос = Новый Запрос;
    	Запрос.Текст = 
    	"ВЫБРАТЬ
    	|	ЭтапРемонт.Номер КАК Номер
    	|ИЗ
    	|	Документ.ЭтапРемонт КАК ЭтапРемонт
    	|ГДЕ
    	|	ЭтапРемонт.Основание = &Основание";
    	Запрос.УстановитьПараметр("Основание", Объект.Ссылка);
    	Выб = Запрос.Выполнить().Выбрать();
    	Выгрузка = Запрос.Выполнить().Выгрузить();
    	Количество = Выб.Количество();
    	Если Количество > 0 Тогда
    		Ссыль = Выгрузка[0].Номер;
    		ЭтаФорма.Элементы.Этап.Заголовок = Документы.ЭтапРемонт.НайтиПоНомеру(Ссыль);
    	КонецЕсли;

    Задача кода - установить текстовое представление документа в надписи на форме, для этого автор сначала получает номер требуемого документа, потом проверяет выборку на наличие записей, а потом по полученному номеру ищет ссылку методом НайтиПоНомеру.
    Говнокод в том, что:
    1. Ссылку можно получить сразу же в запросе
    2. Выгружать выборку в коллекцию значений нет никакой необходимости, можно всё необходимое взять из выборки, она для этого и предназначена.
    Автор - один из франей Абакана.

    VladC, 07 Мая 2019

    Комментарии (3)
  4. 1C / Говнокод #25210

    +2

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    ТекЧас = Строка(Час(Дата2));
    ТекЧас = Прав("00" + ТекЧас, 2);
    ТекМин = Строка(Минута(Дата2));
    ТекМин = Прав("00" + ТекМин, 2);
    ТекВремя = ТекЧас + ":" + ТекМин;

    Это способ получить время в формате "чч:мм" из даты и времени (переменная Дата2).
    Заменяется этим:
    ТекВремя = Формат(Дата2, "ДФ=hh:mm");

    VladC, 21 Декабря 2018

    Комментарии (4)
  5. 1C / Говнокод #23723

    +2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    Процедура ОбработкаЗаполнения(ДанныеЗаполнения, СтандартнаяОбработка)
    	
    	Организация1 = Справочники.Организации.ПустаяСсылка();
    	Организация1 = Справочники.Организации.НайтиПоНаименованию("ХХХХХ Андрей Александрович ИП");
    	Организация = Организация1;
    	ОрганизацияШапка = Организация1;
    	Склад = Справочники.Склады.НайтиПоНаименованию("Сервис центр");
    	Контрагент = Справочники.Партнеры.НайтиПоНаименованию("Розничный покупатель");
    	ВидОперации = Перечисления.ВидРемонта.ПлатныйРемонт;
    	
    КонецПроцедуры;

    Строка 3. Непонятно для чего введена переменная Организация1 (реквизита такого нет).
    Строка 3. Непонятно для чего нужна инициализация пустой ссылкой, если в следующей строке происходит присваивание другого значения.
    Строка5. Почему бы сразу не инициализировать реквизит нужным значением?

    VladC, 12 Февраля 2018

    Комментарии (2)
  6. 1C / Говнокод #19435

    −38

    1. 001
    2. 002
    3. 003
    4. 004
    5. 005
    6. 006
    7. 007
    8. 008
    9. 009
    10. 010
    11. 011
    12. 012
    13. 013
    14. 014
    15. 015
    16. 016
    17. 017
    18. 018
    19. 019
    20. 020
    21. 021
    22. 022
    23. 023
    24. 024
    25. 025
    26. 026
    27. 027
    28. 028
    29. 029
    30. 030
    31. 031
    32. 032
    33. 033
    34. 034
    35. 035
    36. 036
    37. 037
    38. 038
    39. 039
    40. 040
    41. 041
    42. 042
    43. 043
    44. 044
    45. 045
    46. 046
    47. 047
    48. 048
    49. 049
    50. 050
    51. 051
    52. 052
    53. 053
    54. 054
    55. 055
    56. 056
    57. 057
    58. 058
    59. 059
    60. 060
    61. 061
    62. 062
    63. 063
    64. 064
    65. 065
    66. 066
    67. 067
    68. 068
    69. 069
    70. 070
    71. 071
    72. 072
    73. 073
    74. 074
    75. 075
    76. 076
    77. 077
    78. 078
    79. 079
    80. 080
    81. 081
    82. 082
    83. 083
    84. 084
    85. 085
    86. 086
    87. 087
    88. 088
    89. 089
    90. 090
    91. 091
    92. 092
    93. 093
    94. 094
    95. 095
    96. 096
    97. 097
    98. 098
    99. 099
    100. 100
    ВЫБРАТЬ
    	
    	СправочникНоменклатура.Ссылка КАК Ссылка,
    	СправочникНоменклатура.Артикул КАК Артикул,
    	СправочникНоменклатура.Код КАК Код,
    	СправочникНоменклатура.Наименование КАК Наименование,
    	СправочникНоменклатура.ПометкаУдаления КАК ПометкаУдаления,
    	СправочникНоменклатура.Родитель КАК Родитель,
    	СправочникНоменклатура.ЭтоГруппа КАК ЭтоГруппа,
    	СправочникНоменклатура.Ссылка КАК Номенклатура,
    	СправочникНоменклатура.Ссылка КАК ЭлементСписка,
    	СправочникНоменклатура.ВидНоменклатуры КАК ВидНоменклатуры,
    	СправочникНоменклатура.ТипНоменклатуры КАК ТипНоменклатуры,
    	
    	ВЫБОР КОГДА СправочникНоменклатура.ТипНоменклатуры = ЗНАЧЕНИЕ(Перечисление.ТипыНоменклатуры.Набор) ТОГДА
    		ЕСТЬNULL(ВариантыКомплектацииНоменклатуры.ВариантРасчетаЦеныНабора, ЗНАЧЕНИЕ(Перечисление.ВариантыРасчетаЦенНаборов.ПустаяСсылка))
    	ИНАЧЕ
    		ЗНАЧЕНИЕ(Перечисление.ВариантыРасчетаЦенНаборов.ПустаяСсылка)
    	КОНЕЦ КАК ВариантРасчетаЦеныНабора,
    	ВариантыКомплектацииНоменклатуры.СодержитТовары КАК СодержитТовары,
    	ВариантыКомплектацииНоменклатуры.СодержитУслуги КАК СодержитУслуги,
    	
    	СправочникНоменклатура.ЕдиницаИзмерения КАК ЕдиницаИзмерения,
    	СправочникНоменклатура.ЦеноваяГруппа КАК ЦеноваяГруппа,
    	СправочникНоменклатура.Качество КАК Качество,
    	СправочникНоменклатура.ИспользованиеХарактеристик КАК ИспользованиеХарактеристик,
    	ВЫБОР
    		КОГДА СправочникНоменклатура.ТипНоменклатуры В (ЗНАЧЕНИЕ(Перечисление.ТипыНоменклатуры.Товар), ЗНАЧЕНИЕ(Перечисление.ТипыНоменклатуры.МногооборотнаяТара))
    			ТОГДА ИСТИНА
    		КОГДА СправочникНоменклатура.ТипНоменклатуры В (ЗНАЧЕНИЕ(Перечисление.ТипыНоменклатуры.Набор)) И ЕСТЬNULL(ВариантыКомплектацииНоменклатуры.СодержитТовары, ЛОЖЬ) И Не ЕСТЬNULL(ВариантыКомплектацииНоменклатуры.СодержитУслуги, ЛОЖЬ)
    			ТОГДА ИСТИНА
    		ИНАЧЕ ЛОЖЬ
    	КОНЕЦ КАК ЭтоТовар,
    	ВЫБОР
    		КОГДА СправочникНоменклатура.ТипНоменклатуры В (ЗНАЧЕНИЕ(Перечисление.ТипыНоменклатуры.Услуга), ЗНАЧЕНИЕ(Перечисление.ТипыНоменклатуры.Работа))
    			ТОГДА ИСТИНА
    		КОГДА СправочникНоменклатура.ТипНоменклатуры В (ЗНАЧЕНИЕ(Перечисление.ТипыНоменклатуры.Набор)) И ЕСТЬNULL(ВариантыКомплектацииНоменклатуры.СодержитУслуги, ЛОЖЬ) И НЕ ЕСТЬNULL(ВариантыКомплектацииНоменклатуры.СодержитТовары, ЛОЖЬ)
    			ТОГДА ИСТИНА
    		ИНАЧЕ ЛОЖЬ
    	КОНЕЦ КАК ЭтоУслуга,
    	ВЫБОР
    		КОГДА СправочникНоменклатура.ИспользованиеХарактеристик = ЗНАЧЕНИЕ(Перечисление.ВариантыИспользованияХарактеристикНоменклатуры.ОбщиеДляВидаНоменклатуры)
    			ТОГДА СправочникНоменклатура.ВидНоменклатуры
    		КОГДА СправочникНоменклатура.ИспользованиеХарактеристик = ЗНАЧЕНИЕ(Перечисление.ВариантыИспользованияХарактеристикНоменклатуры.ИндивидуальныеДляНоменклатуры)
    			ТОГДА СправочникНоменклатура.Ссылка
    		КОГДА СправочникНоменклатура.ИспользованиеХарактеристик = ЗНАЧЕНИЕ(Перечисление.ВариантыИспользованияХарактеристикНоменклатуры.ОбщиеСДругимВидомНоменклатуры)
    			ТОГДА СправочникНоменклатура.ВладелецХарактеристик
    		ИНАЧЕ Неопределено
    	КОНЕЦ КАК ВладелецХарактеристик,
    	ВЫБОР
    		КОГДА СправочникНоменклатура.ИспользованиеХарактеристик = ЗНАЧЕНИЕ(Перечисление.ВариантыИспользованияХарактеристикНоменклатуры.НеИспользовать)
    			ТОГДА ЛОЖЬ
    		ИНАЧЕ ИСТИНА
    	КОНЕЦ КАК ХарактеристикиИспользуются,
    	ВЫБОР
    		КОГДА СправочникНоменклатура.ЭтоГруппа
    			ТОГДА ВЫБОР
    					КОГДА СправочникНоменклатура.ПометкаУдаления
    						ТОГДА 13
    					ИНАЧЕ 12
    				КОНЕЦ
    		ИНАЧЕ ВЫБОР
    				КОГДА СправочникНоменклатура.ЕстьТоварыДругогоКачества
    					ТОГДА 4 + ВЫБОР
    							КОГДА СправочникНоменклатура.ПометкаУдаления
    								ТОГДА 1
    							ИНАЧЕ 0
    						КОНЕЦ + ВЫБОР
    							КОГДА СправочникНоменклатура.ИспользованиеХарактеристик = ЗНАЧЕНИЕ(Перечисление.ВариантыИспользованияХарактеристикНоменклатуры.НеИспользовать)
    								ТОГДА 0
    							ИНАЧЕ 2
    						КОНЕЦ
    				ИНАЧЕ -1 + ВЫБОР
    						КОГДА СправочникНоменклатура.ПометкаУдаления
    							ТОГДА 1
    						ИНАЧЕ 0
    					КОНЕЦ + ВЫБОР
    						КОГДА СправочникНоменклатура.ИспользованиеХарактеристик = ЗНАЧЕНИЕ(Перечисление.ВариантыИспользованияХарактеристикНоменклатуры.НеИспользовать)
    							ТОГДА 1
    						ИНАЧЕ 3
    					КОНЕЦ
    			КОНЕЦ
    	КОНЕЦ КАК ИндексКартинки,
    
    	&ВидыЦен КАК ВидЦены,
    	ЕСТЬNULL(ЦеныНоменклатуры.Упаковка, ЗНАЧЕНИЕ(Справочник.УпаковкиЕдиницыИзмерения.ПустаяСсылка)) КАК Упаковка,
    	ВЫРАЗИТЬ(ЕСТЬNULL(ЦеныНоменклатуры.Цена, 0) * ВЫБОР
    			КОГДА &Валюта <> ЦеныНоменклатуры.Валюта
    				ТОГДА ВЫБОР
    						КОГДА ЕСТЬNULL(КурсыСрезПоследнихВалютаЦены.Кратность, 0) > 0
    								И ЕСТЬNULL(КурсыСрезПоследнихВалютаЦены.Курс, 0) > 0
    								И ЕСТЬNULL(КурсыСрезПоследнихВалютаДокумента.Кратность, 0) > 0
    								И ЕСТЬNULL(КурсыСрезПоследнихВалютаДокумента.Курс, 0) > 0
    							ТОГДА КурсыСрезПоследнихВалютаЦены.Курс * КурсыСрезПоследнихВалютаДокумента.Кратность / (КурсыСрезПоследнихВалютаДокумента.Курс * КурсыСрезПоследнихВалютаЦены.Кратность)
    						ИНАЧЕ 0
    					КОНЕЦ
    			ИНАЧЕ 1
    		КОНЕЦ КАК ЧИСЛО(15, 2)) КАК Цена,
    	ВЫРАЗИТЬ(ЕСТЬNULL(СвободныеОстатки.ВНаличииОстаток, 0) / ЕСТЬNULL(ВЫБОР
    	КОГДА ЦеныНоменклатуры.Упаковка.ТипИзмеряемойВеличины = ЗНАЧЕНИЕ(Перечисление.ТипыИзмеряемыхВеличин.Вес) И ЦеныНоменклатуры.Номенклатура.ВесИспользовать

    Вот такой текст запроса в типовой конфигурации, причём он присутствует не в одном месте, а склеивается из разных кусков, в зависимости от условий, по моему неплохой претендент на звание лапши.

    VladC, 12 Февраля 2016

    Комментарии (2)
  7. 1C / Говнокод #12388

    −119

    1. 1
    Торм. колодки передние[?(Докум.ТКП_Процент>0,",ост.толщина","")] [?(Докум.ТКП_Процент>0,Докум.ТКП_Процент,"")][?(Докум.ТКП_Процент>0,"мм","")]

    это содержимое одной ячейки таблицы с видом вывода "Шаблон", мало того что автор вынуждает 3 раза обращаться к полю Докум.ТКП_Процент, так ещё и о конкатенации строк видимо ничего не слышал, упрощается до:
    Торм. колодки передние [?(Докум.ТКП_Процент>0,",ост.толщина "+Докум.ТКП_Процент+" мм","")]

    VladC, 04 Января 2013

    Комментарии (6)
  8. 1C / Говнокод #12383

    −119

    1. 001
    2. 002
    3. 003
    4. 004
    5. 005
    6. 006
    7. 007
    8. 008
    9. 009
    10. 010
    11. 011
    12. 012
    13. 013
    14. 014
    15. 015
    16. 016
    17. 017
    18. 018
    19. 019
    20. 020
    21. 021
    22. 022
    23. 023
    24. 024
    25. 025
    26. 026
    27. 027
    28. 028
    29. 029
    30. 030
    31. 031
    32. 032
    33. 033
    34. 034
    35. 035
    36. 036
    37. 037
    38. 038
    39. 039
    40. 040
    41. 041
    42. 042
    43. 043
    44. 044
    45. 045
    46. 046
    47. 047
    48. 048
    49. 049
    50. 050
    51. 051
    52. 052
    53. 053
    54. 054
    55. 055
    56. 056
    57. 057
    58. 058
    59. 059
    60. 060
    61. 061
    62. 062
    63. 063
    64. 064
    65. 065
    66. 066
    67. 067
    68. 068
    69. 069
    70. 070
    71. 071
    72. 072
    73. 073
    74. 074
    75. 075
    76. 076
    77. 077
    78. 078
    79. 079
    80. 080
    81. 081
    82. 082
    83. 083
    84. 084
    85. 085
    86. 086
    87. 087
    88. 088
    89. 089
    90. 090
    91. 091
    92. 092
    93. 093
    94. 094
    95. 095
    96. 096
    97. 097
    98. 098
    99. 099
    100. 100
    Функция РазобратьКодыНеисправностей(СтрокаКодов, ТипКода)
    	СпрКодов = СоздатьОбъект("Справочник.КодыГарантийныхНеисправностей");
    	ГруппаТ1 = СпрКодов.НайтиПоКоду("0",0);
    	ГруппаТ2 = СпрКодов.НайтиПоКоду("00",0);
    	ГруппаТ3 = СпрКодов.НайтиПоКоду("000",0);
    	Если СтрокаКодов <> "" Тогда
    		Разд1 = Найти(СтрокаКодов, ";");
    		Разд2 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд1), ";") + Разд1;
    		Разд3 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд2), ";") + Разд2;
    		Разд4 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд3), ";") + Разд3;
    		Разд5 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд4), ";") + Разд4;
    		Разд6 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд5), ";") + Разд5;
    		Разд7 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд6), ";") + Разд6;
    		Разд8 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд7), ";") + Разд7;
    		Разд9 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд8), ";") + Разд8;
    		Разд10 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд9), ";") + Разд9;
    		Разд11 = Найти(Прав(СтрокаКодов,СтрДлина(СтрокаКодов) - Разд10), ";") + Разд10;
    		Если ТипКода = "Т1" Тогда
    			Если Разд1 > 1 Тогда
    				СпрКодов.ИспользоватьРодителя(ГруппаТ1);
    				Если СпрКодов.НайтиПоКоду("0/" + Лев(СтрокаКодов,Разд1-1),2)  = 1 Тогда 
    					ПолучТ1 = СпрКодов.ТекущийЭлемент();
    					Возврат ПолучТ1;
    				КонецЕсли;
    			КонецЕсли;
    		ИначеЕсли ТипКода = "Т2" Тогда
    			Если Разд2-Разд1 > 1 Тогда
    				СпрКодов.ИспользоватьРодителя(ГруппаТ2);
    				Если СпрКодов.НайтиПоКоду("00/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд1), Разд2 - Разд1 - 1), 2) = 1 Тогда
    					ПолучТ2 = СпрКодов.ТекущийЭлемент();
    					Возврат ПолучТ2;
    				КонецЕсли;
    			КонецЕсли;
    		ИначеЕсли ТипКода = "Т3_1" Тогда
    			Если Разд3-Разд2 > 1 Тогда
    				СпрКодов.ИспользоватьРодителя(ГруппаТ3);
    				Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд2), Разд3 - Разд2 - 1), 2) = 1 Тогда
    					ПолучТ3_1 = СпрКодов.ТекущийЭлемент();
    					Возврат ПолучТ3_1;
    				КонецЕсли;
    			КонецЕсли;
    		ИначеЕсли ТипКода = "Т3_2" Тогда
    			Если Разд4-Разд3 > 1 Тогда
    				СпрКодов.ИспользоватьРодителя(ГруппаТ3);
    				Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд3), Разд4 - Разд3 - 1), 2) = 1 Тогда
    					ПолучТ3_2 = СпрКодов.ТекущийЭлемент();
    					Возврат ПолучТ3_2;
    				КонецЕсли;
    			КонецЕсли;
    		ИначеЕсли ТипКода = "Т3_3" Тогда
    			Если Разд5-Разд4 > 1 Тогда
    				СпрКодов.ИспользоватьРодителя(ГруппаТ3);
    				Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд4), Разд5 - Разд4 - 1), 2) = 1 Тогда
    					ПолучТ3_3 = СпрКодов.ТекущийЭлемент();
    					Возврат ПолучТ3_3;
    				КонецЕсли;
    			КонецЕсли;
    		ИначеЕсли ТипКода = "Т3_4" Тогда
    			Если Разд6-Разд5 > 1 Тогда
    				СпрКодов.ИспользоватьРодителя(ГруппаТ3);
    				Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд5), Разд6 - Разд5 - 1), 2) = 1 Тогда
    					ПолучТ3_4 = СпрКодов.ТекущийЭлемент();
    					Возврат ПолучТ3_4; 
    				КонецЕсли;
    			КонецЕсли;
    		ИначеЕсли ТипКода = "Т3_5" Тогда
    			Если Разд7-Разд6 > 1 Тогда
    				СпрКодов.ИспользоватьРодителя(ГруппаТ3);
    				Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд6), Разд7 - Разд6 - 1), 2) = 1 Тогда
    					ПолучТ3_5 = СпрКодов.ТекущийЭлемент();
    					Возврат ПолучТ3_5;
    				КонецЕсли;
    			КонецЕсли;
    		ИначеЕсли ТипКода = "Т3_6" Тогда
    			Если Разд8-Разд7 > 1 Тогда
    				СпрКодов.ИспользоватьРодителя(ГруппаТ3);
    				Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд7), Разд8 - Разд7 - 1), 2) = 1 Тогда
    					ПолучТ3_6 = СпрКодов.ТекущийЭлемент();
    					Возврат ПолучТ3_6; 
    				КонецЕсли;
    			КонецЕсли;
    		ИначеЕсли ТипКода = "Т3_7" Тогда
    			Если Разд9-Разд8 > 1 Тогда
    				СпрКодов.ИспользоватьРодителя(ГруппаТ3);
    				Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд8), Разд9 - Разд8 - 1), 2) = 1 Тогда
    					ПолучТ3_7 = СпрКодов.ТекущийЭлемент();
    					Возврат ПолучТ3_7; 
    				КонецЕсли;
    			КонецЕсли;
    		ИначеЕсли ТипКода = "Т3_8" Тогда
    			Если Разд10-Разд9 > 1 Тогда
    				СпрКодов.ИспользоватьРодителя(ГруппаТ3);
    				Если СпрКодов.НайтиПоКоду("000/" + Лев(Прав(СтрокаКодов, СтрДлина(СтрокаКодов) - Разд9), Разд10 - Разд9 - 1), 2) = 1 Тогда
    					ПолучТ3_8 = СпрКодов.ТекущийЭлемент();
    					Возврат ПолучТ3_8; 
    				КонецЕсли;
    			КонецЕсли;
    		ИначеЕсли ТипКода = "Т3_9" Тогда
    			Если Разд11-Разд10 > 1 Тогда
    				СпрКодов.ИспользоватьРодителя(ГруппаТ3);

    ещё один высер от автодилера.
    1. начнём сначала, там есть строки:
    ...
    ГруппаТ1 = СпрКодов.НайтиПоКоду("0",0);
    ..., обращаю внимание автора, что функция НайтиПоКоду не возвращает элемент справочника, она лишь говорит есть он или нет, для позиционирования на элементе не хватает строк типа ГруппаТ1 = СпрКодом.ТекущийЭлемент(); и т.д.
    2. дальше. в функцию передаётся параметр СтрокаКодов - это строка с запятыми в качестве разделителей. я конечно понимаю, что в 1С нет встроенной функции для получения массива частей строки типа split в java, но написать свою вообще не сложно, но видимо у автора не хватило мозг..., извините, времени.
    3. теперь смотрим на подобные (их несколько в функции) блоки кода типа:
    СпрКодов.ИспользоватьРодителя(ГруппаТ1);
    Если СпрКодов.НайтиПоКоду("0/" + Лев(СтрокаКодов,Разд1-1),2) = 1 Тогда
    ПолучТ1 = СпрКодов.ТекущийЭлемент();
    Возврат ПолучТ1;
    КонецЕсли;
    строка СпрКодов.ИспользоватьРодителя(ГруппаТ1); не имеет смысла, т.к. ГруппаТ1 содержит значение 1 (см. п.1), а не ссылку на элемент справочника, код работает только благодаря второму параметру - 2, т.к. он заставляет функцию НайтиПоКоду искать по всему справочнику, используя полный код.

    VladC, 02 Января 2013

    Комментарии (41)
  9. 1C / Говнокод #12325

    −122

    1. 1
    2. 2
    3. 3
    Word.FormFields("Data").Result  = СокрЛП(?(ДатаЧисло(ДатаДок2)<10,"0","")+ДатаЧисло(ДатаДок2)); 
    Word.FormFields("Manth").Result = СокрЛП(МесяцПрописью(ДатаДок2)); 
    Word.FormFields("God").Result   = СокрЛП(ДатаГод(ДатаДок2));

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

    VladC, 21 Декабря 2012

    Комментарии (14)
  10. 1C / Говнокод #12252

    −118

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    Если (ЕстьНомен.НайтиЗначение(Док.Номенклатура) > 0)
    	ИЛИ (ЕстьНомен.НайтиЗначение(Док.Номенклатура.Родитель) > 0) 
    	ИЛИ (ЕстьНомен.НайтиЗначение(Док.Номенклатура.Родитель.Родитель) > 0) 
    	ИЛИ (ЕстьНомен.НайтиЗначение(Док.Номенклатура.Родитель.Родитель.Родитель) > 0) 
    	ИЛИ (ЕстьНомен.НайтиЗначение(Док.Номенклатура.Родитель.Родитель.Родитель.Родитель) > 0) 
    	ИЛИ (ЕстьНомен.НайтиЗначение(Док.Номенклатура.Родитель.Родитель.Родитель.Родитель.Родитель) > 0) Тогда
    		ИтогоРМ = ИтогоРМ+Док.Сумма;
    КонецЕсли;

    ЕстьНомен - это элемент формы Список, т.е. код можно упростить до:

    Если ЕстьНомен.Принадлежит(Док.Номенклатура) =1 Тогда
    ИтогоРМ = ИтогоРМ+Док.Сумма;
    КонецЕсли;

    взято из конфигурации одного автодилера.

    VladC, 06 Декабря 2012

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