multiple values for variable in IF

Anon

hi,

I have a simple login system for my pages and i would like to be able to set a password to each username.

so in english i would like to do this:

if (username1=password1 or username2=password2 or username3=password3)

how would it be done?

i have tried this code just for one password but with multiple usernames but with no success and many other similar pieces of code:

if ($passwd==$pword && ($usrname=="Rigo" || $usrname=="Rigo2" || $usrname=="Rigo3"))

Anon

if ((($username == "user1") && ($passwd == "pass1")) || (($username == "user2") && $passwd == "pass2")) || (($username == "user3") && ($passwd == "pass3"))) {
// Login OK
} else {
// Nope
}

Hope that helps

Anon

hi,

thanks for your reply.

I altered your code a little firstly to match my username variable and secondly so i could specifiy variables for each username and password at top of script.

here is what i replaced the current IF with:

if ((($usrname == "$u1") && ($passwd == "$p1")) || (($usrname == "$u2") && $passwd == "$p2")) || (($usrname == "$u3") && ($passwd == "$p3")))

and an example of the variables:

$u1 = "Name1";
$p1 = "Pass1";

and i got a parse error on the IF statement's line. I also tried it without the "" around the variable in the IF.

Any suggestions on how to fix it?

Anon

right before this statement:

$passwd == "$p2"

you forgot an open paren ( .. just count the parens and you should find this to be the problem.. hope that helps 🙂

Chris

Anon

hi,

it works!!!!!

thanks very much.

you did post it originally without that bracket but i guess i should have checked.

thanks again, very helpful 😉

Anon

Dare I ask why you are hard-coding all those names and passwords in the first place?

Anon

it's used for a simple login script that only three users have access to so it doesn't need to be dynamic. The variables are now in a separate file and so i only have to update one file to change the usernames and passwords.

if there's a better way than that, by all means please tell me!

kparker

Well, I'm certainly willing to be pragmatic when appropriate.
If there really are only three users, and they (and their passwords)
really will never change, and if the code's already done, then
by all means don't mess with it.

Otherwise, you could go anywhere from a simple text file containing
username:password lines (a la the unix '/etc/passwd' file), up to using
a database which is by far the best system if you're already using a
database for the project.