Внедрение зависимостей: определение области (Guice, Spring, Wh независимо) - PullRequest
13 голосов
/ 10 июня 2010

Вот упрощенная версия моих потребностей.

У меня есть программа, в которой каждый объект B имеет свои собственные объекты C и D, внедренные через Guice.Кроме того, объект A вводится в каждый объект C и D.

Что я хочу : чтобы для каждого объекта B его объекты C и D были инъецированы одним и тем же объектом A.

[Edit-Start]

(1) Guice поддерживает режимы "singleton" и "prototype".Тем не менее, мне нужно что-то среднее: мне нужно, чтобы A был одноэлементным WRT для данного объекта B (чтобы C и D, введенные в объект B, имели общий объект A).Для другого объекта B я хочу другой объект A. Так что это одиночный объект, но для ограниченной области программы (фактически, ограниченной области структуры данных).

(2) Я не против решениякоторый использует метод (установщик) - или инъекцию поля.

(3) Я пытался несколько раз, чтобы достичь этого, и всегда чувствовал, что мне нужно только реализовать какую-то специальную штуку из контейнера DI, чтобы сделать эторабота - но это никогда не работало.Таким образом, я ищу подробное решение (а не просто «махание рукой»)

[Edit-End]

В частности, я хочу вывод программы (ниже) будет:

Created C0 with [A0]
Created D0 with [A0]
Created B0 with [C0, D0]
Created C1 with [A1]
Created D1 with [A1]
Created B1 with [C1, D1]

Где он в настоящее время производит следующий вывод:

Created C0 with [A0]
Created D0 with [A1]  <-- Should be A0
Created B0 with [C0, D0]
Created C1 with [A2]  <-- Should be A1
Created D1 with [A3]  <-- Should be A1
Created B1 with [C1, D1]

Я ожидаю, что контейнеры DI позволят такую ​​настройку, но пока мне не повезло внайти решение.Ниже приведен мой код на основе Guice, но приветствуется решение на основе Spring (или других контейнеров DI).

  import java.util.Arrays;
  import com.google.inject.*;

  public class Main {

    public static class Super {
      private static Map<Class<?>,Integer> map = new HashMap<Class<?>,Integer>();

      private Integer value;

      public Super(Object... args) {
        value = map.get(getClass());
        value = value == null ? 0 : ++value;
        map.put(getClass(), value);

        if(args.length > 0)
          System.out.println("Created " + this + " with " + Arrays.toString(args));
      }

      @Override
      public final String toString() {
        return "" + getClass().getSimpleName().charAt(0) + value;
      }
    }

    public interface A { }  
    public static class AImpl extends Super implements A  { } 

    public interface B { }  
    public static class BImpl extends Super implements B {
      @Inject public BImpl(C c, D d) { super(c,d); }
    }

    public interface C { }  
    public static class CImpl extends Super implements C  {
      @Inject public CImpl(A a) { super(a); }
    }

    public interface D { }  
    public static class DImpl extends Super implements D {
      @Inject public DImpl(A a) { super(a); }
    }


    public static class MyModule extends AbstractModule {
      @Override
      protected void configure() {
        bind(A.class).to(AImpl.class);
        bind(B.class).to(BImpl.class);      
        bind(C.class).to(CImpl.class);      
        bind(D.class).to(DImpl.class);      
      }    
    }

    public static void main(String[] args) {
      Injector inj = Guice.createInjector(new MyModule());
      inj.getInstance(B.class);    
      inj.getInstance(B.class);    
    }  
  }

Ответы [ 7 ]

13 голосов
/ 13 июня 2010

Вот одно решение на основе вашего исходного кода - есть три изменения:

  1. Переместите привязки для A, C и D в отдельный субмодуль
  2. Пометить A как синглтон в субмодуле
  3. Используйте метод @Provides в главном модуле, чтобы обеспечить экземпляры BImpl с
    новый дочерний инжектор для каждого запроса - вот где появляется субмодуль

Это работает, потому что одноэлементная привязка для A теперь ограничена каждым дочерним инжектором.

