Избегайте использования InstanceOf с шаблоном посетителя - Java

Question

Я искал об этом подэлементе arround здесь, в Stack Overflow. Я нашел следующее решение:

Explanation of Visitor Pattern

Теперь мое дело всего напоминает этому. Мне нужно, однако, избегать использования «instanceOf».

У меня есть игра, которая имеет башни, названные MonkeyTower, CannonTower, OctoTower ... и некоторые другие классы используют instanceOf для сравнения.

Вот пример класса, который использует instanceOf:

BloonsTower.java

public void mousePressed(Point p) { 
     Tower t = null; 
     selectedTower = towerInfo[ insertTowerIdx ].getTower(); 

     if(selectedTower instanceof MonkeyTower) 
      t = tCreator.createMonkey(); 
     else if(selectedTower instanceof OctoTower) 
      t = tCreator.createOctogonal(); 
     else if(selectedTower instanceof CannonTower) 
      t = tCreator.createCannon(); 
     else if(selectedTower instanceof MortarTower) 
      t = tCreator.createMortar(); 
     setMoney(money - towerInfo[ insertTowerIdx ].getPrice()); 
     t.setPosition(p); 
     world.addTower(t); 
     currentState = new SelectTowerState(); 
    } 

ManipulatorCreator.java

if(t instanceof MonkeyTower) 
     return null; 
    else if(t instanceof OctoTower) 
     return new OctoManipulator(t); 
    else if(t instanceof CannonTower) 
     return null; 
    else if(t instanceof MortarTower) 
     return new MortarManipulator((MortarTower)t); 
    return man; 

И GameWriter:

public void saveFile(File file, int round, int money, int lives, World m) throws IOException { 
    PrintWriter out = new PrintWriter(new BufferedWriter(new FileWriter(file))); 

    out.println(round); 
    out.println(money); 
    out.println(lives); 
    Tower []torres = m.getTowers(); 
    out.println(torres.length); // escrever o nº de torres 
    for(Tower t : torres){ 
     Point p = t.getComponent().getPosicao(); 
     // escrever a posição e o tipo de torre 
     out.print(p.x+"\t" + p.y+"\t"); 
     if(t instanceof MonkeyTower) 
      out.println("macaco"); 
     else if(t instanceof OctoTower) 
      out.println("octo"); 
     else if(t instanceof CannonTower) 
      out.println("canhao"); 
     else if(t instanceof MortarTower) 
      out.println("morteiro"); 
    } 

    out.close(); 
} 

Что я создал класс для посетителей, которая посещает каждую башню:

public class TowerVisitor implements Visitor{ 

    public void visit(MonkeyTower monkey) { 
     // TODO Auto-generated method stub 
    } 

    public void visit(CannonTower cannon) { 
     // TODO Auto-generated method stub 
    } 

    public void visit(MortarTower mortar) { 
     // TODO Auto-generated method stub 
    } 

    public void visit(OctoTower octo) { 
     // TODO Auto-generated method stub 
    } 
} 

И в каждой башне я создал есть метод accept, который возвращает себе

Теперь я застрял в что положить внутри метода visit и как использовать шаблон для переключения всех instanceOf.

Спасибо.

Answer 1

Вы должны использовать FactoryMethod или AbstractFactory шаблон (если вы хотите создать ManipulatorDactory independentemt of Tower) вместо Visitor.

Посмотрите на это Factory Method и Abstract Facotry для получения более подробной информации.

Некоторые примеры кода:

public class FactoryMethodDemo{ 

    public FactoryMethodDemo(){ 

    } 
    public Tower getTower(String type){ 
     if (type.equalsIgnoreCase("monkey")){ 
      return new MonkeyTower(); 
     }else if (type.equalsIgnoreCase("octo")){ 
      return new OctoTower(); 
     }else if (type.equalsIgnoreCase("canon")){ 
      return new CannonTower(); 
     }else if (type.equalsIgnoreCase("mortal")){ 
      return new MortarTower(); 
     } 
     return null; 
    } 
    public static void main(String args[]){ 
     FactoryMethodDemo factory = new FactoryMethodDemo(); 
     Tower tower = factory.getTower(args[0]); 
     System.out.println("Tower:"+tower); 
    } 
} 
abstract class Tower{ 
    Manipulator manipulator = null; 
    public Tower(){ 

    } 
    public String toString(){ 
     return this.getClass().getSimpleName()+":manipulator:"+manipulator; 
    } 
    public Manipulator getManipulator(){ 
     return manipulator; 
    } 
} 
class MonkeyTower extends Tower{ 
    public MonkeyTower(){ 

    } 
} 
class OctoTower extends Tower{ 
    public OctoTower(){ 
     manipulator = new OctoManipulator(); 
    } 

} 
class CannonTower extends Tower{ 
    public CannonTower(){ 

    } 
} 
class MortarTower extends Tower{ 
    public MortarTower(){ 
     manipulator = new MortarManipulator(); 
    } 

} 
class Manipulator{ 

} 
class OctoManipulator extends Manipulator{ 

} 
class MortarManipulator extends Manipulator{ 

} 

выход:

D:\Study\Java>java FactoryMethodDemo monkey 
Tower:MonkeyTower:manipulator:null 

D:\Study\Java>java FactoryMethodDemo mortal 
Tower:MortarTower:manipulator:MortarManipulator@4187a8e0 

D:\Study\Java>java FactoryMethodDemo octo 
Tower:OctoTower:manipulator:OctoManipulator@46fb3d6 

D:\Study\Java>java FactoryMethodDemo canon 
Tower:CannonTower:manipulator:null 

Похожие SE вопросы:

Factory Pattern. When to use factory methods?

What is the basic difference between the Factory and Abstract Factory Patterns?

Answer 2

Вы должны использовать шаблоны ядра Object Oriented Programming, а именно наследование, а не шаблон посетителя. У вас есть несколько различных типов башен и подобные действия (создавать, манипулировать, toString и т. Д.), Которые должны быть реализованы по-разному для каждого типа башни. Классический пример наследования.

public abstract class Tower { 
    public abstract Tower create(); 
    public abstract Manipulator manipulate(); 
} 

--- 

public class MortarTower extends Tower { 
    @Override 
    public MortarTower create() { 
    return new MortarTower(); 
    } 

    @Override 
    public MortarManipulator manipulate() { 
    return new MortarManipulator(this); 
    } 

    @Override 
    public String toString() { 
    return "morteiro"; 
    } 
} 

--- 

public void mousePressed(Point p) { 
    selectedTower = towerInfo[insertTowerIdx].getTower(); 
    setMoney(money - towerInfo[insertTowerIdx].getPrice()); 
    Tower t = selectedTower.create(); 
    t.setPosition(p); 
    world.addTower(t); 
    currentState = new SelectTowerState(); 
} 

И так далее.