log4j

[Guest]

Hallo,

würdet ihr sagen, dass das eine gute methode ist einen logger zu implementieren?
ich habe da sganze statisch gemacht.

mein problem ist, dass mein log manchmal zeilenweise auseinander liegt..(schwer zu beschreiben).
jedes mal eine neue Instanz zu erzeugen sollte doch dann auch das log neu schrieben und dürfte in meinen augen dann falsch sein?

package tclass.util;

import org.apache.log4j.FileAppender;
import org.apache.log4j.Level;
import org.apache.log4j.Logger;
import org.apache.log4j.PatternLayout;

public class Log {

	// ****************************************
	// Logging methods
	// ****************************************
	// ****************************************
	// logger.debug(String); 
	// logger.info("My info-Message");
	// logger.warn("My warn-Message"); 
	// logger.error("My error-Message");
	// logger.fatal("My fatal-Message")
	// ****************************************
	// File Appender Options
	// logger.setLevel(Level.xxxx);
	// ALL | DEBUG | INFO | WARN | ERROR | FATAL | OFF: 
	 

		
	private static Logger logger = Logger.getRootLogger();

	public static Logger getLogger() {
		try {
			PatternLayout layout = new PatternLayout(
					"%d{dd.MM.yyyy HH:mm:ss,SSS} %-5p [%t] %c: %m%nFilename: %F  Linenumber: %L  Methodname: %M%n%n");
			FileAppender fileAppender = new FileAppender(layout, "c:\\SysLog.log",
					false);
			logger.addAppender(fileAppender);
			logger.setLevel(Level.ALL);
		} catch (Exception ex) {
			System.out.println(ex);
		}
		return logger;
	}
}

//implementiert habe ich das so
Logger log = Log.getLogger();
log.info("Inforamation");

 

[fehlerfinder]

ich habe da sganze statisch gemacht.

Da spricht eigentlich nichts dagegen - dafür gibt's doch "static".

mein problem ist, dass mein log manchmal zeilenweise auseinander liegt..(schwer zu beschreiben).

Ein Versuch wäre es trotzdem wert gewesen ;-) - meinst du damit, dass die Log-Ausgabe-Befehle im Programm auseinanderliegen? Das wäre durchaus sehr normal.

jedes mal eine neue Instanz zu erzeugen sollte doch dann auch das log neu schrieben und dürfte in meinen augen dann falsch sein?

Eine interessante Frage - teste das doch einfach mal aus, indem zu zwei Logger-Objekte erstellst, die beide auf denselben Logger/Appender zugreifen. Dazu empfehle ich dann die Verwendung einer Properties-Datei.

PatternLayout layout = new PatternLayout(
     "%d{dd.MM.yyyy HH:mm:ss,SSS} %-5p [%t] %c: %m%nFilename: %F  Linenumber: %L  Methodname: %M%n%n");
FileAppender fileAppender = new FileAppender(layout, "c:\\SysLog.log", false);
logger.addAppender(fileAppender);
logger.setLevel(Level.ALL);

Diesen ganzen "Krempel" kannst du in die bereits oben erwähnte Properties-Datei auslagern, sieht dann z.B. so aus:

log4j.rootLogger=DEBUG, FileSystemOut
log4j.appender.FileSystemOut=org.apache.log4j.FileAppender
log4j.appender.FileSystemOut.file=c:\\SysLog.log
log4j.appender.FileSystemOut.layout=org.apache.log4j.PatternLayout
log4j.appender.FileSystemOut.layout.ConversionPattern=%d{dd.MM.yyyy HH:mm:ss,SSS} %-5p [%t] %c: %m%nFilename: %F  Linenumber: %L  Methodname: %M%n%n

Das müsste so ungefähr hinkommen - ich habe es aus einer eigenen Props-Datei kopiert, die aber etwas anders aussieht. Insbesondere weiß ich nicht, ob du den Pfad so in die Props-Datei eintragen kannst, ich arbeite nur unter Linux. Wenn's Probleme damit gibt, melde dich nochmal.

Ob du im rootLogger statt DEBUG auch ALL eintragen kannst, weiß ich ebenfalls nicht, meine aber gelesen zu haben, dass ALL eher so ein intern verwendeter Level ist. Aber das wird wohl schon passen. ;-)

 

[Guest]

hab jetzt mal folgendes gemacht:

public Logger getInLogger() {
		try {
			PatternLayout layout = new PatternLayout(
					"%d{dd.MM.yyyy HH:mm:ss,SSS} %-5p [%t] %c: %m%nFilename: %F  Linenumber: %L  Methodname: %M%n%n");
			FileAppender fileAppender = new FileAppender(layout, "c:\\SysLog.log",
					false);
			logger.addAppender(fileAppender);
			logger.setLevel(Level.ALL);
		} catch (Exception ex) {
			System.out.println(ex);
		}
		return logger;
	}

//in einer main einer testclasse
Log logger = new Log();
		Logger log = logger.getInLogger();
		log.info("info1");
		Logger log2 = logger.getInLogger();
		log2.info("info2");

heraus kam das:

18.06.2008 10:24:32,125 INFO [main] root: info2
Filename: Script1.java Linenumber: 27 Methodname: testMain

18.06.2008 10:24:32,125 INFO [main] root: info2
Filename: Script1.java Linenumber: 27 Methodname: testMain

wieso kommen da zwei einträge?

 

[maki]

Frage am Rande: Wieso nicht einfach log4j so nutzen wie es gemeint wear, anstatt selbst nochmal etwas drumrum zu Proggen?

 

[Gelöschtes Mitglied 5909]

Frage am Rande: Wieso nicht einfach log4j so nutzen wie es gemeint wear, anstatt selbst nochmal etwas drumrum zu Proggen?

Wenns wenigstens was Sinnvolles wär mit nem Errorcode z.b., aber so...