Guice: come configurare un @Provides e @Singleton in un modulo in questo caso?

Question

Ho un metodo fornitore in un modulo annotato con @Provides:

@Provides 
public ChatServicePerformanceMonitor getChatServicePerfMon() { 
    ... 
} 

e ho annotato il mio ChatServicePerformanceMonitor con @Singleton. Nel mio codice, dove uso questa istanza, non posso fare a meno di iniettarla "passivamente", a causa di un framework che sto usando che sta costruendo la classe di inclusione (non usa Guice, quindi questo è l'unico modo che conosco di per ottenere il riferimento):

chatServicePerfMon = injector.getInstance(ChatServicePerformanceMonitor.class); 

sembra Guice non rispetta l'annotazione @Singleton sulla mia classe ChatServicePerformanceMonitor. Ricevo un'istanza per ogni chiamata a injector.getInstance (ChatServicePerformanceMonitor.class).

Aggiunta del @Singleton al metodo fornitore sembra per risolvere questo problema:

@Provides @Singleton 
public ChatServicePerformanceMonitor getChatServicePerfMon() { 
    ... 
} 

è che il comportamento previsto? Sembra che @Singleton sull'istanza sia tutto ciò di cui ho bisogno.

Answer 1

Se si sta creando il ChatServicePerformanceMonitor in questo modo:

@Provides 
public ChatServicePerformanceMonitor getChatServicePerfMon() { 
    return new ChatServicePerformanceMonitor(); 
} 

allora il tuo livello di classe @Singleton annotazione avrà alcun effetto perché Guice non sta creando l'oggetto, sei. Guice può solo applicare l'ambito sugli oggetti che crea. Non c'è niente di sbagliato nell'aggiungere @Singleton al tuo metodo getChatServicePerfMon().

Se si dispone di un costruttore senza argomenti (o un costruttore @Inject) sulla classe ChatServicePerformanceMonitor e si rimuove il metodo @Provides poi chiamate continue all'iniettore restituiranno lo stesso Singleton.

Answer 2

Nel frattempo questa funzione è disponibile (testata con Guice 4.0).

@Provides i metodi possono ora essere annotati con @Singleton per applicare l'ambito. Vedi https://github.com/google/guice/wiki/Scopes