Code for differentiating "Admin" user from normal user not working

Question

0.00/5 (No votes)

See more:

Hi Guys this is the Login Page with the snipped:

C#

public int m = 0;
string user = Usernametxt.Text;
if (user.Equals("admin"))
{
    m = 1;
}
else { m = 0; }

and another snipped on another form where:

C#

private void HomePagefrm_Load(object sender, EventArgs e)
       {
           Loginfrm objLoginfrm = new Loginfrm();
           //int z=objLoginfrm.m;
           //int k=1;

           if (objLoginfrm.m==1)
           {

              Register_btn1.Enabled = true;
              Report_btn.Enabled = true;
           }
           else
           {

               Register_btn1.Enabled = false;
               Report_btn.Enabled = false;
           }
       }

However, when I run this code I get the buttons disabled for both admin and normal users.
Please Help

Posted 14-Mar-13 1:59am

Arora Varun

Updated 14-Mar-13 3:14am

v5

Add a Solution

Comments

CHill60 14-Mar-13 8:12am

According to your snippet you're never displaying objLoginForm so m is always it's initial value of 0. Not quite sure what the point of if(z.Equals(k))
what's wrong with if(objLoginfrm.m == 1?

Arora Varun 14-Mar-13 8:17am

i even tried
if (objLoginfrm.m==1)
{

Register_btn1.Enabled = true;
Report_btn.Enabled = true;
}
but no...

[no name] 14-Mar-13 8:54am

You are creating a new instance of LoginFrm so m will always be equal to 0.

Arora Varun 14-Mar-13 8:56am

So what should be done in such a scenario...???

[no name] 14-Mar-13 9:00am

Pass a reference to your login page to your homepagefrm.

Arora Varun 14-Mar-13 9:01am

never thought of it.. Thanx il keep it to "admin"...
Aniwaz plz gime a soln techies..

[no name] 14-Mar-13 9:35am

"il keep it to "admin", okay so if you do not have any interest in fixing your problem, then why are you even bothering posting this to begin with? Keeping it to "admin" will not in any way solve your problem.

Arora Varun 14-Mar-13 9:38am

phantom can u help me with an alternative to this situation
??

[no name] 14-Mar-13 9:48am

Help you with what? I already told you what you problem is and what you need to do to fix your problem but you just what to use "admin" instead of actually fixing your problem. So what is it that you would have me do?

Arora Varun 14-Mar-13 9:02am

wel i stil nedd 2 aply d module of validation.. il make it to "admin" but stil there aint any soln. techies...

[no name] 14-Mar-13 9:17am

Please lose the textspk. It's annoying, unprofessional, makes you sound like a child and does not make any sense anyway.

Mike Meinz 14-Mar-13 9:50am

My 5 for your comment - If I could.

[no name] 14-Mar-13 9:57am

Thanks

Arora Varun 14-Mar-13 9:04am

here i think as wat phantom told d problem is with new instance...
any other soln. to this guys??

Arora Varun 14-Mar-13 9:35am

ok well thanks phantom..
and the way project works is first the user logs inn.. then > it goes for the if condition "admin" part n sets m..
then the another form that's redirected is this homepageform
Now i might be using a wrong way for using functions ..
apologies for the inconvenience

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Mike Meinz · Answer 1 · 2013-03-14T03:04:00

You check for "Admin" but what if it was typed in as "admin", "ADMIN", "AdMiN" or any other combination?

According to the documentation for String.Equals: This method performs an ordinal (case-sensitive and culture-insensitive) comparison.

C#

public int m = 0;
string user = Usernametxt.Text.ToLower;  //Convert to all lower-case for comparison
if (user.Equals("admin"))
{
    m = 1;
}
else { m = 0; }

Also, don't you have to use ShowDialog to display the login form?

C#

Loginfrm objLoginfrm = new Loginfrm();
objLoginfrm.ShowDialog; // Show Login Form
//int z=objLoginfrm.m;
//int k=1;

if (objLoginfrm.m==1)
{
...

Arora Varun · Answer 2 · 2013-03-14T07:36:00

Hi guys thanks finally i got an solution to the above query ..
wat i did was..

if (m == 1)
{
    HomePagefrm objForm3 = new HomePagefrm();
    objForm3.Show();
    objForm3.Register_btn1.Visible = true; //Register_btn1.Enabled = true;
    objForm3.Report_btn.Enabled = true;
    this.Hide();
}
else
{
    HomePagefrm objForm3 = new HomePagefrm();
    objForm3.Show();
    objForm3.Register_btn1.Enabled = false;
    objForm3.Report_btn.Enabled = false;
    this.Hide();
}

did it on log in page itself..
Phantom thanks fr your guidance...

Code for differentiating "Admin" user from normal user not working

2 solutions

Solution 1

Solution 2

Add your solution here

Preview 0