Thank you, sci. That is a good point. I think I'll have to go with creating my own LoggerWrapper.
On Thursday, October 19, 2017 at 2:15:16 PM UTC+8, scl wrote: > > If the @Logger causes Guice to inject an instance of a logger class which > was not created by you (i.e. bindInstance) then it should work. Given the > class injected is not final. > > Am 19. Oktober 2017 03:56:35 MESZ schrieb Yan Jiang <[email protected] > <javascript:>>: >> >> Thanks, Luke. >> >> I get it now. >> >> I still get one shot, SLF4j.logger is an interface, and Logback.Logger >> implements that interface. >> So I am thinking if I can do >> >> import slf4j.Logger; >> >> class Student { >> @Logger Logger logger; >> } >> >> and bind(slf4j.Logger.class).to(logback.Logger.class); >> I can still do it, right? >> >> The tricky part is to Provider all the necessary information in >> constructor, right? >> >> On Thursday, October 19, 2017 at 12:15:58 AM UTC+8, Luke Sandberg wrote: >>> >>> Guice AOP only works on objects that Guice constructs. So probably not >>> >>> On Wed, Oct 18, 2017 at 11:56 AM Yan Jiang <[email protected]> wrote: >>> >>>> Hi, all. >>>> >>>> I am thinking to add some pre&post actions for SLF4J, logger.info() >>>> and logger.error(). >>>> Is it possible to do this with Guice method interceptor? >>>> >>>> Currently, I am doing something like: >>>> >>>> Matcher<Class> subClassMatcher = Matchers.subclassesOf(org.slf4j.Logger >>>> .class); >>>> bindInterceptor(packageMatcher, Matchers.any(), new intercepters.Logger >>>> ()); >>>> >>>> >>>> Backend, we are using SLF4J and LogBack. >>>> >>>> >>>> But it doesn't work. >>>> >>>> >>>> >>>> -- >>>> You received this message because you are subscribed to the Google >>>> Groups "google-guice" group. >>>> To unsubscribe from this group and stop receiving emails from it, send >>>> an email to [email protected]. >>>> To post to this group, send email to [email protected]. >>>> Visit this group at https://groups.google.com/group/google-guice. >>>> To view this discussion on the web visit >>>> https://groups.google.com/d/msgid/google-guice/c2ba51d4-9352-45bf-acc6-4c41c4b1dbd3%40googlegroups.com >>>> >>>> <https://groups.google.com/d/msgid/google-guice/c2ba51d4-9352-45bf-acc6-4c41c4b1dbd3%40googlegroups.com?utm_medium=email&utm_source=footer> >>>> . >>>> For more options, visit https://groups.google.com/d/optout. >>>> >>> -- You received this message because you are subscribed to the Google Groups "google-guice" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at https://groups.google.com/group/google-guice. To view this discussion on the web visit https://groups.google.com/d/msgid/google-guice/052de221-75be-479a-92fe-b247899bd2a7%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
