Comment utiliser log4net avec l'Injection de Dépendance

Je suis à essayer de comprendre quelle est la bonne formulation et l'utilisation de log4net est avec une injection de dépendance cadre.

Log4Net utilise la ILog interface mais m'oblige à appeler

LogManager.GetLogger(Reflection.MethodBase.GetCurrentMethod().DeclaringType)

dans chaque classe ou d'une méthode où j'ai besoin d'un journal d'informations. Cela semble aller à l'encontre du Cio principes et les couples de moi à l'aide de Log4Net.

Dois-je en quelque sorte mis dans une autre couche d'abstraction quelque part?

Aussi, j'ai besoin de vous connecter propriétés personnalisées comme le nom d'utilisateur comme ceci:

log4net.ThreadContext.Properties["userName"] = ApplicationCache.CurrentUserName;

Comment puis-je saisir cette sorte que je n'ai pas à vous rappeler de le faire à chaque fois et toujours conserver la méthode actuelle qui est de l'exploitation forestière. dois-je faire quelque chose comme cela ou suis-je totalement à côté de la plaque?

public static class Logger
{
    public static void LogException(Type declaringType, string message, Exception ex)
    {
        log4net.ThreadContext.Properties["userName"] = ApplicationCache.CurrentUserName;
        ILog log = LogManager.GetLogger(declaringType);
        log.Error(message, ex);
    }
}

La question la plus intéressante est qui DI conteneur utilisez-vous?

InformationsquelleAutor Micah | 2009-06-02

62

Je pense que vous n'êtes pas voir la forêt pour les arbres. ILog et LogManager sont à la légèreté de façade près de 1:1 équivalent à Apache commons-logging, et ne sont pas réellement en couple, votre code pour le reste de log4net.

<rant>

J'ai également constaté que presque toujours lorsque quelqu'un crée une MyCompanyLogger wrapper autour de log4net ils manquent le point de mal et perdra important et utile capacités du cadre, de jeter de l'information utile, de perdre les gains de performance possible, même en utilisant les simplifiée ILog interface, ou tout ce qui précède. En d'autres termes, emballage log4net pour éviter l'attelage est un anti-pattern.

</rant>

Si vous sentez le besoin de se l'injecter, faire de votre journal instance accessible via une propriété pour permettre l'injection, mais de créer une instance par défaut de la manière démodée.

