这个 Mysql Helper 类有问题吗？

Question

我的数据库类中有这个函数。 它接受 3 个参数：

• 执行查询的
• 服务器由存储在配置中的名称标识
• sql 查询示例调用中引用的参数数组

：

$toplist = MyDbClass->q('SELECT * FROM movies WHERE score > ?','slaveserver1',array(100));

这里是代码...

/*
 * @param the sql query. may be pure sql or having ? as placeholders for variables that are passed in the 3rd param, not enquoted
 * @param name of the link (slave or master server or other arbitrary database)
 * @param optional array of vars that will be filled in where the ? signs in the query are
 */
public function q($sql,$name,$vars=false) {
    // lets see if the link to the server with name $name has already been initialised, if not lets do it
    if(!isset($this->links[$name])) {
        $this->initialize($name);
    }
    // if variables have been passed, lets fill them into the query
    if($vars !== false) {
        // first real scape them all according to the correct link
        for($i=0;$i<count($vars);$i++) {
            $vars[$i] = mysql_real_escape_string($vars[$i],$this->links[$name]);
        }
        // now escape all actual % signs so they are not used as placeholders vor vsprintf
        $sql = str_replace('%','%%', $sql);
        // no add '' quotes arround every placeholder and fill in
        $sql = str_replace('?','\'%\'', $sql);
        $sql = vsprintf($sql,$args);
    }
    // now execute the parsed query on the correct server
    return mysql_query($sql,$this->links[$name]) or die(mysql_error($this->links[$name]));
}

现在我的问题是：

我的代码有问题吗？特别是：

• 是否有任何情况下在查询中的参数周围放置 '' 引号会导致查询不起作用？
• 有没有一些优雅的方法来防止我的函数以双引号结束，例如 where Score > > ''100 '' 在我的查询中（如果我已经在输入查询中放入了 qutoes...）。
• 你觉得这个功能怎么样？这样做的好方法吗？

Answer 1

我不认为允许一个类操作多个 SQL 连接是一个好主意。应允许此类的每个实例仅使用一个 SQL 连接。或者，如果您尝试实现某种手动负载平衡，则应该从用户透明地使用它，可能只允许“从属”和“主”选择。

也许，您应该将查询分为“读”和“写”查询，因为您可以从任何从属设备“读取”，但应该“写入”所有从属设备。

此外，如果查询调度是出于负载平衡的目的，则需要在类构造函数中打开连接，而不是在查询中延迟打开连接。一个脚本是一种非常短暂的生物。除非您为长期操作编写 CLI 脚本，否则无需延迟连接直到查询真正执行。