PHP if else 混乱

Question

我正在尝试根据 3 个条件使图像出现和消失，

条件 A = 当用户登录并且其用户名适合显示名称（通过使用 GET 函数）时，它应该回显“是”

条件 B = 当用户登录时已登录，并且它的用户名不适合显示名称，那么它应该回显“否”

条件 C = 当用户未登录时，它也应该回显“否”

我用“是”和“否”交换了图像，以便于参考）

（ ，用户有如下设置的 cookie

  setcookie("user", $user, $expire);
  setcookie("loggedin", 1, $expire);

首先，我获取用户登录时设置的 cookie。

  $user1 = $_COOKIE["user"];
  $loggedin = $_COOKIE['loggedin'];
  $user = strtoupper($user1);

然后我得到了我的球员的名字

  $playername = $_GET['player'];

现在我执行了条件

$uplayername = strtoupper($playername);

function showplusicon(){

    global $uplayername;

    if(($loggedin = "1") and ($user == $uplayername)){
        echo "yes";
    }
    else if (($loggedin = "1") and ($user != $uplayername)){
        echo "no";
    }
    else{
        echo "no";
    }
}

我不知道有什么问题，但它一直被注册为条件 B。

Answer 1

单个等号赋值，而不是比较。

if(($loggedin == "1") and ($user == $uplayername)){
   ...

由于你实际上只有两个输出状态，所以你不应该需要 3 个条件；删除条件B。

Answer 2

变量 $loggedin 在您的函数 showplusicon() 中未知。您需要将其与 global $uplayername 一起添加为 global。

function showplusicon(){

   global $loggedin, $uplayername;

   // etc
}

---

由于这已被接受但并不完全完整，因此我将补充一点，正如其他人指出的那样，需要使用 == 相等运算符而不是 = 赋值运算符。

if(($loggedin == "1")
             ^^^^

Answer 3

$loggedin = "1"

当然这应该是：

$loggedin == "1"

否则我会 echo $user 和 $uplayername 看看它们是否不同。

Answer 4

首先第一件事是：
$loggedin = "1" 是一个坏主意，因为您实际上是给 $loggedin 值“1”而不是进行比较。如果您确定数据类型，请使用 == 甚至 ===。

此外，$loggedin 在 showplusicon() 范围内不可用，因为您没有像使用 $ 那样将其声明为全局变量上层玩家名称。

修复上面列出的问题，它应该会工作得更好一些。

Answer 5

如果您在理解自己的代码逻辑时遇到问题，一个简单的方法是将条件分配给自述变量以适应它：

$userIsLoggedIn = $loggedin == "1";
$userIsPlayer = $user == $uplayername;

这些变量可以让您轻松地从一开始就调试代码，

var_dump($userIsLoggedIn, $userIsPlayer);

从而找到实际的错误：

1. 变量 $loggedin 未定义
2. if 子句正在设置一个值 (=)，而不是比较它（== 或 = ==）。

然后，您可以另外使用更具可读性的代码流来使您的决定更加明显：

if ($userIsLoggedIn) 
{ // user is logged in
    if ($userIsPlayer)
    { // user is player
       ...
    }
    else
    { // user is not player
       ...
    }
} 
else 
{ // user is not logged in
    ...
}

根据您想要输出的内容，甚至可以简化：

if ($userIsLoggedIn && $userIsPlayer)
{
    echo 'yes';
} else
{
    echo 'no';
}

希望这对您有帮助。

Answer 6

您的主要问题是变量的全局范围：

<?php 
//Get cookie info
$cookie['user'] = $_COOKIE["user"];
$cookie['loggedin'] = (isset($_COOKIE['loggedin'])&&$_COOKIE['loggedin']=='1')?TRUE:FALSE;

//Set user array
$user['user'] = strtoupper($cookie['user']);
$user['loggedin'] = $cookie['loggedin'];
$user['player'] = $_GET['player'];
$user['uplayername']=strtoupper($user['player']);


function showplusicon(){
    //Made $user array available within function
    global $user;

    if($user['loggedin'] === TRUE && $user['user'] == $user['uplayername']){
        echo "yes";
    }else{
        echo "no";
    }
}
?>