Static fields of the parent class should not be accessed through child class instances JAVA-W1030
Anti-pattern
4 months ago
—
5 months old
EXIT_ON_CLOSE should be accessed through WindowConstants, not JFrame
46 * Create the frame.
47 */
48 public ShapeDisplayer() {
49 setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 50 setBounds(100, 100, 650, 650);
51 final GridBagLayout gridBagLayout = new GridBagLayout();
52 gridBagLayout.columnWidths = new int[] { 1, 0 };
Description
Non-private static members of the parent class are accessible by child classes. However, it is a bad practice to do so, because it obscures where a value was actually declared. Always use only the declaring class to access static members.
Bad Practice
class SomeClass {
static Object staticData = null;
}
class SomeChildClass {
public void method() {
// Accessing the static value declared within the parent class through the child class.
SomeChildClass.staticData = this;
}
}
Recommended
class SomeChildClass {
public void method() {
// We are now accessing the static value through the parent class.
SomeClass.staticData = this;
}
}