ASP.NET中存储与检索Session的方式是否正确？自定义会话类实现合理吗？

嘿，我来帮你捋捋ASP.NET里Session存储和检索的常规路子，顺便看看你的代码有没有可以调整的地方～

首先先给你吃个定心丸：你在Controller里存储Session的方式是完全正确的——在list方法里通过Session["memberid"] = 12345;存储值，这就是ASP.NET Framework里存储Session的标准操作之一。

接下来看看你用静态类SessionData检索Session的思路，方向是对的，但有几个小细节可以优化，同时也给你补充几种更常用的常规写法：

1. 你的代码里的小笔误（先修正） #

在SessionData.MemberTableId()方法中，HttpContext.Current.Session[memberid]这里的memberid应该是字符串常量"memberid"，不然会编译报错（因为没定义这个变量）。修正后更健壮的写法可以用常量统一维护Key：

public static class SessionData
{
    // 用常量存Key，避免拼写错误，也方便后续统一修改
    private const string MemberIdSessionKey = "memberid";
    
    public static int MemberTableId()
    {
        // 一定要加空值判断！Session可能过期或被清除
        if (HttpContext.Current.Session[MemberIdSessionKey] == null)
        {
            // 这里可以根据业务需求处理：比如抛异常、返回默认值，或者跳转登录
            throw new InvalidOperationException("用户ID不存在于Session中，请重新登录");
        }
        return Convert.ToInt32(HttpContext.Current.Session[MemberIdSessionKey]);
    }
}

2. ASP.NET中检索Session的几种常规方式 #

除了你用静态类的方式，还有几种更贴合ASP.NET开发习惯的方案，你可以根据场景选：

方案一：直接在Controller中使用自带的Session属性 #

你的TestController继承自Controller，而Controller本身已经封装了Session属性，所以可以直接在Action里取，不需要额外的静态类，简单直接：

public ActionResult iteminfo(string prodcode)
{
    int memberid;
    if (Session["memberid"] != null)
    {
        memberid = Convert.ToInt32(Session["memberid"]);
    }
    else
    {
        // 处理Session不存在的情况，比如跳转到登录页
        return RedirectToAction("Login", "Account");
    }
    // 后续业务逻辑...
}

方案二：用Controller扩展方法封装（适合多Controller复用） #

如果多个Controller都需要获取MemberId，写个扩展方法会比静态类更贴合Controller的使用场景：

public static class ControllerExtensions
{
    private const string MemberIdSessionKey = "memberid";
    
    public static int GetMemberId(this Controller controller)
    {
        if (controller.Session[MemberIdSessionKey] == null)
        {
            throw new InvalidOperationException("用户Session已失效，请重新登录");
        }
        return Convert.ToInt32(controller.Session[MemberIdSessionKey]);
    }
}

然后在Controller里就能这样用：

public ActionResult iteminfo(string prodcode)
{
    int memberid = this.GetMemberId();
    // 后续业务逻辑...
}

方案三：用ActionFilter提前注入（适合大量Action需要的场景） #

如果很多Action都需要MemberId，可以写个ActionFilter，在Action执行前自动获取并注入参数，减少重复代码：

public class InjectMemberIdAttribute : ActionFilterAttribute
{
    private const string MemberIdSessionKey = "memberid";
    
    public override void OnActionExecuting(ActionExecutingContext filterContext)
    {
        var session = filterContext.HttpContext.Session;
        if (session[MemberIdSessionKey] != null)
        {
            int memberId = Convert.ToInt32(session[MemberIdSessionKey]);
            // 把MemberId注入到Action的参数中
            filterContext.ActionParameters["memberId"] = memberId;
        }
        else
        {
            // Session失效，直接跳转到登录页
            filterContext.Result = new RedirectToRouteResult(new RouteValueDictionary
            {
                { "controller", "Account" },
                { "action", "Login" }
            });
        }
        base.OnActionExecuting(filterContext);
    }
}

然后在Action里就能直接用参数：

[InjectMemberId]
public ActionResult iteminfo(string prodcode, int memberId)
{
    // 直接用memberId即可，不用再手动取Session
    // 后续业务逻辑...
}

3. 额外的安全&健壮性建议 #

• 一定要做空值检查：Session可能因为过期、服务器重启（InProc模式下）等原因丢失，直接Convert.ToInt32会抛出异常，必须先判断是否存在。
• 注意Session存储模式：默认是InProc（进程内存储），如果你的项目是Web农场/集群环境，要改用StateServer或SQLServer模式，避免Session跨服务器丢失。

总的来说，你的静态类思路是可行的，补上空值检查和常量Key后会更健壮，同时也可以根据自己的业务场景选择更贴合ASP.NET常规写法的方案～

内容的提问来源于stack exchange，提问作者SK Yogi