Unnecessary else
Categories:
Created by Microsoft, Described by waldo
Description
else
should not be used when the last action in the then
part is an exit
, break
, skip
, quit
, error
.
Bad code
procedure SomeProcedure()
begin
if IsAdjmtBinCodeChanged() then
Error(AdjmtBinCodeChangeNotAllowedErr, ...)
else
Error(BinCodeChangeNotAllowedErr, ...);
end;
Good code
procedure SomeProcedure()
begin
if IsAdjmtBinCodeChanged() then
Error(AdjmtBinCodeChangeNotAllowedErr, ...)
Error(BinCodeChangeNotAllowedErr, ...);
end;
Last modified October 9, 2023: Merge pull request #235 from microsoft/dependabot/npm_and_yarn/postcss-8.4.31 (8c58628)
by Jeremy Vyska