2

我试图减少代码中选择语句、更新等的数量,所以我想我会尝试为此创建一个函数,但是我很困惑我的努力在第一个障碍中失败了。

我已经非常全面地搜索了这个问题(我觉得),并且在我找到的示例中找不到任何差异。

这是我的功能,带有打印语句,可帮助我诊断问题。

function select_statement($action, $table, $where){
print $action.' - ';

switch ($action){
    case 'select':
        print 'select used - ';
        $thequery = 'SELECT * FROM '. $table . ' WHERE '. $where;
    case 'insert':
        print 'insert used - ';
        $thequery = 'INSERT INTO '. $table;
    case 'update':
        print 'update used - ';
        $thequery = 'UPDATE ' . $table . ' SET ';
    }
print $thequery;
mysql_query($thequery);

}

这是调用该函数的行:-

$logins = select_statement('select', 'users', 'user_id=1');//calls function

这是结果:-

select - select used - insert used - update used - UPDATE users SET 

如您所见,代码正在触发每个打印语句,并且似乎忽略了“case”语句。

我真的不确定我在这里做错了什么?

4

1 回答 1

5

你忘了使用break. 没有它,每个 case 语句都将“落空”到下一个并继续运行。在每条语句break的末尾停止执行`;case

switch ($action){
    case 'select':
        print 'select used - ';
        $thequery = 'SELECT * FROM '. $table . ' WHERE '. $where;
        break;
    case 'insert':
        print 'insert used - ';
        $thequery = 'INSERT INTO '. $table;
        break;
    case 'update':
        print 'update used - ';
        $thequery = 'UPDATE ' . $table . ' SET ';
        break;
    }
于 2012-05-14T13:28:07.310 回答