You're still creating a new instance of the listener factory at every
get(), as guice isn't managing the provider. You need to get an
instance of type MouseActionListenerFactory rather than the Provider.
So
Injector injector = Guice.createInjector(new MouseActionModule());
System.out.println(injector.getProvider
(MouseActionListenerFactory.class).get());
System.out.println(injector.getProvider
(MouseActionListenerFactory.class).get());
However, if your original code is complete, and all the provider does
is to create a new instance of MouseActionListenerFactoy, remove the
provider completely:
public class MouseActionModule extends AbstractModule {
@Override
protected void configure() {
bind(MouseActionListenerFactory.class).in(Singleton.class);
}
}
public static void main(String []args){
Injector injector = Guice.createInjector(new MouseActionModule());
System.out.println(injector.getInstance
(MouseActionListenerFactory.class));
System.out.println(injector.getInstance
(MouseActionListenerFactory.class));
}
On Aug 17, 3:04 am, zhaoyi <[email protected]> wrote:
> I have tried that remove the second binding from the Module and use
> injector.getProvider(x), but I still get different instance. See my
> code below:
>
> Injector injector = Guice.createInjector(new MouseActionModule());
>
> mouseActionFactoryProvider = injector.getProvider(
> MouseActionFactoryProvider.class).get();
> System.out.println(mouseActionFactoryProvider.get());
> System.out.println(mouseActionFactoryProvider.get());
> System.out.println(mouseActionFactoryProvider.get());
> System.out.println(mouseActionFactoryProvider.get());
>
> Each time I call mouseActionFactoryProvider.get(), I will get the new
> instance.
>
> On Aug 15, 10:55 am, Max Bowsher <[email protected]> wrote:
>
> > zhaoyi wrote:
> > > This is my code. Whenever I call mouseActionFactoryProvider.get(), it
> > > will return create an instance. It seems that the @Singleton doesn't
> > > work here.
>
> > > @Singleton
> > > public class MouseActionFactoryProvider implements
> > > Provider<MouseActionListenerFactory> {
>
> > > �...@override
> > > @Singleton
> > > public MouseActionListenerFactory get() {
> > > return new MouseActionListenerFactory();
> > > }
> > > }
>
> > Scope annotations apply to providers as a whole, annotating the get()
> > method with @Singleton is erroneous and ignored.
>
> > > public class MouseActionModule extends AbstractModule {
>
> > > �...@override
> > > protected void configure() {
> > > bind(MouseActionListenerFactory.class).toProvider(
> > >
> > > MouseActionFactoryProvider.class).in(Singleton.class);
> > > bind(MouseActionFactoryProvider.class).in(Singleton.class);
> > > }
>
> > > }
>
> > What you have done here is:
>
> > 1) Bind MouseActionListenerFactory, in singleton scope, to be produced
> > via your Provider.
>
> > 2) ENTIRELY SEPARATELY bind a different instance of the Provider as a
> > second singleton.
>
> > > public static void main(String []args){
> > > Injector injector = Guice.createInjector(new
> > > MouseActionModule());
> > > mouseActionFactoryProvider = injector
> > > .getInstance(MouseActionFactoryProvider.class);
>
> > > }
>
> > And here you retrieve the singleton created by the second binding. Any
> > calls you make to its get() method are NOT MANAGED by Guice, so each one
> > runs the get() method afresh.
>
> > You should REMOVE the second binding from your Module, and you should
> > call injector.getProvider(x) in the case above where you are currently
> > calling injector.getInstance(x).
>
> > Max.
>
> > signature.asc
> > < 1KViewDownload
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"google-guice" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/google-guice?hl=en
-~----------~----~----~----~------~----~------~--~---
