Ma condition if / else ne fonctionne pas

Version imprimable

Bonjour

Je suis débutant en javascript.
Voilà, je récupère dans deux champs html input, les valeurs 'ms' et 'app'.
Lorsque je quitte le focus de 'app', je lance la fonction suivante :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 function control_exo() { var mass = document.getElementById('ms').value; var appr = document.getElementById('app').value; if ((mass > '90308')||((mass <= '90308')&&((appr == '0')||(appr == '')))) { calcul(); } else { confirm('Test'); } }
Mon problème : Pour toutes les conditions, j'obtiens le message confirm 'Test'.
Comprends pas... :cry:

10/10/2007, 10h59
SpaceFrog

tes comparaisons sont faites sur des stings pas sur des integers ... c'est normal ?
10/10/2007, 11h01
kryogen

hiii je suis débutant, c'est quoi des stings et des integers ? :oops:

encore faut il savoir ce qu'il y a dans tes variables et d'ou elle viennent
Code:

1 2 3 4 5 6 7 function control_exo() { var mass = Number(document.getElementById('ms').value); var appr = Number(document.getElementById('app').value); if ((mass > 90308)||((mass <= 90308)&&((appr == 0)||(appr == ''))))
integer = numérique entier naturel
string = alphabétique

ok merci j'ai compris :lol:

J'ai fait cela et ça fonctionne :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
function control_exo()
{
var mass = document.getElementById('ms').value;
var appr = document.getElementById('app').value;
 
if ((mass > parseInt('90308'))||((mass <= parseInt('90308'))&&((appr == parseInt('0'))||(appr == parseInt('')))))
  {
  calcul();
  }
 
else
  {
  confirm('Test');
  }
}

Y a peut être mieux ?

Merci en tout cas je me retrouve plus éclairé ! :salut:

10/10/2007, 11h14
SpaceFrog

:cry:

:roll:

dans le genre je prend le problème à l'envers :king:

:mouarf3:
10/10/2007, 11h23
kryogen

oups désolé je suis allé trop vite je n'avais pas vu ta soluce !
Merci :king:
10/10/2007, 11h24
Shinuza

Salut,

Qu'est ce que tu veux faire avec :

Code:

(mass > parseInt('90308'))||((mass <= parseInt('90308'))

Parce que l'expression dit que mass est plus grand que 90308 ou plus petit/egal que 90308, donc ça peut être n'importe quoi :aie:

Code:

((mass > 90308 || mass <= 90308) && appr == 0)

Marchera très bien, sachant que "" est casté en 0 dans une conversion string -> numérique.

Il reste plus qu'a savoir à quoi tu veux comparer "mass"
10/10/2007, 18h35
kryogen
Non, l'expression complète est :
Code:

1 2 ((mass > parseInt('90308'))||((mass <= parseInt('90308'))&&((appr == parseInt('0'))||(appr == parseInt('')))))
Ce qui est différent.

De toute façon mon problème est résolu ! yihha ! :yaisse2:
10/10/2007, 18h55
Shinuza

Bon, reprennons, ton expression est composée de deux sous expressions, la première fais ce que je décris dans mon post précedent, et tu verras que quelque soit la valeur de mass, le test fonctionne, donc la partie

Code:

((mass > parseInt('90308'))||((mass <= parseInt('90308'))

Ne sert à rien!
11/10/2007, 10h28
kryogen
bah non je vais dire pas d'accord.

C'est :
Code:

1 2 (mass > parseInt('90308'))
ou (||)
Code:

1 2 ((mass <= parseInt('90308'))&&((appr == parseInt('0'))||(appr == parseInt('')))
Regarde bien les parenthèses, ... ;)
11/10/2007, 18h39
Shinuza

Mea culpa, j'avais mal copié l'expression.

Par contre inutile de tester deux valeurs sur appr, une seul suffit :king: