2009-06-26 29 views
5

Logback kullanmaya başladım ve bir şeyler yapmanın daha iyi yollarının olup olmadığını bilmek istiyorum. Bu kodu vardır:Geri dönüş kullanımı ve baskı listeleri

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

Bugüne kadar bazı şüphelerim var:

  • Her sınıf bir logger oluşturmak için bir this.logger = LoggerFactory.getLogger(this.getClass().getName()); olacaktır.
  • Her yöntemin, bir yöntem çağrıldığında bunu bilmesi için this.logger.debug("method()"); olacaktır.

Bu iyi görünmüyor. Bunu çözmenin bir yolu var mı?

Ayrıca bu çizgide .log ile bir listesini yazdırmak istiyorum: this.logger.debug("names: {}", xxxxx);

xxxxx listesini yazdırmak için bir şey ile değiştirilmelidir. Anonim bir sınıf mı?

Okumak için teşekkürler!

+0

İlk soru, http://en.wikipedia.org/wiki/Aspect-oriented_programming için bir ders kitabı örneğidir, ancak kendime gerçek bir cevap yazmak için yeterince aşina değilim. –

cevap

4

AspectJ ve log4j'u kullanarak bunu kullanabilirsiniz. Kodunuzu javac yerine ajc derleyicisi ile derleyin ve daha sonra java ile çalıştırılabilir.

Sınıf yolunda aspectjrt.jar ve log4j.jar olması gerekir.

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() + "]"); 
     } 
    } 
} 

Kontrol dışarı TraceMethodCalls aramaları nasıl değiştirileceği ile ilgili AspectJ belgelerine. this.logger.debug("names: {}", xxxxx);

varsayılan olarak slf4j/logback tarafından desteklenen:

Ayrıca bu çizgide .log listesini yazdırmak istediğiniz İlişkin

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

. Sadece örnek

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] 

için

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

yapmak Yoksa özellikle farklı bir şey istiyoruz?

+0

Baskı listesi sorusuna yanıt eklendi – jitter

+1

Herhangi bir nesnenin {} -construction ile yazdırılabildiğini unutmayın. ToString() öğesine bir çağrıdır. Bundan başka bir işleme gerek duyarsanız, özel bir toString yöntemiyle, günlüğe kaydetmek istediğiniz nesneyi tutan bir sarıcı oluşturun. Bkz. Http://www.mail-archive.com/[email protected]/msg02259.html –

+0

@ Thorbjørn Ravn Andersen: Anonim bir sınıfla yapmak istedim çünkü sınıfta toString() yöntemi var nesneyi kullananı kullanır. – Macarse

İlgili konular