Discussion :

[feelwatt]

Bonjour,
Pour faire un calcul, j'ai ceci
avec un OR (||)
mais la deuxi�me condition ne fonctionne pas

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

if (document.getElementById('height').value!='' || document.getElementById('height').value!=0) {h=document.getElementById('height').value;} else {h=1;}

Ce qui est �tonnant c'est que cela m'arrive en php aussi des fois mais pas toujours.
Quand ca ce passe je duplique le test avec un else if
Vous auriez une id�e ?

Merci

[danielhagnoul]

Il faut manipuler un nombre pas un texte.

Exemple :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
const elemHeight = document.querySelector('#height');
 
let 
    h = 0, // un Number !
    elemHeightValue = Number(elemHeight.value); // on transforme le String en Number
 
if (elemHeightValue && elemHeightValue !== 0) {
    h = elemHeightValue;
} else {
    h = 1;
}

[Beginner.]

La deuxi�me condition pour toi c'est document.getElementById('height').value!=0 ?

Je ne sais pas ce que tu entends par "la deuxi�me condition ne fonctionne pas" mais si comme je le pense document.getElementById('height').value est une string alors je ne comprends pas pourquoi tu la compares au nombre 0... Peut-�tre que tu veux la comparer avec la string "0" ? Si oui ben il manque juste les guillemets...

PS : Tu utilises plusieurs fois document.getElementById('height').value ce serait mieux de le stocker dans une variable ce serait plus rapide et peut-�tre m�me plus lisible car moins long...


EDIT : Ah Daniel a r�pondu.

[feelwatt]

Bonjour,
ok, merci des r�ponses.

- document.getElementById('height').value est bien un nombre.
- document.getElementById('height').value!=0 est bien la deuxi�me condition
- Je ne r�ussi pas � r�cup�rer la value avec

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

const elemHeight = document.querySelector('#height');

Faudrait-il faire ceci ?

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

const elemHeight = document.querySelector(document.getElementById('height').value);

- Oui, pour mettre les document.getElementById('....').value dans une variable (faut que j'en prenne l'habitude)

On peut faire ca ?

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
height=document.getElementById('height').value;
if (height == 0) {h=1;} else {h=height;}

Qui g�re la valeur 0 et la valeur '' (vide)

[Beginner.]

- document.getElementById('height').value est bien un nombre.

J'en doute, peux-tu nous dire quel est l��l�ment HTML qui a pour ID "height" ? Je pense que c'est une input ou un textearea, non ? Si oui alors il s'agit bien d'une string mais une string peut correspondre � un nombre bien s�r mais il faudra une conversion dans certains cas, cela d�pend de l'usage que tu veux en faire...

document.getElementById('height').value!=0 est bien la deuxi�me condition

Oui mais l� je pense que tu compares une string � un nombre et cela ne fonctionne pas... Quand dans une input (texte) tu as un 0, il s'agit de la string "0" et non du nombre 0.

Alors il te faut convertir l'un des deux... Par exemple tu peux faire comme d�j� dit : document.getElementById('height').value != "0"

Je ne r�ussi pas � r�cup�rer la value avec

const elemHeight = document.querySelector('#height');

Faudrait-il faire ceci ?

const elemHeight = document.querySelector(document.getElementById('height').value);

Ni l'un ni l'autre, tu peux faire simplement : const valueOfElemHeight = document.getElementById('height').value ;

Ou bien si tu fait : const elemHeight = document.getElementById('height') ;

Alors tu peux r�cup�rer "value" comme �a : elemHeight.value.

On peut faire ca ?

1 2	height=document.getElementById('height').value; if (height == 0) {h=1;} else {h=height;}

Qui g�re la valeur 0 et la valeur '' (vide)

Non mais dis-nous en fran�ais ce que tu veux faire...

Si height vaut 0 tu veux fixer h � 1 ? Sinon tu veux que h=height ?
Mais quand la chaine est vide, que veux-tu faire ?


Attention height pourrait ne pas �tre un nombre, je te conseille (si ce n'est d�j� le cas) d�utiliser une input de type number avec une valeur min (fix� � 1 puisque tu ne veux pas d'un height � 0) et �ventuellement une valeur max...

[SpaceFrog]

document.getElementById('height').value est bien un nombre.

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
+document.getElementById('height').value // est bien un nombre.
1*document.getElementById('height').value // est bien un nombre.

Si la saisie dans l'input est num�rique [0-9.] ....

[feelwatt]

Salut,
Merci pour les pr�cisions.

c'est une input ou un textearea, non ? Si oui alors il s'agit bien d'une string

D'accord, je n'avais pas pens� � �a.

Donc:

document.getElementById('height').value != "0"

Et j'opte donc pour

je te conseille (si ce n'est d�j� le cas) d�utiliser une input de type number avec une valeur min (fix� � 1 puisque tu ne veux pas d'un height � 0) et �ventuellement une valeur max...

Avec un min="0".
Comme �a �a �vite d'avoir � g�rer les chaines vide.

Merci pour ces pr�cisieux infos.