Requête qui fonctionne en dur mais pas avec des variables

Version imprimable

Bonjour à tous,

Je rencontre un soucis sur une requête qui fonctionne correctement dans PHPMyAdmin quand je mets des valeurs en dur.
Mais une fois que je la mets dans mon PHP et que j'essaye de la faire fonctionner avec des variables (des valeurs de checkbox) elle ne fonctionne pas et me retourne un résultat vide (sans erreur).
Je ne comprends pas d'ou peut venir le soucis !
Voici le code en question :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
<?php
 
        if(isset($_POST['color-choice']) AND isset($_POST['type_choice']))
        {
            if(!empty($_POST['color-choice']) and !empty($_POST['type_choice']))
            {
                $couleur = implode(',', $_POST['color-choice']);
 
                $type = implode(",", $_POST['type-choice']);
 
                $req = $sqlQuery->query("SELECT lieux.nom FROM lieux INNER JOIN lieux_couleur ON lieux.ID = lieux_couleur.id_lieux INNER JOIN couleur_vin ON couleur_vin.ID = lieux_couleur.id_couleur WHERE (couleur_vin.ID IN ( " . $couleur . ") AND lieux.id_type_lieu IN( " . $type . "))");
                while ($data = $req->fetch())
				{
                    echo $data["nom"] . '</br>';
                }
            }
        }
		?>

Merci de votre aide !

10/12/2017, 11h28
al1_24

Quand on construit une requête par concaténation de chaines de caractères, la première chose à faire lorsque ladite requête ne s'exécute pas correctement c'est d'afficher le texte de la requête.
Dans la plupart des cas, l'erreur apparait clairement.
10/12/2017, 11h34
matthieu24

Merci de votre réponse,

Comment fait-on pour afficher le texte de la requête ?

Je suis novice en développement je n'ai pas encore tous les automatismes.

Merci pour votre aide en tout cas ;)

On peut faire comme ça.
Code:

1 2 3 4 $query = "SELECT lieux.nom FROM lieux INNER JOIN lieux_couleur ON lieux.ID = lieux_couleur.id_lieux INNER JOIN couleur_vin ON couleur_vin.ID = lieux_couleur.id_couleur WHERE (couleur_vin.ID IN ( " . $couleur . ") AND lieux.id_type_lieu IN( " . $type . "))"; print $query; $req = $sqlQuery->query($query);
D'après moi il manque des apostrophes avant l'affectation des valeurs. Le IN dans une requête SQL sert quand on a une liste de valeurs. Dans ton cas un = suffirait.
Code:

1 2 $query = "SELECT lieux.nom FROM lieux INNER JOIN lieux_couleur ON lieux.ID = lieux_couleur.id_lieux INNER JOIN couleur_vin ON couleur_vin.ID = lieux_couleur.id_couleur WHERE (couleur_vin.ID IN ( '" . $couleur . "') AND lieux.id_type_lieu IN( '" . $type . "'))";
Du moment que tu utilises " pour délimiter une chaîne de caractères, tu peux mettre un nom de variable dedans. PHP remplacera le nom de la variable par sa valeur.
Code:

1 2 $query = "SELECT lieux.nom FROM lieux INNER JOIN lieux_couleur ON lieux.ID = lieux_couleur.id_lieux INNER JOIN couleur_vin ON couleur_vin.ID = lieux_couleur.id_couleur WHERE (couleur_vin.ID IN ( '$couleur') AND lieux.id_type_lieu IN( '$type'))";
Il y aurait encore plein de choses à dire mais comme tu débutes je vais te laisser digérer ça.

Merci de ta réponse !

Alors j'ai essayé cela
Code:

1 2 3 $query = "SELECT lieux.nom FROM lieux INNER JOIN lieux_couleur ON lieux.ID = lieux_couleur.id_lieux INNER JOIN couleur_vin ON couleur_vin.ID = lieux_couleur.id_couleur WHERE (couleur_vin.ID IN ( " . $couleur . ") AND lieux.id_type_lieu IN( " . $type . "))"; print $query; $req = $sqlQuery->query($query);
Le résultat est toujours vide.

Tu me dit qu'un égal suffirait car le IN concercne plusieurs valeurs, mais si je coche plusieurs checkbox, j'ai bien plusieurs valeurs non ?

