Discussion :

[Lenezir]

Salut ^^
J'ai essayé de faire un petit moteur de recherche afin de trouver des entrées dans ma BDD, mais quand j'exécute la requête, elle me fait planter le PC
Pourriez-vous me dire ce que vous en pensez ?
Merci ^^

Code :

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

1
2
3
4
5
6
7
// La partie HTML
<form method="post" action="<?php $PHP_SELF ?>">
	<td align="center" class="lien" height="20">
		<input type="text" name="search" size="15">
		<input type="submit" name="image" value="OK">
	</td>
</form>

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
// La partie requête
$saisie = $_POST['search'];
	if(isset($saisie))
	{
		$recherche = "SELECT repertoire.id_rep, repertoire.id_pers, personnel.nom, fonction.fonction, service.service 
		FROM repertoire, personnel, fonction, service 
		WHERE repertoire.id_pers = personnel.id_pers 
		AND personnel.id_fonction = fonction.id_fonction 
		AND personnel.id_service = service.id_service 
		AND personnel.nom LIKE '".$saisie."' 
		OR fonction.fonction LIKE '".$saisie."' 
		OR service.service LIKE '".$saisie."' 
		OR repertoire.num_int LIKE '".$saisie."' 
		OR repertoire.num_ext LIKE '".$saisie."' 
		OR repertoire.num_fax LIKE '".$saisie."' 
		OR repertoire.num_port LIKE '".$saisie."'";
		$recherche_finale = mysql_query($recherche) or die(mysql_error());
	}

[nicocsgamer]

Tu as essayé de lancer cette requête sur ton SGBD sans passer par PHP ?

Et si tu simplifie ta requête ?

[Lenezir]

Oui j'ai essayé de la lancer par MySQL et ça me fait quand même tout planter ^^
Et comment la simplifier ?

[kernigansnotdead]

Salut,

a ta place, j'utiliserais pas le 'or die' et j'utiliserais echo mysql_error();

De meme je mettrais un echo juste avant $recherche = "SELECT repertoire....
-> echo $recherche = "SELECT repertoire....
juste pour tester

voila, ça t'aidera peut-etre, pour le reste, la requete à l'air correcte

KND

[guidav]

Deux petites remarques sur ta syntaxe :
1 - tu peux écrire "OR fonction.fonction LIKE '$saisie' " plutôt que OR fonction.fonction LIKE '".$saisie."' (c'est valable sur toutes les lignes). Les variables sont évaluées dans les doubles quotes, et c'est nettement plus lisible.
2 - sur le fond de ta requête : si $saisie ne comporte pas de joker, tu ferais mieux d'utiliser l'opérateur = plutôt que LIKE.
Sinon, tu peux ajouter des jokers dans ta requête de cette façon : "OR fonction.fonction LIKE '$saisie%' ".

Sinon, tu dis que ça plante, mais quels sont les symptômes ?

[Lenezir]

Bon ben merci ^^
Alors la requête doit être trop lourdre (en effet les 4 tables contiennent chacune plus de 200 entrées ^^)

EDIT : désolé j'ai vu ton message trop tard ^^
MySQL ralenti tout et fini par tout planter.
Mais j'ai trouvé un autre moyen de faire la requête (par choix de table sur la page principale).
Merci ^^

[guidav]

Non, 200 entrées c'est rien du tout, j'ai des tables de 4 millions d'entrées avec MySQL, ça doit venir d'ailleurs. Tu as essayé de réécrire ta requête avec la syntaxe JOIN ?

[Lenezir]

Connais pas cet élément
Mais j'ai trouvé une meilleure solution.
J'ai fais un peu comme Google, avec des boutons radio pour le choix de la table :

Code :

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

1
2
3
4
5
6
7
8
9
10
// Formulaire
<form method="post" action="<?php $PHP_SELF ?>">
				<td align="center" class="lien" height="20">
					<input type="text" name="search" size="15" />
					<input type="submit" name="image" value="OK" /><br/>
					<input type="radio" name="table" value="service.service" />Services
					<input type="radio" name="table" value="fonction.fonction" />Fonctions
					<input type="radio" name="table" value="personnel.nom" checked />Noms
				</td>
			</form>

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
// La requête
$saisie = $_POST['search'];
	$champ = $_POST['table'];
	if(isset($saisie))
	{
		$recherche = "SELECT repertoire.id_rep, repertoire.id_pers, personnel.nom, fonction.fonction, service.service 
		FROM repertoire, personnel, fonction, service 
		WHERE repertoire.id_pers = personnel.id_pers 
		AND personnel.id_fonction = fonction.id_fonction 
		AND personnel.id_service = service.id_service 
		AND ".$champ." LIKE '%".$saisie."%'";
		$recherche_inter = mysql_query($recherche) or die(mysql_error());
	}