目前,我有以下代码:
- 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;
- }
- }