Discussion :

[a028762]

Bonjour, je souhaite ajouter à mes classes persistantes (Mysql)
une gestion d'erreur un peu construite
sans la gérer à l'extérieur de ces classes ...
Apparamment , la déclaration d'un handler hors d'une méthode ne passe pas ... Cela signifie-t-il que je suis condamné à faire cette déclaration à chaque fonction ?
Merci de votre écoute

[lenono]

Bonjour,

Oui, à chaque fonction.
Exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
$this->result = @mysql_query($query);
if (!$this->result) {
  $this->show_error(mysql_error(),$query,$display_error);
  @mysql_free_result($this->result);
  return false;			
} else{
  ... tu envoies ton résultat
}

et

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
private function show_error($error,$query='',$display_error) {
  if($display_error){
    echo "<p><strong>Error:</strong> " . $error . " : -->" . $query . "</p>";
  }
}

Ce n'est qu'un exemple, bien sûr !

Bon courage !

[stealth35]

Bonjour,

Oui, à chaque fonction.
Exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
$this->result = @mysql_query($query);
if (!$this->result) {
  $this->show_error(mysql_error(),$query,$display_error);
  @mysql_free_result($this->result);
  return false;			
} else{
  ... tu envoies ton résultat
}

et

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
private function show_error($error,$query='',$display_error) {
  if($display_error){
    echo "<p><strong>Error:</strong> " . $error . " : -->" . $query . "</p>";
  }
}

Ce n'est qu'un exemple, bien sûr !

Bon courage !

beurk c'est quoi c'est @ partout ? ..., de plus ça sert a rien de faire un else après un return, et pourquoi ne pas utilise les exceptions ?

[lenono]

C'est un exemple stealth35 pour bien comprendre.

Ceci dit ma pédagogie ne reflète en rien ma méthode de travail, ni le "best practice" et me disant que ce forum est lu par des débutants (comme nous l'avons tous été au moins un jour), cela permet une approche progressive de la gestion des erreurs et donc des exceptions.

[a028762]

Bon, c'est ce que je fais déjà , rarement d'ailleurs avec les @
L'idée est de construire un handleur d'erreur qui dans certains cas enverrait un mail à l'exploitant du site, ou de stocker ces erreurs (celles qui ne sont pas déjà dans les LOG APache) dans un fichier ...
Une règle que j'applique est de ne pas masquer des messages d'erreur si on n'a rien à offrir de mieux à l'utilisateur ....
rien n'est plus troublant de ne rien afficher ...