2009-06-26 17 views
5

Estoy empezando a utilizar logback y quiero saber si hay mejores formas de hacer algo. tengo este código:Uso de Logback y listas de impresión

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; 
} 

tengo algunas dudas hasta el momento:

  • Cada clase tendrá un this.logger = LoggerFactory.getLogger(this.getClass().getName()); para crear un registrador.
  • Cada método tendrá un this.logger.debug("method()"); para saber cuándo se llama a un método.

Eso no se ve bien. ¿Hay alguna manera de resolverlo?

También quiero imprimir una lista en el .log en esta línea: this.logger.debug("names: {}", xxxxx);

la xxxxx debe ser reemplazado con algo para imprimir la lista. Una clase anónima?

¡Gracias por leer!

+0

La primera pregunta es un caso de libro de texto para http://en.wikipedia.org/wiki/Aspect-oriented_programming, pero no estoy lo suficientemente familiar como para escribir una respuesta real. –

Respuesta

4

Usando AspectJ y log4j puede usar esto. Compila tu código con el compilador ajc en lugar de javac y luego ejecuta normalmente con el ejecutable de Java.

Necesita tener el aspectjrt.jar y log4j.jar en el 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() + "]"); 
     } 
    } 
} 

Mira la documentación AspectJ sobre cómo cambiar las llamadas TraceMethodCalls.

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

En cuanto a la

También quiero imprimir una lista en el .log en esta línea: this.logger.debug("names: {}", xxxxx);

Eso es apoyado por slf4j/logback por defecto . Sólo hacer

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

por ejemplo

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 quieres algo específicamente diferente?

+0

Respuesta agregada a la pregunta de la lista de impresión – jitter

+1

Tenga en cuenta que cualquier objeto se puede imprimir con la {} -construcción. Es una llamada a toString(). Si necesita otra representación más que eso, cree un contenedor que contenga el objeto que desea registrar, con un método personalizado toString. Ver http://www.mail-archive.com/[email protected]/msg02259.html –

+0

@ Thorbjørn Ravn Andersen: Estaba dispuesto a hacerlo con una clase anónima porque el método toString() en la clase que tengo usa el de un objeto. – Macarse