函数中的php转换

我有以下代码：

if (isset($_SESSION['fb_token'])){
    if ($queryusers['user_roll'] != '3'){
        header('location:' . $UrlBase."login");
        exit();
    }
}

---

我想把它变成一个更短的函数，在我的项目中，我将经常使用这段代码，我认为有一种更简单的方法来调用代码，你能帮我吗？谢谢大家!

我认为代码是不言自明的：

function shortCheck ($qu, $ub) {
    if (isset($_SESSION['fb_token']) && $qu != '3') {
        header('location: ' . $ub."login");
        exit();
    }
}

然后你这样称呼它：

if (shortCheck($queryusers['user_roll'], $UrlBase)) {
    echo 'Hello admin!';
} else {
    echo 'Not admin!';
}

编辑： 关于命名变量，您是对的，但这只是一个示例，而不是生产代码。例如，我给它们命名的方式是，注释是不必要的，非常长而且是描述性的，所以提出问题的人应该以他/她认为最好的方式重命名它

---

也许我不明白这个问题，但总结起来，对我来说，这听起来像是我的检查，我想重复使用多次。这就是函数被发明的原因，有人建议为任务创建一个简单的类，就像这一个，所以我动态地写了这个

写一节课file@ShreyasAchar这正是我想要的，但是我如何将这些代码转换成一个类，用几句话来检查用户是否是管理员呢。。roleif isset$_会话['fb_令牌]]和&$queryusers['user_roll']！='3'{是shorter@kerbholz-如果您可以使用该语法，可能会更容易理解，但不幸的是，这不起作用。例如，我想说，如果shortCheck是真的hello admin，否则不是admin，我如何使用您的代码？我知道有一个问题要求使用较短的版本，您实际上无法做多少事情来缩短它，但使用abbrevia在我看来，像$qu和$ub这样的选项并不是解决方案。如果你在几年后回到这段代码，你还记得$qu和$ub代表什么吗？我不记得。这意味着需要额外的工作来查找它。缩写变量名并不会使你的代码更快。所以我认为$qu应该是$userRole，$ub应该是$urlBase…还有你忘了把它缩写，在function@KIKOSoftware我想和你一样！我希望有更好的答案。我不认为有“更好”的答案是可能的。你想检查两件事，做两件事，这意味着你需要一些代码。如果Bozidar纠正了代码中的错误，那就太好了。哦，还有一个错误：$qu！=“3”应该是$qu！=“3”。我无法编写更好的代码，因为您的代码太不完整了。没有什么可处理的。