comparing session variable value to a string

Question

I am comparing the session variable to a string to check if the login type is administrator or not.

Code i am using :

if (Session["loggedInUsername"] == null)
        {
            btnLogin.Text = "Sign In";
            lblWelcome.Text = "Welcome!";
            hypManageRestaurants.Enabled = false;
            hypManageReviews.Enabled = false;
            hypPostReviews.Enabled = false;

        }
        else
        {
            if (Session["loggedInUserType"] == "Administrator")
            {
                hypManageRestaurants.Enabled = true;
                hypManageReviews.Enabled = true;
                hypPostReviews.Enabled = true;
            }
            else
            {
                hypManageRestaurants.Enabled = false;
                hypManageReviews.Enabled = false;
                hypPostReviews.Enabled = true;
            }
            lblWelcome.Text = "Welcome " + Session["loggedInUsername"];

            btnLogin.Text = "Sign Out";
        }

So first I am checking if any user has logged in or not. If the user logs in successfully, the session variable "loggedInUsername" will have the value of the username. If the "loggedInUsername" session variable is not empty, it will check "loggedInUserType" session variable for the type of the logged in user.

Here comes the weird thing, the value of the "loggedInUserType" is exactly "Administrator" without the "", at the if function where I am comparing the session variable to the string "Administrator" is being skipped and goes to the else.

All session variables are getting values when the user logs in.

Below is the data which I am using to login and this record is the only record which have a type of "Administrator".

link to image

Is there any other method to compare a session variable to a string

How do you mean it's being skipped? Is it going into the else statement? It's probably a casting issue. Try either Object.ReferenceEquals(a,b) or "Administrator".Equals(Session["..."]) — Thinking Sites, May 21 '12 at 14:17
Use a [Membership provider](http://msdn.microsoft.com/en-us/library/yh26yfzy.aspx) instead. — Tim Schmelter, May 21 '12 at 14:19

score 5 · Answer 1 · answered May 21 '12 at 14:17

5

Try

if(Convert.ToString(Session["loggedInUserType"]) == "Administrator) ...

answered May 21 '12 at 14:17

Carl Winder

938
8
18

This method also works :) but I dont know if i can mark 2 answers xD – drinu16 May 21 '12 at 14:24

score 5 · Accepted Answer · answered May 21 '12 at 14:18

5

Cast the object type value to a string

((string)Session["loggedInUserType"]) == "Administrator"

answered May 21 '12 at 14:18

Jaimal Chohan

8,530
6
43
64

score 1 · Answer 3 · answered May 21 '12 at 14:17

1

Try this:

if (Session["loggedInUserType"].ToString().Trim()
        .Equals("Administrator", StringComparison.InvariantCultureIgnoreCase))

answered May 21 '12 at 14:17

Marco

56,740
14
129
152

score 1 · Answer 4 · answered May 21 '12 at 14:20

The Session collection returns values of type Object, so when you compare that to a string you will be comparing the values of the object references, not comparing the string values.

Cast the object reference to string:

if (((string)Session["loggedInUserType"]) == "Administrator")

score 0 · Answer 5 · answered May 21 '12 at 14:17

0

Are you sure that all whitespace has been trimmed from the end of the Session["loggedInUserType"]?

answered May 21 '12 at 14:17

Dave Lawrence

3,843
2
21
35

score 0 · Answer 6 · answered May 21 '12 at 14:20

0

if (Session["loggedInUserType"].ToString() == "Administrator")

answered May 21 '12 at 14:20

daryal

14,643
4
38
54

score 0 · Answer 7 · edited Oct 19 '16 at 15:12

0

You can do this :

string session = (string)Session["loggedInUserType"]

if (session == "Administrator")

or your Session can be in a specific class with getters.

edited Oct 19 '16 at 15:12

Roman Marusyk

23,328
24
73
116

answered Oct 19 '16 at 14:47

SarahB

135
1
11

score 0 · Answer 8 · answered Feb 19 '18 at 05:57

0

if(Convert.ToString(Session["loggedInUserType"]) == "Administrator)

In this way, no need to check null value becuase Convert.ToString handle Null value return "" empty string

answered Feb 19 '18 at 05:57

Kashif Faraz

321
6
19

comparing session variable value to a string

8 Answers8

Linked