Javascript: if-else statement in document.getElementById.innerHTML not working

Question

I am printing out values of a values-array in a div. I have the following code which is working very well:

...
...
document.getElementById("div").innerHTML = "Value: " + values[i];

<div id="div">The Value will be displayed here.</div>

However some of the values in my values-array are negative, but I would like to only show positive numbers of the array. So if a number is negative, it should say 0 instead.

I tried the following but its not working:

...
...
document.getElementById("div").innerHTML = "Value: " + if (values[i] < 0) {0} else {values[i]};

Any clues or hints? I am new to javascript, help is appreciated.

"Value: " + (values[i] < 0 ? 0 : values[i]); FYI - stackoverflow.com/questions/10270351/… — Cheery
– Cheery, Commented Nov 19, 2014 at 23:00
try document.getElementById("div").innerHTML = "Value: " + (values[i]<0)? 0:values[i]; — blurfus
– blurfus, Commented Nov 19, 2014 at 23:01

Oriol · Accepted Answer · 2014-11-19 23:03:17Z

3

Also consider Math.max:

document.getElementById("div").innerHTML = "Value: " + Math.max(0, values[i]);

answered Nov 19, 2014 at 23:03

Oriol

291k71 gold badges459 silver badges535 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Sudhir Bastakoti · Accepted Answer · 2014-11-19 23:01:57Z

3

use ternary operator, as:

var vals = (values[i] < 0 ) ? 0 : values[i];
document.getElementById("div").innerHTML = "Value: " + vals;

answered Nov 19, 2014 at 23:01

Sudhir Bastakoti

100k15 gold badges163 silver badges169 bronze badges

Comments

blurfus · Accepted Answer · 2014-11-19 23:05:41Z

2

if statements do not 'return' a value; they indicate a code execution path, as in:

if this happens
   do this
else
  do that

you want to concatenate the a value depending on a condition, in your case, the ternary operator fits nicely as they return a possible value given a condition

var returnedValue = (condition)? value-if-true : value-if-false;

In your case:

var myValue = (values[i] < 0)? 0 : values[i]; document.getElementById("div").innerHTML = "Value: " + myValue;

Or putting it all together:

document.getElementById("div").innerHTML = "Value: " + (values[i] < 0)? 0 : values[i];

answered Nov 19, 2014 at 23:05

blurfus

14.1k9 gold badges63 silver badges69 bronze badges

Collectives™ on Stack Overflow

Javascript: if-else statement in document.getElementById.innerHTML not working

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related