CODEWITHSUNDEEP
c#asp.netsession-variables
drinu16
drinu16

Reputation: 795

comparing session variable value to a string

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

Upvotes: 6

Views: 29134

Answers (8)

Kashif Faraz
Kashif Faraz

Reputation: 331

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

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

Upvotes: 0

SarahB
SarahB

Reputation: 155

You can do this : 

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

if (session == "Administrator")

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

Upvotes: 0

Guffa
Guffa

Reputation: 700680

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")

Upvotes: 1

daryal
daryal

Reputation: 14929

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

Upvotes: 0

Jaimal Chohan
Jaimal Chohan

Reputation: 8645

Cast the object type value to a string

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

Upvotes: 5

Carl Winder
Carl Winder

Reputation: 948

Try

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

Upvotes: 5

Dave Lawrence
Dave Lawrence

Reputation: 3929

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

Upvotes: 0

Marco
Marco

Reputation: 57593

Try this:

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

Upvotes: 1

Related Questions