Erreur PHP ou MySQL ?

Version imprimable

04/01/2021, 16h20
Denis Placé

Erreur PHP ou MySQL ?
Bonjour,

Je veux insérer la valeur prise par la variable de session $_SESSION["FruitOrdre_$i"] dans la table feuille-fruit-desordre mais l'insertion de mon enregistrement ne s'effectue pas.
Dans mon code, j'ai

Code:

$SQLinsertPlant = insertPlant();

sachant que la fonction insertPlant est la suivante :
Code:

1 2 3 4 5 function insertPlant() { $SQLinsertPlant = $pdo->query("INSERT INTO `feuille-fruit-desordre` (`id`, `fruit`) VALUES (NULL, '".$_SESSION["FruitOrdre_$i"]."') "); return $SQLinsertPlant; }
Je ne rencontre pas de message d'erreur mais l'insertion ne se fait pas et si je fais appel à vous, c'est parce que je ne vois pas mon erreur.
Êtes-vous plus réveillé(e)s que moi ?
04/01/2021, 19h05
Soundboy39

Bonjour,

Qu'est-ce qui est retourné par $pdo->query() ?
Un objet instance de la class PDOStatement ou FALSE ?

Sur base de la lecture de ton code ci-dessous :
Code:

1 2 3 4 5 function insertPlant() { $SQLinsertPlant = $pdo->query("INSERT INTO `feuille-fruit-desordre` (`id`, `fruit`) VALUES (NULL, '".$_SESSION["FruitOrdre_$i"]."') "); return $SQLinsertPlant; }
Que vaut la variable $i ? Elle n'est pas déclarée dans ta fonction.
Et que vaut $pdo ? Non déclaré également au sein de ta fonction.

Ton code est plutôt mauvais; toutes mes excuses d'être aussi direct :

* Tu utilises des variables non déclarées,
* Tu ne vérifies pas p.ex. avec un isset() pour voir si $_SESSION contient effectivement ta variable,
* Tu ne protèges pas ton code (en ce qui concerne les SQL injections),
* Tu retournes quelque chose mais qu'est-ce ? un objet ? un booléen ?

Vu que ta fonction ne contient qu'une seule ligne (le pdo->query), je te suggère en l'état de supprimer ta fonction et de remplacer son appel par, justement, le $pdo->query que je suppose initialisé à ce moment-là.

Bonne soirée.

05/01/2021, 11h45
Denis Placé

Bonjour à tous 2 et merci pour vos interventions.
Je regarde ça et reviens vers vous asap.
07/01/2021, 09h15
Denis Placé

Citation:

Envoyé par Soundboy39

Bonjour,

Qu'est-ce qui est retourné par $pdo->query() ?
Un objet instance de la class PDOStatement ou FALSE ?

Bonjour Sounboy39,

Merci pour ton intervention mais je suis incapable de répondre à ta question. Il n'y a aucun message de retourné.

Bonjour cavo789,

Désolé pour le délai mais j'ai dû en urgence m'ateler à d'autres tâches.

Citation:

Envoyé par cavo789

Que vaut la variable $i ? Elle n'est pas déclarée dans ta fonction.

En effet, elle n'est as déclarée dans la fonction. C'est juste une avriable que j'incrémente et qui m'a permis précédemment de créer des variables de session différentes dans lesquelles j'ai stocké des valeurs.
Quand $i=0, j'ai par exemple $_SESSION["FruitOrdre_1"] qui contient Peuplier, $_SESSION["FruitOrdre_2"] qui contient Aulne, etc.

Citation:

Envoyé par cavo789

Et que vaut $pdo ?

C'est la connexion à la BdD

Citation:

Envoyé par cavo789

Ton code est plutôt mauvais; toutes mes excuses d'être aussi direct :

Aucun problème, je suis un amateur pur et dur et n'ai aucune prétention technique dans ce domaine, ce n'est pas mon métier. Je me borne juste à maintenir en vie un site pour une communauté.

Citation:

Envoyé par cavo789

Tu utilises des variables non déclarées

Si tu entends par là que les variables n'existent pas, hé bien elles existent. J'ai limité le code car il aurait fallu que j'envoie 5 fichiers de plusieurs centaines de lignes et j'ai eu peur de noyer le poisson.

Citation:

Envoyé par cavo789

Tu ne vérifies pas p.ex. avec un isset() pour voir si $_SESSION contient effectivement ta variable

Exact, j'ai oublié ça. Je le modifie ainsi mais ne suis pas sûr de mon coup.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 function insertPlant() { if (isset($_SESSION["FruitOrdre_$i"])) { $SQLinsertPlant = $pdo->query("INSERT INTO `feuille-fruit-desordre` (`id`, `fruit`) VALUES (NULL, '".$_SESSION["FruitOrdre_$i"]."') "); } else { $SQLinsertPlant ="Probleme"; } return $SQLinsertPlant; // provoque la fin de la fonction et le renvoi de la valeur qui la suit }
Citation:

