Java抽象工厂和单例[重复]

Question

我想实现一个抽象工厂的例子，但是具体工厂必须作为单例。

这里举Banas先生的例子：http://www.newthinktank.com/2012/09/abstract-factory-design-pattern/我应该修改UFONemyShipFactory和UFOBossEnemyShipFactory吧？

我为 UFOEnemyShipFactory 尝试了一些东西，但我不确定是否正确：

public class UFOEnemyShipFactory implements EnemyShipFactory{

    private UFOEnemyShipFactory(){};

    private static UFOEnemyShipFactory firstInstance = null;

    public static UFOEnemyShipFactory getInstance(){
    
        if(firstInstance == null){
        
            synchronized(UFOEnemyShipFactory.class){
            
                if(firstInstance == null){
                
                    firstInstance = new UFOEnemyShipFactory();
                
                }
            
            }
        
        }
    
        return firstInstance;
    
    }

    public ESWeapon addESGun(){
        return new ESUFOGun();
    }

    public ESEngine addESEingine() {
        return new ESUFOEngine();
    }

这似乎有点奇怪，我认为我没有在正确的类中应用所需的修改。如果我完全错了，请给我一个简短的解释（为什么我错了，我必须修改哪个类以及为什么？）

提前致谢。

Answer 1

恐怕我没有关于您的 AbstractFactory 问题的完整信息，但我可以帮助您解决单身问题...

通常，当您尝试单例设计模式时，您会指定 private 构造函数，这样其他人就无法使用默认构造函数来创建自己的实例。

作为补充观察，您已经陷入了经典的“双重检查”反模式（不能做的事情）。我在这里写了更多：What's wrong with the following getInstance() method 基本上，不要在同步块之外检查，同步，然后再次检查。看起来它是安全的，但事实并非如此。为此：

public static UFOEnemyShipFactory getInstance(){
    if(firstInstance == null){
        synchronized(UFOEnemyShipFactory.class){
            if(firstInstance == null){
                firstInstance = new UFOEnemyShipFactory();
            }
        }
    }
    return firstInstance;
}

写这个：

public static synchronized UFOEnemyShipFactory getInstance(){
    if(firstInstance == null){
                firstInstance = new UFOEnemyShipFactory();
    }
    return firstInstance;
}

或者很多人会争辩得更好：

private static final UFOEnemyShipFactory firstInstance = new UFOEnemyShipFactory();

public static UFOEnemyShipFactory getInstance(){
    return firstInstance;
}

也是一个风格点。虽然理论上您可以随意命名，但“firstInstance”会提出问题。有“第二个实例”吗？但是，“实例”（至少在 java 意义上）通常意味着“唯一的”。最好只是“实例”。如果您的想法是让 2 个单例在某处闲逛，但它们是不同的，您可以将它们命名为 ufoBossInstance 和 ufoGruntInstance，例如在参考页面的命名之后。