用静态块覆盖静态字段是不好的做法吗？

Question

我想创建数据结构来捕捉以下想法：

在游戏中，我希望有一个通用的 Skill 类，它可以捕获技能 ID、冷却时间、法力消耗等一般信息。

然后我想拥有定义实际交互和行为的特定技能。所以这些都是从基类Skill扩展而来的。

最后，每个玩家都会有这些特定技能的实例，所以我可以查看每个玩家的技能状态，玩家最近是否使用过等等。

所以我有一个抽象超类Skill，它定义了一些所有技能共有的静态变量，然后对于扩展Skill 的每个单独技能，我使用一个静态块来重新分配静态变量。所以我有以下模式：

class A {
    static int x = 0;
}

class B extends A {
    static {
        x = 1;
    }
}

...

// in a method
A b = new B();
System.out.println(b.x);

上面打印出 1，这正是我想要的行为。我唯一的问题是系统抱怨我以非静态方式访问静态变量。但是我当然不能以这种方式访问​​它，因为我只想将技能视为Skill，而不知道它到底是哪个子类。所以我每次这样做时都必须抑制警告，这让我想到这里是否有更好/更整洁的设计模式。

我曾考虑过将相关变量设为非静态变量，但因为它们在特定技能的所有个实例中应该是静态的，所以我觉得它应该是一个静态变量...

Answer 1

您通常应该避免这样使用全局状态。如果您确定 x 字段将在基类的 all 子类型的 all 实例之间共享，那么放置此类字段的正确位置可能是基类以外的地方。它可能在其他一些配置对象中。

但是即使使用您当前的配置，它也没有任何意义，因为任何修改静态变量的子类都会使该变量对所有类可见。如果子类B 将x 更改为1，然后子类C 将其更改为2，那么B 也可以看到新值。

我认为您在问题中描述的方式，每个子类都应该有自己独立的静态字段。而在抽象基类中，您可以定义一个由每个子类实现的方法，以便访问每个字段：

abstract class A {

    public abstract int getX();
}

class B extends A {
    public static int x = 1;

    public int getX() {
        return x;
    }
}

class C extends A {
    public static int x = 2;

    public int getX() {
        return x;
    }
}

Answer 2

正如一些答案和 cmets 已经指出的那样，您的方法不会按照您想要的方式工作，因为每个静态块都会更改所有扩展 A 的类的静态变量。

改用接口和实例方法：

public interface A {
    int getX();
}

-

public class B implements A {

    private static final int X = 1;

    @Override
    public int getX() {
        return X;
    }
}

-

A myInstance = new B();
System.out.println(myInstance.getX()); // prints "1"