Envoyé par cavo789

Tu ne protèges pas ton code (en ce qui concerne les SQL injections)

C'est également vrai mais ça dépasse largement mon domaine de compétence.

Citation:

Envoyé par cavo789

Tu retournes quelque chose mais qu'est-ce ? un objet ? un booléen ?

Aucune idée, comment savoir ?

Citation:

Envoyé par cavo789

Vu que ta fonction ne contient qu'une seule ligne (le pdo->query), je te suggère en l'état de supprimer ta fonction et de remplacer son appel par, justement, le $pdo->query que je suppose initialisé à ce moment-là.

Là encore, j'ai joué à l'apprenti-sorcier en mettant ce code dans un fichier à part, un peu sur ce que j'ai cru comprendre du fonctionnement d'un modèle MVC pour lequel on met les fonctions dans le modèle, non ?

07/01/2021, 09h41
cavo789
Bonjour Denis

Citation:

Envoyé par Denis Placé

En effet, elle n'est as déclarée dans la fonction. C'est juste une avriable que j'incrémente et qui m'a permis précédemment de créer des variables de session différentes dans lesquelles j'ai stocké des valeurs.

Oui mais non ;-)
Reprends ta fonction (celle que j'ai commentée) : tu utilises la variable $i dans ta fonction; au sein de ta fonction cette variable n'existe pas; elle n'y est ni déclarée ni assignée. Elle vaut donc null.

Ce que tu me dis c'est que tu as déclaré et initialisé cette variable ailleurs; ok, très bien mais ... dans ta fonction, cette variable est inconnue.

Et c'est la même chose pour ma question "Et que vaut $pdo ?"

Au sein de ta fonction, $pdo est inconnu d'où problème; c'est quoi ? ben c'est rien. Et donc pas du tout ta connexion à la base de données.

Citation:

Envoyé par Denis Placé

Aucune idée, comment savoir ?

Ben... vu que tu es le programmeur, tu devrais le savoir ;-) (je taquine)

Avec un code propre, quand tu fais appel à une fonction comme "getFirstName()" p.ex. tu attends quoi ? Ici, j'ai pris un exemple simple : une chaîne de caractères. Il n'est pas logique de recevoir un chiffre.

Ta fonction devrait retourner; selon un principe "clean code" (code propre), un boolean je suppose. Tu es intéressé de savoir si ton insert à fonctionné.

Que retourne $pdo->query ? Regarde la doc. Tu reçois un recordset qui retourne les valeurs (comme si je fais un SELECT FROM; je reçois les records).

Toi tu veux faire quoi ? Un INSERT. Ne devrais-tu pas alors t'intéresser à la fonction pdo->exec() plutôt (https://www.php.net/manual/en/pdo.exec.php) ?
Cette fonction va retourner, voir dans la doc, un entier : le nombre de records ayant été impacté. Si aucun n'a été impacté, retourne 0.

Donc, si zéro, ouch!, il y a eu un souci. Ton INSERT est supposé ajouté un enregistrement.

Du coup, si $pdo->exec(....) retourne le chiffre 1, ta fonction est réussie et peut retourner la valeur true. Sinon, ouch.

Voici une proposition de code (PHP 7)
Code:

1 2 3 4 5 6 7 8 function insertPlant(string $fruit): void { $rows = $this->pdo->exec("INSERT ..."); if (1 !== $rows) { throw new \Exception("Un problème est survenu"); } }
L'idée : tu appelles ta fonction depuis une classe qui a déjà initialisé la propriété $this->pdo (afin qu'elle soit globale).

La fonction est appelée avec un paramètre qui correspond à ton $_SESSION["FruitOrdre_$i"].

Cette fonction ... ne retourne rien. Car, en fait, on n'est pas intéressé de savoir si elle retourne true ou false. On veut (clean coding) qu'elle plante s'il y a eu un problème afin qu'on puisse réagir. En programmation propre, les erreurs ne doivent jamais être silencieuses; il faut les capturer immédiatement et les gérer (avec un try catch p.ex.).

Je sais que j'ai été très certainement très technique ici; l'idée est de te donner quelques pistes.

Bonne journée.

Re-bonjour cavo789,

Suite à ton intervention, j'ai lu et tenté de comprendre ce que tu dis......
J'ai donc modifié ainsi :
Code:

