Welcher ist besser? Fall wenn oder wenn sonst?

Question

Mein Fall ist wie folgt aus:

Fall, wenn so:

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; 
} 

wenn sonst ist wie folgt:

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; 
} 

ich versucht habe es. Die ganze Art der obigen Arbeit. Aber hier frage ich, welcher ist besser?

Answer 1

Das kann nicht stimmen, der erste Code sollte nicht funktionieren! In SQL CASE ist ein Ausdruck, keine Anweisung. Sie können keine Bedingung auf den THEN Teil setzen, nur einen einzelnen Wert. MySQL wertet Booleschen Ausdrücken als 1 \ 0, so dass Ihr Fall ist im Grunde wie folgt:

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

Die meisten RDBMS wird einen Fehler werfen.

Sie können es wie folgt formatiert:

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 = ?  

Solange man über diese Dinge nicht Performance-Probleme haben, denken Sie nicht, es wird nur Sie Zeit verschwenden. Wählen Sie dasjenige aus, das für Sie leichter zu pflegen ist.

Answer 2

Abgesehen von dem von @sagi aufgeworfenen Problem ist der zweite Ansatz aus einem etwas anderen Grund besser. Im ersten Beispiel verwenden Sie die Variable $action direkt in der Abfragezeichenfolge, die möglicherweise einen Angriffsvektor für eine SQL-Injektion öffnet, da sie außerhalb des Methodenbereichs liegt und möglicherweise nicht sicher ist. Während im zweiten Beispiel die Variable $condition in der Methode kontrolliert und sicher zu verwenden ist.

Abgesehen davon würde ich mit Lesbarkeit gehen, solange die Leistung kein Problem ist. Aus diesem Grund könnte ich dies sogar in 3 separate, klar benannte Funktionen aufteilen, die eine sehr ähnliche SQL haben. Auf diese Weise ist immer klar, was passiert und jeder Fall kann unabhängig von einer Methode "wachsen", wo immer mehr Fälle und Wenns hinzugefügt werden.

edit: Auch können Sie die Funktion eindeutig für das, was es tut, z.

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