java - 拥有 'setter'/实例变量会导致单例 DAO 出现竞争条件吗？

Question

假设下面的类是一个单例

public class myDAO {
    //determines the tableName and Id column names to based on itemType
    private String itemType;
    public void setItemType(String itemType) {...} 

    //Following methods use itemType to customize their queries
    public boolean findObj(int id){...} 
    public boolean deleteObj(int id){...}
    public boolean updateObj(Obj obj){...}
    //etc...
}

最近使用 setter 重构了代码，从而为 DAO 提供了一些状态。我看到这在 Spring 的配置文件中配置为 singleton-scope。我有一种奇怪的感觉，这可能会导致潜在的竞争条件。对吗？

我真的不确定这是否确实如此，但如果这是真的，恕我直言，弄清楚这将是一场噩梦。我知道这可能不是最好的设计，但我只是想知道当并发线程查询不同的 itemTypes 时这是否会导致竞争条件。我正在考虑将范围更改为 prototype 但不确定在这种情况下是否确实需要。

Answer 1

如果多个线程正在调用 setItemType(...) 然后调用 findObj(...) 并且期望找到具有 id 的对象并且 与项目类型然后是的，你的奇怪感觉是正确的，这是一个潜在的竞争条件，不会在多线程应用程序中工作。

您应该将 itemType 传递给您的每个方法:

public boolean findObj(String itemType, int id){...} 
public boolean deleteObj(String itemType, int id){...}
public boolean updateObj(String itemType, Obj obj){...}

如果 itemType 在 DAO 由 Spring 构建后仅设置一次，那么这没问题，但应该作为单线程完成的 Spring 配置的一部分完成。

如果您必须向所有或大多数方法添加一个 itemType 参数，那么在我看来您不应该再使用单例。相反，您应该考虑拥有一个 DAO 工厂或其他东西，并拥有多个 DAO 实例，每个实例都有自己的项目类型。