- 1
- 2
- 3
СтрокаТовары.Качество = СтрокиОст.Качество;
СтрокаТовары.ЕдиницаИзмерения = СтрокиОст.ЕдИзм;
СтрокаТовары.ЕдиницаИзмеренияМест = СтрокиОст.ЕдИзмМест;
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
−98
СтрокаТовары.Качество = СтрокиОст.Качество;
СтрокаТовары.ЕдиницаИзмерения = СтрокиОст.ЕдИзм;
СтрокаТовары.ЕдиницаИзмеренияМест = СтрокиОст.ЕдИзмМест;
Поражает сам подход. А почему бы не выдумать для (поля/объекта/коллекции) ещё одно уникальное имя?! Ведь столько ещё свободных и красивых уникальных имён в мире есть! К тому же "ЕдИзм" - это ведь короче чем "ЕдиницаИзмерения", не правда ли? Дурак перепутает только, правда? Писать ведь будет удобней, правда?
Ладно... про ЗаполнитьЗначенияСвойств писать уже баян. Но ведь СтрокиОст - это строка какой-то таблицы? А в исходной таблице это называлось "ЕдиницаИзмерения", наверное, да? Так зачем лишний раз придумывать псевдоним?!!! Зачем? Почему просто не соединить таблицы друг с другом в запросе? Почему?! Почему нужно постоянно дрочить циклами?!
Циклы надо применять тогда, когда нельзя по-другому, а это узкий круг задач, например те, где нужен CAST которого в 1С нет, или распределение одной таблицы на другую по какому какой нибудь "хитровыдуманной и уникальной" бизнес логике. Даже списание по ФИФО уже придумали как сделать запросом... Не хочу! Хочу циклы...
planaric 20.02.2016 18:00 # 0