1 2 3 4 5 6 7 8 9 10 function insertPlant(string $fruit): void { $rows = $this->pdo->exec("INSERT INTO `feuille-fruit-desordre` (`id`, `fruit`) VALUES (NULL, '".$fruit."') "); if (1 !== $rows) { throw new \Exception("Un problème est survenu"); } } insertPlant(string $fruit);
J'ai bien entendu vérifié que $fruit existe bien et contient ma bonne chaîne de caractères qui, parfois, peut valoir bota-1234.jpg (mais c'est normal ;)).
Malheureusement, le traitement du script s’arrête (et donc l'insertion ne se fait pas) et aucun message d'erreur ne s'affiche.

Citation:

Envoyé par cavo789

Je sais que j'ai été très certainement très technique ici; l'idée est de te donner quelques pistes.

Euh.... légèrement technique ;) J'ai de la lecture pour plusieurs jours, en tenant compte du travail de compréhension que cela entraîne.

Merci de t'occuper de mon cas.

07/01/2021, 19h36
cavo789

Ton script est-il gigantesque car si pas, poste le ici pour qu'on puisse t'aider.

Dans la fonction tu fais appel à $this->pdo mais as-tu défini cette propriété ? L'as-tu initialisée ? J'ai supposé dans mon dernier message que tu étais dans une classe php et j'ai supposé que tu avais initialisé la variable....

Pour "rien ne se passe" pense à afficher les erreurs php. Cherche des infos sur error_reporting().
08/01/2021, 08h36
Denis Placé

Encore merci pour ton implication,

Citation:

Envoyé par cavo789

Dans la fonction tu fais appel à $this->pdo mais as-tu défini cette propriété ? L'as-tu initialisée ? J'ai supposé dans mon dernier message que tu étais dans une classe php et j'ai supposé que tu avais initialisé la variable....

Euh ... non, pas de classe à l'horizon :oops: Je regarde ça de mon côté pour tenter de comprendre comment ça marche et reviens vers toi.

Citation:

Envoyé par cavo789

Pour "rien ne se passe" pense à afficher les erreurs php. Cherche des infos sur error_reporting().

J'ai

Code:

error_reporting(E_ALL);

dans mon modèle donc normalement... ça devrait afficher les erreurs (je pense).

