Когда запускать карту в службе, вызванной контроллером весной с впрыском зависимостей?

Question

У меня есть класс контроллера, который вызывается первым в моем приложении. Там я планировал получить значение из Карты из класса Service.

Вот контроллер:

@Controller 
public class AppController { 

    public Service doSomethingWithTheMap(String key) { 
     return ServiceImpl.getMapValueFor(key).exec(); 
    } 
} 

я получаю вопросы, потому что во время инициализации, а во надетой ценностей на карте Службы, чтобы быть более точным, я требовать BeanFactory, поскольку значения в карте Служащих реализации. Выполнение этого в статическом блоке приведет к тому, что BeanFactory будет нулевым, потому что он не вводится, но я бы догадался.

Таким образом, в результате этого вызова initMap() я чувствую себя немного ... должно быть лучшее решение.

Любые подсказки кто-нибудь?

Должен признать, что я новичок в весне и, может быть, я все испортил. FYI Карта пришла мне в голову после бесконечных, если еще проверяет, какая служба вызывается на основе ввода String. Поэтому я заменил его на карту и простой один вкладыш в контроллере.

ServiceImpl.getMapValueFor(key).exec(); 

Вот класс обслуживания:

@Service 
public class ServiceImpl { 

private static Map<String, Service> map; 

private static ApplicationContext context; 

@Autowired 
public void setApplicationContext(ApplicationContext factory) { 
    this.context = factory; 
} 

public static Service getMapValueFor(String key) { 
    if (map == null) { 
     initMap(); 
    } 
    return map.get(key); 
} 

private static void initMap() { 
    /* 
    * FIXME: We can not init the map in a static block or directly 
    * initialize it since the factory is not injected until execution of a 
    * static block and will be null. 
    */ 
    BeanFactory factory = context; 
    map = new HashMap<String, Service>(); 
    map.put("key", factory.getBean(SomeService.class)); 

} 
} 

Answer 1

Первое, что я хочу сказать, что у вас есть ошибка, потому что вы используете HashMap без каких-либо синхронизации! - Не волнуйтесь, многие (если не большинство) разработчики java совершили ту же ошибку.

Если вы не упростили код, ваша служба больше похожа на команду, чем на сервис; Услуга - одноэлементный. Невозможно, чтобы службы имели методы без аргументов, но я бы сказал, что это необычно. Вы уверены, что не должны использовать прототипы вместо синглтонов?

Обычно количество услуг ограничено, и если у вас есть несколько сервисов того же типа, вы должны использовать @Qualifier при их автообновлении. В любом случае этот код выглядит изворотливым для меня, поэтому, возможно, вам стоит попытаться объяснить проблему на более высоком уровне, потому что может быть лучший способ, чем ваш текущий код.

Answer 2

Ваш класс обслуживания ServiceImpl должен реализовать интерфейс org.springframework.context.ApplicationContextAware, чтобы получить экземпляр контекста приложения Spring.

Answer 3

Это очень простое решение, оно использует тот факт, что имя @Bean - это имя метода, который его создает, вам, вероятно, понадобится лучшая стратегия. Идея заключается в том, чтобы скрыть getBean внутри класса поставщика, который затем может быть Autowired (и проверенного)

import org.springframework.beans.BeansException; 
import org.springframework.context.ApplicationContext; 
import org.springframework.context.ApplicationContextAware; 
import org.springframework.context.annotation.AnnotationConfigApplicationContext; 
import org.springframework.context.annotation.Bean; 

public class Main { 

    public static void main(String[] args) { 

     ApplicationContext ctx = new AnnotationConfigApplicationContext(Config.class); 
     CallableProvider provider = ctx.getBean(CallableProvider.class); 
     System.out.println(provider.getCommand("aCommand").call()); 
     System.out.println(provider.getCommand("bCommand").call()); 
    } 

    public static class Config { 
     @Bean 
     public ACommand aCommand() { 
      return new ACommand(); 
     } 
     @Bean 
     public BCommand bCommand() { 
      return new BCommand(); 
     } 

     @Bean 
     public CallableProvider callableProvider() { 
      return new CallableProvider(); 
     } 
    } 

    public static class CallableProvider implements ApplicationContextAware { 
     private ApplicationContext context; 

     @Override 
     public void setApplicationContext(ApplicationContext applicationContext) throws BeansException { 
      this.context = applicationContext; 
     } 

     public Command getCommand(String name) { 
      return context.getBean(name, Command.class); 
     } 
    } 

    public static class ACommand implements Command { 
     // autowire stuff 
     @Override 
     public String call() { 
      return "A"; 
     } 
    } 

    public static class BCommand implements Command { 
     // autowire stuff 
     @Override 
     public String call() { 
      return "B"; 
     } 
    } 

    public interface Command { 
     String call(); 
    } 
}