[Примечание: вы всегда можете кэшировать экземпляр субмодуля в поле
main-module, если вы не хотите создавать его для каждого запроса B]

  import java.util.*;
  import com.google.inject.*;

  public class Main {

    public static class Super {
      private static Map<Class<?>,Integer> map = new HashMap<Class<?>,Integer>();

      private Integer value;

      public Super(Object... args) {
        value = map.get(getClass());
        value = value == null ? 0 : ++value;
        map.put(getClass(), value);

        if(args.length > 0)
          System.out.println("Created " + this + " with " + Arrays.toString(args));
      }

      @Override
      public final String toString() {
        return "" + getClass().getSimpleName().charAt(0) + value;
      }
    }

    public interface A { }  
    public static class AImpl extends Super implements A  { } 

    public interface B { }  
    public static class BImpl extends Super implements B {
      @Inject public BImpl(C c, D d) { super(c,d); }
    }

    public interface C { }  
    public static class CImpl extends Super implements C  {
      @Inject public CImpl(A a) { super(a); }
    }

    public interface D { }  
    public static class DImpl extends Super implements D {
      @Inject public DImpl(A a) { super(a); }
    }

    public static class MyModule extends AbstractModule {
      @Override
      protected void configure() {} 

  // >>>>>>>>
      @Provides
      B builder( Injector injector ) {
        return injector.createChildInjector( new SubModule() ).getInstance( BImpl.class );
      }
  // <<<<<<<<
    }

  // >>>>>>>>
    public static class SubModule extends AbstractModule {
      @Override
      protected void configure() {
        bind(A.class).to(AImpl.class).in( Scopes.SINGLETON );
        bind(C.class).to(CImpl.class);      
        bind(D.class).to(DImpl.class);      
      }    
    }
  // <<<<<<<<

    public static void main(String[] args) {
      Injector inj = Guice.createInjector(new MyModule());
      inj.getInstance(B.class);    
      inj.getInstance(B.class);    
    }  
  }
3 голосов
/ 11 июня 2010

Теперь я не знаю, нужно ли вам создавать BImpl, CImpl и DImpl с помощью Guice (например, для обеспечения AOP), но если нет, то это просто:

public static class MyModule extends AbstractModule {
  @Override
  protected void configure() {
    bind(A.class).to(AImpl.class);
  }

  @Provides
  protected B provideB(A a) {
    C c = new CImpl(a);
    D d = new DImpl(a);
    return new BImpl(c, d);
  }
}

В качестве альтернативы (и я знаю, что вы не указали это в своем вопросе), если вы можете связать каждый экземпляр B, который вы используете, с другой аннотацией привязки, вы можете использовать такой закрытый модуль, который вы ' d добавить один раз на привязку аннотации при создании инжектора:

public static class MyOtherModule extends PrivateModule {
  private final Annotation annotation;

  public MyOtherModule(Annotation annotation) {
    this.annotation = annotation;
  }

  @Override
  protected void configure() {
    bind(A.class).to(AImpl.class).in(Scopes.SINGLETON);
    bind(C.class).to(CImpl.class);
    bind(D.class).to(DImpl.class);
    bind(B.class).annotatedWith(annotation).to(BImpl.class);
    expose(B.class).annotatedWith(annotation);
  }
}

main для этого выглядит так:

public static void main(String[] args) {
  Injector inj = Guice.createInjector(new MyOtherModule(Names.named("first")),
      new MyOtherModule(Names.named("second")));
  inj.getInstance(Key.get(B.class, Names.named("first")));
  inj.getInstance(Key.get(B.class, Names.named("second")));
}

Полагаю, есть и другие возможности.

