Discussion :

[Gannox]

Bonjour,

Je suis en train de faire des statistiques sur l'exploitation d'un portail captif et je me sers de requêtes SQL pour y parvenir. J'affiche tout ces résultats dans une page Web et je me sers occasionnellement d'AJAX pour l'affichage. C'est le cas juste ici :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
<?php
include "connexion2.php";
 
$sql = "SELECT DISTINCT username FROM radacct";
$req = mysqli_query($connect, $sql) or die('Erreur SQL : '.mysql_error());
 
echo '<form name="choix"><select name="utilisateurs" onchange="infos()">';
echo '<option value="">Sélectionner un nom</option>';
while ($row = mysqli_fetch_row($req))
{
echo "<option value='".$row[0]."'>".$row[0]."</option>";
}
echo '</select></form><br />';
echo '<div id="infos"></div><br />';
?>

Jusqu'ici, ça fonctionne, j'ai bel et bien mes utilisateurs dans ma liste déroulante. Je veux lors de la sélection d'un utilisateur, afficher le nombre de connexions de celui-ci.

Fonction JS :

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
function infos()
{
i = document.choix.utilisateur.selectedIndex;
if (i == 0)
{
return;
}
utilisateur = document.choix.utilisateurs.options[i].value;
xmlhttp = new XMLHTTPRequest();
xmlhttp.onreadystatechange = function() {
if (xmlhttp.readyState == 4 && xmlhttp.status == 200) {
document.getElementById("infos").innerHTML = xmlhttp.responseText;
}
}
xmlhttp.open("GET","test.php?username="+utilisateur,true);
xmlhttp.send();
}

test.php :

Code php :

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

1
2
3
4
5
6
7
8
<?php
include "connexion2.php";
$nomUtilisateur = $_GET['username'];
$sql = "SELECT COUNT(*) FROM radacct WHERE username = " + $nomUtilisateur + ";";
$req = mysqli_query($connect, $sql) or die('Erreur SQL : '.mysql_error());
$rows = mysql_fetch_array($req);
return $rows[0];
?>

Le problème est le suivant : je n'ai aucun affichage de résultat quel que soit l'utilisateur sélectionné. Merci de m'aider à la résolution de ce petit problème.

[Bovino]

Code php :

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

1
2
3
$req = mysqli_query($connect, $sql) or die('Erreur SQL : '.mysql_error());
$rows = mysql_fetch_array($req);
return $rows[0];

Heu... tu utilises mysqli ou mysql

Sinon, le return ne sert à rien : tu n'es pas dans une fonction et tu n'as aucun élément à qui renvoyer un résultat ! En revanche, comme tu n'affiches rien (echo), ben JavaScript ne récupère rien !

[Gannox]

J'ai bien compris la différence entre les 2 extensions PHP, j'ai également compris le principe du return ici inutile. Par contre, ce que je comprends moins, c'est quel echo montre un dysfonctionnement ?

A moins que ce ne soit un echo manquant dans la 3ème portion de code que j'ai inscrite dans le 1er post.

J'ai remplacé le return par un

Code :

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

echo $rows[0];

Ca ne change rien au problème apparemment.

[andry.aime]

Bonjour,

1-

$sql = "SELECT COUNT(*) FROM radacct WHERE username = " + $nomUtilisateur + ";";

Les types différents d'un entier doivent être entourés de quote dans une requête SQL et vire le ";" . Regarde aussi l'utilisation de aussi la fonction mysqli_real_escape_string.
2- Remplace les fonctions mysql_* par mysqli_*
3- Montre nous le code HTML généré de la page.

A+.

[Bovino]

Oh... j'avais pas vu ça !

Code php :

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

$sql = "SELECT COUNT(*) FROM radacct WHERE username = " + $nomUtilisateur + ";";

En PHP, la concaténation, c'est ".", pas "+"

[andry.aime]

En PHP, la concaténation, c'est ".", pas "+"

[Gannox]

Laquelle de ces 2 versions est la plus "convenable" ? Pour autant qu'elles soient syntaxiquement correctes (la 1ère me semble être la bonne)

Code php :

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

1
2
3
$sql = "SELECT COUNT(*) FROM radacct WHERE username = '".$nomUtilisateur."'";
 
$sql = "SELECT COUNT(*) FROM radacct WHERE username = '$nomUtilisateur'";

