execute jquery function inside an IF statement?

Question

I'm trying to execute a jquery function inside an IF statement.

basically, I am getting the value of a select option and if the value of selected option is what I want (htl in this example) then I want to execute the function!

but when I wrap the code within an IF statement, i get syntax error which I have no idea what's causing that issue.

This is my entire code:

var ascending = false;
$('.page_navigation .sortBy').change(function () {    
    var vals = $(this).val();
    if (vals == "htl") {    
        ///// I need to put the code bellow here    
    }
    var sorted = $('.mypro').sort(function (a, b) {
        return (ascending == (convertToNumber($(a).find('.prod-price').html()) < 
                  convertToNumber($(b).find('.prod-price').html()))) ? 1 : -1;
    });
    ascending = ascending ? false : true;

    $('#myCont').html(sorted);
});
var convertToNumber = function (value) {
    return parseFloat(value.replace('£', ''));
}

could someone please advise on this issue?

I tried this and I get the syntax error:

var ascending = false;

$('.page_navigation .sortBy').change(function(){

    var vals = $(this).val();
    if(vals == "htl") {


    var sorted = $('.mypro').sort(function(a,b){
        return (ascending ==
               (convertToNumber($(a).find('.prod-price').html()) < 
                convertToNumber($(b).find('.prod-price').html()))) ? 1 : -1;
    });
    ascending = ascending ? false : true;

    $('#myCont').html(sorted);
});
var convertToNumber = function(value){
     return parseFloat(value.replace('£',''));
}
}

@ArunPJohny, yeah, because I haven't put the function inside the IF statement. — H.HISTORY
– H.HISTORY, Commented Jul 17, 2015 at 10:38
You seems missing a } to close the if Move the } at the most bottom after $('#myCont').html(sorted); — fuyushimoya
– fuyushimoya, Commented Jul 17, 2015 at 10:40

Liam · Accepted Answer · 2015-07-17 10:41:02Z

Your closing the if wrong. You need to put the closing brace (of the if statement) inside the function call, so:

var ascending = false;

$('.page_navigation .sortBy').change(function(){

    var vals = $(this).val();
    //closing brace matches this opening one
    if(vals == "htl") {    
          var sorted = $('.mypro').sort(function(a,b){
          return (ascending ==
               (convertToNumber($(a).find('.prod-price').html()) < 
                convertToNumber($(b).find('.prod-price').html()))) ? 1 : -1;
       });
       ascending = ascending ? false : true;

       $('#myCont').html(sorted);
   //put the brace inside the function...i.e. close the if brace
   }
});
var convertToNumber = function(value){
     return parseFloat(value.replace('£',''));
}

The incorrect bit is:

var convertToNumber = function(value){
     return parseFloat(value.replace('£',''));
//WHY's this brace here!
}
}

blex · Accepted Answer · 2015-07-17 10:44:18Z

2

Limit the scope of your if block to $('#myCont').html(sorted); ie

if(vals == "htl") {
    var sorted = $('.mypro').sort(function(a,b){
        return (ascending ==
               (convertToNumber($(a).find('.prod-price').html()) < 
                convertToNumber($(b).find('.prod-price').html()))) ? 1 : -1;
    });
    ascending = ascending ? false : true;
    $('#myCont').html(sorted);
}

edited Jul 17, 2015 at 10:44

blex

25.7k6 gold badges48 silver badges78 bronze badges

answered Jul 17, 2015 at 10:43

Tushar Gupta

16k1 gold badge33 silver badges53 bronze badges

Comments

andrew · Accepted Answer · 2015-07-17 10:49:16Z

0

This is exactly why you should be coding with an IDE and not in a text editor

Syntax errors will be highlighted instantly and you will save yourself a lot of time

take a look at the screenshot from netbeans:

enter image description here

answered Jul 17, 2015 at 10:49

andrew

9,6039 gold badges36 silver badges62 bronze badges

Collectives™ on Stack Overflow

execute jquery function inside an IF statement?

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related