Exemple ci-dessous; non testé!
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 <?php class Fruit { // Properties private $pdo; private function __construct() { // Variables à définir ! $this->pdo = new PDO($dsn, $user, $pass, $driver_options); } public function insertPlant(string $fruit): void { $rows = $this->pdo->exec("INSERT INTO `feuille-fruit-desordre` (`fruit`) VALUES ('".$fruit."')"); if (1 !== $rows) { throw new \Exception("Un problème est survenu"); } } } $fruit = new Fruit(); $fruit->insert('Fraise'); unset($fruit); ?>
C'est juste pour illustrer la création d'une classe et son utilisation.

Note: j'ai viré "id" de ton SQL car je suppose qu'il s'agit d'un champ de type auto-incrément dans ta table; il ne faut donc pas le mettre dans le SQL, ton gestionnaire de base de données le traitant seul en ajoutant +1 à chaque fois.

Même si ça coince toujours de la même façon, je sens que je commence à comprendre un peu le truc.
On va dire que mon fichier appelant le modèle est :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
<?php
include "../php/PDO_conn.php"; // Inclusion du fichier de connexion à la base de données
require('modele.php');
 
    $fruitOrdre = 'bota-0187-tpm.jpg';
 
    insertPlant(string $fruit);
 
    $fruit = new Fruit();   // créé un objet
    $fruit->insertPlant('$fruitOrdre');
    unset($fruit);
?>

modele.php :

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
32
33
34
35
36
37
38
39
40
41
<?php
error_reporting(E_ALL);   // permet d'afficher les erreurs donc utile pour debugage
 
if(session_id() == '')
      {
        session_start(); // A mettre sur chaque page utilisant la session
      }
 
function connectBDD()
	{
		/* Variables de connexion */
		try
			{
			    include "../php/PDO_conn.php"; // Inclusion du fichier de connexion à la base de données
			}
		catch(Exception $e)
			{
			    die('Erreur : '.$e->getMessage());
			}
	}
 
class Fruit
    {
        // Properties
        private $pdo;
 
        private function __construct() {
            // Variables à définir !
            $this->pdo = new PDO($pdo_conn['hostname'], $pdo_conn['database'], $pdo_conn['password'], $driver_options);
            }
 
          public function insertPlant(string $fruit): void
          {
            $rows = $this->pdo->exec("INSERT INTO `feuille-fruit-desordre` (`fruit`) VALUES ('".$fruitOrdre."')");
 
            if (1 !== $rows) {
                throw new \Exception("Un problème est survenu");
            }
          }
    }
?>

Dans la connexion à la BdB, je n'ai pas modifié ton $driver_options. Je ne sais pas à quoi ça sert mais ce n'est peut-être pas la seule erreur.
PS: Je sais qu'il y a un doublon dans la connexion à la BdD mais ce n'est pas le problème pour l'instant.

08/01/2021, 15h13
Denis Placé

En ligne 51 de mon 1er fichier, ça ne serait pas plutôt

Code:

$fruit->insertPlant('$FruitOrdre');

J'essaie mais ça ne change rien.
08/01/2021, 15h24
cavo789

Oui, pardon, erreur de typo dans mon exemple. Le nom de la fonction (ligne 14) est bien insertPlant et donc bien $fruit->insertPlant
08/01/2021, 15h37
Denis Placé

Ok mais ça ne fonctionne pas quand-même.
Comme je le disais, dans la connexion à la BdD, je n'ai pas modifié ton $driver_options. Je ne sais pas à quoi ça sert mais ce n'est peut-être pas la seule erreur.
09/01/2021, 09h24
Denis Placé

PS: Je viens de simplifier le code que j'ai posté hier à 11h34 afin de ne pas "noyer le poisson".

Je viens de modifier mon fichier appelant ainsi:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
<?php
	require "../php/PDO_conn.php"; // Inclusion du fichier de connexion à la base de données
 
	require('modele4.php');
 
    $fruit = new Fruit();
    $fruit->insertPlant('peuplier');
 
    $fruitOrdre = 'bota-0187-tpm.jpg';
    unset($fruit);
    echo "ici";
?>

Tandis que modele.php vaut :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
<?php
error_reporting(E_ALL);
 
if(session_id() == '')
      {
        session_start();
      }
 
class Fruit
    {
        public function insertPlant(string $fruit)
          {
              $prep = $pdo->prepare("INSERT INTO `feuille-fruit-desordre` (`fruit`) VALUES (:fruit)     ");
              $prep->bindParam(':fruit', $fruit);
              $prep->execute();
          }
    }
?>

La ligne 7 du fichier appelant faut planter le script et entraine une erreur HTTP ERROR 500.
Par ailleurs, mon site fonctionne en dehors de cette page donc c'est bien cette ligne qui entraine le problème (ou ce qui se trouve dans le modèle et qui y fait référence) mais je ne vois pas quoi. :?

12/01/2021, 11h27
cavo789

Bonjour Denis

Es-tu en PHP 7 ? Si pas, dans ton modèle, ligne 11, change vers publicfunction insertPlant($fruit). (supprimer "string" donc)

Note: c'est difficile de t'aider en ce moment parce que ton code me paraît "complexe" dans le sens où, là, à la lecture, je continue à me dire que tu fais des erreurs que je pensais avoir déjà mentionné.

Dans ta fonction insertPlant, $pdo existe-t-il ? Si tu regardes ton modèle, cette variable n'est jamais initialisée.

Je parlais de clean_code mais c'est même au-delà ici : une classe PHP (ta classe Fruit) est supposée être autonome càd que pdo doit y être instanciée ce qui n'est pas le cas. Sauf erreur de ma part, ton code ne pourra jamais fonctionner en tant que tel.

Fût une époque où l'on pouvait déclarer des variables globales; tu la déclarais dans un fichier (comme globale) et tu la récupérais dans un autre fichier mais ça, c'est à l'opposé du clean code.

Ceci pour dire : à lire tes bouts de code, perso, je pense que cela ne fonctionnera pas et qu'il te faut d'abord mettre tout ça au propre. Une classe n'est pas un modèle, c'est un fichier qui est supposé ne contenir que, et strictement que, la déclaration de la classe; pas de "if ..." comme tu l'as mis p.ex. (concrètement tes lignes 9 à 18 uniquement).
12/01/2021, 11h52
Denis Placé

Merci cavo789 pour cette nième intervention,

Citation:

Envoyé par cavo789

Es-tu en PHP 7 ?

Non, je suis en 5.5.38 donc je supprime string.

Citation:

Envoyé par cavo789

$pdo existe-t-il ?

Je dirais oui car je l'initialise dans ma connexion à la base de données, connexion qui s'effectue correctement.

Code:

require "../php/PDO_conn.php";

Si j'insère

Code:

var_dump($pdo);

J'obtiens object(PDO)#1 (0) { }

Citation:

Envoyé par cavo789

Une classe n'est pas un modèle, c'est un fichier qui est supposé ne contenir que, et strictement que, la déclaration de la classe; pas de "if ..." comme tu l'as mis p.ex. (concrètement tes lignes 9 à 18 uniquement).

Ce que j'appelle "fichier appelant" est le contrôleur et j'ai mis la classe dans le modèle. Je n'aurais pas dû ?

Comme tu le dis, cette histoire est mal partie donc tant pis, je vais laisser tomber le MVC et considérer qu'elle est résolue.
Un grand merci à tous et surtout à toi.