Question

我想知道下面是否有更好的方法来编写我的代码。基本上有两个问题，我应该使用elseif吗？我是否需要在某处关闭与数据库的连接？

谢谢

更新代码

if ($result && mysql_num_rows($result) === 1) {


    $member = mysql_fetch_assoc($result);

    if ($member['disabled']) {
        header("location: not-allowed.php");
        exit();
        }

    if (!$member['verified']) {
        header("location: please-verify.php");
        exit();
        }
}

Answer 1

if ($result && mysql_num_rows($result) === 1)
{
    ...
}
else die('Wrong username and/or password');

Answer 2

不要重复自己

if (!($result && mysql_num_rows($result) == 1)) redirect("login-failed.php");
$member = mysql_fetch_assoc($result);
if ($member['disabled']) redirect("not-allowed.php");
if (!$member['verified']) redirect("please-verify.php");
if ( ($member['expires']) && ($member['expires'] <= time()) ) redirect("expired.php");
if ( ($_SESSION['SESS_TOKEN'] == $_POST['token']) ) redirect("member-index.php");

function redirect($location){
    header("location: $location");
    exit();
}

不，如果您使用的是exit或return，则无需使用其他内容。

不，不必手动关闭mysql连接。

我应该使用elseif并在某处关闭数据库连接吗？

2 个答案: