probleme sur formulaire Résolu

Question

Bonjour,
J'ai un petit problème sur mon formulaire au niveau des calculs, comment faire pour que si le total de mon formulaire et inferieur a 60 on rajoute 5 au total
 mais si le resultat est supérieur a 60 on garde le total de départ.

Je mets ce bout de code car le code intégral est très long.
 
function total_com()
{

tot = 0;

if (document.acheter2.totsaut.value != "") {
tot = tot + parseInt(document.acheter2.totsaut.value);
 }
 else
 { tot = tot;}
 tot= tot + acompte();
document.acheter2.acompte.value = 30*document.acheter2.nbsaut.value + acompte();
document.acheter2.solde.value = tot-30*document.acheter2.nbsaut.value-acompte();
return tot;
}

Si quelqu'un pouvait m'aider ça serait vraiment cool :)
Je vous remercie par avance :)

Dalida · Answer

salut,

je te propose :
function total(val)
{
      if (Math.parseInt(val) > 60)
      {
            return parseInt(val) + 5;
      }
      else
      {
            return parseInt(val);
      }
}

Orkaell · Answer

Salut :)

Merci de repondre aussi rapidement :)
J'essaye ça tout de suite

Orkaell · Answer

J'ai incorporé ce bout de code dans mon script mais ça ne marche pas... 
Faut-il l'ajouter tout simplement ou bien remplacer certaine fonction ?  :-s

Dalida · Answer

non c'est une fonction toute bête qui ne fait qu'ajouter 5 à une valeur si elle dépasse 60.
il faut que tu l'appelles depuis l'endroit du code où tu as besoin de majorer ta valeur.

Orkaell · Answer

ça ne fonctionne pas :-/ 
Il doit y avoir certaine chose qui parasite, ou des termes differents pour le total et val.... ou peut etre que je l'ai mal inseré 

Tu as une idée ? ou peux-tu m'en dire plus ?

Dalida · Answer

explique moi ta fonction.
je n'ai pas compris ce que tu fais ni d'où vienne les valeurs.

Orkaell · Answer

c'est un formulaire de commande, chaque option a un prix, les prix sont cummulés puis calculés.
Mais si la commande et inferieure a 60€ le client paie des frais de port, donc dans ce cas là il faut rajouter 5€ a la totalité.

mon total et recuperé dans le bout de code que j'ai mis en debut de post je le remets ici 
function total_com()
{
tot = 0;
if (document.acheter2.totsaut.value != "") {
tot = tot + parseInt(document.acheter2.totsaut.value);
 }
 else
 { tot = tot;}
 tot= tot + acompte();
document.acheter2.acompte.value = 30*document.acheter2.nbsaut.value + acompte();
document.acheter2.solde.value = tot-30*document.acheter2.nbsaut.value-acompte();
return tot;
}

donc je pense que les 5e doivent être rajouté là dedans... Mais je ne sais aps comment faire :-s

ilan27 · Answer

Bonsoir, c'est normal, la fonction qu'a proposée Mathieu est correcte, mais ne retourne pas ce qu'il faut: la fonction renvoie +5 si c'est superieur à 60, or si j'ai bien compris c'est le contraire...:
Je la reprends  en corrigeant:

function total(val)
{
      if (Math.parseInt(val) < 60)
      {
            return parseInt(val) + 5;
      }
      else
      {
            return parseInt(val);
      }
}

Et tu dois appeler cette fonction:
tot=total(tot);

Orkaell · Answer

Oui c'est bien le contraire qu'il faut :)

Mais le problème c'est que je ne sais pas ou et comment l'appeler cette fonction :-s  
J'ai mis en place la fonction total(val) 
mais comment l'appeler dans mon code ? :-s

je suis désolée mes connaissance en script sont tres limité... j'ai su modifier ce qu'il fallait mais rajouter une fonction, là je suis perdue :-/

ilan27 · Answer

Cette fonction arrondit, je ne sais pas pourquoi, sinon tu as celle la:

function total(val)
{
      if (val > 60)
      {
            return (val + 5);
      }
      else
      {
            return val;
      }
}

