|
![]() |
#1 |
Участник
|
Цитата:
Сообщение от RedFox
![]() Стоить хотя бы взглянуть на код IF HasErrorOutbnd THEN BEGIN и IF HasErrorInbnd THEN BEGIN. Различие только в одной проверке: (NewItemDocLine."Location Code" <> OldItemDocLine."Location Code").
А вот примение функции CallItemTracking2 так и не нашёл. ![]() Вобщем было бы не так жалко и смешно, если бы не грусно.. Её раз убеждаюсь, что код не только на переменные, но даже на "простое дублирование" даже не проверяют... "Различие только в одной проверке" - да, не спорю. Но такое в наве Вы найдете повсеместно. А функции, которые отличаются типом одного параметра! Тоже не редкость! Здесь нет возможности создавать перегруженные методы и использовать прочие обыденные радости жизни, как в любом более-менее нормальном языке... В наве иногда проще и полезнее продублировать код, чем городить, простите, хрень, из кучи проверок. Увы... |
|
![]() |
#2 |
Участник
|
Цитата:
Цитата:
А функции, которые отличаются типом одного параметра! Тоже не редкость! Здесь нет возможности создавать перегруженные методы и использовать прочие обыденные радости жизни, как в любом более-менее нормальном языке...
![]() Цитата:
В наве иногда проще и полезнее продублировать код, чем городить, простите, хрень, из кучи проверок. Увы...
Или легче написать дублирующий код, чем попытаться разобраться в существующем? При этом этот когд ПРОСТО НЕ НУЖЕН (функция CallItemTracking2)! Я мог даже сказать какой кодеюнит был "прообразом" и где эта функция работает (когда доделывал код) ![]() |
|
![]() |
#3 |
Участник
|
Цитата:
Сообщение от RedFox
![]() Тоесть в итоге Вы частично согласились, что лучше попытаться написать немного оптимальнее код, кем плодить функции?
Или легче написать дублирующий код, чем попытаться разобраться в существующем? При этом этот когд ПРОСТО НЕ НУЖЕН (функция CallItemTracking2)! Я мог даже сказать какой кодеюнит был "прообразом" и где эта функция работает (когда доделывал код) ![]() Вопрос - в чем проблема "подкорректировать код триггера PrintRecords для Item Shipment Header? |
|