Apesar do fato de eu estar aqui há algum tempo, esta é a minha primeira pergunta sobre SO, portanto, seja gentil comig

Estou a usarASP.NET MVC 3 e quero criar um @ personalizaPrincipal para que eu possa armazenar um pouco mais de informações sobre o usuário atual do que o padrão, portanto, não é necessário acessar o banco de dados com muita frequência. É algo bastante padrão que eu estou procurando. Digamos apenas o endereço de email e o ID do usuário em primeira instânci

ecidi armazenar o objeto no cache, pois sei que não é recomendável armazená-lo na sessã

Eu também não quero continuar transmitindo oUser, então eu queria substituir oUser objeto no controlador. Então eu posso simplesmente irUser.UserId e tenha a garantia de algo.

Assim criei um objeto personalizado como este:

public class MyPrincipal : IPrincipal
{
    public MyPrincipal(IIdentity ident, List<string> roles, string email, Guid userId)
    {
        this._identity = ident;
        this._roles = roles;
        this._email = email;
        this._userId = userId;
    }

    IIdentity _identity;

    public IIdentity Identity
    {
        get { return _identity; }
    }

    private List<string> _roles;

    public bool IsInRole(string role)
    {
        return _roles.Contains(role);
    }

    private string _email;

    public string Email
    {
        get { return _email; }
    }

    private Guid _userId;

    public Guid UserId
    {
        get { return _userId; }
    }
}

E eu tenho um controlador base como este:

public class BaseController : Controller
    {
        protected virtual new MyPrincipal User
        {
            get
            {
                if (base.User is MyPrincipal)
                {
                    return base.User as MyPrincipal;
                }
                else
                {
                    return new MyPrincipal(base.User.Identity, new List<string>(0), "", Guid.Empty );
                }
            }
        }

        protected override void OnAuthorization(AuthorizationContext filterContext)
        {
            if (User != null)
            {
                if (User.Identity.IsAuthenticated)
                {
                    if (User.Identity is FormsIdentity)
                    {
                        FormsIdentity id = base.User.Identity as FormsIdentity;
                        MyPrincipal principal = (MyPrincipal)filterContext.HttpContext.Cache.Get(id.Name);
                        if (principal == null)
                        {
                            MembershipUser user = Membership.GetUser();

                            // Create and populate your Principal object with the needed data and Roles.
                            principal = new MyPrincipal(id, Roles.GetRolesForUser(id.Name).ToList(), user.Email, (Guid)user.ProviderUserKey);
                            filterContext.HttpContext.Cache.Add(
                            id.Name,
                            principal,
                            null,
                            System.Web.Caching.Cache.NoAbsoluteExpiration,
                            new System.TimeSpan(0, 30, 0),
                            System.Web.Caching.CacheItemPriority.Default,
                            null);
                        }
                        filterContext.HttpContext.User = principal;
                        System.Threading.Thread.CurrentPrincipal = principal;
                        base.OnAuthorization(filterContext);
                    }
                }
            }
        }
    }

Se você der uma olhada, rapidamente perceberá que, se o usuário não estiver conectado, entãoqualque ligue para oUser objeto @ precisará executar esse código:

return new MyPrincipal(base.User.Identity, new List<string>(0), "", Guid.Empty );

e isso me parece terrivelmente ineficiente, embora apenas crie objetos vazios para as coisas que estão faltand

Funciona bem

Então, eu acho que quero saber se isso está realmente bem e devo parar de ser tão anal sobre desempenho e eficiência, ou se meus medos estão corretos; nesse caso, o que devo fazer? [Por favor, não diga "Conseguindo uma vida, companheiro!"]