Tu dois insérer cette fonction en haut des autres (du moment qu'elle est avant l'instruction qui l'utilisera), et tu dois insérer "tot=total(tot);"  je pense avant return tot; .

Dalida · Answer

regarde ce que t'écris ilan27 !
Et tu dois appeler cette fonction:
tot=total(tot);

donc si tu veux ajouter les frais de port à la fin tu codes :
tot=total(tot);
return tot;
ou en une seule ligne :
return total(tot);

ilan27 · Answer

Oui, j'avais mal compris le vrai sens de ParseInt, il faut en effet le mettre. Mais pour la double instruction, j'ai trouvé ça plus clair pour comprendre le truc, mais après, on peut tout réduire en effet, dans ce cas je pense que ceci marche aussi:

function total(val){
return Math.parseInt(val)>60? Math.parseInt(val): (Math.parseInt(val)+5);
}

Orkaell · Answer

WOW  merci bcp ça fonctionne comme il faut !! :)))

Un grand merci à tout les 2 :)

J'aurais une autre question
dans mon total les centimes ne s'affichent pas... y a une soluce pour palier au problème ?

ilan27 · Answer

aaaaah, je le savais bien  ;)
Essaye Math.parseDouble  a la place de Math.parseInt

Orkaell · Answer

hum.. je le mets ou ? :-))

ilan27 · Answer

A la place de ParseInt...:

function total(val) 
{ 
if (Math.parseDouble(val) < 60) 
{ 
return Math.parseDouble(val) + 5; 
} 
else 
{ 
return Math.parseDouble(val); 
} 
} 

Ca marche?

Orkaell · Answer

J'ai mis ça donc dans mes codes :
function total(val)
{
if (val < 60)
{
return (val + 5);
}
else
{
return val;
}
} 

function total_com()
{
tot = 0;
if (document.acheter2.totsaut.value != "") {
tot = tot + parseInt(document.acheter2.totsaut.value);
 }
 else
 { tot = tot;}
 tot= total(tot) + acompte();
document.acheter2.acompte.value = 30*document.acheter2.nbsaut.value + acompte();
document.acheter2.solde.value = tot-30*document.acheter2.nbsaut.value-acompte();
return tot;
}
Et là la fonction des frais de port fonctionne nikel, en revanche je n'ai pas les centimes qui s'affichent, et si j'essaie ce que m'a donné Ilan ça ne fonctionne plus :-s

Dalida · Answer

si cela ne fonctionne pas :
function total(val)
{
	if (typeof(val) == 'number')
	{
		if (val < 60)
		{
			return val + 5;
		}
		return val;
		}
	return 0;
	}
} 
on se contente de tester le type, retourne '0' en cas de saisie incorrecte.
j'ai virer les 'else' (inutiles puisque 'return' est une sortie de fonction).

ilan27 · Answer

Essaye comme ceci, j'ai rajouté des ".0" pour dire: ce sont des réels, il faut donc afficher quelque chose après la virgule


function total(val)
{
if (val < 60)
{
return (val + 5.0);
}
else
{
return val;
}
} 

function total_com()
{
tot = 0.0;
if (document.acheter2.totsaut.value != "") {
tot = tot + parseInt(document.acheter2.totsaut.value);
 }
 else
 { tot = tot;}
 tot= total(tot) + acompte();
document.acheter2.acompte.value = 30*document.acheter2.nbsaut.value + acompte();
document.acheter2.solde.value = tot-30*document.acheter2.nbsaut.value-acompte();
return tot;
}

Orkaell · Answer

J'ai essaye vos 2 proposition et ça ne fonctionne pas :-/

Dalida : si je remplace ta derniere proposition les totaux ne s'affichent plus

Ilan : si je mets se code là a la place de celui que j'ai ça ne change rien. Tjs pas de centimes

Dalida · Answer

je crois que j'm'ai trompé !
je n'ai jamais utilisé cette fonction...
function total(val)
{
	if ((typeof val) == 'number')
	{
		if (val < 60)
		{
			return val + 5;
		}
		return val;
		}
	return 0;
	}
}

Dalida · Answer

et pour forcer la valeur de retour en type Double ça doit être 'parseDouble()'.
function total(val)
{
	if ((typeof val) == 'number')
	{
		if (val < 60)
		{
			ret = val + 5;
		}
		ret = val;
		}
	ret = 0;
	}
	return parseDouble(ret);
}

ilan27 · Answer

