switch pour calculette

Version imprimable

Bonjour,

Voila, on doit faire en cour un tp : une calculette en javascript.

Pour le moment, tout va bien, mais je suis à la phase débugage (au cas ou l'utilisateur entre ex-pré des mauvaise valeurs).

Bref la je fais la division par zéro qui évidament n'existe pas, et je ferais le reste sur le même principe.
Je fais en fait une fonction total qui va calculé ce que l'utilisateur a entré. Dans cette fonction il y a un switch qui va donc servir à faire ce débugage en fonction de ce que l'utilisateur a entré.

voila mon code (enfin juste la fonction total). C'est le switch qui "bug" (oui entre guillemet car je n'est aucune erreur).
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 function total(r){ switch(r){ case r.search("/0"):{ document.getElementById('calcul').value='Division par zéro impossible !'; break; } default:{ if(n1 != ''){ document.getElementById('calcul').value=Math.pow(n1, r); n1=''; } else{ document.getElementById('calcul').value=eval(r); }// fin du else } // fin du défault } }
En faite j'utilise la fonction search pour cherche si dans l'input de saisit il ya a "/0" (division par zéro). Mais évidement ça ne marche pas :(

J'ai essayé tout ça déjà :
Code:

1 2 3 4 5 6 case r.search("/0"):{} case r.search("/0")!=-1:{} case r.search("/0")<0:{} case r.search("/0")>0:{} case r.search("/0")!=0:{}
quelqu'un peut-il m'aider svp ?

11/11/2009, 17h51
SpaceFrog

Citation:

ex-pré

c'est un fitur parking ?
exprès ...

bref tu dois faire un switch (true)
11/11/2009, 17h55
FotoXe33

Oui c'est les correction google chromium et g pas fais gaffe autan pour moi...

Sinon un switch "true" c-a-d ?
11/11/2009, 18h10
SpaceFrog
Code:

1 2 3 function total(r){ switch(r){ case r.search("/0"):{
que va te retourner r.search("/0") ??? true ou false

ce n'est donc pas sur r qu'il faut swicther !!! mais sur true
Code:

1 2 3 function total(r){ switch(true){ case r.search("/0"):{
11/11/2009, 18h36
emmanuel.remy
Salut,

Citation:

Oui c'est les correction google chromium et g pas fais gaffe autan pour moi...

Comme quoi google chromium a des gros progrès à faire :roll:

Sinon, as tu lu la définition de la fonction search ?

Citation:

The search() method searches for a match between a regular expression and a string. This method returns the position of the match, or -1 if no match is found.

Donc tu comprends que ton switch a du plomb dans l'aile car quand tu écris
Code:

1 2 3 4 switch(r){ case r.search("/0"):{ ...
alors tu compare r qui est ton expression avec la position du /0. :aie:

Ensuite pour t'éviter un écueil supplémentaire, que penses tu du cas "3/0.5" ?

ERE
11/11/2009, 18h55
FotoXe33

Mais ce que tu me dis n'est pas possible ! je ne peut switcher sur true, car je n'aurais pas que ce test, j'aurais dans ce même switch environ un cinquantaine de test donc voila quoi :s
11/11/2009, 18h58
FotoXe33

Citation:

Envoyé par emmanuel.remy

alors tu compare r qui est ton expression avec la position du /0. :aie:

Ensuite pour t'éviter un écueil supplémentaire, que penses tu du cas "3/0.5" ?

ERE

A merde... Comment on "recherche" un ou plusieurs caractères alors ? et oui c'est vrai pour le 0.5, mais puis-je rechercher "/0_et_rien_d_autre_apres" ?
11/11/2009, 19h13
emmanuel.remy

Re,

Citation:

Pour le moment, tout va bien, mais je suis à la phase débugage (au cas ou l'utilisateur entre ex-pré des mauvaise valeurs).

A mon avis tu avais oublié les tests unitaires :scarymov: !

Citation:

Comment on "recherche" un ou plusieurs caractères alors ? et oui c'est vrai pour le 0.5, mais puis-je rechercher "/0_et_rien_d_autre_apres" ?

Cela s'appelle une expression régulière.

C'est un TP très formateur que tu fais :mouarf:

ERE
11/11/2009, 19h16
FotoXe33

AAAAAAAAAAAAA je hais ce mot c'est mon cochemard ! Pas les RegExp pitié... Bon allé heureusement qu'on a 4 semaine pour le faire =_=

Bon, mal grès moi j'ai utilisé des RegExp, mais ça ne marche pas plus...

Voila à présent ma fonction :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
function total(r){
		switch(r){
			case regue = new RegExp("$(/0)+", "gi"):{
			document.getElementById('calcul').value='Division par zéro impossible !'.fontcolor('red');
			break;
			}
			default:{
			 if(n1 != ''){
				document.getElementById('calcul').value=Math.pow(n1, r);
				n1='';
			 }
			 else{
				document.getElementById('calcul').value=eval(r);
			 }// fin du else 
			} // fin du défault
		}
}

Pas d'erreur de script, juste une fonctin qui marche pas =_=

12/11/2009, 13h32
SpaceFrog
tu as toujours un souci dans le case du switch

tu lui passes r il doit donc tester l'egalité entre r et le case ....
Code:

1 2 3 4 switch(r){ case ('coucou'): alert('r vaut coucou'); // le case teste si r=="coucou" break;
il faut dans ton cas faire un switch true !!!

switch(true){
case (r=='coucou'): alert('r vaut coucou'); // le case teste si r=="coucou"
break;[/CODE]
c'est a pru près la meêm chose sauf que le switch true te permet de tester autre chose que r==
12/11/2009, 20h04
FotoXe33

J'ai du mal a voir ce switch true, je suis débutant et j'ai vraiment du mal a concevoir ça... Mais sinon, ok pour le case "coucou" mais est-ce que cela marche avec des expressions régulières ? Car en JS il faut faire ce p**** de new RegExp dans une variable donc je suis un peu perdu...

il suffit que le test regExp retourne true ou false ...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
<script type ='text/javascript'>
function foo(bar){
switch (true){
				case (/[0-9]+/.test(bar)): alert('numérique');
				                         break;
				case (/[a-z]+/i.test(bar)): alert('alphabetique');
				                         break;
				case (/[*+-\/]+/.test(bar)): alert('operateur');
				                         break;
				default: alert('autre');                         
			 }
}
</script>
<body>
<input type='text' value='' id='saisie' />			 
<input type='button' value='go' onclick="foo(document.getElementById('saisie').value)"  />
</body>

13/11/2009, 02h01
kernelfailure

Javascript répond par "Infinity" à une division par zéro. Le résultat est donc isNaN. Ca ne peut pas te suffir ?

pseudo code
ev = eval(r)
if isNaN(ev) return "ERROR" else return ev
/pseudo code

Sinon, pour gagner en souplesse de contrôle, si tu as un peu de temps, penche toi sur le design pattern Interpreter pour réaliser ta calculatrice. Ca peut t'aider à envisager ton tp sous un autre angle.

ERRATUM :oops:
isNaN(100/0) => false
(100/0 == Infinity) => true

Infinity est une constante, -Infinity aussi. donc :
pseudo code
ev = eval(r)
if( isNaN(ev) || (Math.abs(ev) == Infinity)) return "ERROR" else return ev
/pseudo code
15/11/2009, 11h38
FotoXe33

Merci a vous deux, je vais continuer sur ces pistes et je tiens informé l'évolution pour ne pas ré-ouvrir 10000 post :p

Bon sa avance mais j'ai un problème avec la solution du switch(true) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 function total(r){ switch(true){ case ($(/0)+.test(r)):{ document.getElementById('calcul').value='Division par zéro impossible !'.fontcolor('red'); break; } default:{ if(n1 != ''){ document.getElementById('calcul').value=Math.pow(n1, r); n1=''; } else{ document.getElementById('calcul').value=eval(r); }// fin du else } // fin du défault } // fin du switch } // fin de la fonction
Voila le code firebug :

Citation:

ajout is not defined
[Break on this error] <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"\r\ncalculette.html (ligne 1)
eff is not defined
[Break on this error] <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"\r\ncalculette.html (ligne 1)
loga is not defined
[Break on this error] <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"\r\ncalculette.html (ligne 1)
caree is not defined
[Break on this error] <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"\r\ncalculette.html (ligne 1)
racine is not defined

Alors que lorsque j'enlève (ou que je commente) le switch, tout marche (sauf total =_=)

Une idée ?

15/11/2009, 13h03
marts

Curieuse façon de faire !
Pourquoi ne pas simplement utiliser les exceptions ? C'est fait pour.
15/11/2009, 14h13
FotoXe33

Qu'entends-tu par les "exceptions" ?
15/11/2009, 17h31
marts

Je voulais parler des blocs try-catch.
Mais je viens de me rendre compte que le javascript ne génère pas d'erreur en cas de division par 0 !
Donc oublies ça.
A mon avis la meilleure solution est celle de kernelfailure : une division par 0 donne Infinity comme résultat.
15/11/2009, 19h37
kernelfailure

Pas besoin des {} pour tes blocs case et default.

La syntaxe de ton regex curieuse :

Code:

/$(\/0)+/.test(r)

Ce sera déjà plus juste.
Mais ^ = debut
$ = fin

Que veux tu faire exactement ici ?
Ne serait-ce pas :

Code:

/\/[-]?[0\.]+$/.test(r)
15/11/2009, 20h37
FotoXe33

Ce que je souhaite faire dans ce regexp c'est vérifier si dans mon champ de texte il y a à la suite le signe "/" et le signe "0" mais rien d'autre derrière (la division par zéro quoi...).
Comme je n'y connais rien en RegExp, je suis allez voir un peu sur le net, et j'ai "pondu" ce code, qui si vous le dites doit être faux...
Quand à la syntaxe de mon switch, c'est le prof qui nous l'as donné :p il a du faire quelques fautes de frappes, ça lui arrive...
17/11/2009, 20h18
FotoXe33

La syntaxe que tu m'as donnée est bonne, mais je n'ai rien compris... Du coup, je cherche a reproduire une pour rechercher le mot "cos" tout attaché, mais avec possibilité qu'il y est des caractères avant et après...
18/11/2009, 01h49
kernelfailure

/.../ => RegExp
/\// => Qui commence avec un '/' (back slashed, bien sur)
/\/[-]?/ => Suivit du caractère '-' de 0 ou 1 fois
/\/[-]?[0\.]+/ => Suivit des caractères '0' et '.' (back slashed le '.') de 1 à n fois. Ici c'est pas tip top, car par exemple '00.0...00.' match :mrgreen:
/\/[-]?[0\.]+$/ => jusqu'à la fin

Match :
/0
/.0
/0.0
/00.00
/0.00
Aussi en négatif : /-0, /-.0 etc...

On backslash la plus part des caractères pour les distinguer des opérateurs.
Ici '/' est la bordure et '.' est 'any'.

Il y a une bonne explication des RegExp sous wiki ;)