1 голос
/ 10 июня 2010
  import java.util.*;
  import com.google.inject.*;
  import com.google.inject.name.*;

  public class Main {

    public static class Super {
      private static Map<Class<?>,Integer> map = new HashMap<Class<?>,Integer>();

      private Integer value;

      public Super(Object... args) {
        value = map.get(getClass());
        value = value == null ? 0 : ++value;
        map.put(getClass(), value);

        if(args.length > 0)
          System.out.println("Created " + this + " with " + Arrays.toString(args));
      }

      @Override
      public final String toString() {
        return "" + getClass().getSimpleName().charAt(0) + value;
      }
    }

    public interface A { }  
    public static class AImpl extends Super implements A  { } 

    public interface B { }  
    public static class BImpl extends Super implements B {
      @Inject public BImpl(C c, D d) { super(c,d); }
    }

    public interface C { }  
    public static class CImpl extends Super implements C  {
      @Inject public CImpl( A a) { super(a); }
    }

    public interface D { }  
    public static class DImpl extends Super implements D {
      @Inject public DImpl(A a) { super(a); }
    } 


    public static class MyModule extends AbstractModule {
      @Override
      protected void configure() {
        CachingScope cachedScope = new CachingScope();
        bind(C.class).to(CImpl.class);      
        bind(D.class).to(DImpl.class);  
        bind(B.class).to(BImpl.class).in(new ClearingScope(cachedScope));
        bind(A.class).to(AImpl.class).in(cachedScope);
      }
    }

    public static class CachingScope implements Scope {
        List<CacheProvider<?>> providers = new LinkedList<CacheProvider<?>>();
        public <T> Provider<T> scope(Key<T> key, Provider<T> unscoped) {
            CacheProvider<T> t = new CacheProvider<T>(unscoped);
            providers.add(t);
            return t;
        }

        public void clear() {
            for(CacheProvider c : providers) {
                c.clear();
            }
        }
    }

    public static class ClearingScope implements Scope {
        CachingScope scopeToClear;
        ClearingScope(CachingScope scopeToClear) {
            this.scopeToClear = scopeToClear;
        }
        public <T> Provider<T>  scope(Key<T> key, Provider<T> unscoped) {
            return new ClearingProvider<T>(unscoped, scopeToClear);
        }
    }

    public static class CacheProvider<T> implements Provider<T> {
        T t;
        Provider<T> unscoped;
        CacheProvider(Provider<T> unscoped) {
            this.unscoped = unscoped;
        }
        public T get() {
            if(t == null) {
                t = unscoped.get();
            }
            return t;
        }

        public void clear() {
            t = null;
        }
    }
    public static class ClearingProvider<T> implements Provider<T> {
        Provider<T> unscoped;
        CachingScope scopeToClear;
        ClearingProvider(Provider<T> unscoped, CachingScope scopeToClear) {
            this.unscoped = unscoped;
            this.scopeToClear = scopeToClear;
        }
        public T get() {
            scopeToClear.clear();
            return unscoped.get();
        }
    }


    public static void main(String[] args) {
      Injector inj = Guice.createInjector(new MyModule());
      inj.getInstance(B.class);    
      System.out.println("--");
      inj.getInstance(B.class);    
    }  
  }

Ну, это была забавная игра в API. Мне не слишком нравится это решение, но я думаю, что оно работает. Теперь у вас есть две новые области, CachingScope, которая хорошо кэширует результаты. И область очистки, которая очищает кэш, когда ему нужен новый объект. Понятия не имею, насколько надежно это решение, я думаю, что это будет не очень, если речь идет о B, которые хотят вводить B. Я немного удивлен, что не смог заставить что-то подобное работать с детскими инъекторами, но иногда я могу быть немного толще.

1 голос
/ 10 июня 2010

PrivateModule и / или Scopes могут помочь, но я не уверен.Если нет, вам, возможно, придется написать собственного провайдера для ваших объектов A.

0 голосов
/ 10 июня 2010

Рассматривали ли вы изменить свой дизайн? Если для C и D требуется один и тот же экземпляр A, это говорит о том, что между этими двумя классами существует некоторая общая ответственность.

Рассмотрите возможность объединения их в один класс или извлечения частей, которые управляют экземпляром A, в новый класс.

0 голосов
/ 10 июня 2010

Я не знаком с хитростью, только весна.Я не думаю, что возможно настроить DI-движок так, чтобы он достиг того, чего вы пытаетесь достичь.Я вижу 2 решения:

  1. сделать объект B зависимым от (A, C, D) и ввести A в C и D во время выполнения.
  2. заставляет B зависеть только от A, а A зависит от C и D.
0 голосов
/ 10 июня 2010

Не уверен насчет Guice, но Spring не будет иметь с этим проблем, поскольку бины могут иметь разные области действия , например, singleton (только один экземпляр создан, по умолчанию), prototype (создается новый экземпляр bean-компонента) каждый раз, когда на него ссылаются и т. д.

Например, следующая конфигурация XML приведет к одному экземпляру Foo и трем Bar экземплярам.

<bean id="Foo" class="com.name.Foo"/>

<bean id="Bar1" class="com.name.Bar">
    <property name="foo" ref="Foo"/>
</bean>

<bean id="Bar2" class="com.name.Bar">
    <property name="foo" ref="Foo"/>
</bean>

<bean id="Bar3" class="com.name.Bar">
    <property name="foo" ref="Foo"/>
</bean>

Где эта конфигурация должна привести к 3 экземплярам Bar, каждый из которых имеет свой экземпляр Foo.

<bean id="Foo" class="com.name.Foo" scope="prototype" />

<bean id="Bar1" class="com.name.Bar">
    <property name="foo" ref="Foo"/>
</bean>

<bean id="Bar2" class="com.name.Bar">
    <property name="foo" ref="Foo"/>
</bean>

<bean id="Bar3" class="com.name.Bar">
    <property name="foo" ref="Foo"/>
</bean>

Похоже, У Guice те же понятия, что и у , с аннотацией @Singleton.

...