10

我需要检查 cookie 是否存在价值。但我想知道是否有一些快速而好的方法可以这样做，因为如果我需要检查 3 个 cookie，用ifor来检查似乎很糟糕try。

如果 cookie 不存在，为什么它不会为我的变量分配空字符串？相反，它显示Object reference not set to an instance of an object.

我的代码（它有效，但对于这项任务来说似乎太大了，我认为应该有更好的方法来做到这一点）

// First I need to asign empty variables and I don't like this
string randomHash = string.Empty;
string browserHash = string.Empty;
int userID = 0;

// Second I need to add this huge block of try/catch just to get cookies
// It's fine since I need all three values in this example so if one fails all fails
try
{
    randomHash = Convert.ToString(Request.Cookies["randomHash"].Value);
    browserHash = Convert.ToString(Request.Cookies["browserHash"].Value);
    userID = Convert.ToInt32(Request.Cookies["userID"].Value);
}
catch
{
    // And of course there is nothing to catch here
}

正如你所看到的，我有这么大块只是为了得到饼干。我想要的是这样的：

// Gives value on success, null on cookie that is not found
string randomHash = Convert.ToString(Request.Cookies["randomHash"].Value);
string browserHash = Convert.ToString(Request.Cookies["browserHash"].Value);
int userID = Convert.ToInt32(Request.Cookies["userID"].Value);

编辑也许我可以以某种方式覆盖.Value我喜欢的方法？

4

1 回答 1

13

只需检查 cookie 是否为空：

if(Request.Cookies["randomHash"] != null)
{
   //do something
}

注意：“更好”的做法是编写既可读又可靠的好代码。它不会分配空字符串，因为这不是 C# 的工作方式，您正在尝试调用对象 ( )Value上的属性- 您不能使用空对象，因为没有什么可使用的。nullHttpCookie

转换为int你仍然需要避免解析错误，但你可以使用这个内置方法：

int.TryParse(cookieString, out userID);

这带来了另一点？为什么要将用户 ID 存储在 cookie 中？这可以由最终用户更改 - 我不知道您打算如何使用它，但我认为这是一个很大的安全漏洞是否正确？

或带有一点辅助功能：

public string GetCookieValueOrDefault(string cookieName)
{
   HttpCookie cookie = Request.Cookies[cookieName];
   if(cookie == null)
   {
      return "";
   }  
   return cookie.Value;
}

然后...

string randomHash = GetCookieValueOrDefault("randomHash");

或者使用扩展方法：

public static string GetValueOrDefault(this HttpCookie cookie)
{
   if(cookie == null)
   {
      return "";
   }  
   return cookie.Value;  
}

然后...

string randomHash = Request.Cookies["randomHash"].GetValueOrDefault();
于 2012-08-24T12:57:50.610 回答