内存中剩余的静态类...如何最好地实现这一目标

本文关键字：实现 目标 何最好 静态类 内存 | 更新日期: 2023-09-27 18:32:07

我创建了一个非常简单的帮助程序类，可以在我的 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 类的实例，如下所示：

私有静态用户消息 _userMessage { get; set; }

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

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，但我应该如何纠正呢？

内存中剩余的静态类...如何最好地实现这一目标

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

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字段，并且您无法保证将显示当前用户的消息。

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

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;
}

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

在这种情况下，简单的 ACID 支持的数据库（实际上几乎是市场上的任何数据库）非常适合这种情况。

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

希望这有帮助。

静态变量将在 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
   ...
}

我可能还会使用户消息构造函数private。

相关文章：