Discussion :

[nico78200]

Bonjour à tous,

Je ne comprend pas j'ai une méthode, qui me sert pour les 'INSERT, UPDATE et DELETE' et qui fonctionne correctement jusqu’à maintenant:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
 
	public function dbExec($req,$condition) {
		try {
			$bdd = new PDO('mysql:host='.$this->dbhost.';dbname='.$this->db, $this->dbuser, $this->dbpass);
			$bdd->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
		} catch(Exception $log) {
			echo 'Erreur: '.$log->getMessage().'<br />';
			echo 'Code: '.$log->getCode();
		}
		if (empty($log)) {
			$requete = $bdd->prepare($req);
			if (!is_array($condition)) {
				$condition = array();
			}
			try {
				$do = $requete->execute($condition);
			} catch(Exception $log) {
				echo 'Erreur: '.$log->getMessage().'<br />';
				echo 'Code: '.$log->getCode();
			}		
			// Si on trouve une reponse			
			if ($do != false) {
				return true;
			} else {
				return false;
			}
		}
	}

Cette méthode fonctionne correctement pour faire des 'UPDATE' mais avec le code suivant le try/catch ne me renvoi aucune erreur et la méthode me renvoi TRUE alors que rien n'est modifié dans la BDD... C'est étrange! Voici le code en question:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
	// Mettre à jour une note
	if ($_POST['updateNote']) {
		if ( ($_POST['id'] != '') && ($_POST['noteTitre'] != '') && ($_POST['noteContent'] != '') && ($_POST['noteKeywords'] != '') ) {
			// Recuperer le timestamp
			$time = time();
			if (!$Core->dbExec('UPDATE `notes` SET `titre`=?, `note`=?, `keywords`=?, `ts_lastmodif`=? WHERE `id_creator`=? AND `id`=?', array($_POST['noteTitle'], $_POST['noteContent'], $_POST['noteMetaKeywords'], $_SESSION['userid'], $time, $_POST['id'])) ) {
				$txt = '<p>Erreur lors de l\'enregistrement des modifications...</p>';
				$Core->newErrorMessage($txt);
			} else {
				$txt = '<p>Les modifications sont correctement enregistrés!</p>';
				$Core->newMessage($txt);
			}					
		} else {
			$txt = '<p>Erreur: Impossible de d\'enregistrer les modifications...</p>';
			$Core->newErrorMessage($txt);				
		}	
	}

Je ne comprend pas pourtant quand je teste coté serveurs les données que j'enregistre sont bien différentes de celles présentes dans ma DB (ne serai-ce que le timestamp) et pourtant rien ne change dans ma DB....

J'en perd mon latin et je suis fatigué alors ça n'arrange rien, si quelqu'un peut me filer un coup de main, il est le bienvenu!

Merci

Nico

[vorace]

dans ta liste de paramètres, tu as inversé $time et $_SESSION['userid'],
mes $time avant $_SESSION['userid']...

[nico78200]

dans ta liste de paramètres, tu as inversé $time et $_SESSION['userid'],
mes $time avant $_SESSION['userid']...

Un grand merci Vorace, erreur stupide de fatigue... Juste pour la forme pourquoi je n'ai aucun message d'erreur alors que pourtant j’exécutais un UPDATE sur une entrée inexistante dans ma DB?

[rikemSen]

Parce que ne requête qui ne retourne pas de résultat n'est pas forcément une requête fausse. Alors il n'affiche pas d'erreur. (:

[nico78200]

Parce que ne requête qui ne retourne pas de résultat n'est pas forcément une requête fausse. Alors il n'affiche pas d'erreur. (:

Oui attention ce commentaire est un résidu d'une autre méthode dbQuery() pour faire des SELECT dans la DB, mais dans le cas présent la requête PDO execute() est censé renvoyer un bool point barre.... Je l'ai ré-éecrite avec des opérateurs de type '===' .

Bonne journée

