Discussion :

[kryogen]

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 :

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
 
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...

[SpaceFrog]

tes comparaisons sont faites sur des stings pas sur des integers ... c'est normal ?

[kryogen]

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

[SpaceFrog]

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

Code :

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

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

[kryogen]

ok merci j'ai compris

J'ai fait cela et ça fonctionne :

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
 
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é !

[SpaceFrog]

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

[Shinuza]

Salut,

Qu'est ce que tu veux faire avec :

Code :

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

(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

Code :

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

((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"