java - Spring bean 的静态方法。

Question

关闭。这个问题是 opinion-based 。目前不接受答案。

---

想要改进这个问题吗？更新问题，以便 editing this post 可以用事实和引文来回答它。 .

已关闭 4 年前。

Improve this question

public class QuestionStaticObject {
    private static String staticprefix;
    private static BeanFactory factory;

    @Value("${account.prefix}")
    public void setPrefix(String prefix) {
        staticprefix = prefix;
    }

    public static String getPrefix() {
        return staticprefix;
    }

    @Autowired
    public void setFactory(BeanFactory f) {
        factory = f;
    }

    public static BeanFactory getFactory() {
        return factory;
    }

    public static String foo1(String id) {
        String uuid = getPrefix() + id + "-" + System.currentTimeMillis() / 1000;
        return uuid;
    }

    public static boolean foo2() throws GFMException {  
        OtherObject obj = getFactory().getBean(OtherObject.class);
        return obj.foo1();
    }
}

我的团队是在上面编写 Java 代码的团队。但我认为这段代码非常危险，因为如果 spring bean 的加载较晚，那么静态方法将引用 null 工厂或 null 'staticprefix'。

您对此有何看法？

Answer 1

即使我认为编写这样的静态内容是错误的，我也不认为这个特定的代码是危险的。

setPrefix 会在对象实例化后立即调用，因此不存在 NPE 风险。

该对象默认是单例。所以也不存在并发调用的风险。但这是糟糕的代码设计。