Essayez Catch Block dans l'emballage PDO Classe?

Question

J'ai écrit un wrapper PDO soigné en classe. J'essaie de comprendre où est le meilleur endroit pour mettre Try/Catch bloquer? Dans le wrapper de classe PDO ou d'autres fichiers?

Par exemple:

public function fetchData($sql, $params = array(), $fetchMode = 'all') { 
    try { 
     $this->sth = $this->db->prepare($sql); 
     if (count($params) > 0) 
      $this->setBind($params); 

     $this->sth->execute(); 
     $fetch = ($fetchMode == 'single') ? "fetch" : "fetchAll"; 
     return $this->sth->$fetch(); 
    } catch (PDOException $e) { 
     return $data['error'] = array(
         'status' => 'error', 
         'message' => $e->getMessage() 
        ); 
    } 
} 

Ou dois-je faire comme ça, sans try/catch bloc:

public function fetchData($sql, $params = array(), $fetchMode = 'all') { 
     $this->sth = $this->db->prepare($sql); 
     if (count($params) > 0) 
      $this->setBind($params); 

     $this->sth->execute(); 
     $fetch = ($fetchMode == 'single') ? "fetch" : "fetchAll"; 
     return $this->sth->$fetch(); 
} 

Exemple de requête:

try { 
    $SQL = "SELECT * FROM table"; 
    $result = $db->fetchData($SQL); 
    print_r($result); 
} catch (PDOException $e) { 
    $data['error'] = array(
     'status' => 'error', 
     'message' => $e->getMessage() 
    ); 
} 

Mise à jour: Ce qui serait plus efficace? Par exemple, je ne veux pas répéter try/catch block dans toutes les fonctions ou les fichiers .... et je suis toujours capable de retourner différents "fail-message" sur chacun d'entre eux.

Answer 1

Cela dépend ..

En tant qu'utilisateur de votre classe, j'attendre lors de l'appel fetchData une valeur de retour avec un tableau de données ou faux ou un tableau vide quand il n'y a pas de données ou une erreur. Je ne m'attendrais pas à un tableau avec des informations d'erreur.

• Lorsque vous utilisez un try/catch dans la classe que vous vous devez saisir les informations d'erreur et rendre disponible comme une propriété/méthode pour vérifier, et laisser fetchData return false, indiquant qu'il ya une erreur. Avoir aucun essai/catch dans votre classe et laisser à l'utilisateur de votre classe peut être une bonne stratégie dans certaines situations. Si l'utilisateur de votre classe a une belle configuration de gestionnaire de excepention global qui avorte gracieusement, par exemple

Mise à jour

Votre code, avec bloc try/catch

class myPDO 
{ 
    private $error; 

    public function fetchData($sql, $params = array(), $fetchMode = 'all') { 
    try { 
     $this->sth = $this->db->prepare($sql); 
     if (count($params) > 0) 
      $this->setBind($params); 

     $this->sth->execute(); 
     $fetch = ($fetchMode == 'single') ? "fetch" : "fetchAll"; 
     return $this->sth->$fetch(); 
    } catch (PDOException $e) { 
     $this->error = array(
         'status' => 'error', 
         'message' => $e->getMessage() 
        ); 

     return false; 
    } 
    } 

    public function error() 
    { 
    return $this->error; 
    } 

} 

Votre code pour non essayer/attraper le bloc reste le même.