Bug sous IE seulement !

**Sebe** · 29/01/2007, 23h50

Bonjour,

Il n'y pas si longtemps, je me suis lancé un défi ... celui de faire un composant pour Joomla!(CMS).

Et dernièrement, j'ai remarqué que j'avais un bug visible que sur IE ... n'étant pas un grand programeur, je viens à vous pour peut-être y trouver la solution !

Voyons d'abord mon composant fonctionner ici et c'est au moment de cliquer sur le bouton 'Evaluation' que l'erreur se passe:

Ligne: 262
Car: 2
Erreur: Cet objet ne gère pas cette propriété ou cette méthode
Code: 0

La ligne 262 est celle-ci:

Code :

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

1
2
 
test = datetest[2]+''+datetest[1]+''+datetest[0]; //alert ("La date du test " + test);

La fonction entière est:

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
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
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
// **************************** Fonction 'Apparition d'un bouton de soumission' ****************************
function valid(){
who = document.forms.test.iduser.value; //alert ("Id de l'utilisateur est " + who);
fcrecup = document.forms.test.fcrecup.value; //alert ("La FC après récupération est " + fcrecup);
recup = document.forms.test.recup.value; //alert ("Qualificatif de la récupération " + recup);
dateserveur = document.forms.test.dateserveur.value.split("-"); //alert ("La date du serveur est " + dateserveur);
serveur = dateserveur[2] +''+dateserveur[1]+''+dateserveur[0]; //alert ("La date du serveur " + serveur);
datetest = document.forms.test.datetest.value.split("-"); //alert ("La date du test est " + datetest);
test = datetest[2]+''+datetest[1]+''+datetest[0]; //alert ("La date du test " + test);
fcp1 = document.forms.test.fcp1.value; //alert ("La FC du palier 1 est de " + fcp1);
fcp3 = document.forms.test.fcp3.value; //alert ("La Fc au palier 3 est de " + fcp3);
vitesse1 = document.forms.test.vitesse1.value; //alert ("La vitesse au palier 1 est " + vitesse1);
vitesse2 = document.forms.test.vitesse2.value; //alert ("La vitesse au palier 2 est " + vitesse2);
vitesse3 = document.forms.test.vitesse3.value; //alert ("La vitesse au palier 3 est " + vitesse3);
vma = document.forms.test.vma.value; //alert ("La VMA est de " + vma);
vo2max = document.forms.test.vo2max.value; //alert ("La V02max est de " + vo2max);
pma = document.forms.test.pma.value; //alert ("La PMA est " + pma); // pma = ((85 + 70.0280112044818) /2);
 
if (test > serveur){
alert ("Le test ne peut s'être déroulé à une date qui n'a pas encore existé ! ! !");
document.forms.test.datetest.focus();
return(true);
}else if (fcp3 < 20 || fcp3 > 220) {
alert ("Vous n'avez pas introduit de valeur pour la FC au palier 3");
document.forms.test.fcp3.focus();
return;
} else {
fcM = fcp3; //alert ("La FCmax est de " + fcM);
document.forms.test.fcmax.readonly = false;
document.forms.test.fcmax.value = String( fcM );
document.forms.test.fcmax.readonly = true;
}
if ((vma <= 0) || (vo2max <= 0)) {
alert ("Avez-vous entré le temps au Palier 3 ?");
document.forms.test.tempsp3.focus();
return;
} else {
ticaint = (Math.round((pma*vma/100)*1000)/(3.6*(pma*vma/100)));
// alert ("La tica est de " + ticaint + " à mettre en heure !");
var heure = Math.floor(ticaint / 3600);
var minute = Math.floor((ticaint - (heure * 3600)) / 60);
var seconde = Math.floor(ticaint - (heure * 3600) - (minute * 60));
// alert (heure + ":" + minute + ":" + seconde);
heure = (heure < 10) ? "0"+heure : heure ;
minute = (minute < 10) ? "0"+minute : minute;
seconde= (seconde< 10) ? "0"+seconde : seconde;
tica1000 = heure + ":" + minute + ":" + seconde;
// alert ("Tica1000 est de " + tica1000);
document.forms.test.tica1000.readonly = false;
document.forms.test.tica1000.value = String( tica1000 );
document.forms.test.tica1000.readonly = true;
}
if ((recup != "Excellente") && (recup != "Très bonne") && (recup != "Bonne") && (recup != "Moyenne") && (recup != "Mauvaise") && (recup != "Très mauvaise") || (fcrecup <= 0)){
alert ("Avez-vous entré votre FC après 5' de repos ?");
document.forms.test.fcrecup.focus();
return;
}
if ((vitesse3 != " ") && (vitesse2 != " ") && (vitesse1 != " ") && (vitesse3 > vitesse2) && (vitesse2 > vitesse1)) {
//alert ("La FC du palier 1 est de " + fcp1);
//alert ("La Fc au palier 3 est de " + fcp3);
//alert ("La vitesse au palier 1 est " + vitesse1);
//alert ("La vitesse au palier 2 est " + vitesse2);
//alert ("La vitesse au palier 3 est " + vitesse3);
//alert ("La PMA est " + pma);
B30 = ((fcp3 - fcp1)/(vitesse3 - vitesse1));
//alert ("B30 est de " + B30);
B31 = (fcp3 - ((fcp3 - fcp1)/(vitesse3 - vitesse1)) * vitesse3);
//alert ("B31 est de " + B31);
fcca = (Math.round(B30*pma*(vitesse3/100)+B31));
//alert ("La valeur de la fcca est de "+ fcca);
document.forms.test.fcca.readonly = false;
document.forms.test.fcca.value = String( fcca );
document.forms.test.fcca.readonly = true;
}else{
alert ("Vous avez commis une erreur dans la prise de temps aux différents paliers ! ");
return (false);
}
if (who <= 0){
alert ("Malheureusement, vous n'êtes pas enregistré ... vous ne pourrez donc pas conserver vos données en mémoire et obtenir un plan d'entraînement !");
return;
}else {
document.getElementById('btn_val').style.visibility = 'visible';
}
}
// **************************** Fin de la fonction 'Apparition d'un bouton de soumission' ****************************