Comme pour contextuels de l'état dans chaque message de log, vous devez ajouter une propriété globale dont ToString() décide de ce que vous cherchez. Comme un exemple pour la taille de segment de mémoire:
```
public class TotalMemoryProperty
{
    public override string ToString()
    {
        return GC.GetTotalMemory(false).ToString();
    }
}
```
Puis de le brancher en cours de démarrage:
```
GlobalContext.Properties["TotalMemory"] = new TotalMemoryProperty();
```
- +1 pour venir bien dehors et dire a son mauvais juju pour le faire.
- Votre diatribe est la voie de base ici. Ce mec a une question sur la façon d'injecter une instance de ILog à ses objets qui sont à l'intérieur d'un conteneur. Il y a beaucoup de raisons de le faire.
- La diatribe a été dirigé son approche de la création d'une couche d'abstraction entre log4net et son code, et en particulier à l'exemple de bloc de code à la fin de la question.
- Il peut être un anti-modèle, dans votre opinion, mais j'ai trouvé qu'il y a autant d'opinions là-bas qu'il y a des ingénieurs de l'écriture de code. Mon avis, bien sûr 🙂
- Je me rends compte que c'est un vieux post, mais pouvez-vous m'expliquer pourquoi une telle enveloppe est "mauvais"? ILog et LogManager résider dans le log4net espace de noms, et par l'utilisation de ces types tout au long de mon code n'est pas ce couplage moi de log4net? Ce qui me manque ici?
- et LogManager sont déjà un wrapper, ils ont juste arriver à être construit dans le cadre de log4net de l'assemblée. log4net est vrai de la surface est dans le Configurator, Repository, Logger, Appender et Formatter types, et vous ne devriez pas avoir de couplage, sauf dans votre App.config.
- Quant à savoir pourquoi c'est mauvais: ILog avec le mode recommandé de l'utiliser est une meilleure conception de toute la maison wrapper j'ai jamais vu. Ce n'est pas que je suis intrinsèquement opposés à l'enrobage log4net, c'est que dans mon expérience, le remède est pire que la maladie.
- Habillage de log4net est bien pratique. Le problème n'est pas l'habillage de la bibliothèque, c'est de faire si mal.
- C'était plutôt le point de la partie de la réponse à l'intérieur de diatribe tags 🙂 API conception tend à être un talent faible pour beaucoup de développeurs d'applications, afin de diriger les dit les développeurs pour envelopper les choses résultats inférieurs wrapper Api. Ma recommandation est d'utiliser la bibliothèque d'encapsulation livré avec log4net si possible: utiliser un autre enregistreur de remplacer l'usine et de fournir une mise en œuvre différente de ILog. Séparer le lot de wrapper dans sa propre assemblée, si vous devez, mais penser à long et dur avant d'écrire un nouveau.
- Mais à l'aide de ILog et LogManager t couple vous pour le reste de log4net, car il vous oblige à avoir un using log4net dans votre code, et de disposer d'une référence à log4net.dll. log4net.ILog et SomeStd.ILog peuvent avoir le même nom de base et des membres, mais ils sont, néanmoins, les deux classes différentes.
- Mais cela semble être rarement pertinentes dans la pratique. Il semble peu probable que vous serez en échangeant dans une autre structure de journalisation, et si ce besoin n'venir, c'est probablement un une genre de chose. Son pas comme vous avez besoin d'un maximum de flexibité là, au moins IME.
InformationsquelleAutor Jeffrey Hantin
2

Comment j'ai fait c'était de créer ma propre interface, et avait une classe implémente l'interface utilisée Log4Net, et injecté de cette classe. De cette façon, vous ne sont pas liés à Log4Net...il vous suffit de créer une autre classe pour une nouvelle enregistreur et injecter cette classe.

InformationsquelleAutor CSharpAtl
2

Si vous êtes vraiment préoccupés par le fait d'avoir plusieurs enregistreurs, vous pouvez toujours déclarer un mondial de l'enregistreur et l'appeler pour tous vos enregistrement. L'inconvénient de cette est que vous perdez le construit dans la capacité à identifier la classe que le journal a été émis, mais vous pouvez également injecter votre propre message personnalisé dans le journal pour identifier la classe.

Cela dépend de la façon robuste vous voulez que l'enregistrement de l'être. Vous pouvez également simplement mettre l'enregistreur de la classe principale et de laisser toutes les exceptions non gérées bulle jusqu'à elle pour la journalisation.
- Mais sûrement, vous voulez généralement à se connecter plus que juste des exceptions?
- Suggérez-vous le pattern Singleton ici? Je ne l'espère pas.
InformationsquelleAutor Dillie-O
2

Une autre façon de faire serait de câbler l'log4net ILog interface contenant la séquence d'enregistrement comme ceci: (à l'aide de Château dans un ASP.NET ci-dessous à titre d'exemple)
```
container.Register(Component.For<ILog>().LifeStyle
    .PerWebRequest.UsingFactoryMethod(() => LogManager.GetLogger(
    MethodBase.GetCurrentMethod().DeclaringType)));
```
et juste constructeur-injecter ILog chaque fois que vous en avez besoin.
- Cela fonctionne mieux, je pense, que vous obtenez le droit de l'enregistreur pour la mise en œuvre de la classe en cours de construction: Component.For<ILog>().UsingFactoryMethod((kernel, model, cc) => LogManager.GetLogger(cc.Handler.ComponentModel.Implementation))
InformationsquelleAutor santos

Vous devez vous connecter pour publier un commentaire.