Unnecessary ELSE

ELSE should not be used when the last action in the THEN part is an EXIT, BREAK, SKIP, QUIT, ERROR.

Bad code

IF IsAdjmtBinCodeChanged THEN
    ERROR(AdjmtBinCodeChangeNotAllowedErr,...)
ELSE
    ERROR(BinCodeChangeNotAllowedErr,...);

Good code

IF IsAdjmtBinCodeChanged THEN
    ERROR(AdjmtBinCodeChangeNotAllowedErr,...)
ERROR(BinCodeChangeNotAllowedErr,...);

Feedback

Was this page helpful?

Sorry to hear that. Please tell us how we can improve.