Peut-être aller vous pouvoir m'aider à résoudre mon problème ... merci

**Ricou13** · 30/01/2007, 00h31

Salut,

Voici plusieurs points qui peuvent t'aider à trouver ton bug :

1 - Je ne pense pas que la syntaxe "document.forms.test.xxx.value" soit bonne. "forms" est une collection (et non un objet classique) et doit donc être utilisé avec un index :

Code :

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

document.forms[0].xxx.value

Je crois que cette syntaxe fonctionne aussi :

Code :

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

document.forms["test"].xxx.value

2 - il y a plus simple pour récupérer tes données du formulaire. Tu modifies l'appel sur le onclick par

Code :

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

<input name="submit" type="button" onClick="valid(this.form);" value="Evaluation" >

Ce qui te permet d'envoyer directement une référence au bon formulaire. Ainsi tu ajoutes un paramètre à ta fonction

Code :

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

function valid(MyForm){

Et tes récupérations de données sont codées correctement et simplifiées :

Code :

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

1
2
3
4
who = MyForm.iduser.value; //alert ("Id de l'utilisateur est " + who);
fcrecup = MyForm.fcrecup.value; //alert ("La FC après récupération est " + fcrecup);
recup = MyForm.recup.value; //alert ("Qualificatif de la récupération " + recup);
...

3 - Pour ton pb (si le point précédent ne le résoud pas), il ne vient certainement pas de la ligne 262 mais d'une ligne précédente (voire celle d'au-dessus). Comme tu as mis plein d'alertes, tu les réactives et tu notes celle qui s'affiche en dernier (avant le bug). Ton erreur se situera entre la dernière alerte et la ligne indiquée (262). Ce qui va énormément simplifié la zone de recherche.

**Ricou13** · 30/01/2007, 00h43

Il se peut aussi que split ne soit pas acceptée parce que c'est une méthode d'un objet "String".
Or, dans ta syntaxe directe, il se peut qu'IE considère l'objet appelant split comme un objet "input" ou "tag" ou ...
De fait, ce ne serait pas un objet "String" et la méthode "split" ne peut être appliquée. D'où le message "[...]ne gère pas [...]cette méthode."

Teste quand même le point no 2 du post précédent (ça ne peut pas te faire de mal, au contraire), puis modifie l'appel du split comme suit

Code :

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

