ASP.NET MVC会话

这是不好的做法吗?

using System.Web.Mvc;

namespace WebInventaris.Utils.Session
{
    public static class SessionControllerExtensions
    {
        public static void SetSessionVar<T>(this Controller controller, string field, T value)
        {
            controller.Session[field] = value;
        }

        public static T GetSessionVar<T>(this Controller controller, string field)
        {
            return (T) controller.Session[field];
        }
    }
}

我试图将会话与控制器分离,但对这些方法没有什么好的感觉。