在特定条件下拦截和忽略方法调用的最有效方法是什么？

Question

我有一个类Logger，其中包含许多用于各种用户活动记录的静态方法。比如：

public static class Logger
{
    public static void FileDownload(int fileId, int userId) 
    {
        // Do stuff
    }

    // ... and a number of similar additional methods
}

所以我现在想忽略为特定角色的用户记录活动。

我的想法是，为了避免重写大量代码并出于 DRY 的目的，实现一个我可以在 Logger 类上使用的自定义属性，该属性将在每个方法调用中验证是否用户处于特定角色，在这种情况下，我想完全忽略方法调用。

我无法解决的问题是如何拦截和中止我的属性中的方法调用。

是否有可能，或者是否有其他更有效的方法来拦截方法调用并在某个条件为真时忽略它？

Answer 1

一种有效的方法和DRY 是将这些方法从static 更改为实例方法，然后从类Logger 继承并覆盖其方法以不执行任何操作。

你的Logger 类现在看起来像这样：

public class Logger
{
    public virtual void FileDownload(int fileId, int userId) 
    {
        // Do stuff
    }
    // ... and a number of similar additional methods
}

子类将如下所示：

public class IdleLogger : Logger
{
    public override void FileDownload(int fileId, int userId) 
    {
        // Do NOTHING
    }
    // ... and a number of similar additional methods
}

那么你可以说：

var myLogger =  IsUserInNoLoggingRole  ? new IdleLogger() : new Logger();

...你就完成了！简单、高效、DRY 和优雅。

Answer 2

一般来说，您的日志记录应该是完整的，这取决于需要根据用户类型决定显示活动的报告应用程序。话虽如此（我相信你有充分的理由去做“if then else”），这可以通过创建一个通用的日志记录方法（logMessage）来实现

int t = User.LogRequired? logMessage(<parameters here>) : 0;

Answer 3

为什么不在实际记录任何内容之前简单地检查一下？

public static class Logger
{
 public void LogSensetiveInformation(string message)
 {
  if (!AllowedLogging(HttpContext.Current)
  {
    return;
  }

  LogMessage(message);// write message to some storage

 }
 public void Log(string message)
 {
  LogMessage(message);// write message to some storage
 }
}

这样您就不需要以任何特殊方式注释代码（这可能无论如何都不可能，因为所有类型的用户都可能使用相同的方法）。