if else syntax -

[daniel]

I'm trying to do a simply if else thing but for some reason it doesn't
work. The commented code works - as you can see I'm using jquery but
the problem only happens when I convert the two .click functions into
one with an if.. else statement. The commented section works but i
would like to get the simpler one following working.

/* $("#unsubscribe").hide();

$("#subscribe h4 a").click(function(){
$("#subscribe").hide();
$("#unsubscribe").fadeIn();
return false;
});
$("#unsubscribe h4 a").click(function(){
$("#unsubscribe").hide();
$("#subscribe").fadeIn();
return false;
});*/


$("#unsubscribe").hide();
var formswitch = 0;

$(".left h4 a").click(function(){
if (formswitch == 0) {$("#subscribe").hide();
$("#unsubscribe").fadeIn();
var formswitch = 1;
return false;}
else {$("#unsubscribe").hide();
$("#subscribe").fadeIn();
return false;}
});

 

[Bjoern Hoehrmann]

* daniel wrote in comp.lang.javascript:

I'm trying to do a simply if else thing but for some reason it doesn't
work. The commented code works - as you can see I'm using jquery but
the problem only happens when I convert the two .click functions into
one with an if.. else statement. The commented section works but i
would like to get the simpler one following working.

var formswitch = 0;

$(".left h4 a").click(function(){
if (formswitch == 0) {$("#subscribe").hide();
$("#unsubscribe").fadeIn();
var formswitch = 1;

Presumably you did not mean to specify the 'var' here; other than that
there seems little wrong with this, though I would strongly suggest to
use proper indentation at least when asking others to debug your code,
and you might want to put the "return false" at the end of the function
rather than duplicate it.

 

[daniel]

I'm not sure which one it was but that was the fix. I think the silly
mistake of putting the var in there was it.

 

[Thomas 'PointedEars' Lahn]

I'm not sure which one it was but that was the fix. I think the silly
mistake of putting the var in there was it.

Which only leaves the silly mistake of using jQuery in its current form to
be corrected.

http://groups.google.com/groups?as_q=jquery&as_ugroup=comp.lang.javascript&scoring=d&filter=0


PointedEars