Skip to content
ptittof57 edited this page Aug 15, 2015 · 6 revisions
<title>SA1409: RemoveUnnecessaryCode</title> <script src="script/helpstudio.js" type="text/javascript"></script> <script src="script/StandardText.js" type="text/jscript"></script>
<script type="text/jscript">WritePageTop(document.title);</script>

TypeName

RemoveUnnecessaryCode

CheckId

SA1409

Category

Maintainability Rules

Cause

A C# file contains code which is unnecessary and can be removed without changing the overall logic of the code.

Rule Description

A violation of this rule occurs when the file contains code which can be removed without changing the overall logic of the code.

For example, the following try-catch statement could be removed completely since the try and catch blocks are both empty.

try

{

}

catch (Exception ex)

{

}

The try-finally statement below does contain code within the try block, but it does not contain any catch blocks, and the finally block is empty. Thus, the try-finally is not performing any useful function and can be removed.

try

{

this.Method();

}

finally

{

}

As a final example, the unsafe statement below is empty, and thus provides no value.

unsafe

{

}

How to Fix Violations

The fix a violation of this rule, remove the unnecessary code, or fill in the code with additional statements.

How to Suppress Violations

[SuppressMessage("StyleCop.CSharp.MaintainabilityRules", "SA1409:RemoveUnnecessaryCode", Justification = "Reviewed.")]
Clone this wiki locally