Essaye comme ceci: (et les autres fonctions sont de quel ordre? tu es sûre qu'elles tournent bien?)

function total(val) 
{ 
if (val < 60.0) 
{ 
return parseDouble(val + 5.0); 
} 
else 
{ 
return parseDouble(val); 
} 
} 

function total_com() 
{ 
tot = 0.0; 
if (document.acheter2.totsaut.value != "") { 
tot = tot + parseInt(document.acheter2.totsaut.value); 
} 
else 
{ tot = tot;} 
tot= total(tot) + acompte(); 
document.acheter2.acompte.value = 30*document.acheter2.nbsaut.value + acompte(); 
document.acheter2.solde.value = tot-30*document.acheter2.nbsaut.value-acompte(); 
return tot; 
}

Orkaell · Answer

Bouhh ça ne marche tjs pas... :-( 
quand je mets ces codes les totaux ne s'affichent plus.

Dalida · Answer

salut,

en fait 'parseDouble()' n'est pas une fonction JS, il faut utiliser 'parseFloat()'.
voici une version qui fonctionne, j'ai adapté un peu.
et voila le Javascript.

par contre tu ne testes pas la valeur d'entrée donc si tu saisies des lettres à la place de la valeur : ça plante !

Orkaell · Answer

je viens de tester ta proposition dalida et les résultats ne s'affichent plus ! :-(

Dalida · Answer

oui désolé je faisais des modifs et j'ai mis en ligne une version qui ne fonctionnait pas !
tu peux essayer à nouveau, c'est corrigé !

Orkaell · Answer

J'ai essayé tjs pareil les totaux ne s'affichent plus ! :-(

Je commence a desespérer....

Dalida · Answer

vide le cahe de ton navigateur.
il y a une 'alert' (boite de dialogue) qui s'affiche.

Orkaell · Answer

Les résultats s'affichent dans IE mais pas dans FF mais le pb des centimes n'est tjs pas réglé

Dalida · Answer

tu parles de mon exemple ?

Orkaell · Answer

non qd j'integre ton dernier code proposé dans mon formulaire

Dalida · Answer

il faut adapter !!!
je ne sais pas comment tu as organisé ton code !
il faut que tu remplaces 'parseInt' par 'parseDouble' et pour attribuer des valeurs il faut utiliser 'document.getElementById()'.

Orkaell · Answer

J'ai fait ça :function total(val)
{
	if (val < 60.0)
	{
		return parseFloat(val + 5.0);
	}
	else
	{
		return parseFloat(val);
	}
}

function total_com()
{
	tot = 0.0;
	if (document.getElementById('totsaut').value != "") {
		tot = tot + parseFloat(document.getElementById('totsaut').value);
	}
	else
	{
		tot = tot;
	}
	tot= total(tot) + acompte();
	document.getElementById('acompte').value = 30*document.getElementById('nbsaut').value + acompte();
	document.getElementById('solde').value = tot-30*document.getElementById('nbsaut').value - acompte();

	return tot

Dalida · Answer

et 'acompte()' retourne quoi ?

Orkaell · Answer

J'ai ça apres le code donné audessus

function acompte()
{
	return 10;
}

Dalida · Answer

mais dans ton code à toi il y avait bien une fonction 'acompte()' ???!!!

j'ai ajouté ça car il n'y avait rien dans ton code !!!

dis moi exactement ce que tu as comme '<input>' dans ton formulaire, ce que tu attends comme valeurs saisies et les calculs qu'il faut effectuer.

Orkaell · Answer

Il y a plein de input et plein de valeur a récupérer... 
Je peux peut etre t'envoyer le fichier complet par mail , ça serait plus simple non ?

Orkaell · Answer

J'ai ça en code

Total de ma commande :

€

Acompte et frais versés à la commande :

€

Orkaell · Answer

J'ai ça que j'ai abrégé car c'est assez long
function acompte() {
valac=0;
	if (document.acheter2.totcarteA.value != "") {
valac =  valac + parseInt(document.acheter2.totcarteA.value);
}
else 
{ valac = valac; }
if (document.acheter2.totcarteB.value != ""){
valac=  valac + parseInt(document.acheter2.totcarteB.value);
}
else
{ valac =valac;}
return valac;	
}

ilan27 · Answer

Ca a l'air d'un détail, mais mets un ';' à la fin de return tot; peut-être que l'erreur est là

Orkaell · Answer

J'ai rajouté lle ; comme tu m'as dit, mais ça ne change rien... le total ne s'affiche tjs pas dans FF, dans IE ça s'affiche, mais tjs pas le signe de centimes :-s

ilan27 · Answer

Pour ce qui est du non-affichage sous FF, essaye de rédiger ainsi:
<input type="text" name="total" readonly="true" size="6" onfocus="javascript:total_com();">

Orkaell · Answer

pareil mm resultat dans les 2 browsers :-/

ilan27 · Answer

Ca alors!!, bon est-ce que ta fonction acompte() est avant la fonction total_com()? Sinon,mets la avant

Orkaell · Answer

J'ai passé la fonction acompte avec la fnction total et tjs pareil, ça passe sans les centimes sur IE mais dans FF pas de résultat ..... argghhh :-/

ilan27 · Answer

Ok, si tu peux, remets exactement les codes dans l'ordre, on y verra plus clair :)

Orkaell · Answer

C'est fait et c'est tjs le même problème... je désespère ça fait 2 jours qu'on est là dessus glups :-(

Orkaell · Answer

Tu ne veux pas que je t'envoie les fichiers pour que tu puisses les voir dans leur integralité ? et peut être que tu auras l'oeil ! :))

Orkaell · Answer

Voilà c'est envoyé :)

ilan27 · Answer

Ok. J'ai vu la fameuse page, et je pense que u devrais ecrire non pas "return tot", mais:
document.form....value=tot;  (là où il y a ton input text...)
car sinon ça ne s'affiche pas, enfin je pense.

Dalida · Answer

salut,

en fait il manquait les identifiants ('id="blabla"') sur les balises que appellées par 'document.getElementById()'.
et pour les virgules il fallait ajouter des 'parseFloat()' dans 'acompte()'.

ilan27 · Answer

Donc ça s'affiche correctement?

Orkaell · Answer

Bonjour  :)

Bon alors les résultats s'affichent et avec des virgules !!! un grand merci a vous ! :)

Cependant il n'affiche plus le bon resultat... :s 
Par exemple si je commande une option a 0,52 il me dit que je doit payer 6,04 alors que normalement le prix devrait etre a 5,52 (dont 0,52 de commande + frais de port a 5€ = 5,52 !) 

Coment ça se fait ça ??

Orkaell · Answer

ça vient peut etre des arrondies et de chiffres apres la virgule ?

Dalida · Answer

salut,

ça sent surtout l'erreur de calcul !
dans ton il vaudrait peut être mieux commencer par écrire tous les scénarios sur une feuille (acompte, pas acompte, soldes, quantités) et d'en déduire les actions principales et les actions redondantes pour ensuite coder les fonctions.

Orkaell · Answer

arghh.... mais ça peut venir d'ou ce probleme, car des fois ça calcule tres bien et des fois c'est aps ça ! :-s
alors qu'au debut tout etait juste

Orkaell · Answer

C'est bon j'ai trouvé l'erreur ! et tout est nickel a present !! WOW !! trop contente !! :-)))
Je vous remercie tous les 2 !!! et vous fait un gros bisous !!!! :)))))

