Contents
V6091. Suspicious getter/setter implementation. The 'A' field should probably be returned/assigned instead.
The analyzer has detected a getter/setter that accesses a field different from the one mentioned in the name.
Such errors usually result from inattention or inaccurate use of autocomplete or copy-paste.
Consider the following example:
public class Vector2i
{
private int x;
private int y;
public void setX(int x)
{
this.x = x;
}
public int getX()
{
return x;
}
public void setY(int y)
{
this.y = y;
}
public int getY()
{
return x; // <=
}
}Fixed code:
public class Vector2i
{
private int x;
private int y;
public void setX(int x)
{
this.x = x;
}
public int getX()
{
return x;
}
public void setY(int y)
{
this.y = y;
}
public int getY()
{
return y;
}
}To implement methods like that, it is better to use the means provided by the IDE or code generation provided by the Lombok library.
You can look at examples of errors detected by the V6091 diagnostic. |
Was this page helpful?
