Un morceau de code semble bloquer la suite

**CinePhil** · 25/02/2019, 11h58

Bonjour,

Soit un formulaire d'inscription d'un étudiant qui comprend de nombreux champs.

Suite à la saisie du numéro d'étudiant ou de son numéro INE, je lance un script Ajax de récupération des informations déjà existantes en BDD.

J'arrive sans problème à récupérer et afficher un paquet d'informations mais l'une de ces récupérations semble bloquer la suite.

Première partie du code qui fonctionne :

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
45
46
function recupererEtudiant(champ)
{
	var chemin = baseChemin();
	var objData = {};
	var numEtudiant;
	var message;
	var adrel;
 
	if ($("#etudiantId").val() == "" && $("#" + champ).val().trim() != "")
	{
		// Si l'étudiant n'est pas déjà identifié 
		// et si le numéro ou l'INE de l'étudiant n'est pas vide, on cherche l'étudiant portant ce numéro
 
		numEtudiant = $("#" + champ).val().trim();
		objData = {numEtudiant: numEtudiant, param: champ};
 
		$.ajax({
			url: chemin + '/fr/Inscription/aj_recup_etudiant',
			data: objData,
			dataType: 'json',
			cache: 'false',
			type: 'POST',
			success: function (infos_etudiant)
			{
				/* On compare les infos déjà saisies aux infos récupérées
				 * On remplit les champs encore vides
				 * On affiche une alerte si un champ non vide a une valeur différente de l'information récupérée
				 */ 
 
				/* *** Identifiant de l'étudiant *** */
				$("#etudiantId").val(infos_etudiant.etuId);
 
				/* *** Nom usuel de l'étudiant *** */
				if($("#nomUsuel").val().trim() == "")
				{
					// Nom usuel vide : on affecte la valeur récupérée
					$("#nomUsuel").val(infos_etudiant.etuNomUsuel);
				}
				else if($("#nomUsuel").val().trim() != infos_etudiant.etuNomUsuel)
				{
					message = "Le nom usuel déjà saisi (" + $("#nomUsuel").val().trim()
						+ ") de l'étudiant n° " + numEtudiant
						+ " est différent du nom enregistré en base de données : " + infos_etudiant.etuNomUsuel;
					alert(message);
				}
// Une série d'autres récupérations du même style fonctionne

Ensuite, j'ai un sous-tableau dans le JSON qui permet de récupérer l'adresse électronique :

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
				/* *** Adrel *** */
				if(typeof infos_etudiant.adrel !== 'undefined')
				{
					// Une adrel trouvée pour l'étudiant
					adrel = infos_etudiant['adrel']['0'];
 
					if($("#adrel").val().trim() == "")
					{
						// Adrel non encore saisie => on affiche celle récupérée
						$("#adrel").val(adrel.adrelRacine);
						$("#idAdrel").val(adrel.adrelId);
						$("#domaineInternet").val(adrel.adrelDomaine);
						$("#domaineInternetId").val(addrel.adrelIdDomaine);
					}
					else if($("#adrel").val().trim() != adrel.adrelRacine)
					{
						message = "L'adresse électronique déjà saisie (" + $("#adrel").val()
							+ ") de l'étudiant n° " + numEtudiant
							+ " est différente de celle enregistrée en base de données : " 
							+ adrel.adrelRacine + "@" + adrel.adrelDomaine;
					}
				}
				alert("test");

La récupération fonctionne, l'adrel s'affiche bien mais le alert("test"); que j'ai placé n'affiche rien et la suite du code de récupération ne s'affiche pas.

Si je commente toute cette partie de récupération de l'adrel, le alert("test"); s'affiche et la suite du code s'exécute.

Je ne comprends pas ce qui peut bloquer.

Une idée à me soumettre ?

Invité · 25/02/2019, 13h41

Bonjour,

1- oui : regarde dans la console.

Ça raconte quoi ?

2- Mets d'autres console.log et/ou alert à l'intérieur des if, pour voir où passe le code.

N.B.

Code :

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

1
2
3
4
						message = "L'adresse électronique déjà saisie (" + $("#adrel").val()
							+ ") de l'étudiant n° " + numEtudiant
							+ " est différente de celle enregistrée en base de données : " 
							+ adrel.adrelRacine + "@" + adrel.adrelDomaine;

Si je comprends bien :
on peut taper un email bidon, et on obtient le VRAI EMAIL enregistré en BDD ??
Ca craint un peu, question "sécurité", non ?

Perso, je ne mettrais pas la dernière ligne...

**CinePhil** · 25/02/2019, 14h23

Ca craint un peu, question "sécurité", non ?

Sur le plan général, oui.
Sur le plan pratique, réel, ça reste une appli interne (inaccessible de l'extérieur) réservée aux personnes gestionnaires de nos publics en formation. Donc le risque est faible.

2- Mets d'autres console.log et/ou alert à l'intérieur des if, pour voir où passe le code.

C'est justement ce que j'ai fait, ce qui m'a permis de détecter que c'est ce morceau de code qui bloque la suite alors qu'il fonctionne normalement.

En l'occurrence, l'exécution passe par // Adrel non encore saisie => on affiche celle récupérée donc normalement ensuite on sort de if($("#adrel").val().trim() == "") puis de if(typeof infos_etudiant.adrel !== 'undefined') et ça devrait afficher l'alert mais non.

Invité · 25/02/2019, 14h29

1- Moi, quand je ne sais pas, je fais ça :

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
				/* *** Adrel *** */
console.log( 'typeof infos_etudiant.adrel : ' + typeof infos_etudiant.adrel );
				if(typeof infos_etudiant.adrel !== 'undefined')
				{
console.log( 'OK, infos_etudiant.adrel est défini' );
					// Une adrel trouvée pour l'étudiant
					adrel = infos_etudiant['adrel']['0'];
console.log( 'adrel : ' + adrel );
 
					if($("#adrel").val().trim() == "")
					{
console.log( 'OK, #adrel.val pas vide' );
console.log( 'adrel : ' + adrel );
console.log( 'adrel.adrelRacine : ' + adrel.adrelRacine );
console.log( 'adrel.adrelId : ' + adrel.adrelId );
console.log( 'adrel.adrelDomaine : ' + adrel.adrelDomaine );
console.log( 'adrel.domaineInternetId : ' + adrel.adrelIdDomaine );
						// Adrel non encore saisie => on affiche celle récupérée
						$("#adrel").val(adrel.adrelRacine);
						$("#idAdrel").val(adrel.adrelId);
						$("#domaineInternet").val(adrel.adrelDomaine);
						$("#domaineInternetId").val(addrel.adrelIdDomaine);
					}
					else if($("#adrel").val().trim() != adrel.adrelRacine)
					{
console.log( 'OK, #adrel.val différent de adrel.adrelRacine' );
						message = "L'adresse électronique déjà saisie (" + $("#adrel").val()
							+ ") de l'étudiant n° " + numEtudiant
							+ " est différente de celle enregistrée en base de données : " 
							+ adrel.adrelRacine + "@" + adrel.adrelDomaine;
console.log( 'message : ' + message );
					}
				}
				alert("test");

... et je regarde ce qui s'affiche dans la console.

2- D'autre part :

Code :

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

					else if($("#adrel").val().trim() != adrel.adrelRacine)

Ca ne devrait pas plutôt être :

Code :

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

					else if($("#adrel").val().trim() != adrel.adrelRacine + "@" + adrel.adrelDomaine)

?

3- Quant à la variable JS "message", tu ne l'affiches pas ?

**CinePhil** · 25/02/2019, 14h49

Trouvé !

$("#domaineInternetId").val(adrel.addrelIdDomaine); => adreIdDomaine avec un seul d ça fonctionne mieux !

**Doksuri** · 25/02/2019, 15h44

Salut,

je sais que le probleme est resolu, mais c'etait pour completer le "debuggage" de jreaux62
plutot que de faire des console.log('toto' + var); je passe par les arguments du log
=> concatener avec + si ta variable est un object ou un tableau => ca la castera en string, et tu n'auras que "object" comme info de debug

console.log('-------------------\ntoto', var); avec ca, tu as un dump de la variable, quelque soit son type, et pas non plus besoin de console.log() chaque element de ta variable

au lieu de :

Code javascript :

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

1
2
3
4
5
6
console.log( 'OK, #adrel.val pas vide' );
console.log( 'adrel : ' + adrel );
console.log( 'adrel.adrelRacine : ' + adrel.adrelRacine );
console.log( 'adrel.adrelId : ' + adrel.adrelId );
console.log( 'adrel.adrelDomaine : ' + adrel.adrelDomaine );
console.log( 'adrel.domaineInternetId : ' + adrel.adrelIdDomaine );

tu as :

Code javascript :

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

1
2
console.log( 'OK, #adrel.val pas vide' );
console.log( 'adrel : ', adrel );//et dans la console, tu peux "explorer" la variable

Un morceau de code semble bloquer la suite

AJAX

Vue hybride

Discussions similaires

Partager

Partager