yes
this is my module configuration :
@Override
protected void configure() {
MapBinder<DrawGraphType, IDrawGraphService> mapBinder =
MapBinder.newMapBinder(
this.binder(), DrawGraphType.class,
IDrawGraphService.class);
mapBinder.addBinding(DrawGraphType.DOT).to(DrawDotServiceImpl.class);
mapBinder.addBinding(DrawGraphType.TEXT).to(drawTextServiceImpl.class);
}
jeremie
On 20 juil, 18:52, Sam Berlin <[email protected]> wrote:
> Did you include this part
> --> configure() { // your module configuration
> > MapBinder<GraphType, IDrawGraphService> mapBinder =
> > MapBinder.newMapBinder(binder(), GraphType.class,
>
> IDrawGraphService.class);>
> mapBinder.addBinding(GraphType.DOT).to(DotGraphServiceImpl.class);
> > mapBinder.addBinding(GraphType.TEXT).to(TextGraphServiceImpl.class);
> > }
>
> --
> in your module that gets installed?
>
> sam
>
>
>
> On Tue, Jul 20, 2010 at 11:46 AM, Jerem's <[email protected]> wrote:
> > hey thanks sam for you answer (very detailled answer :) ).
>
> > But I still have an exception :
>
> > 1) No implementation for
> > java.util.Map<fr.irisa.cairn.simpleGraph.drawGraph.module.DrawGraphType,
> > fr.irisa.cairn.simpleGraph.drawGraph.service.IDrawGraphService> was
> > bound.
> > while locating
> > java.util.Map<fr.irisa.cairn.simpleGraph.drawGraph.module.DrawGraphType,
> > fr.irisa.cairn.simpleGraph.drawGraph.service.IDrawGraphService>
> > for parameter 0 at
>
> > fr.irisa.cairn.test.simpleGraph.runTimeTest.Launch.ClientImpl.<init>(Client
> > Impl.java:
> > 20)
> > at
>
> > fr.irisa.cairn.test.simpleGraph.runTimeTest.Launch.ClientModule.configure(C
> > lientModule.java:
> > 9)
>
> > 1 error
> > at
>
> > com.google.inject.internal.Errors.throwCreationExceptionIfErrorsExist(Error
> > s.java:
> > 354)
> > at
>
> > com.google.inject.InjectorBuilder.initializeStatically(InjectorBuilder.java
> > :
> > 152)
> > at com.google.inject.InjectorBuilder.build(InjectorBuilder.java:105)
> > at com.google.inject.Guice.createInjector(Guice.java:92)
> > at com.google.inject.Guice.createInjector(Guice.java:69)
> > at com.google.inject.Guice.createInjector(Guice.java:59)
> > at
> > fr.irisa.cairn.test.simpleGraph.runTimeTest.Launch.Main.main(Main.java:
> > 24)
>
> > My ClientImpl class look like that :
>
> > 1) No implementation for
> > java.util.Map<fr.irisa.cairn.simpleGraph.drawGraph.module.DrawGraphType,
> > fr.irisa.cairn.simpleGraph.drawGraph.service.IDrawGraphService> was
> > bound.
> > while locating
> > java.util.Map<fr.irisa.cairn.simpleGraph.drawGraph.module.DrawGraphType,
> > fr.irisa.cairn.simpleGraph.drawGraph.service.IDrawGraphService>
> > for parameter 0 at
>
> > fr.irisa.cairn.test.simpleGraph.runTimeTest.Launch.ClientImpl.<init>(Client
> > Impl.java:
> > 20)
> > at
>
> > fr.irisa.cairn.test.simpleGraph.runTimeTest.Launch.ClientModule.configure(C
> > lientModule.java:
> > 9)
>
> > 1 error
> > at
>
> > com.google.inject.internal.Errors.throwCreationExceptionIfErrorsExist(Error
> > s.java:
> > 354)
> > at
>
> > com.google.inject.InjectorBuilder.initializeStatically(InjectorBuilder.java
> > :
> > 152)
> > at com.google.inject.InjectorBuilder.build(InjectorBuilder.java:105)
> > at com.google.inject.Guice.createInjector(Guice.java:92)
> > at com.google.inject.Guice.createInjector(Guice.java:69)
> > at com.google.inject.Guice.createInjector(Guice.java:59)
> > at
> > fr.irisa.cairn.test.simpleGraph.runTimeTest.Launch.Main.main(Main.java:
> > 24)
>
> > Jeremie
>
> > On 20 juil, 15:43, Sam Berlin <[email protected]> wrote:
> > > If you prefer the old method, use getInstance(Key.get(...)). It does
> > > exactly what you're looking for. Just swap the annotation from the local
> > > variable to inside the Key.get method.
>
> > > FWIW, though, injecting the injector is bad practice (because you aren't
> > > explicitly defining what your dependencies are). MapBinder is the
> > preferred
> > > way to do something like this. It's very easy to use. Your case would
> > be:
> > > GraphType enum { DOT, TEXT } // new enum class
> > > ...
> > > configure() { // your module configuration
> > > MapBinder<GraphType, IDrawGraphService> mapBinder =
> > > MapBinder.newMapBinder(binder(), GraphType.class,
> > IDrawGraphService.class);
> > > mapBinder.addBinding(GraphType.DOT).to(DotGraphServiceImpl.class);
> > > mapBinder.addBinding(GraphType.TEXT).to(TextGraphServiceImpl.class);
> > > }
> > > ...
> > > �...@inject Client(Map<GraphType, IDrawGraphService> services) { // your
> > > client classes
> > > IDrawGraphService dotService = services.get(GraphType.DOT); // if
> > you
> > > want dot
> > > IDrawGraphService textService = services.get(GraphType.TEXT); // if
> > you
> > > want text
> > > ...
> > > }
>
> > > no need to even use binding annotations.
>
> > > sam
>
> > > On Tue, Jul 20, 2010 at 9:17 AM, Jerem's <[email protected]> wrote:
> > > > hmm.
>
> > > > it looks complicated.
>
> > > > At he beginning I just wanted to do that :
>
> > > > @ToDot
> > > > IDrawGraphService drawDotService =
> > > > drawGraphInjector.getInstance(IDrawGraphService.class);
>
> > > > @ToText
> > > > IDrawGraphService drawTextService =
> > > > drawGraphInjector.getInstance(IDrawGraphService.class);
>
> > > > So it was easy to choose an implementation. (just need an annotation).
> > > > I thought that Google guice is cool.
> > > > But guice annotation doesn't work on local variable.
>
> > > > So if we can't do that easily. I think I prefer the old method.
>
> > > > jérémie
>
> > > > On 19 juil, 21:07, Sam Berlin <[email protected]> wrote:
> > > > > Ahh -- sorry, i didn't get that you were wanting the client to choose
> > > > which
> > > > > one to use. In that case, take a look at MapBinder @
> > > >http://code.google.com/p/google-guice/wiki/Multibindings. Set up a
> > > > > MapBinder where you bind a Map<Class<? extends Annotation>,
> > > > > IDrawGraphService>, and add a binding into that MapBinder for each of
> > > > your
> > > > > implementations. Then your client injects Map<Class<? extends
> > > > Annotation>,
> > > > > IDrawGraphService> and calls map.get(TheAnnotationItWants.class) and
> > gets
> > > > > the proper implementation. (You can also use enums for the keys, or
> > any
> > > > > arbitrary other type, which might be clearer.)
>
> > > > > sam
>
> > > > > On Mon, Jul 19, 2010 at 2:48 PM, Jerem's <[email protected]> wrote:
> > > > > > Yeah I know that. but It is not really what I want.
> > > > > > If I do that :
>
> > > > > > @Inject
> > > > > > public Client(@ToDot IDrawGraphService service) {
> > > > > > .. }
>
> > > > > > service can only be a DrawDotServiceImpl. but how can I have a
> > > > > > DrawTextServiceImpl?
> > > > > > I wuld like that the client can choose the implementation.
> > > > > > Here, he can't.
>
> > > > > > may I can build a new class client2.
>
> > > > > > @Inject
> > > > > > public Client2(@ToText IDrawGraphService service) {
> > > > > > .. }
>
> > > > > > But I don't like to have two kinds of ciient.
>
> > > > > > Jérémis
>
> > > > > > On 19 juil, 19:37, Sam Berlin <[email protected]> wrote:
> > > > > > > Add an annotation to your injected variable, like:
>
> > > > > > > �...@inject public Client(@YourBindingAnnotation IDrawGraphService
> > > > service)
> > > > > > {
> > > > > > > .. }
>
> > > > > > > That tells Guice that when it injects the constructor with
> > variables,
> > > > it
> > > > > > > should inject the graph service that was annotated with that
> > binding
> > > > > > > annotation.
>
> > > > > > > sam
>
> > > > > > > On Mon, Jul 19, 2010 at 12:45 PM, Jerem's <[email protected]>
> > > > wrote:
> > > > > > > > Thanks sam.
>
> > > > > > > > that's not cool that Annotation don't work for local variable.
>
> > > > > > > > I try the second version. IT works. thanks :).
> > > > > > > > But I don't really like it.
>
> > > > > > > > So I try the first version. Bit it is not really clear in my
> > mind.
> > > > > > > > I create a client class :
>
> > > > > > > > public class Client {
>
> > > > > > > > private IDrawGraphService service;
>
> > > > > > > > �...@inject
> > > > > > > > public Client(IDrawGraphService service) {
> > > > > > > > super();
> > > > > > > > this.service = service;
> > > > > > > > }
>
> > > > > > > > public void executeService(ISimpleGraph graph, String
> > > > path)
> > > > > > throws
> > > > > > > > IOException {
> > > > > > > > service.draw(graph, path);
> > > > > > > > }
> > > > > > > > }
>
> > > > > > > > but Don"t really see hos decide hos to decided is the service
> > is a
> > > > > > > > drawTextServiceImpl or a drawDotServiceImpl
>
> > > > > > > > jérémie
>
> > > > > > > > On 19 juil, 17:39, Sam Berlin <[email protected]> wrote:
> > > > > > > > > Guice cannot detect annotations on local variables this way
> > > > (although
> > > > > > > > it's
> > > > > > > > > an interesting idea!). You have two options here:
>
> > > > > > > > > 1) Make the variable on the class wherever this code is in
> > and
> > > > add
> > > > > > an
> > > > > > > > extra
> > > > > > > > > module that has requestInjection(YourClass.this) in its
> > configure
> > > > > > method.
> > > > > > > > > Then Guice will inject that variable as part of its Injector
> > > > > > creation.
> > > > > > > > > 2) Use injector.getInstance(Key.get(IDrawGraphService.class,
> > > > > > > > ToDot.class))
> > > > > > > > > instead. That tells Guice that you want to retrieve the
> > instance
> > > > > > defined
> > > > > > > > by
> > > > > > > > > the Key "@ToDot IDrawGraphService", as opposed to just
> > > > > > > > "IDrawGraphService".
>
> > > > > > > > > sam
>
> > > > > > > > > On Mon, Jul 19, 2010 at 11:21 AM, Jerem's <
> > [email protected]>
> > > > > > wrote:
> > > > > > > > > > Hello I try to use annotatoij on a local variable :
>
> > > > > > > > > > this is my module :
>
> > > > > > > > > > public class DrawModule extends AbstractModule {
>
> > > > > > > > > > �...@override
> > > > > > > > > > protected void configure() {
>
> > this.bind(IDrawGraphService.class).annotatedWith(ToDot.class).to(DrawDotSer
> > > > > > > > viceImpl.class);
> > > > > > > > > > }
> > > > > > > > > > }
>
> > > > > > > > > > and my Annotation look like that :
>
> > > > > > > > > > @Retention(RUNTIME)
> > > > > > > > > > @Target({ LOCAL_VARIABLE, FIELD, PARAMETER })
> > > > > > > > > > @BindingAnnotation
> > > > > > > > > > public @interface ToDot {
>
> > > > > > > > > > }
>
> > > > > > > > > > and I try to do that :
>
> > > > > > > > > > Injector drawGraphInjector = Guice.createInjector(new
> > > > > > DrawModule());
> > > > > > > > > > @ToDot
>
> ...
>
> plus de détails »
--
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.
