Question

在我的ASP.NET MVC应用程序中，我大量使用User.Identity.GetUserId()。但是，我想知道这是否有严重的性能损失。

或者，我相信我可以这样做：在View中，我可以将当前用户的id分配给第一页加载中的隐藏字段。然后，在进行AJAX调用时，我可以将隐藏的字段值传递给控制器的操作。这样，我就不需要使用User.Identity.GetUserId()方法来检索当前用户的用户ID。

我想知道是否有人对此有任何想法？

Answer 1

查看GetUserId扩展方法的来源：

/// <summary>
///     Return the user id using the UserIdClaimType
/// </summary>
/// <param name="identity"></param>
/// <returns></returns>
public static string GetUserId(this IIdentity identity)
{
    if (identity == null)
    {
        throw new ArgumentNullException("identity");
    }
    var ci = identity as ClaimsIdentity;
    if (ci != null)
    {
        return ci.FindFirstValue(ClaimTypes.NameIdentifier);
    }
    return null;
}

/// <summary>
///     Return the claim value for the first claim with the specified type if it exists, null otherwise
/// </summary>
/// <param name="identity"></param>
/// <param name="claimType"></param>
/// <returns></returns>
public static string FindFirstValue(this ClaimsIdentity identity, string claimType)
{
    if (identity == null)
    {
        throw new ArgumentNullException("identity");
    }
    var claim = identity.FindFirst(claimType);
    return claim != null ? claim.Value : null;
}

每次调用该扩展程序时，都会搜索ClaimTypes.NameIdentifier声明的标识。

性能影响并不大（IMO），但隐藏的用户信息泄露（如果只需点击view source即可看到它们，实际上并不隐藏）不是一个好主意。

如果您担心多次调用并在多个位置需要它，那么您可以将它延迟加载到控制器或基本控制器中的属性后面。

private string userId
public string UserId {
    get {
        if(userid == null) {
            userid = User.Identity.GetUserId();
        }
        return userid;
    }
}

您还可以创建一个服务来封装该信息。

User.Identity.GetUserId（）的负担是什么？

1 个答案: