org.sonar.l10n.squidjava.rules.squid.S1444.html Maven / Gradle / Ivy

Go to download


  There are no good reasons to declare a field "public" and "static" without declaring it "final". Most of the time this is a cheap but crappy way to share a state among several objects. With this approach, any object can do whatever it wants with the shared state like setting it to ... null.


Noncompliant Code Example

public class Greeter {
  public static Foo foo = new Foo(...);
  ...
}



Compliant Solution

public class Greeter {
  public static final Foo foo = new Foo(...);
  ...
}