避免使用重载mutator的instanceof

Question

我有两个继承自Player的类：SimplePlayer和InterleavedPlayer。

B类有以下方法：setPlayer（SimplePlayer player）和setPlayer（InterleavedPlayer player）

A类有一个Player字段，它被设计为SimplePlayer或InterleavedPlayer。 A类想要在这些玩家上调用B.setPlayer（），而不关心对象所属的Player的子类。

我为子类重载了B.setPlayer（），以避免在广义的B.setPlayer（播放器播放器）方法中使用instanceof。

但是在A调用B.setPlayer（）时，我得到一个未找到符号的错误，指的是B.setPlayer（Player）。

我应该在这里用什么设计方法来解决问题？ （实际上是一个玩家，因此它在理论上“知道”它传递给B的东西。但是，由于A不分青红皂白地对待它的玩家的方式，似乎只有一个玩家才有意义，而不是必须区分SimplePlayer和InterleavedPlayer）。

Answer 1

使用instanceof并不一定是邪恶的。 像所有设计决策一样，它涉及权衡。 我想这里要问的问题是，有问题的代码是否更多地与A或B相结合？

例如，如果创建了一个新的Player子类，会发生什么？ 你需要修改B来添加一个新的setPlayer方法吗？ 如果是这样，在B中通过instanceof调度通用方法是有意义的，因为无论如何你都必须修改B.

Answer 2

您收到错误的原因是我们没有对java中的方法参数进行运行时调度，只是在调用它的对象上。 这意味着您调用它的对象的实际运行时类（ B ）会影响选择哪个函数，但是参数（ Player ）的唯一编译时类会影响函数的选择。

解决这个问题的答案将取决于为什么B需要2个函数，以及是否可以将差异移动到各个玩家类中 ，以及这是否具有设计意义。 即由于B或由于玩家类别而发生额外的计算？

Answer 3

在A中有重载方法：

private Player;

public void setPlayer(SimplePlager player) {
    this.player = player;
    b.setPlayer(player);
}

// similar for InterleavedPlayer

或者，如果调用来自构造函数，您也可以重载它。

Answer 4

你需要发布代码才能修复它...你应该使用mediator，因为你是在两个对象之间翻转调用。

我有两个继承自Player的类：SimplePlayer和InterleavedPlayer

好

class Player{}
class SimplePlayer extends Player {}
class InterleavedPlayer extends Player{}

A类有一个Player字段，它被设计为SimplePlayer或InterleavedPlayer。 A类想要在这些玩家上调用B.setPlayer（），而不关心对象所属的Player的子类。

好

class A { Player field; }

我为子类重载了B.setPlayer（），以避免在广义的B.setPlayer（播放器播放器）方法中使用instanceof。

好

class B { 
    void setPlayer(InterleavedPlayer player) {  out.println( player );  }  
}

“但是在A调用B.setPlayer（）时，我得到一个未找到符号的错误，指的是B.setPlayer（播放器）。” 好。

B需要一个交错的玩家吗？ 在A中它调用B.setPlayer（（InterleavedPlayer）thePlayer），将播放器转换为Interleaved播放器，你可以通过很多不同的方式解决这个问题，但没有代码可以看，它可以是很多单词....

特别是，所需的代码是Player :: setPlayer和Interleaved :: setPlayer