Sinon, voici ci-dessous le code HTML :

Code php :

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
19
20
<label>Nombre total blablabla :</label><br />
<label>Liste des utilisateurs : </label>
<?php
include "connexion2.php";
 
$sql = "SELECT DISTINCT username FROM radacct";
$req = mysqli_query($connect, $sql) or die('Erreur SQL : '.mysql_error());
 
echo '<form name="choix"><select name="utilisateurs" onchange="infos()">';
echo '<option value="">Sélectionner un nom</option>';
while ($row = mysqli_fetch_row($req))
{
echo "<option value='".$row[0]."'>".$row[0]."</option>";
}
echo '</select></form><br />';
echo '<div id="infos"></div><br />';
?>
<br /><br />
<hr />
<label>Autre statistique ... </label>

[andry.aime]

C'est du code PHP que tu donnes là, le code HTML généré c'est ce qui se trouve dans le source du naviagteur.

i = document.choix.utilisateur.selectedIndex;

echo '<form name="choix"><select name="utilisateurs" onchange="infos()">';

As-tu déjà remplace les fonctions mysql_* par mysqli_* aussi?
Et comme j'ai dit plus haut, essaie de voir la fonction mysqli_real_escape_string pour éviter les injections SQL.

A+.

[Gannox]

Bien vu pour les 2 citations mais c'est simplement une erreur de frappe, c'est correct sur le code lui-même.

J'ai déjà remplacé les mysql_* par des mysqli_*. J'ai également essayé ceci :

Code php :

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

1
2
3
4
5
6
7
8
9
10
<?php
$nomUtilisateur = $_GET['username'];
 
$nomUtilisateur = mysqli_real_escape_string($connect, $nomUtilisateur);
 
$sql = "SELECT COUNT(*) FROM radacct WHERE username = '".$nomUtilisateur."'";
$req = mysqli_query($connect, $sql) or die('Erreur SQL : '.mysqli_error());
$rows = mysqli_fetch_array($req);
echo $rows[0];
?>

Sans résultat. J'ai vérifié le code source et tout les éléments HTML s'affichent bel et bien là où je veux. Je montrerai ça le plus vite possible par la suite je bosse sur un autre PC en fait.

[andry.aime]

Bien vu pour les 2 citations mais c'est simplement une erreur de frappe, c'est correct sur le code lui-même.

fait un copier coller de ton code réel mais ne nous embrouille pas.

J'ai déjà remplacé les mysql_* par des mysqli_*. J'ai également essayé ceci :

Code php :

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

1
2
3
4
5
6
7
8
9
10
<?php
$nomUtilisateur = $_GET['username'];

$nomUtilisateur = mysqli_real_escape_string($connect, $nomUtilisateur);

$sql = "SELECT COUNT(*) FROM radacct WHERE username = '".$nomUtilisateur."'";
$req = mysqli_query($connect, $sql) or die('Erreur SQL : '.mysql_error());
$rows = mysql_fetch_array($req);
echo $rows[0];
?>

ça n'a pas l'air en tout cas.

A+.

[Gannox]

Second mauvais copier/coller toutes mes excuses.

[Gannox]

Je relance le sujet pour savoir si quelqu'un aurait une idée de l'erreur en question. De plus, où peut-on lire les erreurs php d'une page Web ? J'ai lu plusieurs articles dans lesquels il fallait modifier le fichier php.ini avec les champs error_log et log_errors et juste après ouvrir le fichier error.log. Mais malgré cela, je n'arrive toujours pas à afficher la moindre erreur (aucun affichage dans le fichier error.log).

Toute aide est la bienvenue

[TigerCX]

Pour les erreurs, simplement ajouter ceci en premier ligne de ton fichier php (après la balise <?php bien sur) : error_reporting(-1);

[Gannox]

C'est fait. Où seront donc affichées les erreurs du coup ? Dans un fichier texte ?

[Gannox]

Au final, problème résolu. Pour ceux que ça intéresse, j'avais écris "XMLHTTPRequest()" au lieu de la syntaxe correcte "XMLHttpRequest()". Autre erreur, mon navigateur n'a pas du tout aimé cette ligne de code :

Code :

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

xmlhttp.send();

Je l'ai remplacée par la ligne suivante :

Code :

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

xmlhttp.send(null);

Et le résultat que j'attendais s'est affiché sans problèmes.