Merci merci beaucoup  !!! ;-)

ilan27 · Answer

De rien! Bon weekend à tous les deux!

Orkaell · Answer

Merci, Bon week end a toi aussi :)

Orkaell · Answer

Bonjour, 

Je viens de me rendre compte qu'il y a un problème dans la fonction qu'on a rajouté pour les 5€ de frais de port 
Vous pouvez m'aider ? ça fait qq jours que je tourne en rond... :-(

En fait la fonction >60 on rajoute + 5 ne fonctionne seulement qu'avec la 1ere option choisie. et pour les autres options on a automatiquement 5 en plus même si la commande dépasse les 60€ 

Vous avez une idée d'où cela peut venir ?

ilan27 · Answer

Je n'ai pas compris: tu veux que ça ne marche que pour la  première option ou c'est justement le problème? Et c'est quoi ce que tu appelles la première option?
Par ailleurs, j'ai jeté un coup d'oeil, essayé, et on m'a affiché un solde négatif, c'est normal?

Orkaell · Answer

Que le solde soit négatif non c'est pas normal, et je voudrais arriver a ce que le >60 = +5 soit applique sur le total et non sur le 1er choix seulement.

Orkaell · Answer

Bonjour, 
Pus personne pour m'aider ? :-s
Je galère encore et tjs pour la même chose... :-(

Dalida · Answer

salut,

essaie de modifier la fonction 'total_com()'.
function total_com()
{
	tot = 0.0;
	if (document.getElementById('totsaut').value != "") {
		tot = tot + parseFloat(document.getElementById('totsaut').value);
	}
	tot = total(tot + acompte());
	return parseFloat(tot);
}

Orkaell · Answer

WOW ça marche !!!!! :-)))
Je viens de rentrer chez moi et j'ai de suite essayé tes modif et ça marche nickel !! :-)))
WOW je suis trop contente merci beaucoup !! ;-))

MERCI de ton aide précieuse ! ;-)

Je te souhaite un bonne nuit :)

Probleme sur formulaire

66 réponses

Discussions similaires

Newsletters