重构命名 bean 以避免代码重复

Question

我检测到我在命名 bean 中系统地重复代码，所以我正在寻找一种方法来重新组织代码。

例如，我有三个命名的 bean，它们重复有关 Collars 的代码：

@Named(value = "dogController")
@ViewScoped
public class DogController implements Serializable {

    private Dog dog;  // Class Dog has a List<Collar>
    private Collar collar = new Collar();  // to be used in an XHTML form
    ....
    public addCollar(){
        //adds collar in dog
    }
    public removeCollar(){
        //removes collar from dog
    }
  }

所以 bean CatController 和 BirdController 也有相同的代码。 Photo 也是如此，它们都重复代码。所以我认为我落后于一个模式。

我的直接步骤是为Dog、Cat 和Bird 创建一个interface HasCollar。然后我把重复的代码移到一个新的bean中：

@Named(value = "collarController")
@ViewScoped
public class CollarController implements Serializable {

    private HasCollar animal;
    private Collar collar = new Collar();  // to be used from the XHTML
    ....
    public addCollar(){
        //adds collar in animal
    }
    public removeCollar(){
        //removes collar from animal
    }
  }

到目前为止一切顺利。我现在的问题是：

• 如何初始化animal？特别是因为同一个 XHTML 可能同时使用 Dog 和 Cat。之前没问题，因为每种类型都有一个 Collar 对象。如果我必须注射它，我不知道如何注射。 （问题似乎与this 相似，但略有不同）。
• 通常我会从 CollarController 扩展命名 bean 并重用所有功能，但这是不可能的，因为命名 bean 已经从另一个类继承。

那么，我该如何重构这种情况以尽可能减少重复代码？

（如果需要更多信息，请告诉我）

Answer 1

考虑Dog 和Collar 之间的关系：a Dog has-a Collar。

换句话说，addCollar() 和 removeCollar() 是 Dog 的操作，因此这些方法属于 Dog：

 public class Dog {
    public addCollar(){
        //adds collar
    }
    public removeCollar(){
        //removes collar
    }
}

Cat、Bird 等也是如此。此时代码仍然重复。

然后控制器可以简单地委托：

@Named(value = "dogController")
@ViewScoped
public class DogController implements Serializable {
    private Dog dog;  // Class Dog has a List<Collar>
    private Collar collar = new Collar();  // to be used in an XHTML form

    public addCollar(){
        //adds collar in dog
        dog.addCollar();
    }
    public removeCollar(){
        //removes collar from dog
        dog.removeCollar();
    }
}

Cat、Bird 等控制器也是如此。

现在通用代码可以放在一个单独的类中如：

public class CollarHandler {
    private List<Collar> collars;
    public addCollar(){
        //adds collar
    }
    public removeCollar(){
        // remove a collar;
    }
}

而Dog 的构造函数采用CollarHandler，其方法委托给处理程序：

public class Dog {
    private CollarHandler collarHandler;
    public Dog(CollarHandler collarHandler) {this.collarHandler = collarHandler; }

    public addCollar(){
        collarHandler.addCollar();
    }
    public removeCollar(){
        collarHandler.removeCollar();
    }
}

Cat、Bird 等也是如此。

没有通用接口也没有继承。每个控制器仍然是独立的，但行为可以共享。