C# -Warning Возможное непреднамеренное сравнение ссылок; чтобы получить сравнение значения, отлитый левую часть к типу «строка»

Question

if ((Session["UserName"] != null && Session["LoginType"] == "Admin") || (Session["UserName"] != null && Session["LoginType"] == "Employee")) 
{ 
    TotalMarketBalance(); 
} 

Answer 1

Я предлагаю изменить if -statement к этому:

if (Session["UserName"] != null && 
    (Session["LoginType"] as string == "Admin" || 
     Session["LoginType"] as string == "Employee") 
    ) 

Session[string key] возвращает object. И если вы сравниваете object с чем-то, используя ==, вы делаете сравнение ссылок. И строковый литерал (например, "Admin") никогда не будет иметь такую ​​же ссылку, как этот объект, даже если этот объект - это строка.

Отбрасывая object в string, компилятор знает, что он должен вызывать методы равенства string, которые сравнивают содержимое строки, а не их ссылки.

Конечно, вы можете сделать прямой перевод ((string)Session["LoginType"]) или позвонить по телефону ToString(). Но первый будет генерировать исключение, если (по какой-то странной причине) объект возврата не строка. Второй будет бросать NullReferenceException, если (по какой-то странной причине) значение по-прежнему null.

Answer 2

Изменение Session["LoginType"] в Session["LoginType"].ToString()

Answer 3

вы также можете бросить в строку:

(string)Session["LoginType"] == "Admin" 

Это как пример делается в документации Reading Values From Session State