JavaScript is not currently enabled, but is required for full CodeSonar manual search and browse functionality.
If you are viewing this file in your hub's Web GUI, enable JavaScript in your browser: you will also need it for GUI functionality.
If you opened this file directly from disk, your browser may be directly suppressing JavaScript functionality: certain browsers perform this suppression on local files (but not files delivered by web servers) for security reasons.
| CodeSonar® 9.0p0 Hot Tips | CONFIDENTIAL | CodeSecure Inc |
One of the following.
| Class Name | Unnecessary Field (C#) | ||||||
|---|---|---|---|---|---|---|---|
| Significance | reliability | ||||||
| Mnemonic | CSHARP.STRUCT.UNFLD | ||||||
| Categories |
|
||||||
| Availability | Available for C# only. |
||||||
| Enabling | Checks for this warning class are enabled by
default. To disable them, add the following WARNING_FILTER rule to the
project configuration file.
WARNING_FILTER += discard class="Unnecessary Field (C#)" |
Japanese (ja)
namespace DocumentationExamples
{
public class ImproperField
{
public static void Main(string[] args)
{ }
private int[] array = new int[100];
private int counter; // "Unnecessary Field (C#)" warning issued here
// - 'counter' should be replaced by local variables.
public void Clear()
{
for (counter = 0; counter < array.Length; counter++)
array[counter] = 1;
counter = 0; // "Unnecessary Field (C#)" warning issued here
// - value never used
}
private int field1; // "Unnecessary Field (C#)" warning issued here
// - field1 only used in constructors
public ImproperField(){
field1 = 0;
field1 = field1+1;
}
}
}
These issues can be resolved by removing the counter field (using a local variable instead), and by marking field1 as readonly.
namespace DocumentationExamples
{
public class ImproperField
{
public static void Main(string[] args)
{ }
private int[] array = new int[100];
public void Clear()
{
for (int counter = 0; counter < array.Length; counter++)
array[counter] = 1;
}
private readonly int field1;
public ImproperField(){
field1 = 0;
field1 = field1+1;
}
}
}
Use variables local to the methods and the constructors instead of fields.
The following configuration file parameters affect checks for this warning class.