Guice协助注入了几个工厂方法和null参数

时间:2014-09-03 10:30:14

标签: java dependency-injection guice

我有这个界面和简单的实现:

public interface Data {    
}

import java.nio.file.Path;
import javax.annotation.Nullable;
import javax.inject.Inject;
import com.google.inject.assistedinject.Assisted;

public class SimpleData implements Data {
    @Inject
    public SimpleData(@Assisted @Nullable Path path) {
    }
}

我想使用guice生成一个Factory使用不同的方法。

import java.nio.file.Path;

import javax.annotation.Nullable;

public interface Factory {
    Data create();
    Data load(@Nullable Path path);
}

但是以下模块配置:

import com.google.inject.Guice;
import com.google.inject.Injector;
import com.google.inject.assistedinject.FactoryModuleBuilder;

public class Main {
    public static void main(String[] args) {
        Injector injector = Guice.createInjector(
                binder -> binder.install(
                            new FactoryModuleBuilder().implement(Data.class, SimpleData.class)
                                                      .build(Factory.class)));
        Data data = injector.getInstance(Factory.class).create();
    }
}

失败:

Exception in thread "main" com.google.inject.CreationException: Guice creation errors:

    1) No implementation for java.nio.file.Path annotated with @com.google.inject.assistedinject.Assisted(value=) was bound.
      while locating java.nio.file.Path annotated with @com.google.inject.assistedinject.Assisted(value=)
        for parameter 0 at SimpleData.<init>(SimpleData.java:10)
      at Factory.create(Factory.java:1)
      at com.google.inject.assistedinject.FactoryProvider2.initialize(FactoryProvider2.java:539)
      at com.google.inject.assistedinject.FactoryModuleBuilder$1.configure(FactoryModuleBuilder.java:335)

    1 error
        at com.google.inject.internal.Errors.throwCreationExceptionIfErrorsExist(Errors.java:435)
        at com.google.inject.internal.InternalInjectorCreator.injectDynamically(InternalInjectorCreator.java:175)
        at com.google.inject.internal.InternalInjectorCreator.build(InternalInjectorCreator.java:109)
        at com.google.inject.Guice.createInjector(Guice.java:95)
        at com.google.inject.Guice.createInjector(Guice.java:72)
        at com.google.inject.Guice.createInjector(Guice.java:62)
        at Main.main(Main.java:9)

3 个答案:

答案 0 :(得分:9)

我使用注释@AssistedInject解决了我的问题。引自javadoc:

  

FactoryModuleBuilder一起使用时,使用@AssistedInject注释的构造函数表示可以注入多个构造函数,每个构造函数都有不同的参数。

所以我将注释和构造函数添加到SimpleData类:

public class SimpleData implements Data {
    @AssistedInject
    public SimpleData(@Assisted Path path) {

    }
    @AssistedInject
    public SimpleData() {

    }
}

我从工厂删除了@Nullable注释:

import java.nio.file.Path;

public interface Factory {
    Data create();
    Data load(Path path);
}

答案 1 :(得分:2)

@Nullable并不意味着如果您没有绑定,则会注入null。只有允许将绑定写入null。如果您没有绑定且没有适用的JIT绑定,则注入将失败。

您工厂的create()方法要求Guice找到@Assisted Path绑定,但显然无法找到它,因为您从未创建过它,因此它失败了

老实说,我不确定是否有一种干净的方式来实施这种违约。理想情况下,您应该使用一些绑定注释标记Path并为其添加默认绑定为null,但@Assisted已经是绑定注释,并且不可能在单个注入点上具有多个绑定注释。您可以尝试为@Assisted Path创建绑定:

binder.bind(Path.class).annotatedWith(Assisted.class).toInstance(null);

但是,我不确定它是否可行,因为Assisted对Guice来说可能是特殊的。即使它会起作用,也不是很干净 - 可能与接受Path的其他辅助工厂发生冲突。

答案 2 :(得分:0)

我希望Guice实现某种内部工厂接口,然后暴露其他东西。像这样:

interface InternalFactory {
    Data load(@Nullable Path path);
}

public interface Factory {
    Data load();
    Data load(@Nullable Path path);
}

class FactoryImpl implements Factory {
    @Inject InternalFactory internalFactory;

    @Override
    public Data load() {
        return load(null); // Pass your defaults here
    }

    @Override
    public Data load(@Nullable Path path) {
        // Sadly you'll have to explicitly forward arguments here, but it's not
        // too bad IMO
        return internalFactory.load(path);
    }
}

public class MyModule extends AbstractModule {
    @Override
    protected void configure() {
        install(new FactoryModuleBuilder()
                .implement(Data.class, SimpleData.class)
                .build(InternalFactory.class));

        bind(Factory).to(FactoryImpl.class);
    }
}