Steeltoe icon indicating copy to clipboard operation
Steeltoe copied to clipboard

Address S134: Control flow statements "if", "switch", "for", "foreach", "while", "do" and "try" should not be nested too deeply

Open bart-vmware opened this issue 3 years ago • 0 comments

Address existing violations of S134: Control flow statements "if", "switch", "for", "foreach", "while", "do" and "try" should not be nested too deeply in the codebase and set severity to Warning in Steeltoe.Debug.ruleset and Steeltoe.Release.ruleset.

To find existing violations, enable the rule (see above) and rebuild src/Steeltoe.All.sln to make them appear in the Output window.

To address the violations, choose from the following on a case-by-case basis:

  • Fix the violation by changing the code to not violate the rule (usually extract method)
  • Suppress the violation in code using #pragma warning disable/restore, preceded by a justification comment if not obvious

bart-vmware avatar Jul 28 '22 10:07 bart-vmware