if else not working js

Question

Here is my Code i just want to change x value on click if it is x then change y ..if it is y then change to x again but after become y value cant change to x .. the control is always on 3rd section .. whats wrong

jQuery(document).ready(function(){
           var x;
          $(".ticktack tr td").click(function(){
          if(x == null){

          alert('section 1' );
            x = 'X';
           }
          else if(x == 'X'){
          alert('section 2');
             x = 'Y';
           }
          else if(x == 'Y'){
          alert('section 3');
                x == 'X';
           }
            $(this).html(x);
          })

        });

jsfiddle.net/fsdrqo1t - In this fiddle section 1 is alerting which is expected — ssilas777
– ssilas777, Commented Feb 7, 2015 at 6:02

Cyrille Armanger · Accepted Answer · 2015-02-07 06:03:05Z

1

Actually you have two mistakes, one ; missing and x == X which should x = X

$(document).ready(function(){
           var x;
          $(".ticktack tr td").click(function(){
              if(x == null){
                    alert('section 1' );
                    x = 'X';
               } else if(x == 'X'){
                    alert('section 2');
                    x = 'Y';
               } else if(x == 'Y'){
                    alert('section 3');
                    x = 'X';
               }
            $(this).html(x);
          });

        });

answered Feb 7, 2015 at 6:03

Cyrille Armanger

7595 silver badges18 bronze badges

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

Comments

Paul Draper · Accepted Answer · 2015-02-07 06:01:09Z

0

alert('section 3');
x == 'X';

should be

alert('section 3');
x = 'X';

JSFiddle

answered Feb 7, 2015 at 6:01

Paul Draper

84.2k53 gold badges216 silver badges303 bronze badges

Comments

Adrien · Accepted Answer · 2015-02-07 06:01:28Z

0

You have an extra '=' in the last else if statement!

x = 'X'; should work :)

answered Feb 7, 2015 at 6:01

Adrien

1,9451 gold badge14 silver badges23 bronze badges

Comments

coderz · Accepted Answer · 2015-02-07 06:04:40Z

0

You added one more = in the third section. Try:

jQuery(document).ready(function(){
               var x;
              $(".ticktack tr td").click(function(){
              if(x == null){

              alert('section 1' );
                x = 'X';
               }
              else if(x == 'X'){
              alert('section 2');
                 x = 'Y';
               }
              else if(x == 'Y'){
              alert('section 3');
                    x = 'X';
               }
                $(this).html(x);
              });

            });

answered Feb 7, 2015 at 6:04

coderz

5,01912 gold badges53 silver badges80 bronze badges

Comments

Shreejibawa · Accepted Answer · 2015-02-07 06:08:20Z

0

Well you are using comparison operator after alert('Section 3'). Instead you must use assignment operator! Try this one:

jQuery(document).ready(function(){
  var x;
  $(".ticktack tr td").click(function(){
  if(x == null){
    alert('section 1' );
    x = 'X';
  } else if(x == 'X') {
    alert('section 2');
    x = 'Y';
  } else if(x == 'Y') {
    alert('section 3');
    x = 'X';
  }
  $(this).html(x);
  });
});

edited Feb 7, 2015 at 6:08

answered Feb 7, 2015 at 6:02

Shreejibawa

1,8681 gold badge28 silver badges38 bronze badges

Comments

Pratik Joshi · Accepted Answer · 2015-02-07 06:23:16Z

0

Change x == 'X'; to x = 'X' as you have to assign a value x=='X' does Nothing. If you put comparions Or ternary operator like x =='X'?If true then do this:else this; in if then it gives you true or false result

answered Feb 7, 2015 at 6:23

Pratik Joshi

11.7k7 gold badges44 silver badges73 bronze badges

Comments

Hardeep Randhawa · Accepted Answer · 2015-02-07 09:54:34Z

0

    Used assignment operator instead of comparison operator in Third section
    try this code.

    jQuery(document).ready(function(){
       var x;
      $(".ticktack tr td").click(function(){
      if(x == null){

      alert('section 1' );
        x = 'X';
       }
      else if(x == 'X'){
      alert('section 2');
         x = 'Y';
       }
      else if(x == 'Y'){
      alert('section 3');
            x = 'X';
       }
        $(this).html(x);
      })

    });

answered Feb 7, 2015 at 9:54

Hardeep Randhawa

478 bronze badges

Collectives™ on Stack Overflow

if else not working js

7 Answers 7

Comments

Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

7 Answers 7

Comments

Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related