将会话处理包装在静态类中

Question

我将所有直接会话交互分离到一个单独的类中并将其设为静态，因为我不想多次创建新对象。 但是，我希望确保不存在并发问题或其他不稳定的意外。

这是代码：

public static class HttpHelper
{

    public static string Get(string key)
    {
        object value = HttpContext.Current.Request.QueryString[key];
        return (value == null) ? null : value.ToString();
    }


    public static string Post(string key)
    {
        object value = HttpContext.Current.Request.Form[key];
        return (value == null) ? null : value.ToString();
    }

    public static string Session(string key)
    {
        object value = HttpContext.Current.Session[key];
        return (value == null) ? null : value.ToString();
    }

    public static void ClearSession(string key)
    {
        HttpContext.Current.Session[key] = null;
    }

    public static void StoreInSession(string key, object value)
    {
        HttpContext.Current.Session[key] = value;
    }

}

Answer 1

如果存在任何功能问题，许多应用程序早就失败了:)

但是，我要指出的是，这个模型不太适合单元测试。 您可能需要考虑使方法成为实例成员并将 Session 提供程序传递到此 HttpHelper 对象的构造函数中。

Answer 2

这是一个类似问题的答案，可能会对您有所帮助 - 它将使您能够避免使用密钥来完全访问会话值并为您提供类型安全的属性：

如何从 ASP.NET 中的任何类访问会话变量？

Answer 3

从概念上讲，你应该没问题。 即使在部分回发（EG、AJAX）场景中，您可能会遇到竞争条件，也应该没问题。 会话状态使用读取器/写入器锁来保持你安全了。

我倾向于在我的项目中做类似的事情，尽管我喜欢将实际有效的会话条目（键等）封装到属性中。 我发现它使应用程序代码更加一致，而且更重要的是，消除了魔术键字符串上的拼写错误的可能性，该拼写错误将使应用程序以极其意想不到的方式运行。

如果您对应用程序状态执行类似的操作，那么您必须确保在设置值之前锁定和解锁值。

Answer 4

我不确定这个包装器是否有用，但我认为您可以进行以下改进
相反，

public static string Get(string key)
{
    object value = HttpContext.Current.Request.QueryString[key];
    return (value == null) ? null : value.ToString();
}

您可以使用

public static string Get(string key)
{
    return HttpContext.Current.Request.QueryString[key];
}

相同的 Post 方法。
如果您可以通过 StoreInSession 方法在会话中存储任何对象，那么您的 Session 方法是否只返回 string ？