32 réponses
Bonjour,
Pour commencer, comme te l'a indiqué Yg_be, il faut correctement utiliser les balises de code
explications disponibles ici :
https://codes-sources.commentcamarche.net/faq/11288-les-balises-de-code
Ensuite, tu sembles utiliser l'ancienne extension mysql ... qui est désormais OBSOLETE.
http://www.commentcamarche.net/faq/43261-php-l-extension-mysql-est-obsolete#top
Il faut donc réécrire ton code en PDO ou en mysqli.
Il serait bien également d'utiliser l'écriture ternaire histoire de rendre ton code un peu plus lisible.
Par exemple;.. à la place de :
écrire :
et lorsque tu as besoin de IF/ELSE .. la bonne présentation du code est la façon suivante (avec les retours à la ligne... et l'indentation )
je t'invite également à lire et à appliquer le contenu de ce lien :
http://www.commentcamarche.net/faq/48399-php-gestion-des-erreurs-debogage-et-ecriture-du-code
Pour commencer, comme te l'a indiqué Yg_be, il faut correctement utiliser les balises de code
explications disponibles ici :
https://codes-sources.commentcamarche.net/faq/11288-les-balises-de-code
Ensuite, tu sembles utiliser l'ancienne extension mysql ... qui est désormais OBSOLETE.
http://www.commentcamarche.net/faq/43261-php-l-extension-mysql-est-obsolete#top
Il faut donc réécrire ton code en PDO ou en mysqli.
Il serait bien également d'utiliser l'écriture ternaire histoire de rendre ton code un peu plus lisible.
Par exemple;.. à la place de :
if($nbM!='0') {
$ok='1';
} else {
$ok='0';
}
écrire :
$ok = $nbM!='0' ? '1' : 0 ;
et lorsque tu as besoin de IF/ELSE .. la bonne présentation du code est la façon suivante (avec les retours à la ligne... et l'indentation )
if( condition ) {
//code si vrai
}else{
// code si faux
}
je t'invite également à lire et à appliquer le contenu de ce lien :
http://www.commentcamarche.net/faq/48399-php-gestion-des-erreurs-debogage-et-ecriture-du-code
Attends...
Tu me montre la fonction dbQuery ... mais toi tu continues à utiliser db->query ...
Alors qu'il est bien indiqué que c'est déprécié..
Ensuite.. dans ton code tu utilises du
sauf que... ta class dispose d'une fonction prévue pour ça
Donc... soit tu fais du mysql et tu n'as donc pas besoin d'utiliser cette usine à gaz ...
Soit tu souhaites l'utiliser.. mais dans ce cas tu le fais jusqu'au bout !
Pour finir, dans les liens que je t'ai donné, lorsque j'aborde la récupération des variables, j'utilise l'écriture TERNAIRE (au lieu de faire des IF/ELSE )
Typiquement.. au lieu d'écrire
ceci suffit :
Pour le reste je n'irai pas plus loin.
Le code que tu nous as posté ne respecte pas la moindre indentation ni aucune norme d'écriture de code.
(même si la norme PSR-2 / PSR-4 n'est pas une obligation.. ça permet quand même d'avoir un code à peu près lisible... )
Tu me montre la fonction dbQuery ... mais toi tu continues à utiliser db->query ...
Alors qu'il est bien indiqué que c'est déprécié..
Ensuite.. dans ton code tu utilises du
$nbM = mysql_num_rows($resMusic);
sauf que... ta class dispose d'une fonction prévue pour ça
// function to count the number of rows in a record set
function recordCount($rs) {
return $this->db->getRecordCount($rs);
}
Donc... soit tu fais du mysql et tu n'as donc pas besoin d'utiliser cette usine à gaz ...
Soit tu souhaites l'utiliser.. mais dans ce cas tu le fais jusqu'au bout !
Pour finir, dans les liens que je t'ai donné, lorsque j'aborde la récupération des variables, j'utilise l'écriture TERNAIRE (au lieu de faire des IF/ELSE )
Typiquement.. au lieu d'écrire
if(isset($_POST['type'])) { $types = $_POST['type']; } else { $type = 'tous'; }
ceci suffit :
$types = isset($_POST['type']) ? $_POST['type'] : 'tous';
Pour le reste je n'irai pas plus loin.
Le code que tu nous as posté ne respecte pas la moindre indentation ni aucune norme d'écriture de code.
(même si la norme PSR-2 / PSR-4 n'est pas une obligation.. ça permet quand même d'avoir un code à peu près lisible... )
Tutti
<?php
$host = "Mon serveur";
$user = "utilisateur";
$password = "mot de passe";
$bdd = "ma base de donnée";
mysql_connect($host, $user, $password) or die ("Connexion au serveur impossible");
// on choisit la bonne base
mysql_select_db($bdd) or die ("Connexion a la base impossible");
$nom = $_POST['nom'];
echo "
<html>
<head>
<title>Résultat de la recherche</title>
</head>
<body>";
if (($Mot == "")||($Mot == "%")) {
// Si aucun mot clé n'a été saisi,
// le script demande à l'utilisateur
// de bien vouloir préciser un mot clé
echo "
Aucun résultat de recherche. Élargissez votre requête pour obtenir plus de résultats.
<p>";
}
else {
// On selectionne les enregistrements contenant le mot clé
$query = "SELECT distinct nom FROM membre
WHERE nom LIKE \"%$nom%\"
";
$result = mysql_query($query);
$row = mysql_fetch_row($result);
$Nombre = $row[0];
// Si aucun enregistrement n'est retourné,
// on affiche un message adéquat
if ($Nombre == "0") {
echo "
<h2>Aucun résultat ne correspond à votre recherche</h2>
<p>
";
}
// Sinon, on affiche le nombre d'enregistrements correspondant
// et les résultats eux-mêmes
else {
$query = "SELECT distinct nom, titre FROM membre
WHERE nom LIKE \"%$nom%\"
ORDER by titre ASC";
$result = mysql_query($query);
// Si un seul enregistrement est trouvé, on affiche un message au singulier
if ($Nombre == "1") {
echo "
<a name=\"#resultat\"><h2>Résultat: Un article trouvé</h2></a>
<p>";
}
// Dans le cas contraire le message est au pluriel...
else {
echo "
<a name=\"#resultat\"><h2>Résultat: $Nombre articles trouvés</h2></a>
<p>";
}
while($row = mysql_fetch_row($result))
{
echo "
<p>\n
<b>$row[2]</b>\n
<br><a href=\"../$row[0]\">Visualiser l'article</a>\n
<p>\n
";
}
}
}
// on ferme la base
mysql_close();
?>
</body>
</html>
J'ai essayé d'adapter mon code en sql. Je n'ai plus de problème d'erreur. Cependant quand je tape un mot ( il m'affiche tous le temps " Aucun résultat ne correspond à votre recherche. ( est ce que j'ai oublié une requète ? )
PS : j'ai limité mon retour que au nom
Visiblement ... tu n'as pas envie de suivre les consignes qu'on te donne.
Je t'ai "fortement" invité à lire ET à appliquer le contenu de ce lien : http://www.commentcamarche.net/faq/48399-php-gestion-des-erreurs-debogage-et-ecriture-du-code
.. mais tu ne sembles pas décidé à le faire...
Tu as également de grosses lacunes sur l'écriture du code html en plus du code php...
bref... ton code devrait ressembler à ceci :
Cordialement,
Jordane
Je t'ai "fortement" invité à lire ET à appliquer le contenu de ce lien : http://www.commentcamarche.net/faq/48399-php-gestion-des-erreurs-debogage-et-ecriture-du-code
.. mais tu ne sembles pas décidé à le faire...
Tu as également de grosses lacunes sur l'écriture du code html en plus du code php...
bref... ton code devrait ressembler à ceci :
<?php
$host = "Mon serveur";
$user = "utilisateur";
$password = "mot de passe";
$bdd = "ma base de donnée";
mysql_connect($host, $user, $password) or die ("Connexion au serveur impossible :" . . mysql_error());
mysql_select_db($bdd) or die ("Erreur :" . mysql_error());
// récupération PROPRE des variables AVANT de les utiliser
$nom = !empty($_POST['nom']) ? $_POST['nom'] : NULL;
if(!$nom){
$error = "Vous devez rensigner un mot pour la recherche";
} else {
// On selectionne les enregistrements contenant le mot clé
$query = "SELECT distinct nom
,titre
FROM membre
WHERE nom LIKE '%$nom%'
ORDER BY titre ASC";
$result = mysql_query($query) or die('Requête invalide : ' . mysql_error());
$Nombre = mysql_num_rows($result); //Retourne le nombre de lignes d'un résultat MySQL
// Si aucun enregistrement n'est retourné,
// on affiche un message adéquat
if ($Nombre == "0") {
$error = "<h2>Aucun résultat ne correspond à votre recherche</h2>";
}
?>
<html>
<head>
<title>Résultat de la recherche</title>
<meta charset="utf-8">
</head>
<body>
<?php
if(!empty($error)){
echo "<p><span class='error'>".$error."</span></p>";
}else{
$txtNbResult = ($Nombre==1) ? "Un article trouvé" : "$Nombre articles trouvés " ;
echo "<p><a name=\"#resultat\"><h2>Résultat: " . $txtNbResult ."</h2></a></p>";
while($row = mysql_fetch_row($result)) {
echo "<div id='resultat'><p>
<b>".$row['titre']."</b>\n
<br><a href=\"../".htmlspecialchars($row['nom'])".\">".htmlspecialchars($row['titre'])."</a>
</p>
</div>";
}
}
// on ferme la base
mysql_close();
?>
</body>
</html>
Cordialement,
Jordane
Tu ne veux toujours pas appliquer les consignes pour poster ton code ??
Il faut utiliser les BALISES DE CODE !!!
Grrrrr !!!!
Mais non.. je ne pense pas que ça soit cette ligne qui pose problème
Sûrement celle d'avant...
Au pire, tu peux déjà la remplacer par :
Il faut utiliser les BALISES DE CODE !!!
Grrrrr !!!!
Mais non.. je ne pense pas que ça soit cette ligne qui pose problème
Sûrement celle d'avant...
Au pire, tu peux déjà la remplacer par :
echo '<p><a name="#resultat"><h2>Résultat: ' . $txtNbResult . '</h2></a></p>';
j'ai rajouté des points à la variable Nombre
aucun interet.
Par contre... tu n'as pas fait la modification que je t'ai donné....( donc à quoi ça sert que je te réponde ??!! )
Si ça ne résout pas le problème... peux tu nous reposter l'intégralité de ton fichier ?
Pense aussi à vérifier que le fichier est bien encodé en utf8 sans bom ( voir chapitre 1 de ce lien pour savoir comment faire : http://www.commentcamarche.net/faq/47069-html-php-caracteres-accentues-et-l-utf8 )
Tutti
J'ai essayé comme tu m'as dis mais ça marchais pas (c'est pour cela que j'avais laissé comme cela). ( j'ai regardé au niveau de l'encodage et tout est bien en utf 8 sans bom.
<?php
$host = "Mon serveur";
$user = "utilisateur";
$password = "mot de passe";
$bdd = "ma base de donnée";
mysql_connect($host, $user, $password) or die ("Connexion au serveur impossible :" . . mysql_error());
mysql_select_db($bdd) or die ("Erreur :" . mysql_error());
// récupération PROPRE des variables AVANT de les utiliser
$nom = !empty($_POST['nom']) ? $_POST['nom'] : NULL;
if(!$nom){
$error = "Vous devez rensigner un mot pour la recherche";
} else {
// On selectionne les enregistrements contenant le mot clé
$query = "SELECT distinct nom ? titre
FROM membre
WHERE nom LIKE '%$nom%'
ORDER BY titre ASC";
$result = mysql_query($query) or die('Requête invalide : ' . mysql_error());
$Nombre = mysql_num_rows($result); //Retourne le nombre de lignes d'un résultat MySQL
// Si aucun enregistrement n'est retourné,
// on affiche un message adéquat
if ($Nombre == "0") {
$error = "<h2>Aucun résultat ne correspond à votre recherche</h2>";
}
?>
<html>
<head>
<title>Résultat de la recherche</title>
<meta charset="utf-8">
</head>
<body>
<?php
if(!empty($error)){
echo "<p><span class='error'>".$error."</span></p>";
}else{
$txtNbResult = ($Nombre==1) ? "Un article trouvé" : "$Nombre articles trouvés " ;
echo '<p><a name="#resultat"><h2>Résultat: ' . $txtNbResult . '</h2></a></p>';
while($row = mysql_fetch_row($result)) {
echo "<div id='resultat'><p>
<b>".$row['titre']."</b>\n
<br><a href=\"../".htmlspecialchars($row['nom'])".\">".htmlspecialchars($row['titre'])."</a>
</p>
</div>";
}
}
// on ferme la base
mysql_close();
?>
</body>
</html>
Je viens de voir une erreur dans ton code
Tu as un point d'interrogation à la place d'une virgule
A remplacer par :
C'est étonnant qu'il ne t'ait pas affiché une erreur....
Car ben.. c'est bien grace à cette requête que tu affiches la liste des résultats trouvés...
via cette boucle
Tu as un point d'interrogation à la place d'une virgule
$query = "SELECT distinct nom ? titre
FROM membre
WHERE nom LIKE '%$nom%'
ORDER BY titre ASC";
A remplacer par :
$query = "SELECT distinct nom , titre
FROM membre
WHERE nom LIKE '%$nom%'
ORDER BY titre ASC";
C'est étonnant qu'il ne t'ait pas affiché une erreur....
Car ben.. c'est bien grace à cette requête que tu affiches la liste des résultats trouvés...
via cette boucle
while($row = mysql_fetch_row($result)) {
echo "<div id='resultat'><p>
<b>".$row['titre']."</b>\n
<br><a href=\"../".htmlspecialchars($row['nom'])".\">".htmlspecialchars($row['titre'])."</a>
</p>
</div>";
}
