Récupérer une valeur à partir d'une requête SQL

**Zebulon777** · 06/12/2018, 14h20

Bonjour à tous,

J'ai un problème tout bête que je n'arrive pas à résoudre depuis une demi-journée.

Je fais une requête SQL qui me renvoie une seule valeur que je voudrai pouvoir utiliser dans le reste du script.

Le petit PHP :

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
21
22
 
<?php
	header('Content-Type: application/json; charset=UTF-8');
 
	 $svname = "...";
	 $usname = "...";
	 $passwd = "...";
	 $dbname = "...";
 
	$dbconn = mysqli_connect($svname, $usname, $passwd, $dbname)
		or die("Connexion impossible : " . mysqli_connect_error());
 
	$query = "SELECT min(year(ST_DATE_MOUV)) FROM stattot ORDER BY ST_DATE_MOUV";
	$result = mysqli_query($dbconn, $query)
		or die("Échec de la requête : " . mysqli_error($dbconn));
	$AnneeStattot = mysqli_fetch_all($result);
	mysqli_free_result($result);
 
	echo json_encode(array($AnneeStattot));
 
	mysqli_close($dbconn);
?>

Le code :

Code javascript :

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
21
22
23
24
25
26
27
28
29
30
31
32
$(document).ready(function () {
...
...
...
// récupération de la plus vieille année disponible dans la table
	var WAN = 0;
	deferred = $.get("./scripts/PHP/TraficJournalier_PLTF_PlusVielleAnnee.php");
	// Retour du serveur (traitement des données JSON)
	deferred.done(function (jsonData) {
		if (jsonData[0].length > 0) {
			WAN = parseInt(jsonData[0][0]); // Ici WAN contient bien une année.
		}
	});
			
	var WAnneeDeb = WAN; // Ici il est à 0 !!!
	i = WAnneeDeb;
	while (i <= yy) {
		$('#Liste_Annee').append($('<option>', {
			value: i,
			text: i
		}));
		i = i + 1;
	};
	var WAnnee = $('#Liste_Annee').val(yy); // Sélection de l'année en cours.
	
	//	dayInit=$('#Liste_Annee').val()+"0101"; // Indispensable pour la compatibilité
	dayInit = $('#Liste_Annee').val();

...
...
...

Je suis certain que ce doit être une grosse bêtise, mais j'ai trop la tête dans le guidon et je ne la trouve pas.

Merci d'avance pour votre aide.

**Celira** · 06/12/2018, 14h32

C'est un problème de concept, plus qu'un problème de code (et davantage JS que PHP, donc ce message va déménager vers le forum JS

)

L'idée principale d'un appel AJAX, c'est qu'il est Asynchrone. Autrement dit, on lance l'appel et on continue le reste du script sans attendre la réponse. La réponse est traitée à part, lorsqu'elle arrive, par un code séparé.

En l’occurrence, le code qui traite la réponse dans la fonction qui se trouve dans deferred.done(). Ce code, et donc l'affectation de WAN, sera effectué à l'arrivée de la réponse. Par contre le reste du code est exécuté tout de suite sans attendre la réponse.
En résumé :

tu lances l'appel,
tu utilises WAN,
la réponse arrive,
tu affectes WAN

Il faut donc que tu déplaces le code qui utilise WAN à l'intérieur de la fonction de traitement de la réponse.

Invité · 06/12/2018, 14h37

Bonjour,

Code php :

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

1
2
3
4
5
	$query = "SELECT MIN(YEAR(ST_DATE_MOUV)) AS min_year FROM stattot ORDER BY ST_DATE_MOUV";
	$result = mysqli_query($dbconn, $query)
		or die("Échec de la requête : " . mysqli_error($dbconn));
	$row = mysqli_fetch_assoc($result); // ou mysqli_stmt_fetch()
	$AnneeStattot = $row['min_year'];

N.B. Je connais mal mysqli_, mais il faudrait que tu regardes la doc de plus près : mysqli_ Interface procédurale et orientée objet

... et contrairement à Celira, je n'ai regardé QUE le code PHP...

**Zebulon777** · 06/12/2018, 14h56

Merci pour vos réponses et de vous pencher sur mon souci.

@Celira : j'y avais bien pensé, mais le souci, c'est derrière plus rien ne fonctionne ! arf !
Alors que si je n'utilise pas WAN, mais que j'affecte directement WAnneeDeb = 2014, là tout fonctionne !
Je comprends bien ce que tu m'expliques, mais je suis paumé quand à trouver une solution, sans tout casser.
Il n'y aurait pas moyen de passer WAN pour que ce soit utilisable dans la suite du code ?

@jreaux62 : merci, j'ai été voir, mais ça na solutionne pas mon problème ou alors je n'ai pas compris...

Invité · 06/12/2018, 15h56

Je n'avais regardé QUE le code PHP...
Mon code est une façon plus "conventionnelle" de récupérer UNE SEULE valeur.
Conserve le tien (sinon, il y aurait d'autres modif. à faire)

... mais le souci est en JavaScript (comme l'a bien vu Celira) :

Code JavaScript :

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

1
2
3
4
5
6
7
8
9
10
	var WAN = 0;
	deferred = $.get("./scripts/PHP/TraficJournalier_PLTF_PlusVielleAnnee.php");
...
	deferred.done(function (jsonData) {
...
			WAN = parseInt(jsonData[0][0]);
...
	});
 
	var WAnneeDeb = WAN;

1- $.get, comme tout appel Ajax, est ASYNCHRONE.

2- Quand tu affectes var WAnneeDeb = WAN;, WAN = 0 car la réponse de $.get n'est pas encore arrivée.

La solution est effectivement de REVOIR la logique de ton code.

**Zebulon777** · 07/12/2018, 08h45

Ok, je pense avoir compris.

Ca va être une question bête, je le sens

: Il n'y a pas un moyen pour passer en synchrone le temps de récupérer la valeur ?

Mais sinon, si j'ai bien compris, il faut que je fasse rentrer tout mon code DANS la fonction deffered.done ?

**popo** · 07/12/2018, 09h14

C'est pas forcément une bonne idée pour l'expérience utilisateur
Mais si tu tiens vraiment à rendre le processus synchrone.

Code jQuery :

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

1
2
3
4
5
6
7
8
9
10
11
var WAN = 0;
$.ajaxSetup({ async: false });
try {
    $.get("./scripts/PHP/TraficJournalier_PLTF_PlusVielleAnnee.php")
            .done(function (jsonData) {
                      WAN = parseInt(jsonData[0][0]);
            });
}
finally {
    $.ajaxSetup({ async: true });
}

**Zebulon777** · 07/12/2018, 09h33

Super, ça fonctionne impec.
Pour que ça serve au plus grand nombre, voici ce que j'ai fait :

Code javascript :

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
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
 
$(document).ready(function () {
 
	var day = new Date();
	day.setDate(day.getDate() - 1); // On sélectionne le jour précédent
	var mm = ((day.getMonth() + 1) < 10) ? '0' + (day.getMonth() + 1) : (day.getMonth() + 1);
	var dd = (day.getDate() < 10) ? '0' + day.getDate() : day.getDate();
	var yy = day.getFullYear();
	var dayInit = yy + mm + dd;
 
	var WAnnee = $('#Liste_Annee').val(yy); // Sélection de l'année en cours.
 
// récupération de la plus vieille année disponible dans la table
	var WAN = 0;
//	deferred = $.get("./scripts/PHP/TraficJournalier_PLTF_PlusVielleAnnee.php");
 
// Permet de passer en synchrone pour récupérer l'année la plus petite.
	$.ajaxSetup({ async: false });
	try {
		$.get("./scripts/PHP/TraficJournalier_PLTF_PlusVielleAnnee.php")
				.done(function (jsonData) {
						  WAN = parseInt(jsonData[0][0]);
				});
	}
	finally {
		$.ajaxSetup({ async: true });
	}
// FIN SYNCHRONE
 
	var WAnneeDeb = WAN;
	i = WAnneeDeb;
	while (i <= yy) {
		$('#Liste_Annee').append($('<option>', {
			value: i,
			text: i
		}));
		i = i + 1;
	};
	var WAnnee = $('#Liste_Annee').val(yy); // Sélection de l'année en cours.
 
console.log("WAnnee2="+WAnnee);	
 
	//	dayInit=$('#Liste_Annee').val()+"0101"; // Indispensable pour la compatibilité
	dayInit = $('#Liste_Annee').val();

Par contre, je veux bien de plus amples explications pour ce qui concerne l'expérience utilisateur ?

**NoSmoking** · 07/12/2018, 10h56

Bonjour,
tu as le chic pour choisir la meilleur solution qui est la pire

Structure ton code cela t'éviteras bien des problèmes de lisibilité, par exemple minimum

Code :

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

1
2
3
4
5
6
7
8
9
10
function getData() {
  $.getJSON("./scripts/PHP/TraficJournalier_PLTF_PlusVielleAnnee.php", function(jsonData) {
    // appel de la fonction gérant le graphe
    afficherGraphe(jsonData);
  });
}
 
function afficherGraphe(data) {
  // le code de ta fonction
}

**Zebulon777** · 07/12/2018, 11h09

Ha revoila mon non fumeur préféré

Oui, tu as raison, mais je vais au plus facile, c'est vrai, car je manque de temps.

Ta solution correspond bien à ce que j'avais compris (je crois), il faut que j'imbrique les fonctions concernées, ce que je mettrai en place dés que possible.

En tout cas, c'est toujours un plaisir d'apprendre comme ça, grâce à vous tous.

**NoSmoking** · 07/12/2018, 11h20

..., mais je vais au plus facile, c'est vrai, car je manque de temps.

Permet moi d'en douter

**Zebulon777** · 07/12/2018, 13h42

Rhooo, tu chipotes

Mais si, si, l'astuce de Popo fonctionne nickel ou alors, je n'ai pas encore découvert et retours de bâton ?

**Watilin** · 07/12/2018, 16h46

Envoyé par Zebulon777

l'astuce de Popo fonctionne nickel ou alors, je n'ai pas encore découvert et retours de bâton ?

Comme popo l’a dit, ça diminue la qualité de l’expérience utilisateur. Si le client a une mauvaise connexion, le gel de l’interface sera perceptible, et dans le pire des cas, le navigateur est inutilisable pendant plusieurs secondes. Ça fait un effet très désagréable.

On pourrait se dire qu’après tout, c’est pas si grave, mais il y a pire :

Les requêtes synchrones sont dépréciées et émettent des avertissements dans la console ;
Certaines des nouvelles fonctionnalités ajax ne supportent pas le mode synchrone, par exemple la propriété responseType.

Autrement dit, l’ajax synchrone n’est pas future-proof. Tu as peut-être l’impression de gagner du temps aujourd’hui, mais pour toi ou la personne qui devra maintenir le code, c’est du travail en plus, donc de la dette technique.

Avec la syntaxe ES2017, tu peux utiliser l’instruction await pour « linéariser » le traitement de la réponse ajax, comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
$(document).ready(async function () {
 
  ...
 
  // requête asynchrone
  // l’instruction await dit au script « d’attendre » l’arrivée de la réponse
  let jsonData = await $.get("./scripts/PHP/TraficJournalier_PLTF_PlusVielleAnnee.php");
  WAN = parseInt(jsonData[0][0]);
 
  ...
 
});

Quelques remarques sur ton code PHP :

Attention à toujours déclarer l’encodage de connexion avec mysqli_set_charset pour éviter les mauvaises surprises.

mysqli_fetch_all renvoie toujours un tableau de tableaux (chaque sous-tableau représentant une ligne de résultat). En l’occurence, tu ne veux qu’une seule ligne de résultat, donc tu peux utiliser mysql_fetch_assoc comme l’a indiqué jreaux62, ou même mysqli_fetch_row qui, dans le cas présent, est plus simple.

Dans ta requête SQL, tu n’as pas besoin de faire ORDER BY car tu récupères toujours un seul résultat. Si tu es obsédé par l’optimisation, tu peux toujours mettre un ORDER BY NULL à la place, mais je ne crois pas que ça change grand chose.

Tu passes un tableau à json_encode, ce qui te rajoute encore un niveau de tableaux. Ce dernier tableau est inutile, tu peux simplement le supprimer.
En fait, tu peux carrément envoyer l’année « nue » sans la formater en json.

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
21
22
23
header('Content-Type: text/plain; charset=utf-8');
 
$svname = '...';
$usname = '...';
$passwd = '...';
$dbname = '...';
 
$dbconn = mysqli_connect($svname, $usname, $passwd, $dbname)
    or die('Connexion impossible : ' . mysqli_connect_error());
 
mysqli_set_charset($dbconn, 'utf8mb4');
 
$query = 'SELECT min(year(ST_DATE_MOUV)) FROM stattot';
$result = mysqli_query($dbconn, $query)
    or die('Échec de la requête : ' . mysqli_error($dbconn));
 
$row = mysqli_fetch_row($result);
mysqli_free_result($result);
$AnneeStattot = $row[0];
 
echo $AnneeStattot;
 
mysqli_close($dbconn);

ET côté client :

Code JS :

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

1
2
let textData = await $.get("./scripts/PHP/TraficJournalier_PLTF_PlusVielleAnnee.php");
WAN = parseInt(textData);

Mais comme mysqli peut aussi envoyer des erreurs, tu dois gérer ce cas en vérifiant si la conversion en nombre a réussi :

Code JS :

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

1
2
3
4
5
6
7
8
9
10
if (Number.isNaN(WAN)) {
  // erreur mysqli
  alert(textData);
}
else {
  // on continue normalement le script
 
  ...
 
}

**Zebulon777** · 17/12/2018, 10h14

Merci beaucoup pour ces explications très claires que je vais m'efforcer de mettre en pratique ASAP.

**Zebulon777** · 17/12/2018, 13h31

Bon, ben ça ne fonctionne pas.
Je ne comprends pas tout ou je n'ai pas les yeux en face des trous ! C'est lundi

Voici ce que j'ai fait, tout bêtement pour commencer :

Code javascript :

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
21
22
23
24
 
$(document).ready( async function () {
 
	var day = new Date();
	day.setDate(day.getDate() - 1); // On sélectionne le jour précédent
	var mm = ((day.getMonth() + 1) < 10) ? '0' + (day.getMonth() + 1) : (day.getMonth() + 1);
	var dd = (day.getDate() < 10) ? '0' + day.getDate() : day.getDate();
	var yy = day.getFullYear();
	var dayInit = yy + mm + dd;
 
	var WAnnee = $('#Liste_Annee').val(yy); // Sélection de l'année en cours.
 
// récupération de la plus vieille année disponible dans la table
	var WAN = 0;
	// requête asynchrone
	// l’instruction await dit au script « d’attendre » l’arrivée de la réponse
	let textData = await $.get("./scripts/PHP/TraficJournalier_PLTF_PlusVielleAnnee2.php");
	WAN = parseInt(textData);	
 
console.log("WAN="+WAN);	
...
...
...
}

Le PHP :

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
21
22
23
24
25
26
27
28
29
30
31
32
33
34
 
<?php
	header('Content-Type: application/json; charset=UTF-8');
 
	 $svname = "...";
	 $usname = "...";
 	 $passwd = "...";
	 $dbname = "...";
 
	$dbconn = mysqli_connect($svname, $usname, $passwd, $dbname)
		or die("Connexion impossible : " . mysqli_connect_error());
 
//	$query = "SELECT min(year(ST_DATE_MOUV)) FROM stattot2 ORDER BY ST_DATE_MOUV";
//	$result = mysqli_query($dbconn, $query)
//		or die("Échec de la requête : " . mysqli_error($dbconn));
//	$AnneeStattot = mysqli_fetch_all($result);
//	mysqli_free_result($result);
 
//	echo json_encode(array($AnneeStattot));
 
	mysqli_set_charset($dbconn, 'utf8mb4');
 
	$query = 'SELECT min(ST_ANNEE_REF) FROM stattot2'; // ST_ANNEE_REF est un smallint
	$result = mysqli_query($dbconn, $query)
		or die('Échec de la requête : ' . mysqli_error($dbconn));
 
	$row = mysqli_fetch_row($result);
	mysqli_free_result($result);
	$AnneeStattot = $row[0];
 
	echo $AnneeStattot;
 
	mysqli_close($dbconn);
?>

Le console.log ne s'affiche jamais.

**Watilin** · 17/12/2018, 17h48

Tu as quelque chose dans l’onglet « réseau » de la console ?

**NoSmoking** · 17/12/2018, 18h42

Quel est ton environnement de test IE ?
• Async functions

**Zebulon777** · 18/12/2018, 08h09

Alors, dans l'onglet réseau je vois les différents temps d'accès aux modules, mais bizarrement je ne vois apparaitre aucun accès aux modules php ?

Je teste avec Chrome ou Firefox et j'ai le même résultat avec les 2.

Edit : j'ai oublié de préciser qu'avec l'ancienne méthode, ça fonctionne.

**Watilin** · 18/12/2018, 13h09

Tu devrais essayer de placer un console.log juste avant le await pour voir si le script arrive jusque-là.

J’ai une semi-hypothèse : la version de jQuery est trop ancienne et $.get ne renvoie pas de promesse. Mais même si c’était le cas, await recevrait un objet jqXHR et le parseInt donnerait NaN, donc tu verrais quand même quelque chose à la fin.

Ça m’étonne que tu ne voies ni requête, ni message d’erreur. Vérifie qu’il n’y a pas de filtres activés dans la console.

**Zebulon777** · 18/12/2018, 13h42

C'est le $(document).ready(async function () { qu'il n'aime pas, mais quand je l'enlève j'ai le message d'erreur Uncaught SyntaxError: await is only valid in async function.

J'ai mis un console.log avant et après le php et rien ne s'affiche quand je laisse le "async" de la fonction.

Edit : il n'y a aucun filtre.

Récupérer une valeur à partir d'une requête SQL

AJAX

Discussions similaires

Partager

Partager