requete de recherche impossible a formuler

par **sadeq** » 11 sept. 2006, 09:06

Le plaisir est pour moi

par **RinaBK** » 09 sept. 2006, 12:54

Oui, effectivement ta suggestion est pas mal du tout !! C’est vrai que elle est mieux formulée et en plus, elle ne prend pas beaucoup de place !! C’est géniale, et merci beaucoup !!!! Je vais dès maintenant utiliser ta manière de faire, car ça me plait !

Aussi, je voulais te montrer en live la page que tu m’as aidé à faire :
• http://www.bk-sims.ch/vetements.php

…et je te remercie encore infiniment pour ton aide qui me fut très précieuse !!! Merci

par **sadeq** » 07 sept. 2006, 13:15

D'abord je suis content pour toi

Ta tout à fait raison de t'organiser comme tu veux quant à l'agencement de ton code.

Mais il me semble que ton code d'exécution et de lecture du résultat de la requête pose 3 problèmes logiques :

1. même si $db n'existe pas (pas de ressource ouverte par le header) une instruction mysql_query est lancée en vain et c'est une perte de temps processeur

2. le mysql_fetch_array qui suit et qui lit le résultat de mysql_query ne sait pas vraiment si il y a eu résultat ou pas et dans le cas où il y en a pas un gros crash survient.

3. la variable $sql est utilisée à double rôle : elle est dans un premier temps porteuse de la chaîne sql et dans un second elle devient le résultat (ressource) de l'exécution de mysql_query. Sachant que les deux types ne sont pas les mêmes cette façon de faire est source de confusion pour le programmeur qui lit et maintient le programme (en dehors de toi)

La corrective est devoir identifier les ressource de lien avec la BD par des noms uniques et de les vérifier avant tout usage pour gérer les erreurs et optimiser le code en évitant de faire appel à des fonctions dépendantes de ressources absentes.

Je suggère ce qui suit :

if (!$db) echo 'Impossible de vous afficher le résultat';   
else {  
    $resultat = mysql_query ($sql, $db) or die ('Impossible de vous afficher le résultat'); 
    if ($resultat && mysql_num_rows($resultat)>0)  
      	while ($row = mysql_fetch_array($resultat)) { 
      		extract($row); 
      		//affichage de la ligne
			echo $nom.'<br />Version : '.$version.'<br />Type : '.$type.'<br />Age : '
				.$age.'<br />Style : <ul><li>'.str_replace(';','<li>',$style).'</ul><br /><br />'; 
   		} //fin while
  	else echo "<p>Aucun téléchargement n'est disponible pour le moment.";
}//fin if !$db

par **RinaBK** » 07 sept. 2006, 12:08

C'est juste !! Je ne pensais plus à ce % pour le jocker !! Heureusement que tu es là pour me reveiller un peu

Donc, suite à ça, j'ai modifié ma requête comme suit :

$sql ='SELECT * FROM download WHERE categorie="vetements" 
	AND (version LIKE "%'.implode('%" || version LIKE "%', $version).'%") 
	AND (type LIKE "%'.implode('%" || type LIKE "%', $type).'%") 
	AND (age LIKE "%'.implode('%" || age LIKE "%', $age).'%")
	AND (style LIKE "%'.implode('%" || style LIKE "%', $style).'%")
	ORDER BY id DESC';

..Et tu sais quoi ?? ça marche !!!! $\:D/$
Bé je dois te dire un super, méga grand MERCI !!! Car je n'aurais jamais réussi toute seule à trouver la bonne métode et en plus j'ai pu découvrire de nouvelles choses !!

Alors encore MERCI !!!

Par contre, pour ta correction sur le code PHP qui execute la requête... C'est tout à fait juste ta correction, car je l'ai apprise exactement comme la correction que tu as faite. Mais je me suis rendu compte que je me retrouvais avec des lignes et des lignes de code.. Et pour finir, j'y voyais plus rien du tout

Donc, j'ai trouvé une autre technique, qui pour l'instant à laire de fonctionner très bien. Sur ma page header.php j'ouvre ma db comme suit :

// Connexion à la db
$db = '';
if (!include('config.php')) { echo 'Impossible de trouver le fichier de configuration'; }
$db = mysql_pconnect(HOTE, NAME, PASSE) or die ('Impossible de se connecter à la db');
mysql_select_db(BASE, $db) or die ('Impossible de se connecter à la db');

ensuite, dans ma page footer.php je referme la connexion comme suit :

 mysql_close($db);

