9

我正在尝试将 C# 5.0 调用方信息与 C# params 关键字结合起来。目的是为日志框架创建一个包装器，我们希望记录器像 String.Format 一样格式化文本。在以前的版本中，该方法如下所示：

void Log(
   string message,
   params object[] messageArgs = null);

我们这样称呼它：

log.Log("{0}: I canna do it cap'n, the engines can't handle warp {1}!", 
        "Scotty", warpFactor);

现在，我们要捕获调用者信息并记录它。所以签名变成：

void Log(
    string message,
    params object[] messageArgs,
    [CallerMemberName] string sourceMemberName = null);

这不会编译，因为参数必须是最后一个参数。所以我试试这个：

void Log(
    string message,
    [CallerMemberName] string sourceMemberName = null,
    params object[] messageArgs);

有没有办法在不提供 sourceMembername 或将 messageArgs 参数显式分配为命名参数的情况下调用它？这样做违背了 params 关键字的目的：

// params is defeated:
log.Log("message", 
        messageArgs: new object[] { "Scotty", warpFactor });
// CallerMemberName is defeated:
log.Log("message", null,
        "Scotty", warpFactor);

有没有办法做到这一点？似乎传递调用者信息的“hacky”方式排除了使用 params 关键字。如果 C# 编译器认识到调用者成员信息参数根本不是真正的参数，那就太棒了。我认为没有必要明确地传递它们。

我的备份将是跳过 params 关键字，调用者将不得不在最后一个示例中使用长签名。

4

4 回答 4

8

我不认为它可以完全按照您想要的方式完成。但是，我可以想到一些可行的解决方法，它们可能会给您带来几乎相同的好处。

  1. 使用中间方法调用来捕获调用者成员名称。第一个方法调用返回一个委托，该委托又可以被调用以提供附加参数。这看起来很奇怪，但它应该可以工作：

    log.Log()("{0}: I canna do it cap'n, the engines can't handle warp {1}!", 
"Scotty", warpFactor);

    这里的一个缺点是可以调用log.Log("something")，期望您的消息将被记录，并且不会发生任何事情。如果您使用 Resharper，您可以通过向该方法添加一个[Pure]属性来缓解这种情况，Log()这样如果有人没有对生成的对象执行任何操作，您就会收到警告。你也可以稍微调整一下这种方法，说：

    var log = logFactory.GetLog(); // <--injects method name.
log("{0}: I canna do it cap'n, the engines can't handle warp {1}!", 
    "Scotty", warpFactor);

  2. 使用 lambda 生成日志消息，并让 string.Format 处理 params 数组：

    log.Log(() => string.Format("{0}: I canna do it cap'n, the engines can't handle warp {1}!", 
    "Scotty", warpFactor));

    这是我通常使用的方法，它有一些附带优势：

    1. 您的日志方法可以捕获在生成调试字符串时产生的异常，因此您不会破坏您的系统，而是会收到一条错误消息：“无法生成日志消息：[异常详细信息]”。

    2. 有时，您传递给格式字符串的对象可能会产生额外的成本，而您只想在需要时产生这些成本：

      log.Info(() => string.Format("{0}: I canna do it cap'n, the engines can't handle warp {1}!", 
    _db.GetCurrentUsername(), warpFactor));

      如果未打开信息级日志记录，您不希望上述代码执行数据库访问。

    附带说明一下，我发现自己经常使用 string.Format 以至于我创建了一个辅助方法来稍微缩短语法：

    log.Log(() => "{0}: I canna do it cap'n, the engines can't handle warp {1}!" 
    .With("Scotty", warpFactor));
于 2014-11-03T17:00:47.957 回答
5

要接受 StriplingWarrior 建议而不是委托，您可以使用流利的语法来完成。

public static class Logger
{
    public static LogFluent Log([CallerMemberName] string sourceMemberName = null)
    {
        return new LogFluent(sourceMemberName);
    }
}

public class LogFluent
{
    private string _callerMemeberName;

    public LogFluent(string callerMamberName)
    {
        _callerMemeberName = callerMamberName;
    }

    public void Message(string message, params object[] messageArgs)
    {

    }
}

然后像这样称呼它

Logger.Log().Message("{0}: I canna do it cap'n, the engines can't handle warp {1}!", "Scotty", 10);

记录器不必是静态的，但它是一种演示概念的简单方法

于 2014-11-03T17:37:43.440 回答
1

好吧，让我提一个选择；您可以使用反射来获得完全相同的名称，如CallMemberName. 这肯定会更慢。假设您不是每毫秒都记录一次，我相信这足以应对压力。

var stackTrace = new StackTrace(); var methodName = stackTrace.GetFrame(1).GetMethod().Name;

于 2014-11-03T17:13:04.897 回答
1

我喜欢跟随 Jim Christophers (@beefarino) 的复数课程来设置我自己的日志记录项目。为此，我将 ILog 接口克隆为 ILogger，我实现了它 - 比如说在一个名为 LoggerAdapter 的类中 - 然后我使用 Jim Christophers LogManager 来拥有一个 GetLogger(Type type) - 方法，该方法返回一个包装的 log4net-logger 'LoggerAdapter'：

namespace CommonLogging
{
    public class LogManager : ILogManager
    {
        private static readonly ILogManager _logManager;

        static LogManager()
        {
            log4net.Config.XmlConfigurator.Configure(new FileInfo("log4net.config"));
            _logManager = new LogManager();
        }

        public static ILogger GetLogger<T>()
        {
            return _logManager.GetLogger(typeof(T));
        }

        public ILogger GetLogger(Type type)
        {
            var logger = log4net.LogManager.GetLogger(type);
            return new LoggerAdapter(logger);
        }
    }
}

下一步是创建一个像这样的通用扩展，将调用者信息设置为 ThreadContext.Property：

public static class GenericLoggingExtensions
{
    public static ILogger Log<TClass>(this TClass klass, [CallerFilePath] string file = "", [CallerMemberName] string member = "", [CallerLineNumber] int line = 0)
        where TClass : class
    {
        ThreadContext.Properties["caller"] = $"[{file}:{line}({member})]";
        return LogManager.GetLogger<TClass>();
    }
}

有了它，我总是可以在编写实际方法之前调用一个记录器，只需调用：

this.Log().ErrorFormat("message {0} {1} {2} {3} {4}", "a", "b", "c", "d", "e");

如果您将 PatternLayout 的 conversionPattern 配置为使用该属性：

<layout type="log4net.Layout.PatternLayout">
  <conversionPattern value="%utcdate [%thread] %-5level %logger - %message - %property{caller}%newline%exception" />
</layout>

您将始终获得带有第一个 .Log() 调用的调用者信息的正确输出：

2017-03-01 23:52:06,388 [7] ERROR XPerimentsTest.CommonLoggingTests.CommonLoggingTests - message a b c d e - [C:\git\mine\experiments\XPerimentsTest\CommonLoggingTests\CommonLoggingTests.cs:71(Test_Debug_Overrides)]
于 2017-03-01T23:53:19.030 回答