Contrôle valeur numérique isNaN

Version imprimable

13/05/2008, 22h02
Kimado

Contrôle valeur numérique isNaN
Bonjour.

En tant que novice du Javascript je me trouves confronté à un nouveau problème :

J'ai un prompt que je récupère dans une variable "v" comme çà. Je veux que cette valeur soit numérique (décimale ou entière).

Pour le vérifier je fais :
Code:

1 2 3 if(isNaN(v)){ alert("BOUH C'EST PAS BIEN !!!"); }
Seulement si je tapes des espaces çà passe, comme si c'était un nombre.

J'aimerais savoir comment faire pour que justement çà ne passe plus. J'ai essayé le v.replace() pour les espaces mais le souci c'est que si je clique sur le "annuler" du prompt v est à null et j'effectue replace dessus donc mon firebug n'aime pas des masses.

J'ai pensé à tester la valeur avec un expression régulière mais j'ai peur que cela pose des problèmes au niveau des décimales pour la virgule qui est tantôt un point tantôt une virgule en fonction de la config du pc.

J'aimerais avoir votre avis et savoir s'il y'a un moyen plus simple de contrôler que c'est un nombre. J'ai cherché sur le net et sur ce forum mais sans succès.

Merci déjà.
13/05/2008, 22h40
htr999

RegExp ?.?
13/05/2008, 23h48
Sacha999
ca devrait faire l'affaire, ca remplace les espace par la letter 'Y' et donc devien forcement non numerique
Code:

1 2 3 4 if(v == null || v =='' || isNaN(v.replace(/\s/g,'Y'))){ alert("BOUH C'EST PAS BIEN !!!"); }

pour être sûr:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
<head>
<script>
function testNumber(num){
	var pattern = new RegExp("^[0-9]+(\.[0-9]+)?$","g");
	var result = pattern.test(num);
 
	if (num.length == 0)
		document.getElementById("affNum").innerHTML = "";
	else{
		if (result)
			document.getElementById("affNum").innerHTML = "ok";
		else
			document.getElementById("affNum").innerHTML = "Not ok";
	}
 
	return result;
}
</script>
</head>
 
<body>
	<input type="text" onkeyup="testNumber(this.value);" /> <span id="affNum"></span>
</body>

14/05/2008, 09h01
Kimado

Ok merci bien.
Je vais essayer tout çà et je vous retiens au courant :P
14/05/2008, 09h04
le_chomeur

+1 pour la réponse de htr999

J'ai essayé la solution de Sacha999, j'obtiens donc dans mon cas, ce code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 //On lit la premiere valeur tant qu'elle n'est pas numérique do{ v1=prompt(msg1); if(v1 =='' || isNaN(v1.replace(/\s/g,'Y'))){ alert('Ce champ doit être une valeur numérique'); } }while(v1 =='' || isNaN(v1.replace(/\s/g,'Y'))); //Si la premiere valeur est saisie (pas de clic sur "Annuler" ), on lit la deuxieme valeur if(v1!=null){ do{ v2=prompt(msg2); if(v2 =='' || isNaN(v2.replace(/\s/g,'Y'))){ alert('Ce champ doit être une valeur numérique'); } }while(v2 =='' || isNaN(v2.replace(/\s/g,'Y'))); }
Tout marche bien mais vu que je suis un peu maniaque, j'ai remarqué que mon deuxième petit problème était toujours là : lorsque je clique sur le bout "Annuler" de la première boîte il m'arrête bien le traitement seulement mon firebug me dit "v1 has no property" (pour le replace). Je pense que c'est normal parce qu'il vaut null mais je ne vois pas comment contourner le problème ?

14/05/2008, 09h17
le_chomeur

Code:

if(v1 && v1 =='' ...)

?
14/05/2008, 09h32
Kimado

Malheureusement çà ne fonctionne pas. :?
14/05/2008, 09h50
le_chomeur

Code:

if (v1 == null && v1 =='' ...)

?

J'ai essayé plein de méthodes, pleins de bidouillages, comme par exemple çà :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 do{ v1=null; ok=1; if((v1=prompt(msg1))==null){ v1="null"; } if((v1 =='' || isNaN(v1.replace(/\s/g,'Y'))) && v1!=null && v1!="null"){ alert('Ce champ doit être une valeur numérique'); ok=0; } }while(ok==0);
Mais la si je clique sur "Annuler" je boucle. :mur:

14/05/2008, 11h04
le_chomeur

en gros tu souhaites afficher le prompt tant que l'utilisateur n'a pas saisie une donnée ?
14/05/2008, 11h07
Kimado

Non, en fait je veux que si l'utilisateur clique sur ok il faut impérativement que la valeur soit numérique et non vide, dans ce cas j'affiche un deuxième prompt pour une deuxième valeur.
S'il clique sur annuler je ne veux pas afficher le second prompt et simplement que le 1er prompt se ferme.

essaye ceci :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
<script type="text/javascript">
function first(){
toto = prompt('Posez ici votre question','');
	if(toto == "titi"){
		second();
	}
	else if(toto != null){
		first();
	}
}
 
function second(){
titi = prompt('Posez ici votre 2 eme question question','');
if(titi == "toto"){
alert("genial");
}
}
 
first();
</script>

en remplaçant juste les if par ton test sur le champs non vide :)

Bien tenté mais le problème reste le même. :roll:

Mais je crois que y'a quelque chose qui cloche mais bien comme il faut. J'ai ce code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 //On lit la premiere valeur tant qu'elle n'est pas numérique var v1=null; do{ var ok="pouetpouet"; v1=prompt(msg1); if(v1!=null){ if(isNaN(v1.replace(/\s/g,'Y')) || (v1 =='')){ ok="prout"; alert('Ce champ doit être une valeur numérique'); } } alert(ok); }while(ok=="prout");
Le "alert(ok)" affiche bien "pouetpouet" mais pourtant la boucle continue. Je comprend vraiment pas la subtilité. Soit j'ai fait une erreur tellement énorme que je la vois pas soit c'est pas mon jour de chance mais y'a un truc.:bug:

EDIT : j'ai également essayé en remplacant "pouetpouet" par 1 et "prout" par 0 ou "pouetpouet" par true et "prout" par false. Même résultat.

essaye ceci :

Code:

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
 
function maQuestion()
{
  var rep1, rep2, out;
  var rexp = /\d+(.{0,1}\d+)+/gi;
 
  out = false;
  rep1 = null;
  rep2 = null;
 
  do
  {
    rexp.test("");
    rep1 = prompt("Entrez une valeur");
    if (rep1==null)
      out = false;
    else
      out = !rexp.test(rep1);   
  }while (out);
 
  if (rep1!=null)
  {
      out = false;
      do
      {
        rexp.test("");
        rep2 = prompt("Entrez une valeur");
        if (rep2==null)
          out = false;
        else
          out = !rexp.test(rep2);   
      }while (out);      
  }
 
  if (rep1!=null && rep2!=null && out)
  {
    //suite du traitement.............
  }
}

14/05/2008, 13h33
Kimado

Bon en fait c'est bon, çà marche impec' mais vu que j'ai tendance à être un peu boulet des fois (si c'était que des fois d'ailleurs). En fait j'avais une autre boucle externe à celle ci qui me rappelais la chose à chaque fois car j'ai oublié une ligne qui empêchait cette ré exécution.

:arf: :dehors:

En tout cas merci bien pour votre aide !:king: