Discussion :

[moimp]

Bonjour,
J'au une difficulté de compréhension du fonctionnement d'un bloc try/catch avec PDO.
J'ai le code suivant:

Code php :

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
function saveFormations($id, $string) {
	global $db;
 
	$query = "
		UPDATE dat_users
		SET formations=:string
		WHERE id=:id
	";
	try {
		$result = $db->prepare($query);
		$result->bindParam('id', $id);
		$result->bindParam('string', $string);
		$result->execute();
	}
	catch (Exception $e) {
		if (IS_LOCAL) die($e->getMessage);
		else return false;
	}
	return true;
}

Dans ce code, où faut-il placer le return true;
Faut-il le mettre à la fin comme ici ou dans le bloc try. Dans ce cas, n'y a t'il pas un risque de renvoyer la valeur true avant que l'exception ne soit lancée?

[Celira]

Il faut réviser le fonctionnement des exceptions

Le principe de base c'est :

1. le script execute tout ce qu'il peut dans le try.
2. Dès que quelque chose dans le try lance une exception, on arrête de traiter le contenu du try, et on commence à traiter le contenu du catch correspondant à l'exception
3. (Optionnel) Si il y a un finally, on traite tout ce qui se trouve dans le finally qu'il y ait une exception ou pas
4. On traite ce qui se trouve après le bloc try/catch(/finally)

Donc dans ton cas :

1. On exécute la requête.
2. Si elle plante, on traite le catch donc soit die, soit return false
3. Si elle ne plante pas, on sort du try/catch et on traite ce qu'il y a après, donc return true.

Ce qui m'a l'air de correspondre à ce que tu attends.

A noter que tu pourrais aussi l'écrire 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
function saveFormations($id, $string) {
    global $db;
 
    $query = "
        UPDATE dat_users
        SET formations=:string
        WHERE id=:id
    ";
    try {
        $result = $db->prepare($query);
        $result->bindParam('id', $id);
        $result->bindParam('string', $string);
        $result->execute();
        // si on arrive ici, c'est que l'exécution s'est bien passée
        return true;
    }
    catch (PDOException $e) {
        if (IS_LOCAL) die($e->getMessage);
        // Si pas local, on ne fait rien (c'est pas terrible, il faudrait faire un log du message d'erreur)
    }
    // si on arrive ici, c'est qu'il y a eu un problème dans le try et qu'on n'est pas arrivés jusqu'au return true
    return false;
}

[grunk]

Tout d'abord ,
faire un die dans un try /catch annule complètement l'intérêt du try/catch qui te permet en principe de gérer correctement un potentiel cas d'erreur. La si erreur tu coupe tout comme un cochon

Ceci étant dit que tu place ton return true à la fin du try ou à la fin de ta fonction n'a pas d'importance, le fonctionnement sera le même.
Je préfère le faire dans le try et dans le catch pour des raison de lisibilité , mais c'est pas obligatoire

Code :

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

1
2
3
4
5
6
7
8
9
10
try {
    $result = $db->prepare($query);
    $result->bindParam('id', $id);
    $result->bindParam('string', $string);
    $result->execute();
    return true;
}
catch (Exception $e) {
    return false;
}

Dans ce cas, n'y a t'il pas un risque de renvoyer la valeur true avant que l'exception ne soit lancée

Non , pas possible, dès qu'une des instructions lève une exception le code passe dans le catch.

[moimp]

Merci à vous deux!