If value exists check
This check could look very trivial but it could definitely help us save from getting future bugs.
If we are writing any code for any ISV solution or be it for any implementation where we have integration scenarios
For any validation check based on a field or any process which we are calling it is always better to have a check if the field value is available only then call our logic.
Dynamics AX / Dynamics 365 FO Technical Consultant
4 年Hm. If you accept empty values in this.CheckNumSeqGroup this code is not needed. If this.CheckNumSeqGroup("") returns false - the method name may be confusing, better to rename it, as you actually accept empty values. Also, it is not good to overwrite next() call(ok may be false or true before calling your method), I think it is a BP error for ISV, your solution should be extensible.