Question

我正在建立一个排名系统，检查用户的XP是否满足某个级别，然后使用新的排名更新数据库。这是我目前的代码。

 <?php
 session_start();
$servername = "localhost";
$username = "USERNAME";
$password = "PASSWORD";
$dbname = "DB";

// Create connection
$conn = new mysqli($servername, $username, $password, $dbname);
// Check connection
if ($conn->connect_error) {
    die("Connection failed: " . $conn->connect_error);
} 
$id=$_SESSION['user']['id'];
$sl = "SELECT * FROM users WHERE id='$id'";
$reult = $conn->query($sl);

    while($ro = $reult->fetch_assoc()) {
if($ro['xp']>2000){
$sql = "UPDATE users SET rank='2' WHERE id='$id'";
if ($conn->query($sql) === TRUE) {
    echo"Success";
} 
} //ENDS THE IF
elseif($ro['xp']>"5000"){
$sel = "UPDATE users SET rank='3' WHERE id='$id'";

if ($conn->query($sel) === TRUE) {
    echo"Success";
}

}//ENDS ELSEIF


    }//WHILE LOOP ENDS
$conn->close();
    ?>

但是，如果满足第一个elseif's，PHP将不允许if继续运行。用PHP做更好的方法吗？

Answer 1

只需在if中添加其他条件：

if ($ro['xp'] > 2000 && $ro['xp'] <= "5000")

Answer 2

只需更改您使用的订单即可。首先检查XP是否> 5000，然后2000。

Answer 3

您实际上带回的数据超出了实际需要的数量（您只使用了所选的xp列）。所以你的陈述：

$sl = "SELECT * FROM users WHERE id='$id'";

实际上可以简化为以下内容：

$sl = "SELECT xp, rank FROM users WHERE id = '$id' AND xp > 2000";

（一般情况下，使用SELECT *不是最佳做法。）

AND子句就在那里，因为您不需要更新{2000}或更少的xp的用户 - 所以不需要检索它们！

为什么我会rank？好吧，我也建议你重组if：

while ($ro = $reult->fetch_assoc()) {
    $newrank = 2; // This is the default since any user retrieved will have xp > 2000
    if ($ro['xp'] > 5000) { // N.B. you have "5000" in quotes here - remove them!
        $newrank = 3;
    }
    if ($newrank != $ro['rank']) {
        // Update the user - no need to update if existing rank same as new!
        $sql = "UPDATE users SET rank='$newrank' WHERE id='$id'";
        if ($conn->query($sql) === TRUE) {
            echo"Success";
        } 
    }
} // end while

P.S。还可以选择一次更新所有用户，而不是逐个用户进行更新：

UPDATE users
   SET rank = CASE WHEN xp > 5000 THEN '3' ELSE '2' END
 WHERE xp > 2000;

如果您有一个单独的排名和XP值表，这可能是一个好主意：

RANKS
--------------------
rank  min_xp  max_xp
1     0       2000
2     2001    5000
3     5001    etc.

然后您可以执行以下操作：

UPDATE users u JOIN ranks r
    ON u.xp >= r.min_xp
   AND u.xp <= r.max_xp
   SET u.rank = r.rank;

如果满足阈值，则更新Sql

3 个答案: