Discussion :

[moimp]

Bonjour,
En remplaçant le paramètre envoyé à la fonction ci-dessous par sa valeur 'FR', phpMyAdmin renvoie une valeur 'EUR'.
Par contre, lorsque j'utilise la fonction avec le même paramètre 'FR' la ligne renvoie true au lieu de la valeur attendue ('EUR'). J'ai beau chercher, je ne comprends pas pourquoi.

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 getDefaultCurrency($countryId) {
	$db = dbConnect();
	var_dump($countryId);	// 'FR'
	var_dump($db);		// 'object(PDO)[1]'
 
	$sql = "
		SELECT currency_code
		FROM lst_countries
		WHERE id=:countryId
		LIMIT 1
	";
	$stmt = $db->prepare($sql);
	$stmt->bindParam(':countryId', $countryId);
	$stmt->execute();
	var_dump($stmt->fetch()['currency_code'] or die($stmt->errorInfo()));	// true
	echo $stmt->fetch()['currency_code'];
	return $stmt->fetch()['currency_code'];
}
 
 
		$defaultCountry = getDefaultCountry(); // FR
		$defaultCurrency= getDefaultCurrency($defaultCountry);
		var_dump($defaultCurrency);exit;	// null

[Séb.]

Code :

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

1
2
3
var_dump($stmt->fetch()['currency_code'] or die($stmt->errorInfo()));	// true
echo $stmt->fetch()['currency_code'];
return $stmt->fetch()['currency_code'];

Tu appelles 3 fois fetch(), tu te déplaces dans ton jeu de résultats, résultat dont le nombre de lignes récupérées est LIMIT 1, donc pour moi là il y a un truc incohérent

Code :

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

var_dump($stmt->fetch()['currency_code'] or die($stmt->errorInfo()));	// true

Pas clair. Tu vas avoir le résultat du or, pas une valeur.

Je me permets de réécrire ta fonction :

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
21
22
23
24
25
26
27
28
29
30
31
function getDefaultCurrency($countryId)
{
    $db = dbConnect();
    var_dump($countryId);	// 'FR'
    var_dump($db);		// 'object(PDO)[1]'
 
    // Il est possible que cette syntaxe <<< ne soit pas supportée si vieux PHP
    $sql = <<<SQL
        SELECT ALL currency_code
        FROM lst_countries
        WHERE id = :countryId
        SQL;
    // LIMIT 1 retiré car je suppose que ta table lst_countries ne donne qu'une 1 ligne par ID
 
    $stmt = $db->prepare($sql);
    if ($stmt === false) {
        exit('Anomalie -- Erreur lors de la préparation de la requête : ' . print_r($db->errorInfo(), true));
    }
 
    $result = $stmt->execute([':countryId' => $countryId]); // bindParam remplacé car pas utile ici
    if ($result === false) {
        exit('Anomalie -- Erreur lors de l\'exécution SQL : ' . print_r($stmt->errorInfo(), true));
    }
 
    $row = $stmt->fetch(); // Je récupère mon unique ligne
    if ($row === false) {
        exit('Anomalie -- Mon unique ligne vaut false');
    }
 
    return $row['currency_code'];
}

[moimp]

Tout d'abord merci.
Ensuite, voici quelques remarques ou questions par rapport à ta réponse:

Tu appelles 3 fois fetch(), tu te déplaces dans ton jeu de résultats, résultat dont le nombre de lignes récupérées est LIMIT 1, donc pour moi là il y a un truc incohérent

Effectivement, je n'avais pas vu, c'est mon erreur principale.

Code :

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

var_dump($stmt->fetch()['currency_code'] or die($stmt->errorInfo()));	// true

Pas clair. Tu vas avoir le résultat du or, pas une valeur.

Deuxième erreur!

Je me permets de réécrire ta fonction :

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
21
22
23
24
25
26
27
28
29
30
31
function getDefaultCurrency($countryId)
{
    $db = dbConnect();
    var_dump($countryId);	// 'FR'
    var_dump($db);		// 'object(PDO)[1]'
 
    // Il est possible que cette syntaxe <<< ne soit pas supportée si vieux PHP
    $sql = <<<SQL
        SELECT ALL currency_code
        FROM lst_countries
        WHERE id = :countryId
        SQL;
    // LIMIT 1 retiré car je suppose que ta table lst_countries ne donne qu'une 1 ligne par ID
 
    $stmt = $db->prepare($sql);
    if ($stmt === false) {
        exit('Anomalie -- Erreur lors de la préparation de la requête : ' . print_r($db->errorInfo(), true));
    }
 
    $result = $stmt->execute([':countryId' => $countryId]); // bindParam remplacé car pas utile ici
    if ($result === false) {
        exit('Anomalie -- Erreur lors de l\'exécution SQL : ' . print_r($stmt->errorInfo(), true));
    }
 
    $row = $stmt->fetch(); // Je récupère mon unique ligne
    if ($row === false) {
        exit('Anomalie -- Mon unique ligne vaut false');
    }
 
    return $row['currency_code'];
}

Ligne 7, j'utilise PHP 7.3 et je n'ai aucun problème avec cette syntaxe, par contre je ne vois pas l'avantage de l'utiliser par rapport aux guillemets dans les cas simples.
Ligne 13, c'est un conseil lu dans un cours pour éviter que la requête continue à scruter après la première réponse.
Ligne 20: Dans quel cas utiliser bindParam ou un argument tableau dans execute? Lorsque ce n'est pas indispensable, j'utilise bindParam quand, j'ai peu de données ou un tableau quand j'en ai beaucoup. Si ce n'est pas judicieux, pourquoi?
Le reste ne me pose pas de problème.

[Séb.]

Ligne 7, j'utilise PHP 7.3 et je n'ai aucun problème avec cette syntaxe, par contre je ne vois pas l'avantage de l'utiliser par rapport aux guillemets dans les cas simples.

Tu envoies une chaîne bizarre au serveur SQL, avec des espaces/tabs avant/après la requête et en début de ligne, alors qu'avec la syntaxe que j'ai utilisée on envoie une chaîne sans espaces inutiles, elle est correctement taillée. Cela se verra par ex. si tu fait un log et que tu écris ta requête dans un fichier ou simplement un <pre>[<?= $sql ?>]</pre>

Ligne 13, c'est un conseil lu dans un cours pour éviter que la requête continue à scruter après la première réponse.

OK je vois, ça peut être utile dans certains cas comme vérifier l'existence d'une ligne parmi des millions.

Ligne 20: Dans quel cas utiliser bindParam ou un argument tableau dans execute? Lorsque ce n'est pas indispensable, j'utilise bindParam quand, j'ai peu de données ou un tableau quand j'en ai beaucoup. Si ce n'est pas judicieux, pourquoi?

bindParam() est utile si tu veux récupérer des résultats de procédure stockée SQL, elle peut aussi, comme bindValue(), permettre de typer tes paramètres pour la bdd (3e argument).
Si on ne fait rien de tout cela execute() est suffisante.

My 2 cents,