具有继承的工厂模式

Question

在与继承一起使用时，Factory模式出现问题，

这是我的代码

public class Animal {
  public int numberOfLegs() { return 2 ;}
}

public class Cat extends Animal {
  public String getSound() {return "Maaaw";}
}
public class Dog extends Animal {
  public String getSound() {return "woof";}
}

public class AnimalFactory {
  public Animal getAnimal(String name){
    Animal an= null ;
    if(name=="cat"){an = new Cat();}
    else if(name=="dog"){an=new Dog();}
    return an ;
  }
}

public class FactoryDemo {

  public static void main(String[] args) {
    AnimalFactory anmF=new AnimalFactory();
    Animal anm=anmF.getAnimal("cat") ;
    System.out.println("legs : "+anm.numberOfLegs()); // working fine
    System.out.println("sound : "+anm.getSound());    // giving error
  }
}

当我运行它时，我无法进入getSound()方法。 它给出了一个错误。

如果我将Animal类定义为Abstract类，这将很好地工作，
但是我要如何处理这样的情况。

Answer 1

您需要为getSound添加抽象方法

public abstract class Animal {
    public int numberOfLegs() { return 2 ;}
    public abstract String getSound();
}

Answer 2

将您的代码更改为：

public abstract class Animal {
    public int numberOfLegs() { 
        return 2;
    }

    public abstract String getSound();
}


public class Cat extends Animal {
    public String getSound() {
        return "Maaaw";
    }
}

public class Dog extends Animal {
    public String getSound() {
        return "woof";
    }
}

public class AnimalFactory {
    public Animal getAnimal(String name) {

        Animal an = null;
        if ("cat".equals(name)) {
            an = new Cat();
        } else if ("dog".equals(name)) {
            an = new Dog();
        } 
        return an;
     }
}

您应添加抽象方法，并在工厂方法中使用equals ，而不要在Objects上使用== 。

Answer 3

您所包含的代码与工厂无关。 如果您引用Factory Method Pattern ，那么作为OP的一部分实现的是不正确的实现。 有两种“工厂”代码设计，一种是我之前指出的工厂方法模式（您的代码绝对不是），而有效Java手册中推荐了工厂，这是Java JDK的首选设计，即valueOf或create*方法。