log4j

[max5432]

Hallo allerseits

In meiner log4j.properties Datei habe ich folgendes angegeben:

log4j.appender.FileAPP.layout.ConversionPattern = [%d / %p]\t(%F\:%L) - %m%n

Wenn ich eine Ausnahme protokolliere, bekomme ich immer die Nummer der Codezeile, die den Eintrag in die log-Datei geschreiben hat: dies ist die Zeile

logger.error(e.getMessage());

im entsprechenden catch-Block.

Ist es möglich, die Codezeile direkt anzugeben, die zur Ausnahme geführt hat? Wie sehe so ein Pattern aus?

Danke.

 

[maki]

[c]logger.error(e);[/c]

 

[max5432]

Danke.

Nun, klappt nicht ganz. Zusätzlich bekomme ich die genaue Angabe der Ausnahme: die Nummer der Codezeile, die zur Ausnahme geführt hat, wird nicht angegeben. Weiterhin wird die Nummer der Codezeile angegeben, in der logger.error(e) steht.


Die genaue Angabe des Ausnahme-Typs hilft sicher, die "schuldige" Codezeile zu finden: jedoch wäre schön, wenn die Codezeile-Nummer explizit angegeben werden könnte.

 

[maki]

Denke dass du F und L falsch verstanden hast, letzteres auf jedenfall.

Die Zeile in der die Exception geworfen wurde bekommst du nur aus dem Stacktrace, L gibt dir nur die Zeile in der die Exception geloggt wurde.
F sollte komplett überflüssig sein, wenn man den Loggern sinnvolle Namen gibt, so wie in der DOku gezeigt:

 import com.foo.Bar;

 // Import log4j classes.
 import org.apache.log4j.Logger;
 import org.apache.log4j.BasicConfigurator;

 public class MyApp {

   // Define a static logger variable so that it references the
   // Logger instance named "MyApp".
   static Logger logger = Logger.getLogger(MyApp.class);
...

Es wird ja sowohl von F und L abgeraten in der Log4J Doku.

 

[max5432]

Wenn ich F und L weglasse, habe ich "von Haus aus" noch weniger Informationen.

Das heisst, dass ich vor dem Loggen an sich Stack-Trace verarbeiten bzw. einzelne Einträge extrachieren und in die Log-Datei passend schreiben muss? Oder, bin ich auf dem Holzweg?

 

[maki]

Du bist auf dem Holzweg imho, vielleciht machst du vorher schon was falsch, L & F braucht man nicht, und wenn doch, macht man was falsch

Zeig mal den Code wie du die Logger initialisierst.

 

[max5432]

Hier ist der Code:

public class DataManager{

static Logger logger = Logger.getLogger(DataManager.class.getName());

// ... 

}

Die log4j.properties:

### Ausgabe in die Datei 'data.log'

# LogRoot-Kategorie
log4j.rootCategory=INFO,FileAPP

# log-file parameter
log4j.appender.FileAPP = org.apache.log4j.RollingFileAppender
log4j.appender.FileAPP.File = data.log
log4j.appender.FileAPP.MaxFileSize = 1000KB
log4j.appender.FileAPP.MaxBackupIndex = 1
log4j.appender.FileAPP.Append = true

# Layout definieren
log4j.appender.FileAPP.layout = org.apache.log4j.PatternLayout

#log4j.appender.FileAPP.layout.ConversionPattern = [%d / %p]\t(%F\:%L) - %m%n
log4j.appender.FileAPP.layout.ConversionPattern = [%d / %p]\t - %m%n

Ich habe inzwischen eine Methode implementiert, die mir StackTrace aufbereitet:

public static String getStackTraceAsString(Exception e) {
        StringBuffer b = new StringBuffer();

        StackTraceElement[] arr = e.getStackTrace();

        for (StackTraceElement ste : arr)
        {
            b.append(ste.getClassName() + " - " + ste.getMethodName()
                    + " - Line Nr. " + ste.getLineNumber() + "\n");
        }

        return b.toString();
    }

Da habe ich schon viel mehr Infos und finde auch die Codezeile-Nummer, die Probleme verursacht hat. Trotzdem würde mich interessieren, ob man das einfacher und besser machen kann.

 

[maki]

...Trotzdem würde mich interessieren, ob man das einfacher und besser machen kann.

Siehe meine erste antwort
Dann noch das Pattern aus der Doku, das war's.

Du solltest den Stacktrace loggen, nicht nur die Message.

Den Aufwand den du treibst macht es nur komplizierter und viel langsamer, aber vor allem könnten dir wichtige Infos verloren gehen.

 

[mvitz]

Einfach den Stacktrace loggen. Dort steht ja die Zeile in der Fehler geworfen wurde. (Evtl. muss dann mit der debug Option von Java kompilieren).

 

[HoaX]

Nimm logger.error(e.getMessage(), e);

bei nur logger.error(e) wird nur toString von e aufgerufen, du bekommst keinen Stacktrace!

 

[max5432]

Nimm logger.error(e.getMessage(), e);

bei nur logger.error(e) wird nur toString von e aufgerufen, du bekommst keinen Stacktrace!

Das war's!

Vielen Dank an alle.

 

[maki]

Nimm logger.error(e.getMessage(), e);

bei nur logger.error(e) wird nur toString von e aufgerufen, du bekommst keinen Stacktrace!

Hab gerade nachgelesen wie das funzt, keine der Logger Methoden (error, debug, etc.) erwartet eine Exception bei einem einzelnen Parameter.
Also wäre alternativ auch das möglich:

logger.error("Meldung deiner Wahl", e);

Dann klappt es auch mit dem Stacktrace.