Problème de syntaxe

Version imprimable

Voir 40 message(s) de cette discussion en une page

Problème de syntaxe

Bonsoir,

J'ai un problème avec une requête SQL.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
<?php
	session_start();
	if(isset($_POST['submit'])){
		if(empty($_POST['recherche'])){
			$erreur = 'Le champ recherche est vide';
		}
		else{
			require 'config.php';
			$connexion_sql = mysql_connect(DB_HOST,DB_LOGIN,DB_PASS); 
			$connexion_bdd = mysql_select_db(DB_BDD);
 
			$recherche = mysql_real_escape_string($_POST['recherche']);
 
			$lecture = mysql_query("SELECT * FROM membre,article WHERE pseudo,titre,nom,prenom LIKE '%$recherche%'") or die;
			$retour = mysql_fetch_array($lecture);
			print_r($retour);
		}
	}
?>
<form method="post">
	<input type="text" name="recherche">
	<input type="submit" name="submit">
</form>

La requête SQL suivante ne fonctionne pas.
$lecture = mysql_query("SELECT * FROM membre,article WHERE pseudo,titre,nom,prenom LIKE '%$recherche%'") or die;

Celle ci "die" à chaque fois.

J'aimerais bien savoir là ou sa cloche. :/

Merci. ;)

16/03/2010, 22h07
zugolin

euhh...
"SELECT * FROM membre,article " ... membre, article ... , c'est une table ?
16/03/2010, 22h56
JeanJean75

Oui monsieur.
16/03/2010, 23h46
zugolin
j'avais fait une recherche sql qui voulait pas fonctionner et j'avais du repéter mes requetes comme ça :
Code:

1 2 3 4 5 6 delete FROM `contact_autres` WHERE `emailcontact` LIKE '%@aliceadsl.fr%' or `emailcontact` LIKE '%@aol.com%' or `emailcontact` LIKE '%@aol.fr%' etc...
(j'suis pas au top, mais si je peux essayer d'aider...)
17/03/2010, 01h45
sabotage

Sur le fond ta requete ne vas pas, tu mets deux tables sans jointure entre elles, tu vas donc recuperer toutes les combinaisons possibles entre les lignes des deux tables.

Sinon pour l'erreur de syntaxe, il faut effectivement écrire la condition entière pour chaque champ.
17/03/2010, 06h11
RunCodePhp

Pour le die, corrige comme ceci :

Code:

$lecture = mysql_query(...) or die('ERREUR : '.mysql_error());
17/03/2010, 11h22
JeanJean75

Plop,

Merci pour vos réponses. ;)

En effet, j'ai ré-écrit la requête en suivant vos conseils :

Code:

$lecture = mysql_query("SELECT nom,prenom,titre,contenu FROM membre,articles WHERE pseudo LIKE '%$recherche%' OR titre LIKE '%$recherche%' OR nom LIKE '%$recherche%' OR prenom LIKE '%$recherche%'") or die('Erreur :'.mysql_error());

Ca marche bien. ;)

Sabotage => Enfaite, je veux faire une sorte de mini moteur de recherche qui va chercher dans plusieurs le terme renseigné par l'utilisateur. ;)
17/03/2010, 12h56
BornBanane

Sans jointure ta requête va péter* dès que tu auras un certain nombre d'utilisateurs et/ou d'articles.

* Prendre énormément de temps, ou ne pas se faire du tout.
17/03/2010, 14h25
JeanJean75

on la fait plus courte

Après renseignements, j'ai fais quelque chose comme ça, mais j'ai toujours ce même problème.

$lecture = mysql_query("SELECT articles.titre,articles.contenu,membre.pseudo FROM membre,articles WHERE membre.pseudo LIKE '%$recherche%' OR articles.titre LIKE '%$recherche%' OR articles.contenu LIKE '%$recherche%'") or die('Erreur :'.mysql_error());

Quand je fais une recherche avec pour mot clé Jean :

Citation:

Résultat de la recherche pour les pseudo :

Jean

Résultat de la recherche pour les pseudo :

JeanJean

Résultat de la recherche pour les pseudo :

Jean

Résultat de la recherche pour les pseudo :

JeanJean

etc.
17/03/2010, 15h09
ThomasR

Bonjour,

Citation:

mais j'ai toujours ce même problème

Quel problème ? Si tu as ces résultats qui s'affiche c'est que ta requête ne "die" plus donc ce n'est plus le même problème :koi: ?

Quel est le comportement attendu ? Tu souhaites éviter les doublons ? Si oui utilises DISTINCT.
17/03/2010, 15h13
JeanJean75

