Question

我创建了一个非常简单的帮助类，我可以在ASP.Net页面中使用它。我们的想法是，它应该是一种非常简单的方法来记录页面错误或成功（而不是表单验证错误），然后将其显示给用户。

在我的公共帮助器类中，我有一个具有某些属性的类，如下所示：

public class UserMessage
{
    public UserMessage()
    {
        Messages = new Dictionary<string, string>();
    }

    public string SummaryMessage;
    public Dictionary<string, string> Messages;
    public bool ShowMessages;
    public bool ShowAsError;
}

然后我有一个变量用于存储UserMessage类的实例，如下所示：

private static UserMessage _userMessage {get;组; }

然后我有两个公共静态方法，一个用于记录消息，另一个用于显示所有消息，如下所示：

public static void LogSummary(string summaryMessage, bool showIndividualMessages, bool showAsError)
{
    _userMessage = new UserMessage();
    _userMessage.SummaryMessage = summaryMessage;
    _userMessage.ShowMessages = showIndividualMessages;
    _userMessage.ShowAsError = showAsError;
}

public static string DisplayUserMessages()
{
    if (_userMessage == null)
        return string.Empty;

    StringBuilder messageString = new StringBuilder();
    messageString.AppendFormat("\n");
    messageString.AppendLine(string.Format("<div class=\"messageSummary {0}\">", (_userMessage.ShowAsError) ? "invalid" : "valid"));
    messageString.AppendLine(string.Format("<h3>{0}</h3>", _userMessage.SummaryMessage));
    messageString.AppendLine("</div>");

    return messageString.ToString();
}

我遇到的问题是_userMessage变量必须是一个静态变量，否则我会收到错误消息“非静态字段需要一个对象引用.......”。变量是静态的问题是保留在内存中，因此如果用户收到错误消息然后访问另一个页面 - 仍会显示错误消息！

我确定这是因为我错过了OOP 101，但我该如何纠正呢？

提前致谢人

Answer 1

将引用作为参数传递给静态成员，或让它返回如下所示的新实例：

public static UserMessage LogSummary(string summaryMessage, bool showIndividualMessages, bool showAsError)
{
    var userMessage = new UserMessage();
    userMessage.SummaryMessage = summaryMessage;
    userMessage.ShowMessages = showIndividualMessages;
    userMessage.ShowAsError = showAsError;
    return userMessage;
}

Answer 2

不要使用静态变量来保留每个用户的消息！ ASP.NET应用程序是多线程的，使用静态变量不是线程安全的。将它们存储在Session中。

public static void LogSummary(string summaryMessage, ...)
{
   HttpContext.Current.Session["userMessages"] = new UserMessage(); 
   ...
}

public static string DisplayUserMessages()
{
   // get the value from session
   var userMessage = (UserMessage)HttpContext.Current.Session["userMessages"];
   // do the work
   // do the clean up
   HttpContext.Current.Session["userMessages"] = null;
   // the messages will not be displayed on next request
}

每个请求都由不同的线程处理，因此用户将覆盖_userMessage字段，并且您无法保证将显示当前用户的消息。

Answer 3

在我的意见中，您尝试用错误的方法来解决问题。考虑到您正在开发服务端组件（ASP.NET），并且每个用户访问您的站点之间必须有完美的隔离，我个人认为没有任何理由不使用后端数据库用于保存错误消息，其中每个记录可以与单个用户唯一ID相关联。

简单ACID支持的数据库（实际上几乎所有市场上的数据库）都非常适合这种情况。

通过这种方式，您可以在需要时从数据库中提取所需的消息，并且不需要担心任何类型的内存问题（至少从这个问题的角度来看）

希望这有帮助。

Answer 4

静态变量将在AppDomain中共享 - 即并发请求将共享同一个实例，因此您的方法存在问题。

您应该考虑将您的用户消息实例放入当前HttpContext以根据您的用例需要获取每个请求的语义。例如，

public class UserMessage
{
   public static UserMessage Current
   {
      get { return HttpContext.Current.Items["_User_Message"] as UserMessage; }
   }

   public static void LogSummary(string summaryMessage, bool showIndividualMessages, bool showAsError)
   {
      var userMessage = new UserMessage();
      userMessage.SummaryMessage = summaryMessage;
      ...  

      HttpContext.Current.Items["_User_Message"] = userMessage;
   }

   public static string DisplayUserMessages()
   {
       var userMessage = UserMessage.Current;
       if (userMessage == null ) return string.Empty;

       ...
   }

   // rest of the code
   ...
}

我可能也会创建UserMessage构造函数private。

静态类留在内存中......如何最好地实现这一点

4 个答案: