Why does this variable always return true?
I am trying to write a login script. After logging in a session variable is set, and on the main page, the isLoggedIn function is run. The problem is, that the variable $loggedIn is always returning true. Can someone help?
function validateUser($user)
{
if(isset($user)){
session_regenerate_id ();//this is a security measure
$_SESSION['logged'] = 1;
$_SESSION['userid'] = $user;开发者_JS百科}
}
//Validates Login
function isLoggedIn()
{
if($_SESSION['logged'] = 1)
return true;
return false;
}
$loggedIn = isLoggedIn();
if($loggedIn){ SHOW CONTENT FOR LOGGED IN USERS }
else { show content for users not logged in }
You are using the assignment operator =
instead of a conditional operator ==
.
It's supposed to be:
if ($_SESSION['logged'] == 1)
return true;
The result of an assignment =
is the right hand side expression, which is 1, which is always true in this case. :)
it's because you're setting $_SESSION['logged'] with the single equals. Use a double equals instead:
function isLoggedIn()
{
if($_SESSION['logged'] == 1)
return true;
return false;
}
function isLoggedIn() { if($_SESSION['logged'] = 1)//assigned value '1' to $_SESSION['logged'] so in this if condition it is checking if there is any value is $_SESSION['logged'], which will be true for all the cases (valid/invalid) return true; return false; }
$loggedIn = isLoggedIn();/those the value for $loggedIn will be always true
change the if($_SESSION['logged'] = 1) to if($_SESSION['logged'] == 1)
NOT THIS if($_SESSION['logged'] = 1) THIS is correct -> if($_SESSION['logged'] == 1)
精彩评论