Баг или не баг?

[Murlin]

Цитата:

Сообщение от dech

Это конечно да, но такое я бы назвал хардкодом.

Да майкрософту тоже надо от кода отказаца типа вот этого
while select inventTrans
where inventTrans.TransType > a && inventTrans.TransType < b

либо отказаться от использования метода index2Symbol

[db]

Цитата:

Сообщение от Murlin

Да майкрософту тоже надо от кода отказаца типа вот этого
while select inventTrans
where inventTrans.TransType > a && inventTrans.TransType < b

либо отказаться от использования метода index2Symbol

да что вы так упелись то
вызов index2Symbol с значением (не индексом) енума в качестве параметра это не осмысленное желание/правило мс, а обычный баг сделанный обычным разработчиком. От того что он работал в это время в мс ничего не меняется - это просто баг (проявляется он только не всегда). Таких кривых вызовов по всему приложению dax2009 ru8 с sl2 слоем около десятка - поправить 10 минут и зарегистрировать для очистки совести в мс. зачем всё усложнять?

[Murlin]

Цитата:

Сообщение от db

да что вы так упелись то
вызов index2Symbol с значением (не индексом) енума в качестве параметра это не осмысленное желание/правило мс, а обычный баг сделанный обычным разработчиком. От того что он работал в это время в мс ничего не меняется - это просто баг (проявляется он только не всегда). Таких кривых вызовов по всему приложению dax2009 ru8 с sl2 слоем около десятка - поправить 10 минут и зарегистрировать для очистки совести в мс. зачем всё усложнять?

Вот я про это и пишу, баг это или не баг...
Либо делаем enum с разрывом, тогда баг, либо без разрыва тогда наш баг.

[dech]

Цитата:

Сообщение от Murlin

Вот я про это и пишу, баг это или не баг...
Либо делаем enum с разрывом, тогда баг, либо без разрыва тогда наш баг.

Делайте енум с разрывом и исправляйте на value2Symbol() и хватит уже морочить всем голову!

[Murlin]

Цитата:

Сообщение от dech

Делайте енум с разрывом и исправляйте на value2Symbol() и хватит уже морочить всем голову!

Так не пойдет... непорядок.

[kair84]

Перечитал топик, и действительно, чуть мозг не взорвался.

index2Symbol() - если в него передавать value - баг, и однократно "просканировать" код на предмет вызова метода, не так уж и сложно

Enum с разрывом - нормальная практика,

Цитата:

inventTrans.TransType > a и inventTrans.TransType < b

в этом случае inventTransType не самый удачный пример, лучше обратить внимание на StatusIssue StatusReceipt и другие, когда элементы Enum-a действительно отражают стадии какого либо процесса, и чтобы добавлять элементы в такие Enum-ы, нужно 10 раз подумать, а если у Вас до, после, или между такими этапами добавляются свои, то лучше создать второй Enum типа StatusIssueSub, и соответственно поле для него, и предшествовать, продолжать или детализировать процесс сколько влезет

[Murlin]

Цитата:

Сообщение от kair84

Перечитал топик, и действительно, чуть мозг не взорвался.

index2Symbol() - если в него передавать value - баг, и однократно "просканировать" код на предмет вызова метода, не так уж и сложно

Enum с разрывом - нормальная практика,

в этом случае inventTransType не самый удачный пример, лучше обратить внимание на StatusIssue StatusReceipt и другие, когда элементы Enum-a действительно отражают стадии какого либо процесса, и чтобы добавлять элементы в такие Enum-ы, нужно 10 раз подумать, а если у Вас до, после, или между такими этапами добавляются свои, то лучше создать второй Enum типа StatusIssueSub, и соответственно поле для него, и предшествовать, продолжать или детализировать процесс сколько влезет

Гипотетическая ситуация совсем не обязательно что мне требуется именно детализация а не добавление нового типа проводок, велосипед изобретаете.
Однократно можно также изменить relation и код про который я писал. Ничего вы не меняете абсолютно. Вопрос был в том что будет если будет обновления лежащие на нижних слоях, опять ничего не меняется.