Ma requête fonctionne dans phpMyAdmin mais pas avec PDO

Version imprimable

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:

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

Code:

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:

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:

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'];
}

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

Citation:

Envoyé par Séb.

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.

Citation:

Code:

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!
Citation:
Je me permets de réécrire ta fonction :
Code:

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.

26/08/2020, 00h00
Séb.

Citation:

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> ;)

Citation:

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.

Citation:

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,