jQuery语法错误？ CSS和动画的 '如果' 和 '其他' 语句

Question

编辑 - 香港专业教育学院改变了我的代码：

这工作得很好：

$(".image-div").click(function() { 
     if ($(this).css('z-index') == 1) 
      $(this).css('z-index','2'); 
     else 
      $(this).css('z-index','1'); 
}); 

而且这也适用：

$(".image-div").click(function() { 
     if ($(this).css('z-index') == 1) 
      $(this).animate({ 
       width: '500px' 
      }, 5000, function() { 
       // Animation complete. 
      }); 
     else 
      $(this).animate({ 
       width: '250px' 
      }, 5000, function() { 
       // Animation complete. 
      }); 
    }); 

但这不要做任何事情：

$(".image-div").click(function() { 
     if ($(this).css('z-index') == 1) 
      $(this).css('z-index','2'); 
      $(this).animate({ 
       width: '500px' 
      }, 5000, function() { 
       // Animation complete. 
      }); 
     else 
      $(this).css('z-index','1'); 
      $(this).animate({ 
       width: '250px' 
      }, 5000, function() { 
       // Animation complete. 
      }); 
    }); 

谢谢

Answer 1

看起来你需要花括号来围绕你的if ...否则，如果没有连接，那么它是浮动的。

[添加] 好吧，任何控件都需要花括号来支持多个语句。罚款不花括号，因为只有一个每个控制后声明了以下工作：

if(condition) doSomething(); 
else doSomethingElse(); 

然而，这将打破：

if(condition) 
    doSomething(); 
    doAnotherThing(); 
else 
    doSomethingElse(); 

为什么？因为如果在这种情况下，只有涉及此：

if(condition) 
    doSomething(); 

然后，你这坐出在中间的地方：

doAnotherThing(); 
else 
    doSoemthingElse(); 

doAnotherThing不包含内部的，如果（），从而既不是你的其他人{}。所以，你的其他人失去了对if和code破解的依赖。它是，恕我直言，总是使用大括号的最佳做法，因为如果你不这样做，你或其他人可能会进来并将doAnotherThing（）行添加到代码中，认为它包含在if（）中，并且不经意间破坏了代码。

也就是说，你的if/else都包含多个语句。所以，从严格意义上的语法点，你的代码需要：

$(".image-div").click(function() { 
    if ($(this).css('z-index') == 1) { 
     $(this).css('z-index','2'); 
     $(this).animate({ 
      width: '500px' 
     }, 5000, function() { 
      // Animation complete. 
     }); 
    } 
    else { 
     var divRef = this; 
     $(this).animate({ 
      width: '250px' 
     }, 5000, function() { 
      $(divRef).css('z-index','1'); 
     }); 
    } 
}); 

Answer 2

是的。否则开始{或结束}之后的if &。

此外，它看起来像在底部双});。