J'ai essayé en rajoutant les apostrophes et le résultat de mon submit est toujours tristement vide alors que ma requête fonction dans la console !!!

Bonjour,

1- facilite-toi la vie en indentant proprement le code :
Code:

1 2 3 4 5 6 7 8 $query = "SELECT lieux.nom FROM lieux INNER JOIN lieux_couleur ON lieux.ID = lieux_couleur.id_lieux INNER JOIN couleur_vin ON couleur_vin.ID = lieux_couleur.id_couleur WHERE (couleur_vin.ID IN ( " . $couleur . ") AND lieux.id_type_lieu IN( " . $type . "))";
ou encore (avec des alias pour les noms des tables) :
Code:

1 2 3 4 5 6 7 8 $query = "SELECT lieux.nom FROM lieux L INNER JOIN LC LC ON L.ID = LC.id_lieux INNER JOIN couleur_vin CV ON CV.ID = LC.id_couleur WHERE (CV.ID IN ( " . $couleur . ") AND L.id_type_lieu IN( " . $type . "))";
2- Si on te dit d'AFFICHER la requête, ce n'est pas pour que tu répondes : "Le résultat est toujours vide".
MONTRE ce qui s'affiche !

10/12/2017, 13h23
matthieu24

Merci de votre réponse,

Le problème c'est que rien ne s'affiche.

1 - Je check des checkbox dans les 2 fieldset.
2 - Je valide le formulaire avec le bouton submit
3- Page blanche, aucun resultat ne s'affiche et pas d'erreur.

Alors que la requête fonctionne en dur !

Désolé si je n'ai pas encore tous les reflex :)

Merci encore !
10/12/2017, 13h51
badaze

Est ce que tu as mis le print $query; ?
si oui et que rien ne s'affiche c'est que le souci est ailleurs.

Pour le IN il ne suffit pas de dire que tu peux cocher plusieurs choix pour que ça marche.
Le IN a une syntaxe propre et ton code ne formate pas le paramétre comme il se doit. D'où ma remarque sur le =.
10/12/2017, 14h08
matthieu24

Oui j'ai bien mis le print $query;

et rien ne s'affiche au submit ni aucune erreur !

Bon.

montre-nous ce que valent $couleur et $type.

Code:

1
2
3
4
5
6
7
$couleur = implode(',', $_POST['color-choice']);
$type = implode(",", $_POST['type-choice']);
 
echo '$couleur : '.$couleur.'<br />';
echo '$type : '.$type.'<br />';
 
exit; // stoppe le script

Je viens de faire l'echo des 2 variables

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
if(isset($_POST['color-choice']) AND isset($_POST['type_choice']))
        {
            if(!empty($_POST['color-choice']) and !empty($_POST['type_choice']))
            {
 
                $couleur = implode(',', $_POST['color-choice']);
 
                $type = implode(",", $_POST['type-choice']);
 
                echo '$couleur : '.$couleur.'<br />';
                echo '$type : '.$type.'<br />';
 
                exit;
 
            }
        }
		?>

Toujours un résultat vide et sans erreur !

Je suis vraiment paumé !

1- PHP

Code:

1
2
3
<?php
// Afficher les erreurs et les avertissements
	error_reporting(E_ALL); 	// en TEST !!

2- PDO : Les erreurs et leur gestion
voir l'exemple :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
<?php
$dsn = 'mysql:dbname=testdb;host=127.0.0.1';
$user = 'dbuser';
$password = 'dbpass';
 
try {
    $dbh = new PDO($dsn, $user, $password);
    $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); // rapport d'erreurs sous forme d'exceptions
} catch (PDOException $e) {
    echo 'Échec lors de la connexion : ' . $e->getMessage();
}
?>

Idem pour les requêtes :

Code:

1
2
3
4
5
6
try {
    $req = "............";
    $dbh->query($req);
} catch (PDOException $e) {
    echo 'Échec requete : ' . $e->getMessage();
}

