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

[kryogen]

oups désolé je suis allé trop vite je n'avais pas vu ta soluce !
Merci

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

[kryogen]

Non, l'expression complète est :

Code :

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

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 !

[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 :

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

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

Ne sert à rien!

[kryogen]

bah non je vais dire pas d'accord.

C'est :

Code :

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

1
2
 
(mass > parseInt('90308'))

ou (||)

Code :

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

1
2
 
((mass <= parseInt('90308'))&&((appr == parseInt('0'))||(appr == parseInt('')))

Regarde bien les parenthèses, ...

[Shinuza]

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

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