Implementing if-else condition inside ajax script

Question

I'm having a problem with this condition. The "(numofeggs < totalsm)" in the condition is only executes. Otherwise, all variables (except "numofeggs < totalsm") are not functional. Is there any ways things to do?

$(document).ready(function() {
  displayDataSM();
  displayTotaleggs();
  $("#btn-input").click(function() {
    var numofeggs = $("#numofeggs").val();
    var size = $("#size").val();
    var price = $("#price").val();
    var farmername = $("#farmername").val();
    var totalsm = "<?php echo $totalSmall ?>";

    if ((numofeggs < totalsm) && (size && price && farmername)) {
      $.ajax({
        url: "ajaxeggsout.php",
        type: "POST",
        async: false,
        data: {
          "btn-input": 1,
          "numofeggs": numofeggs,
          "size": size,
          "price": price,
          "farmername": farmername
        },
        success: function(data) {
          displayDataSM();
          displayTotaleggs();
          $("#numofeggs").val('');
        }
      })
    }
    return false;
  });
});

if((numofeggs < totalsm) && (size && price && farmername)) Shouldn't this be written as if((numofeggs < totalsm) && (size > 0 && price > 0 && farmername.length > 0 )) — Satya
– Satya, Commented Jan 1, 2017 at 13:24
From my program, (numoffeggs<totalsm) only excutes. Your suggestion stops working... — Emerge Paul Faller
– Emerge Paul Faller, Commented Jan 1, 2017 at 13:33

Barmar · Accepted Answer · 2017-01-01 13:49:45Z

1

You need to convert your inputs to numbers. The number 0 is falsy, but the string "0" is truthy. And when you do comparison with <, numbers work differently than strings (e.g. 5 < 12 is true, but "5" < "12" is false).

For the inputs, use Number() to convert them to numbers. And for the variable that comes from PHP, don't put it in quotes.

So do:

var numofeggs = Number($("#numofeggs").val());
var size = Number($("#size").val());
var price = Number($("#price").val());
var farmername = $("#farmername").val();
var totalsm = <?php echo $totalSmall ?>;

edited Jan 1, 2017 at 13:49

answered Jan 1, 2017 at 13:36

Barmar

789k57 gold badges554 silver badges669 bronze badges

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

5 Comments

Emerge Paul Faller Over a year ago

When I remove less than "<" sign, It's working. But the condition of (numofeggs < totalsm) is still I need to...

Emerge Paul Faller Over a year ago

Where should I put this condition (numofeggs < totalsm)? In the last part?

Barmar Over a year ago

See my updated answer, you need to convert more things to numbers.

Emerge Paul Faller Over a year ago

My first program: if(numofeggs && size && price && farmername), is all working when I execute button. My second program: if((numofeggs < totalsm) && (size && price && farmername)), the "size", "price", and "farmername" are not working anymore

Barmar Over a year ago

What do you mean by "not working"? What is the value of all the variables?

Collectives™ on Stack Overflow

Implementing if-else condition inside ajax script

1 Answer 1

5 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

5 Comments

Your Answer

Sign up or log in

Post as a guest

Related