V3008. The 'x' variable is assigned values twice successively. Perhaps this is a mistake.
The analyzer has detected an error that has to do with assigning values to one and the same variable twice in a row, while this variable is not used in any way between the assignments.
Consider this example:
A = GetA(); A = GetB();
The 'A' variable being assigned values twice might indicate a bug. The code should have most probably looked like this:
A = GetA(); B = GetB();
Cases when the variable is used between the assignments are treated as correct and do not trigger the warning:
A = 1; A = Foo(A);
The following is an example of the bug taken from a real-life application:
.... if (bool.TryParse(setting, out value)) _singleSignOn = value; _singleSignOn = false; ....
A correct version of this code should look like this:
.... if (bool.TryParse(setting, out value)) _singleSignOn = value; else _singleSignOn = false; ....
The analyzer might output false positives sometimes. This happens when such variable assignments are used for debugging purposes. For example:
status = Foo1(); status = Foo2();
The false positive in this code can be handled in a number of ways:
- You can suppress it by inserting comment "//-V3008".
- You can forbid the analyzer to output diagnostic V3008 for any case where variable 'status' is used. To do that, insert comment "//-V:status:3008".
- You can remove idle assignments from the code.
- Perhaps this code is incorrect, so we have to check the value of the 'status' variable.
This diagnostic is classified as:
You can look at examples of errors detected by the V3008 diagnostic.