Discussion :

[Sebe]

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]

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]

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]

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]

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]

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 ?

[Ricou13]

Bon, j'ai fait ce bout de code

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
<script type="text/javascript" language="javascript">
function Test(MyForm)
{
	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'
 
}
</script>
</head>
<body>
<form method="post" id="date" name="date" action="">
<input type="text" name="dateserveur" value="2007-01-20" />
<input type="button" value="Test" onclick="Test(this.form);" />
</form>
</body>
</html>

C'est le tiens auquel je n'ai rien changé. Ca marche nickel sous FF 2 et IE 6
La déclaration des tableaux n'est donc pas obligatoire.
Je me demande si ton erreur ne viendrais pas d'encore plus haut :
- vérifie bien toutes tes alertes précédentes
- et la correspondance des libelle des "name" des balises

Sur quelle version de IE ça ne marche pas ?

[Sebe]

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 !

[Sebe]

J'ai reçu par MP ceci:

Essayes avec ça :
*Méthode /substring()/*

La méthode /substring()/ permet de récupérer une sous-chaîne dans une chaîne de caractères en précisant en paramètres les positions des caractères entre lesquels la sous-chaîne doit être récupérée.

/ Chaine = "chaine de caractères";
Resultat = x.substring(position1, position2); /

Les positions indiquées en argument permettent de déterminer les position des caractère entre lesquels la sous-chaîne doit être récupérée. Les arguments /position1/ et /position2/ doivent être compris entre 0 et /n-1/.

J'ai beaucoup de mal à saisir le mode de fonctionnement puis, je ne sais pas si cela sera efficace !

Merci pour cette suggestion

[Ricou13]

Je dois faire la vidange de ma voiture.

je regarde ça après.

Mets moi aussi le fichier html, ça m'évitera de le créer et je serais sûr d'utiliser exactement le même code

[Sebe]

J'ai trouvé un peu par hazard ... en faisant des recherches sur Méthode substring()!

Il se fait que ce qui déconne c'est la variable 'test' ... je la ramplace par 'JourTest' cela fonctionne !

De là à dire pourquoi! J'ai une idée qui est que le nom du formulaire est 'test' mais c'est sans aucune certitude ! ! !

Je joins quand même mon fichier html au cas où il y aurait encore d'autres surprises cachées qu'oeil de lynx trouverait ... je n'ai pas le temps de faire la MàJ sur le serveur car je vais bosser mais ce soir après 22h00,je m'en occupe !

Merci pour tout

[Sebe]

Qu'est ce qui est le mieux ? string () ou substring () ?

Code :

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

1
2
3
4
 
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'
JourTest = datetest[2] + '' + datetest[1] + '' + datetest[0]; alert ("La date du test " + JourTest); // Erreur Sous IE déclaré ici

ou

Code :

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

1
2
3
4
5
ValeurDateTest = MyForm.datetest.value; alert ("La date du test (ValeurDateTest) est " + ValeurDateTest);//'30-01-2007'	
var jour = ValeurDateTest.substring(0,2); alert ("Le jour du test " + jour);
var mois = ValeurDateTest.substring(3,5); alert ("Le mois du test " + mois);
var annee = ValeurDateTest.substring(6,10); alert ("L'année du test " + annee);
JourTest = annee+mois+jour; alert ("La date du test " + JourTest);

C'est juste une question de qualité !

Merci

[E.Bzz]

Qu'est ce qui est le mieux ? string () ou substring () ?

Ni mieux ni moins bien :
string() convertit une variable de type X en string()
substring() extrait une sous-chaine d'une chaine (comme son nom l'indique, d'ailleurs)

A+

[Ricou13]

Il se fait que ce qui déconne c'est la variable 'test' ... je la ramplace par 'JourTest' cela fonctionne !

Je l'avais même pas vu !

"Il faut toujours faire l'effort de nommer ses variables avec des noms clairs et compréhensibles" pour :

- la compréhension du code
- la maintenance
- éviter les noms utilisés pour 2 destinations différentes
- comprendre directement ce que représente la variable

Personnellement, tous mes formulaires commencent par "frm" ("frmToto", "frmTata"...) et les seuls noms de variables que je m'autorise à réutiliser sont pour des variables dont la durée de vie est très courte : "Temp" pour un calcul, strSQL pour mes requètes SQL...

De plus, dans les fonctions il faut également déclarer ses variables pour qu'elles restent locales et qu'il n'y ait pas de conflit avec des variables globales.

Bref, une convention de nommage peut sembler plus lourde mais elle simplifie infiniment plus la maintenance du code

Voilà, c'était la pensée informatico-philosophique du jour

[Ricou13]

Personnellement, j'utiliserais plutot le "split"

- il fait la même chose en une seule ligne
- s'il manque un zéro non significatif dans la chaine de la date, par exemple "8-9-2007" pour le 8 septembre, le code fonctionnera toujours contrairement à substring

[E.Bzz]

"Il faut toujours faire l'effort de nommer ses variables avec des noms clairs et compréhensibles" pour :

- la compréhension du code
- la maintenance
- éviter les noms utilisés pour 2 destinations différentes
- comprendre directement ce que représente la variable

Si je puis me permettre d'abonder :

- ne jamais créer de conflit avec les mots réservés du langage (on est en plein dedans) : pas de <form name="form">, <select name="select">, var date = ... etcetcetc

A+

[E.Bzz]

... et +1 pour le split()

[Ricou13]

Si je puis me permettre d'abonder

heu... "abonder", ça à un rapport avec le "bondage" ?

Non parce que là, ça commence à dévier...

[E.Bzz]

heu... "abonder", ça à un rapport avec le "bondage" ?

Ben ça dépend essentiellement ..... du lecteur

[Sebe]

Personnellement, j'utiliserais plutot le "split"

- il fait la même chose en une seule ligne
- s'il manque un zéro non significatif dans la chaine de la date, par exemple "8-9-2007" pour le 8 septembre, le code fonctionnera toujours contrairement à substring

Oui mais comme je suis un vrai malade, j'ai une fonction qui replace la date '00-00-0000' peut-importe les entrées ! ! !

En tout cas merci à tous pour ce coup de main