哪一个更好？情况何时或如果其他？

Question

我的情况是这样的：

情况是这样的：

public function get_opportunity($customer_id, $group_id, $action)
{
    $sql = "SELECT COUNT(transaction_id) AS total_transaction 
            FROM `transaction` 
            WHERE 
                CASE 
                    WHEN '$action' = 'view_all' THEN transaction_id IS NOT NULL
                    WHEN '$action' = 'view_group' THEN group_id IN($group_id) 
                    ELSE transaction_created_by = ? 
                END 
            ";

    $result = $this->db->query($sql, array($customer_id))->result_array();

    return ($result[0]['total_transaction']) ? $result[0]['total_transaction'] : 0;
}

如果是这样的话：

public function get_opportunity($customer_id, $group_id, $action)
{
    if($action == "view_all")
        $condition = " transaction_id IS NOT NULL";
    else if($action == 'view_group')
        $condition = " group_id IN($group_id)";
    else
        $condition = " customer_id = ?";

    $sql = "SELECT COUNT(transaction_id) AS total_transaction 
            FROM `transaction` 
            WHERE $condition";

    $result = $this->db->query($sql, array($customer_id))->result_array();

    return ($result[0]['total_transaction']) ? $result[0]['total_transaction'] : 0;
}

我试过了。上述工作的所有类型。但在这里我问，哪个更好？

Answer 1

除了@sagi 提出的问题，实际上第二种方法更好，原因有些不相关。在第一个示例中，您直接在查询字符串内使用变量$action，这可能会打开 SQL 注入的攻击向量，因为它来自方法范围之外并且可能不安全。而在第二个示例中，变量$condition 在方法中受到控制并且可以安全使用。

除此之外，只要性能不是问题，我会考虑可读性。出于这个原因，我什至可以将其拆分为 3 个独立的、明确命名的函数，而不是它们具有非常相似的 SQL。这样一来，发生的事情总是很清楚，并且每个案例都可以独立“增长”，而不是使用一种方法来添加越来越多的案例和 if。

edit：您也可以清楚地命名该函数的功能，例如

• getAllTransactionCounts()
• getTransactionCountForCustomers($transactionIds)
• getTransactionCountForCustomer($customerId)

Answer 2

这不对，第一个代码应该不起作用！在 SQL 中，CASE 是一个表达式，而不是一个语句。您不能对 THEN 部分设置条件，只能设置一个值。 MySQL 将布尔表达式计算为 1\0 ，所以你的情况基本上是这样的：

WHERE CASE... THEN (Cond) -- equals to 1/0 
              THEN (Cond) -- equals to 1/0
              ELSE (Cond) -- -- equals to 1/0 

大多数 RDBMS 都会抛出错误。

你可以这样格式化：

SELECT COUNT(transaction_id) AS total_transaction 
            FROM `transaction` 
            WHERE ('$action' = 'view_all' AND transaction_id IS NOT NULL) OR 
                  ('$action' = 'view_group' THEN group_id IN($group_id)) OR
                  transaction_created_by = ?       

只要你没有性能问题，不要考虑这些事情， 它只会浪费你的时间。选择一个更容易维护的。