1
2
3
4
5
recup = MyForm.recup.value; //alert ("Qualificatif de la récupération " + recup);
ValeurDateServeur = MyForm.dateserveur.value;
dateserveur = ValeurDateServeur.split("-"); //alert ("La date du serveur est " + dateserveur);
serveur = dateserveur[2] +''+dateserveur[1]+''+dateserveur[0]; //alert ("La date du serveur " + serveur);
datetest = MyForm.datetest.value.split("-"); //alert ("La date du test est " + datetest);

En faisant ça, on crée un objet "String" : ValeurDateServeur, auquel il n'y a pas de raison de ne pouvoir appliquer la méthode "split"

**Sebe** · 30/01/2007, 11h35

J'ai fait tout ce qui a été demandé mais j'ai toujours l'erreur !
Voici le code avec les messages envoyés après l'alert:

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
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
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
function valid(MyForm){
	who = MyForm.iduser.value; //alert ("Id de l'utilisateur est " + who); //'62'
	fcrecup = MyForm.fcrecup.value; //alert ("La FC après récupération est " + fcrecup); //'99'
	recup = MyForm.recup.value;  //alert ("Qualificatif de la récupération " + recup); //'Excellente'
 
	ValeurDateServeur = MyForm.dateserveur.value; //alert ("La date du serveur (ValeurDateServeur) est " + ValeurDateServeur); //'30-01-2007'
	dateserveur = ValeurDateServeur.split("-"); //alert ("La date du serveur (dateserveur) est " + dateserveur); //'30,01,2007'
	serveur = dateserveur[2] +''+dateserveur[1]+''+dateserveur[0]; //alert ("La date du serveur " + serveur); //'20070130'
 
	ValeurDateTest = MyForm.datetest.value; alert ("La date du test (ValeurDateTest) est " + ValeurDateTest);//'30-01-2007'
	datetest = ValeurDateTest.split("-"); alert ("La date du test (datetest) est " + datetest); //'30,01,2007'
	test = datetest[2] + '' + datetest[1] + '' + datetest[0]; alert ("La date du test " + test); // Erreur Sous IE déclaré ici
 
	fcp1 = MyForm.fcp1.value; //alert ("La FC du palier 1 est de " + fcp1);
	fcp3 = MyForm.fcp3.value; //alert ("La Fc au palier 3 est de " + fcp3);
	vitesse1 = MyForm.vitesse1.value; //alert ("La vitesse au palier 1 est " + vitesse1);
	vitesse2 = MyForm.vitesse2.value; //alert ("La vitesse au palier 2 est " + vitesse2);
	vitesse3 = MyForm.vitesse3.value; //alert ("La vitesse au palier 3 est " + vitesse3);
	vma = MyForm.vma.value; //alert ("La VMA est de " + vma);
	vo2max = MyForm.vo2max.value; //alert ("La V02max est de " + vo2max); 
	pma = MyForm.pma.value; //alert ("La PMA est " + pma); // pma = ((85 + 70.0280112044818) /2);
 
	if (test > serveur){
		alert ("Le test ne peut s'être déroulé à une date qui n'a pas encore existé ! ! !");
		document.forms.test.datetest.focus();
		return(true);
		}else if (fcp3 < 20 || fcp3 > 220) {
			alert ("Vous n'avez pas introduit de valeur pour la FC au palier 3");
			document.forms.test.fcp3.focus();
			return;
			} else {
				fcM = fcp3; //alert ("La FCmax est de " + fcM);
				document.forms.test.fcmax.readonly = false;
				document.forms.test.fcmax.value = String( fcM );
				document.forms.test.fcmax.readonly = true;
			}
		if ((vma <= 0) || (vo2max <= 0)) {
			alert ("Avez-vous entré le temps au Palier 3 ?");
			document.forms.test.tempsp3.focus();
			return;
			} else {
				ticaint = (Math.round((pma*vma/100)*1000)/(3.6*(pma*vma/100)));
				// alert ("La tica est de " + ticaint + " à mettre en heure !");
				var heure = Math.floor(ticaint / 3600);
				var minute = Math.floor((ticaint - (heure * 3600)) / 60);
				var seconde = Math.floor(ticaint - (heure * 3600) - (minute * 60));
				// alert (heure + ":" + minute + ":" + seconde);
				heure = (heure < 10) ? "0"+heure : heure ;
				minute = (minute < 10) ? "0"+minute : minute;
				seconde= (seconde< 10) ? "0"+seconde : seconde; 
				tica1000 = heure + ":" + minute + ":" + seconde;
				// alert ("Tica1000 est de " + tica1000);
				document.forms.test.tica1000.readonly = false;
				document.forms.test.tica1000.value = String( tica1000 );
				document.forms.test.tica1000.readonly = true;
				} 
		if ((recup != "Excellente") && (recup != "Très bonne") && (recup != "Bonne") && (recup != "Moyenne") && (recup != "Mauvaise") && (recup != "Très mauvaise") || (fcrecup <= 0)){
			alert ("Avez-vous entré votre FC après 5' de repos ?");
			document.forms.test.fcrecup.focus();
			return;
			}
		if ((vitesse3 != " ") && (vitesse2 != " ") && (vitesse1 != " ") && (vitesse3 > vitesse2) && (vitesse2 > vitesse1)) {
			//alert ("La FC du palier 1 est de " + fcp1);
			//alert ("La Fc au palier 3 est de " + fcp3);
			//alert ("La vitesse au palier 1 est " + vitesse1);
			//alert ("La vitesse au palier 2 est " + vitesse2);
			//alert ("La vitesse au palier 3 est " + vitesse3);
			//alert ("La PMA est " + pma);
			B30 = ((fcp3 - fcp1)/(vitesse3 - vitesse1));
			//alert ("B30 est de " + B30);
			B31 = (fcp3 - ((fcp3 - fcp1)/(vitesse3 - vitesse1)) * vitesse3);
			//alert ("B31 est de " + B31);
			fcca = (Math.round(B30*pma*(vitesse3/100)+B31));
			//alert ("La valeur de la fcca est de "+ fcca);
			document.forms.test.fcca.readonly = false;
			document.forms.test.fcca.value = String( fcca );
			document.forms.test.fcca.readonly = true;
			}else{
		alert ("Vous avez commis une erreur dans la prise de temps aux différents paliers ! ");
		return (false);
		}
		if (who <= 0){
			alert ("Malheureusement, vous n'êtes pas enregistré ... vous ne pourrez donc pas conserver vos données en mémoire et obtenir un plan d'entraînement !");
		return;
		}else {
			document.getElementById('btn_val').style.visibility = 'visible';
			}
}

