Discussion :

[Marvelll]

Bonjour,

J'ai crée une classe spécifique pour paramétrer tous les logs afin d'éviter de mettre 5 ligne de code à chaque endroit où je mets mon log.

je crée une classe Log.java et je mets à l'intérieur:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
public static void error(String texte){
		Logger log = Logger.getLogger("org");
		PatternLayout layout = new PatternLayout("%d %-5p %c - %F:%L - %m%n");
		try {
			log.addAppender(new FileAppender(layout, "./log/Adoc.log"));
		} catch (IOException e) {
			e.printStackTrace();
		}
		log.info(texte);
		log.setLevel(Level.ERROR);
	}

et je l'appelle dans un fichier TableOfContents.java :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Log.error("Fichier Table des matières non chargé");

Et je regarde dans mon fichier Adoc.log, pb :

2010-02-02 15:05:23,411 INFO org - Log.java:57 - Fichier Table des matières non chargé



Il me spécifie l'endroit où se situe le log (Log.java) et non pas TableOfContents.java or je voudrais récupérer la ligne qui pose problème dans TableOfContents.java.

Merci de votre aide.

[tchize_]

Bonjour,

J'ai crée une classe spécifique pour paramétrer tous les logs afin d'éviter de mettre 5 ligne de code à chaque endroit où je mets mon log.

un log ca prend une ligne, pas besoin de créer une méthode statique pour ça, sinon, tu perd en performance et, comme tu viens de la voir, en informations.


c'est comme ça qu'on fait du logging:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
public class MaClasse{
    private static Logger log = Logger.getLogger("org");
    // ou alternativement,
    // private static Logger log = Logger.getLogger(MaClasse.class);
 
 
    // ....
    // dans une méthode quelconque, quand on a besoin de logguer un erreur
    log.error("ceci est une erreur");
    // dans une méthode quelconque, quand on a besoin de logguer une info de base
    if (log.isDebugEnabled())
      log.debug("ceci est une information de debug complexe: "+unMessageDeDebugLongAGenerer());
}

Il n'est de plus pas recommandé d'utiliser %F et %l, ces informations sont longues et complexes à générer, ca ne se justifie que pour les logs d'erreur (une fois toutes les lustres) mais pas pour des logs de debug ou d'info qui peuvent etre nombreux aux cours de l'exécution de l'applicaiton (et plomber donc les performances)

[Marvelll]

Mais si je fais avec votre méthode, je perds les informations qui me sont absolument vitale comme la date et le numéro de la ligne.

En clair, je dois avoir dans mon log lorsque j'ai une erreur le maximum d'informations sur l'erreur. Par exemple, si je file l'appli que je développe à un gars et qu'il me le retourne en me disant que ça ne marche pas. J'aimerais bien savoir ce qui ne marche pas et à quel moment, ça a planté.

[tchize_]

fichier: suffit de donner au logger le nom de ta classe (cf commentaire en haut de ma classe)
numéro de ligne: dans 99.9% des cas inutiles. Avec le nom de classe et le message de log, il est toujours possible de retrouver la ligne qui l'a générer, il est rare que deux endroit dans une même classe génèrent exactement le même message de log. Et si c'est le cas, ben change en ai pour qu'il n'y aie plus ambiguité. Il faut bien réaliser que mettre le numéro de ligne et le nom de fichier,ça bouffe des ressources à calculer. Donc à savoir.

Dans tous les cas, je vois pas en quoi ma méthode (cf code) a avoir avec les données que tu affiche. Ta méthode n'a pas à définir les appender et le pattern à chaque appel. C'est configuré une fois pour toutes au démarrage de log4j.

[Marvelll]

Dans tous les cas, je vois pas en quoi ma méthode (cf code) a avoir avec les données que tu affiche. Ta méthode n'a pas à définir les appender et le pattern à chaque appel. C'est configuré une fois pour toutes au démarrage de log4j.

Je n'ai rien défini au démarrage de log4j. Où est-ce que tu définis les appender et le pattern???

[tchize_]

log4j.xml ou log4j.properties, dans la pluspart des cas (voir la doc log4j là dessus)