获得更多“粒度”来自 MVC 迷你分析器

Question

如果这对任何人都有用，我很乐意将其变成社区维基。

我的 MVC3 应用程序中有一些缓慢的页面，并且由于我的代码中似乎几乎没有执行时间，所以我想看看是否可以找到更多有关花费了这么长时间的信息。并不是说我成功了，而是我一路走来获得了更多的智慧。

对于具有 MVC 经验的人来说，这里没有什么是不明显的。基本上，我创建了自己的 ActionFilterAttribute，如下所示：

public class ProfilerAttribute : ActionFilterAttribute
{
    IDisposable actionStep = null;
    IDisposable resultStep = null;

    public override void OnActionExecuting(ActionExecutingContext filterContext)
    {
        actionStep = MiniProfiler.Current.Step("OnActionExecuting " + ResultDescriptor(filterContext));
        base.OnActionExecuting(filterContext);
    }

    public override void OnActionExecuted(ActionExecutedContext filterContext)
    {
        if (actionStep != null)
        {
            actionStep.Dispose();
            actionStep = null;
        }
        base.OnActionExecuted(filterContext);
    }

    public override void OnResultExecuting(ResultExecutingContext filterContext)
    {
        resultStep = MiniProfiler.Current.Step("OnResultExecuting " + ResultDescriptor(filterContext));
        base.OnResultExecuting(filterContext);
    }

    public override void OnResultExecuted(ResultExecutedContext filterContext)
    {
        if (resultStep != null)
        {
            resultStep.Dispose();
            resultStep = null;
        }
        base.OnResultExecuted(filterContext);
    }

    private string ResultDescriptor(ActionExecutingContext filterContext)
    {
        return filterContext.ActionDescriptor.ControllerDescriptor.ControllerName + "." + filterContext.ActionDescriptor.ActionName;
    }

    private string ResultDescriptor(ResultExecutingContext filterContext)
    {
        var values = filterContext.RouteData.Values;

        return String.Format("{0}.{1}", values["controller"], values["action"]);
    }

这似乎工作得很好，就我而言，我了解到大部分时间实际上花在生活的 ResultExecuting 部分，而不是在我的操作中。

但是，我对这种方法有一些疑问。

1）这是一种请求安全的做事方式吗？我猜不会，因为 actionfilter 仅在 Global.asax.cs 的 RegisterGlobalFilters() 方法中创建一次。如果同时出现两个请求，actionStep 和 resultStep 将毫无价值。这是真的吗？如果是这样，比我了解更多的人可以贡献一个聪明的方法来处理这个问题吗？在本地机器分析期间对我有用，但可能不太适合部署在多人同时发出请求的服务器上。

2）有没有办法更深入地了解结果执行过程？或者我应该接受渲染视图等需要花费时间吗？在我自己的应用程序中，我确保所有数据库访问在我的操作方法结束之前完成（在我的例子中使用 NHibernate Profiler），并且我喜欢保持我的视图简洁明了；不过，任何对导致渲染速度变慢的原因的深入了解仍然是有用的。我猜想，如果我的部分执行速度较慢，那么在我的模型对象中使用 Mini Profiler 会出现在这里。

3) ResultDescriptor 方法可能是邪恶且有毒的。它们在我的测试中为我工作，但可能需要被更强大的东西取代。我只是选择了第一个版本，它给了我一些有用的东西。

对此的任何其他评论也将非常受欢迎，即使他们是“这是一个坏主意，独自死去”。

Answer 1

这看起来是一个很酷的主意。我相信这不是一种请求安全的做事方式。

您可以像这样将其链接到 HttpContext.Items

HttpContext.Items.Add("actionstep", actionStep);
HttpContext.Items.Add("resultstep", resultStep);

然后以类似的方式检索它

actionStep = HttpContext.Items["actionstep"];
resultStep = HttpContext.Items["resultstep"];

显然放入您自己的空值检查等。

每个用户/请求的 HttpContext 都是不同的。

关于 HttpContext.Current.Session.SessionID 需要记住的一点是，我有时会忘记它是当前 HTTP 请求的 SessionId（即每次按 F5 或以其他方式发出新请求时它都会更改） 。另一件需要记住的重要事情是，在任何时间，所有 HttpContext.Current.Session.SessionID 值都必须是唯一的（即每个用户或请求一个），它们可以重复使用，因此，不要将它们视为仅使用一次的 GUID。

Answer 2

MiniProfiler 程序集中已有一个操作筛选器属性，用于对操作进行分析。它位于 StackExchange.Profiling.MVCHelpers 命名空间中，称为 ProfilingActionFilter。您还可以扩展它来分析您的观点。

它使用与 @Dommer 描述的相同方法，但不是直接存储 IDisposable，而是将 Stack 存储在 HttpContext.Current.Items 中。您可以对视图执行相同的操作。

以下是操作分析的代码：

/// <summary>
/// This filter can be applied globally to hook up automatic action profiling
/// 
/// </summary>
public class ProfilingActionFilter : ActionFilterAttribute
{
    private const string stackKey = "ProfilingActionFilterStack";

    /// <summary>
    /// Happens before the action starts running
    /// 
    /// </summary>
    public override void OnActionExecuting(ActionExecutingContext filterContext)
    {
        if (MiniProfiler.Current != null)
        {
            Stack<IDisposable> stack = HttpContext.Current.Items[(object) "ProfilingActionFilterStack"] as Stack<IDisposable>;
            if (stack == null)
            {
                stack = new Stack<IDisposable>();
                HttpContext.Current.Items[(object) "ProfilingActionFilterStack"] = (object) stack;
            }
            MiniProfiler current = MiniProfiler.Current;
            if (current != null)
            {
                RouteValueDictionary dataTokens = filterContext.RouteData.DataTokens;
                string str1 = !dataTokens.ContainsKey("area") || string.IsNullOrEmpty(dataTokens["area"].ToString()) ? "" : (string) dataTokens["area"] + (object) ".";
                string str2 = Enumerable.Last<string>((IEnumerable<string>) filterContext.Controller.ToString().Split(new char[1] { '.' })) + ".";
                string actionName = filterContext.ActionDescriptor.ActionName;
                stack.Push(MiniProfilerExtensions.Step(current, "Controller: " + str1 + str2 + actionName, ProfileLevel.Info));
            }
        }
        base.OnActionExecuting(filterContext);
    }

    /// <summary>
    /// Happens after the action executes
    /// 
    /// </summary>
    public override void OnActionExecuted(ActionExecutedContext filterContext)
    {
        base.OnActionExecuted(filterContext);
        Stack<IDisposable> stack = HttpContext.Current.Items[(object) "ProfilingActionFilterStack"] as Stack<IDisposable>;
        if (stack == null || stack.Count <= 0) return;
        stack.Pop().Dispose();
    }
}

希望有所帮助。

Answer 3

您可以将 ExecuteCore 方法包装在 Controller 上。 :)