Autant pour moi. A force de me battre contre ce problème, j'ai cru que je vous en avait déjà parlé. :mouarf:

J'ai essayé avec DISTINCT, mais j'ai aucun changement. :/
17/03/2010, 15h55
RunCodePhp

Si tu ne fais pas de jointure, ça ne va pas l'faire :?

Faudrait que tu regarde la structure de tes tables "membre" et "article" pour faire la jointure.

Un truc du genre peut être : ( avec JOIN ou INNER JOIN)

Code:

SELECT ... FROM membre JOIN article ON article.membre_id = membre.membre_id WHERE ...

Ensuite, faut voir ce que tu recherche exactement.
Si c'est uniquement des personnes en recherche par leur pseudo (donc ne pas vouloir les articles), alors faut pas faire allusion à la table "article" dans la requête, sinon il y aura autant de lignes que d'articles créés pour chaque personne.

Si tu recherche tous les articles d'une personne d'après son pseudo, et bien si cette personne a plusieurs articles, il est normal que le pseudo se répète.

Faut savoir ce que tu veux faire : Soit rechercher des personnes, soit des articles.
17/03/2010, 16h34
JeanJean75

Je veux bien faire des jointures, mais je crois qu'il va falloir que je réorganise un peu mes tables non ?

Exemple avec membre et articles :

http://img682.imageshack.us/img682/5659/sanstitreko.png

http://img341.imageshack.us/img341/7...stitre2hmx.png

Je suis obligé d'utiliser les id des membres de la table membre partout ailleurs non ? (Au lieu de remettre le pseudo dans la table articles)

Désolé d'être long à la détente, mais les jointure c'est nouveau pour moi. :/
17/03/2010, 17h09
RunCodePhp

Citation:

Je veux bien faire des jointures, mais je crois qu'il va falloir que je réorganise un peu mes tables non ?

Et bien théoriquement oui, car ta table "article" n'a aucun élément pour lier les membres et leur articles, du moins, la méthode n'est pas correcte.

Ta table "article" un champ "auteur", et leur données (valeurs) sont des noms. Ce n'est pas correcte.
Faudrait remplacer tous ces noms de la table "article" par leur identifiants respectifs.
1 -> à la place de Jean
2 -> à la place de Test

Mais pour faire bien, il faudrait nommer correctement tout ça, pour enlever toute ambiguïté.
Comme la table contenant les membres a pour nom "membre", il est plus simple de remplacer "id" par "membre_id".
De même que remplacer "id" par "article_id" de la table "article".

Ensuite, remplacer le nom "auteur" de la table "article" par "membre_id". (le même nom que la table "membre" car ce seront les même données communes).

Pour finir, suffit de reprendre la requête que j'ai donnée plus haut pour faire la jointure :

Code:

SELECT ... FROM membre JOIN article ON article.membre_id = membre.membre_id WHERE ...

On peu simplifier la syntaxe en utilisant des alias (c'est très courant) :

Code:

SELECT a.titre, m.pseudo FROM membre m JOIN article a ON a.membre_id = m.membre_id WHERE m.pseudo LIKE '%machin%'
17/03/2010, 17h26
JeanJean75

D'accord.

Je suis entrain d'éditer toutes mes tables sur ton modèle. ;)
Je profite du topic pour te demander comment modifier cette table :

http://img509.imageshack.us/img509/2062/sanstitretr.png

Si je transforme "expediteur" et "destinataire" en membre_id, je risque d'avoir des problèmes.

Vois tu une autre possibilité ?
17/03/2010, 20h07
RunCodePhp

Citation:

Si je transforme "expediteur" et "destinataire" en membre_id, je risque d'avoir des problèmes.

On ne peut pas avoir 2 fois le même nom dans une même table évidemment, donc tu peux conserver "destinataire" et "expediteur".
Par contre, il faudra que ça soit leur identifiants respectifs (comme pour la table article), des couples d'IDs en faite.

Vu que ça concerne des IDs, les nommer : destinataire_id et expediteur_id serait plus "parlant".

Bref ... le tout c'est d'avoir une cohérence au niveau des noms, des données, aussi qu'un simple coup d'oeil suffit pour savoir qu'est ce que cela concerne et contient.
22/03/2010, 22h07
JeanJean75

Bon voilà, j'ai arrangé les tables et la requête SQL à la page qui permet de lire les articles.

http://img440.imageshack.us/img440/2629/sanstitreii.png

Code:

$lecture = mysql_query('SELECT articles.titre,articles.contenu,membre.membre_id FROM membre,articles WHERE membre.membre_id=articles.membre_id ORDER BY articles.timestamp DESC LIMIT 0,5') or die('Erreur');

Cependant, il faut que je refasse la requête pour la page qui permet d'envoyer des articles.

Actuellement j'ai ça :

Code:

mysql_query("INSERT INTO articles (id,auteur,titre,contenu,timestamp) VALUES ('','$pseudo','$titre','$contenu','$timestamp')") or die(mysql_error());

Je suis obligé de faire deux mysql_querry. Un pour récupérer l'id_membre du membre et l'autre pour insérer l'article.

A moins qu'une astuce me permette de faire qu'une requête ! :)