Il y a juste un truc qui me choque pour la ligne en question ... le message de l'alerte ne s'affiche pas (donc pas de valeur non plus) mais l'erreur indique la ligne suivante qui est vide !

Sinon, toujours aucun problème avec FF ... je n'ai pas essayé avec d'autres comme Opéra ! Je n'ai pas non plus fait la M-à-J sur le site (testé en local) mais je peux le faire s'il le faut !

A+

**Ricou13** · 30/01/2007, 11h43

oui mais tu as un bloc identique (avec ValeurDateServeur) juste au dessus et où les alert ne sont pas activées.

Donc, si ça plante au dessus, tu ne verras jamais tes alertes.

réactive-les à partir de "recup = "

L'alert de "recup" devrait fonctionner

**Ricou13** · 30/01/2007, 11h46

Au fait, tu ne devrais pas faire une déclaration préalable du type

Code :

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

1
2
var dateserveur = Array();
var datetest = Array();

pour lui dire qu'il s'agit de tableaux ?

**Sebe** · 30/01/2007, 12h37

Envoyé par Ricou13

oui mais tu as un bloc identique (avec ValeurDateServeur) juste au dessus et où les alert ne sont pas activées.

Donc, si ça plante au dessus, tu ne verras jamais tes alertes.

réactive-les à partir de "recup = "

L'alert de "recup" devrait fonctionner

Cela a été fait mais par étape ... j'ai mis après l'alert ce qu'il m'avait été renvoyé.

Bien que je ne pense pas que l'erreur puisse venir de plus haut, j'ai joint le fichier avec toutes les fonctions JS que j'utilise ... je suis complètement perdu sur ce truc (cela fonctionne avec FF et pas IE, pour une variable et pas une autre, etc)

En tout cas merci pour ce précieux coup de main !

**E.Bzz** · 30/01/2007, 15h18

... et +1 pour le split()

Bug sous IE seulement !

JavaScript

Vue hybride

Discussions similaires

Partager

Partager