Une fois la connexion et la deconnexion sur la db faite, je peu travailler avec mes tables dans la page... (Bon, lorsque j'utilise cette technique, il est bien clair que je travaille seulement avec 1 seule base de donnée)
Donc, dans mes page, je n'ai plus qu'à faire le code suivant pour travailler sur la db :

$sql ='SELECT * FROM download';
$sql = mysql_query($sql, $db);
if (!$db) {
	echo 'Impossible de vous afficher le résultat';
}
else {
	while ($row = mysql_fetch_array($sql)) {
		extract($row);
		.....
	}
}

Ainsi mon code est beaucoup plus petit, le contrôle d'erreur sur la db est prenoncé au cas ou il y aurait un problème et les codes qui se placeront autour, seront nettement plus visible et accessible que si j'ai une connexion qui me prend 20 lignes en exagérant. Bon, comme j'ai pu le constater, cette méthode n'est pas la méthode classic et courrante, mais je trouve que lorsque l'on a beaucoup de ligne de code sur une page, c'est plus simple pour s'y retrouver et voir du premier coup d'oeil ce qui se passe...

Mais penses-tu que cette méthode peut comporter des problèmes suivant certains cas ?

par **sadeq** » 07 sept. 2006, 09:40

Oaw, t'as tout compris c'est très bien et en plus t'as tout bien modifié comme je l'ai imaginé.

En ce qui concerne la requête, c'est vrai que je n'ai pas pensé au problème que t'as soulevé :

si l’on a un vêtement qui a uniquement "sport" et un autre qui a "sport;soiree", si l’on sélectionne uniquement "sport" le vetement qui se présente sous "sport;soiree" ne s’affichera pas

Mais il est possible de le résoudre en profitant de LIKE pour lui demander de chercher le mot à l'intérieur de la chaîne "style" de la table.

Exemple :
La condition : style LIKE "%sport%"
Cherche le mot "sport" quelque soit sa position dans la chaîne "style" et c'est le caractère générique % qui modifie le comportement de LIKE ainsi.
Cette condition retourne vrai si :

style = "sport;soiree"
style = "sport"
style = "decontracte_bas;sport;soiree"
style = "decontracte_bas;maillotdebain;sport"

Et reste fausse si le mot n'existe pas dans la chaîne.

Maintenant, il faut modifier la requête en concéquence sans toucher ni à la base de données ni au formulaire ni au programme php.

Ceci dit je te passe tout ce que j'ai fait qui correspond exactement à ce que tu as fait.

Mais avant il faut que je revienne sur un morceau de ton code php qui me semble à réécrire dans la manière la plus courante.
Il s'agit de la partie php qui exécute la requête :

$sql = mysql_query($sql, $db);
echo ' '.$sql.' '.mysql_error().' ';
$fichierNB = mysql_num_rows($sql);
if ($fichierNB == 0) { ?>Aucun téléchargements n'est disponibles pour le moment.<? }
while ($row = mysql_fetch_array($sql)) {
extract($row);
echo $nom.' Version : '.$version.' Type : '.$type.' Age : '.$age.' Style : '.$style.' ';
}

Suggestion de reformulation :

//Pour le débogage : afficher la requête
echo '<hr>Débogage: (veuillez copier cette requête et l\'exécuter directement sous Mysql)<br><pre>'.$sql.'</pre><hr>';

//Connexion 
$con = mysql_connect("localhost", "root", "")or die ("Erreur: ".mysql_errno()." - ".mysql_error()); 
if ($con){
	//Ouverture de la base 
	$db = mysql_select_db("test", $con) or die ("Erreur: ".mysql_errno()." - ".mysql_error()); 
	if ($db){
		//Exécution de la requête
		$resultat = mysql_query($sql, $con) or die ("Erreur: ".mysql_errno()." - ".mysql_error()); 
		//si la requête est exécutée
		if ($resultat){
			//Nombre de lignes retournées
   			$fichierNB = mysql_num_rows($resultat); 
   			if ($fichierNB > 0) {
				//lire le résultat 
   			 	while ($row = mysql_fetch_array($resultat)) { 
      						extract($row); 
      						//affichage de la ligne
							echo $nom.'<br />Version : '.$version.'<br />Type : '.$type.'<br />Age : '
							.$age.'<br />Style : <ul><li>'.str_replace(';','<li>',$style).'</ul><br /><br />'; 
   				} //fin while
  			} else { echo "<p>Aucun téléchargement n'est disponible pour le moment."; } 
		}//fin if $resultat
	}//fin if $db
}//fin if $con

Voici donc ce que j'ai fait:

requete de recherche impossible a formuler

Répondre

Étendre la vue Revue du sujet : requete de recherche impossible a formuler