[RunCodePhp]

Salut

mais dans le cas présent la requête PDO execute() est censé renvoyer un bool point barre

Tout a fait.

Il me semble qu'il y aurait plus simple et plus fiable en procédant ainsi :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
public function dbExec($req,$condition) {
	try {
		$bdd = new PDO('mysql:host='.$this->dbhost.';dbname='.$this->db, $this->dbuser, $this->dbpass);
		$bdd->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
                //
		$requete = $bdd->prepare($req);
		if (!is_array($condition)) {
			$condition = array();
		}
		try {
			return $requete->execute($condition);
		}
		catch(Exception $e) {
			echo 'Erreur: '.$e->getMessage().'<br />';
			echo 'Code: '.$e->getCode();
		}
	}
	catch(Exception $e) {
		echo 'Erreur: '.$e->getMessage().'<br />';
		echo 'Code: '.$e->getCode();
	}
	return FALSE;
}

Entre autre, il n'est pas utile de continuer le script si le 1er try/catch de connexion échou, du coup une vérif de $log devient par la même occasion inutile.
La suite du code, de l'exécution de la requête serait bien mieux dans ce 1er try/catch à mon avis.


Aussi, une vérif est faite sur $do après le 2ème bloc try/catch.
Là aussi, s'il y a erreur il vaut mieux ne pas poursuivre le script.

S'il y a une erreur de syntaxe sur la requête par exemple, une exception sera levée et seul le code dans le catch sera exécuté.
Du coup, $do n'existera pas à la suite du code, et ceci devrait provoquer une toute autre erreur, vois tu ?


Donc attention au déroulement du code, c'est un peu dommage de mettre en place une gestion des exceptions en recréant d'autres erreurs en cascades, même minimes.

[stealth35]

un seul try...catch suffis :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
public function dbExec($req, $condition)
{
    try 
    {
        $bdd = new PDO('mysql:host='.$this->dbhost.';dbname='.$this->db, $this->dbuser, $this->dbpass);
        $bdd->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
        $requete = $bdd->prepare($req);
 
        if(!is_array($condition))
        {
            $condition = array();
        }
 
        return $requete->execute($condition);
    }
    catch(PDOException $e)
    {
        echo 'Erreur: ', $e->getMessage(), '<br />';
        echo 'Code: ', $e->getCode();
    }
 
    return false;
}

[RunCodePhp]

un seul try...catch suffis :

Oui, parce qu'en cas d'erreur le message est exactement le même, donc pourquoi pas.


Mais en conservant les 2 try/catch, les 2 niveaux, on peu améliorer le débogage en y rajoutant la requête SQL par exemple dans le 2ème catch.
On ciblera mieux la nature de l'erreur à mon sens.


Il me semble qu'il est là l'avantage des try/catch, c'est d'attraper les erreurs le plus tôt possible afin de la gérer autrement ou/et alors informer le codeur/développeur de la vrai nature de l'erreur, et aussi où elle a eu lieu.
Ces messages d'erreurs risquent d'être peu explicites d'ailleurs, un $e->getMessage() c'est maigre bien souvent, m'enfin.


D'ailleurs, là j'ai un doute, mais il faudrait voir lorsqu'une exception est levée si le message est enregistré dans les logs (php_errors.log).
A mon avis ce n'est pas le cas, et ce serait un manque (voir gros manque).

[stealth35]

D'ailleurs, là j'ai un doute, mais il faudrait voir lorsqu'une exception est levée si le message est enregistré dans les logs (php_errors.log).
A mon avis ce n'est pas le cas, et ce serait un manque (voir gros manque).

non c'est a toi de faire ça.

[nico78200]

Merci beaucoup à tous pour votre aide, oui en effet j'ai bien compris le pourquoi du comment grâce à vos explications.

Pour le débogage, je vais ajouter une méthode dans mon application pour les logger proprement dans un fichier.

Bonne continuation