2009-06-26 11 views
5

Sto iniziando a utilizzare il logback e voglio sapere se ci sono modi migliori per fare qualcosa. ho questo codice:Logback utilizzo e liste di stampa

public class ClassA { 
    private List<String> l; 
    private Logger logger; 

    public ClassA(){ 
     this.logger = LoggerFactory.getLogger(this.getClass().getName()); 
    } 
.... 
    public List<String> method() { 
     this.logger.debug("method()"); 
     List<String> names; 

     try { 
      names = otherClass.getNames(); 
     } catch (Exception e) { 
      String msg = "Error getting names"; 
      this.logger.error(msg); 
      throw new ClassAexception(msg, e); 
     } 

     this.logger.debug("names: {}", xxxxx); 
     return names; 
} 

ho qualche dubbio finora:

  • Ogni classe avrà un this.logger = LoggerFactory.getLogger(this.getClass().getName()); per creare un logger.
  • Ogni metodo avrà un this.logger.debug("method()"); per sapere quando viene chiamato un metodo.

Questo non sembra buono. C'è un modo per risolverlo?

Anche io voglio stampare un elenco nel log in questa linea: this.logger.debug("names: {}", xxxxx);

il xxxxx dovrebbe essere sostituito con qualcosa per stampare l'elenco. Una lezione anonima?

Grazie per la lettura!

+0

La prima domanda è un caso da manuale per http://en.wikipedia.org/wiki/Aspetto- orientato_programmazione, ma non sono abbastanza familiare con me stesso per scrivere una risposta reale. –

risposta

4

Utilizzando AspectJ e log4j è possibile utilizzare questo. Compilare il codice con un compilatore ajc invece di javac e quindi eseguirlo normalmente con l'eseguibile java.

È necessario avere l'aspectjrt.jar e log4j.jar sul classpath.

import org.aspectj.lang.*; 
import org.apache.log4j.*; 

public aspect TraceMethodCalls { 
    Logger logger = Logger.getLogger("trace"); 

    TraceMethodCalls() { 
     logger.setLevel(Level.ALL); 
    } 

    pointcut traceMethods() 
     //give me all method calls of every class with every visibility 
     : (execution(* *.*(..)) 
     //give me also constructor calls 
     || execution(*.new(..))) 
     //stop recursion don't get method calls in this aspect class itself 
     && !within(TraceMethodCalls); 

    //advice before: do something before method is really executed 
    before() : traceMethods() { 
     if (logger.isEnabledFor(Level.INFO)) { 
      //get info about captured method and log it 
      Signature sig = thisJoinPointStaticPart.getSignature(); 
      logger.log(Level.INFO, 
         "Entering [" 
         + sig.getDeclaringType().getName() + "." 
         + sig.getName() + "]"); 
     } 
    } 
} 

Consultare la documentazione di AspectJ su come modificare le chiamate di TraceMethodCalls.

// e.g. just caputre public method calls 
// change this 
: (execution(* *.*(..)) 
// to this 
: (execution(public * *.*(..)) 

Per quanto riguarda la

Anche io voglio stampare un elenco nel log in questa linea: this.logger.debug("names: {}", xxxxx);

che è supportato da slf4j/logback per default . Basta fare

logger.debug("names: {}", names); 

ad esempio

List<String> list = new ArrayList<String>(); 
list.add("Test1"); list.add("Test2"); list.add("Test3"); 
logger.debug("names: {}", list); 

//produces 
//xx::xx.xxx [main] DEBUG [classname] - names: [Test1, Test2, Test3] 

O volete qualcosa di specificamente diverso?

+0

Aggiunta risposta alla lista di stampa domanda – jitter

+1

Si noti che qualsiasi oggetto può essere stampato con la {} -costruzione. È una chiamata a toString(). Se è necessario un rendering diverso da quello, creare un wrapper che contenga l'oggetto che si desidera registrare, con un metodo toString personalizzato. Vedere http://www.mail-archive.com/[email protected]/msg02259.html –

+0

@ Thorbjørn Ravn Andersen: ero disposto a farlo con una classe anonima perché il metodo toString() nella classe che ho usa quello dall'oggetto. – Macarse