目前,我有以下代码：
if(isset($_GET['mid']) && !empty($_GET['mid'])) {
$mid = $_GET['mid'];
if(is_numeric($mid) && $mid > 0) {
if(isset($_GET['op']) && !empty($_GET['op'])) {
$op = $_GET['op'];
if($op == 'info') {
}
if($op == 'cast') {
}
}
}
}
但我认为if语句中的if语句太“复杂”了……
你会以不同的方式处理吗？你会如何让它变得更简单？
[编辑]一般承认的答案：
好吧,我已经学到了一些我不知道的小细节和新的PHP函数.我认为没有正确的方法可以按照我的要求行事.我显然是以错误的方式使用一些PHP函数,我会解决这个问题.
在我看来,像这样的输入应该使用PHP过滤器函数进行验证/清理,因此我将Arkh的答案标记为已接受.
但是,对于特定的大学项目(PHP代码完全不相关),我将使用他的答案与Tatu的帮助函数构思混合使用.但对于一个不同的项目,我会将他的回答与Ignacio的课堂理念混合使用,因为它看起来更好,更有条理.
我会在中间使用带有FILTER_SANITIZE_NUMBER_FLOAT过滤器的
filter_input.像这样的东西：
$mid = filter_input(INPUT_GET,'mid',FILTER_SANITIZE_NUMBER_FLOAT);
$op = filter_input(INPUT_GET,'op');
if($mid > 0){
switch($op){
case 'info':
// Some code
break;
case 'cast':
// Some more code
break;
default:
break;
}
}
