5

我有一个类，它有一个构造函数，其中所有参数都由 GUICE 注入。

Public class Order {

    private final ClassOne classOneObj;
    private final ClassTwo classTwoObj;

    @Inject
    public order(ClassOne classOneObj, ClassTwo classTwoObj){
    this.classOneObj = classOneObj;
    this.classTwoObj = classTwoObj;
    }
}

现在，我想再添加一个无法注入的字段（例如 int status）变量。

首先使用所有注入的参数创建一个对象，然后设置无法使用 setter 方法注入的新字段，这是一种好习惯吗？

我想出了另一种方法，我创建了一个工厂类，如下所示：

public class OrderFactory {

    private final ClassOne classOneObj;
    private final ClassTwo classTwoObj;

    @Inject
    public order(ClassOne classOneObj, ClassTwo classTwoObj){
    this.classOneObj = classOneObj;
    this.classTwoObj = classTwoObj;
    }

   //getter methods for all the above variables

    public  ClassOne getclassOneObj(){
          return classOneObj;
    }
    ....

    public Order createOrder(int status) {
        return new Order(status, classOneObj, classTwoObj);
    }
}

然后新的 Order 类看起来像

public class Order {

    int status
    private final ClassOne classOneObj;
    private final ClassTwo classTwoObj;


    public order(int status, ClassOne classOneObj, ClassTwo classTwoObj){
    this.status = status
    this.classOneObj = classOneObj;
    this.classTwoObj = classTwoObj;
    }

    //getter methods for all these member variables
}

现在要创建订单对象，我将首先创建一个 OrderFactory 对象，然后使用“createOrder”方法创建订单对象。

我最终要编写样板代码。不确定这是否是一个好习惯。如果这种方法是正确的或者有任何更好的方法来解决这个问题，任何人都可以提出建议吗？

我在 Google Guice 中阅读，发现有一个功能 @Assisted 可用于此类情况的辅助注射。但我发现这很复杂，无法说服自己是否应该在我的情况下这样做。

提前感谢您的任何建议或指导。

4

2 回答 2

4

你的工厂方法非常好。请不要使用设置器：如果可以使字段不可变，则它应该是不可变的，无论它是否使实例化“方便”。

您可以采取的另一种方法是辅助注射，它可以解决这个确切的问题。这样，您只需定义工厂接口，Guice 就神奇地为您提供了它的实现：

class Order {
  interface Factory {
    Order create(Status status);
  }

  @AssistedInject Order(
      ClassOne one, 
      ClassTwo two, 
      @Assisted Status status) {
  }
}

模块代码：

bind(Order.Factory.class).toProvider(
    FactoryProvider.newFactory(Order.Factory.class, Order.class));

然后客户端注入Factory并使用它，就像他们在您的示例中所做的那样。

于 2013-11-02T00:14:33.787 回答
-1

你通常会注入一些需要花费一些精力来构建的东西。如果您只是注入一个 int 字段，最好只在对象上调用一个 setter 方法（其中注入了一些更复杂的依赖项）。此外，如果一个字段经常更改，正如一个名为“状态”的字段所暗示的那样，那么它也不是一个很好的注入候选者。

于 2013-11-01T22:49:53.817 回答