Merci. ;)
23/03/2010, 09h35
RunCodePhp

Citation:

Je suis obligé de faire deux mysql_querry. Un pour récupérer l'id_membre du membre et l'autre pour insérer l'article.

Et pourquoi donc ?
Ca n'a pas de sens d'ailleurs, car sur quel élément te réfèreras tu pour récupérer l'ID du membre (à part son ID) ?

Si un membre parvient à éditer un formulaire pour créer un article, son ID est connu non ? En général on le stock dans une session.
Ou alors, c'est une autre personne qui créer un article pour le compte d'un membre, dans cette condition le formulaire de création doit offrir la possibilité de choisir le membre, un ID (parmi une liste).

L'un comme l'autre tu as son ID.

Tu dois avoir un problème en amont, comme par exemple n'avoir sous le coude que son pseudo, et non son ID.
Faut donc procéder un peu différemment.

Quand un membre s'identifie par exemple, faut conserver son ID en mémoire quelque part (session), c'est l'élément le plus important, plus que le pseudo, car tu en auras besoin constamment (théoriquement).

Petite remarque en passant.
Vu que c'est une requête pour insertion, et que l'id_article est un auto_increment, il n'est pas utile de la renseigner dans la requête :

Code:

INSERT INTO articles (membre_id, titre, contenu, timestamp) VALUES ('$membre_id', '$titre', '$contenu', '$timestamp') ...etc ...

Ici, on laisse la Bdd générer un ID unique à l'article.

Oui en effet, je n'avais pas pensé à utiliser les sessions.

Donc de retour sur mon moteur de recherche.

Pour rappel, voilà le code :

<

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
?php
session_start();
if(isset($_POST['submit'])){
	if(empty($_POST['recherche'])){
		$erreur = 'Le champ recherche est vide';
	}
	else{
		require 'config.php';
		$connexion_sql = mysql_connect(DB_HOST,DB_LOGIN,DB_PASS); 
		$connexion_bdd = mysql_select_db(DB_BDD);
 
		$recherche = mysql_real_escape_string($_POST['recherche']);
 
		$lecture = mysql_query("SELECT distinct articles.titre,articles.contenu,membre.pseudo FROM membre,articles WHERE membre.pseudo LIKE '%$recherche%' OR articles.titre LIKE '%$recherche%' OR articles.contenu LIKE '%$recherche%'") or die('Erreur :'.mysql_error());
		while($retour = mysql_fetch_array($lecture)){
			if(isset($retour['pseudo']) && $retour['pseudo'] !== null){
				echo $retour['pseudo'].
				'<br /><br />';
			}
		}
	}
}
?>
<form method="post">
	<input type="text" name="recherche">
	<input type="submit" name="submit">
</form>

Alors là c'est assez marrant.

Si je recherche Jean, j'ai Jean qui s'affiche. Si je cherche un autre terme au pif j'ai Jean et Test qui s'affichent et ça je sais pas pourquoi O_o

24/03/2010, 08h15
RunCodePhp

Citation:

Si je recherche Jean, j'ai Jean qui s'affiche. Si je cherche un autre terme au pif j'ai Jean et Test qui s'affichent et ça je sais pas pourquoi

C'est surement parce que ta requête n'a pas de jointure entre les 2 tables "membre" et "articles".

Aussi, faudrait savoir ce que tu espère avec le DISTINCT que tu as mis.
Faudrait même savoir le but des recherches.
- Si c'est pour rechercher tous les articles d'un ou plusieurs membres, il ne faut pas de DISTINCT (ou de GROUP BY), car tu va perdre des lignes, des articles.
- Si c'est pour rechercher un ou plusieurs membres, indépendamment du nombre d'articles, alors un GROUP BY sera nécessaire.

Ce sont 2 requêtes différentes.

Code:

SELECT articles.titre,articles.contenu,membre.pseudo FROM membre JOIN articles ON articles.membre_id = membre.membre_id WHERE ... etc ...

Ceci dit, cette requête je l'avais déjà donnée plus haut, et en plus, avec alias et sans alias.
Ben alors ... :lol:

Voir 40 message(s) de cette discussion en une page