Our website uses cookies to enhance your browsing experience.
Accept
to the top
>
>
>
V6091. Suspicious getter/setter...
menu mobile close menu
Additional information
toggle menu Contents

V6091. Suspicious getter/setter implementation. The 'A' field should probably be returned/assigned instead.

Aug 10 2020

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.