I want to addClass to element if value less than zero, value is negative number -0.3, but what i tried so far not working.

var rate = $('#view-ad-rating');
var text = parseInt($(rate).text());

if (text < 0) {
  $(rate).addClass('viewAdRateBad');
} else {
  //$(rate).addClass('viewAdRateGood');
}

.viewAdRateBad {
  color: red;
}

.viewAdRateGood {
  color: green;
}

<script src=".1.1/jquery.min.js"></script>
<a id="view-ad-rating">-0.3</a>

I want to addClass to element if value less than zero, value is negative number -0.3, but what i tried so far not working.

var rate = $('#view-ad-rating');
var text = parseInt($(rate).text());

if (text < 0) {
  $(rate).addClass('viewAdRateBad');
} else {
  //$(rate).addClass('viewAdRateGood');
}

.viewAdRateBad {
  color: red;
}

.viewAdRateGood {
  color: green;
}

<script src="https://ajax.googleapis./ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<a id="view-ad-rating">-0.3</a>

I also tried this but no success:

if (text <= -1) {

How can solve this?

• javascript
• jquery

Share Improve this question Follow asked Jan 10, 2018 at 16:13 Jack The BakerJack The Baker 1,90333 gold badges2727 silver badges6464 bronze badges 18

• 5 Try parseFloat() instead of parseInt(). The "Int" means "integer". – Pointy Commented Jan 10, 2018 at 16:14
• @Pointy But the number can be positive, like 11 or 65 , no problem? – Jack The Baker Commented Jan 10, 2018 at 16:15
• FYI an int is a whole number, so when you try to parse -0.3 it is rounding to 0. – Jim Wright Commented Jan 10, 2018 at 16:15
• 5 Why would you name a number 'text'? – Jared Smith Commented Jan 10, 2018 at 16:17
• 1 @tourtravel what JaredSmith try to say to you, is that your variable name is not really the best one. You can use text() to retrieve it, but for exemple, it would be better if your variable "text" was named "adRatingValue" for example. – MathKimRobin Commented Jan 10, 2018 at 16:22

 |  Show 13 more ments

1 Answer 1

Sorted by: Reset to default 8

Use parseFloat instead of parseInt because parseInt just parses the first part as 0 and it is not negative. Working code:

var rate = $('#view-ad-rating');
var text = parseFloat($(rate).text()); // <- HERE

if (text < 0) {
  $(rate).addClass('viewAdRateBad');
} else {
  //$(rate).addClass('viewAdRateGood');
}

.viewAdRateBad {
  color: red;
}

.viewAdRateGood {
  color: green;
}

<script src="https://ajax.googleapis./ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<a id="view-ad-rating">-0.3</a>

parseInt("-0.3")
//returns -0

parseFloat("-0.3")
//returns -0.3

parseInt("-0.9") // note it doesn't round but just parses the first part
//returns -0