j'ai trouvé ma réponse en passant par une 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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
<?php
 
        function implodePost($param)
        {
            if(isset($_POST[$param]) && !empty($_POST[$param]) )
            {
                return implode(",", $_POST[$param]);
            }
            else
            {
                return null;
            }
        }
 
        if(isset($_POST['hidden']))
		{
			if (isset($_POST['color-choice']) or isset($_POST['type-choice']))
			{
                $couleur = implodePost("color-choice");
                $type = implodePost("type-choice");
                $where = "";
 
                if( $couleur !== null and $type !== null )
                {
                    $where = "WHERE ((couleur_vin.ID IN ( " . $couleur . ") AND lieux.id_type_lieu IN( " . $type . ")))";
                }
				elseif ( $couleur !== null and $type == null )
                {
                    $where = "WHERE ((couleur_vin.ID IN ( " . $couleur . ")))";
                }
				elseif( $couleur == null and $type !== null )
                {
                    $where = "WHERE ((lieux.id_type_lieu IN( " . $type . ")))";
                }
 
                $query = "SELECT lieux.nom FROM lieux INNER JOIN lieux_couleur ON lieux.ID = lieux_couleur.id_lieux INNER JOIN couleur_vin ON couleur_vin.ID = lieux_couleur.id_couleur $where GROUP BY lieux.nom";
 
                $req = $sqlQuery->query($query);
 
                while ($data = $req->fetch())
                {
                    echo $data["nom"] . '</br>';
                }
			}
			else
			{
				echo 'Veuillez cocher au minimum 1 case svp';
			}
		}
		?>

dossier clos ! Merci de vos aides :D

Citation:

Toujours un résultat vide et sans erreur !
Je suis vraiment paumé !

Avec:

Citation:

echo '$couleur : '.$couleur.'<br />';
echo '$type : '.$type.'<br />';

Tu ne peux pas savoir si les variables sont vides ou si on n'entre pas dans la boucle. Il vaudrait mieux faire:
Code:

1 2 3 4 5 6 7 8 if(isset($_POST['color-choice']) AND isset($_POST['type_choice'])) echo "<hr>IS SET OK<hr>"; { if(!empty($_POST['color-choice']) and !empty($_POST['type_choice'])) $couleur = implode(',', $_POST['color-choice']); $type = implode(",", $_POST['type-choice']); echo "<hr> $couleur --- $type<hr>"; {
Avec ça tu saura si les conditions sont validées ou pas.

Autres points: Comme dit plus haut toujours encadrer d'apostrophes le nom des tables sinon la plupart des requêtes ne fonctionneront pas.

Et enfin pour voir les erreurs dans une requête ajouter:
Code:

1 2 3 4 5 $req = $sqlQuery->query($query); if(!$req) { echo "<hr>$mysqli->error<hr>"; }
En conclusion une question bête: $sqlQuery contient vraiment le nom de la connexion à la base de données?

As-tu vu que depuis le début, tu avais écrit $_POST['type_choice'] avec un underscore ?? :weird:
Et là, tu dis que ça fonctionne avec..... $_POST['type-choice'] .... un tiret !

DEPUIS LE DEBUT, l'erreur provient simplement de cet underscore à la place d'un tiret !!

Je t'ai montré comment AFFICHER LES MESSAGES D'ERREUR, dans mon post #12 !

Du coup, ta fonction ne résoud rien (et ne sert, à vrai dire, à rien non plus).

Ton code se résume à :

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
if(isset($_POST['hidden']))
{
	if ( !empty($_POST['color-choice']) && !empty($_POST['type-choice']) )
	{	
		// construction du WHERE
		$arrayWhere = array();
		if ( !empty($_POST['color-choice']) )
		{
               		$colorList = implode(',', $_POST['color-choice']);
               		$arrayWhere[] = " couleur_vin.ID IN ( " . $colorList . ") ";
		}
		if ( !empty($_POST['type-choice']) )
		{
               		$typeList = implode(',', $_POST['type-choice']);
               		$arrayWhere[] = " lieux.id_type_lieu IN( " . $typeList . ") ";
		}
		$clauseWhere = ( !empty($arrayWhere) )? " WHERE " . implode( " AND ", $arrayWhere ) : "";
 
		$query = "SELECT lieux.nom FROM lieux 
			INNER JOIN lieux_couleur ON lieux.ID = lieux_couleur.id_lieux 
               		INNER JOIN couleur_vin ON couleur_vin.ID = lieux_couleur.id_couleur 
               		" . $clauseWhere . "
               		GROUP BY lieux.nom";
 
		$req = $sqlQuery->query($query);
		// ......
	}
	else
	{
		echo 'Veuillez cocher au minimum 1 case svp';
	}
}