javascript如果條件不起作用？

Question

我坐在像下面的代碼片段前幾小時，不能讓它像我想要的那樣運行。

基本上這個代碼創建一個右鍵點擊的導航菜單，但點擊切換器應該關閉這個功能，並在下一次點擊它再次打開。

一切工作正常（如預期的），只是第12行if if語句（如果（切換器％2 == 0））不能像預期的那樣工作，意味着它內部的代碼總是得到執行，無論var切換器是否均勻甚至沒有。我還嘗試了其他條件，如「> 0」等，但其中的代碼始終得到執行。

$(document).ready(function() { 
    /* Set switcher to zero */ 
    switcher = 0; 
    /* If switch gets clicked increment var switcher*/ 
    $('#guidenavschalter').click(function() { 
     switcher++; 
     return false; 
    }); 

    /* If var switcher is even execute following code, if not do nothing of this*/ 
    if (switcher % 2 == 0) { 
     /* do not display right click browser menu */ 
     document.oncontextmenu = function() { 
      return false; 
     }; 
     /* if click within #page excluding area of #newid */ 
     $('#page:not(#newid)').mousedown(function (e) { 
      /* if right click */ 
      if (e.button == 2) { 
       /* if #newid already exist display it again */ 
       if ($('#newid').length) { 
        $('#newid').css({ 
         "display": 'block' 
        }); 
        $('#newid').css({ 
         "top": e.pageY + 'px' 
        }); 
        $('#newid').css({ 
         "left": e.pageX + 'px' 
        }); 
        /* if it does not exist create and display #newid */ 
       } else { 
        var $div = $('#block-bookoblock-book-outline').clone().attr('id', 'newid'); 
        $('body').append($div); 
        $('#newid').css({ 
         "top": e.pageY + 'px' 
        }); 
        $('#newid').css({ 
         "left": e.pageX + 'px' 
        }); 
        $('#newid').css({ 
         "position": 'absolute' 
        }); 
        return false; 
       } 
      } 
      /* if left click hide #newid */ 
      if (e.button == 0) { 
       $('#newid').css({ 
        "display": 'none' 
       }); 
      } 
      return true; 
     }); 
    } 
}); 

Answer 1

我不認爲這是條件switcher % 2 == 0這是有問題的在這裏。 如果是真的，你已經掛鉤了事件，但是是否有一個Else語句解除了這些事件，以便恢復原始功能？即右鍵單擊創建默認上下文菜單。

更新：

爲了恢復原來的功能，稱之爲

document.oncontextmenu = null; 

中的其他部分。

另外，您只需要定義$('#page:not(#newid)').mousedown(function (e) {一次（if/else之外），然後使用切換變量來確定是否調用該功能。

總之，你需要以下

$(document).ready(function() { 
    /* Set switcher to zero */ 
    switcher = 0; 
    /* If switch gets clicked increment var switcher*/ 
    $('#guidenavschalter').click(function() { 
     switcher++; 
     return false; 
    }); 

    document.oncontextmenu = function() { 
     if (switcher % 2 == 0) { 
      return false; 
     } else { 
      return true; 
     } 
    }; 
    /* if click within #page excluding area of #newid */ 
    $('#page:not(#newid)').mousedown(function (e) { 
     if (switcher % 2 == 0) { 
      // do stuff 
     } else { 
      // do nothing 
     } 
    }); 

}); 

Answer 2

switcher = 0; // created outside the click event handler 

，你就增加點擊事件處理程序內的值。因此它總是零。

你應該通過Scoping in JavaScript

從評論，您有興趣瞭解更多關於變量的作用域在Javascript然後

退房這個SO answer

Answer 3

你的代碼基本上是

switcher = 0; 

... some irrelevant code here (the callback is not executed right now) 

if (switcher % 2 == 0) { 

所以難怪測試總是通過。

你可能想要的是把if回調裏面，所以，它的每一次測試你點擊：

var switcher = 0; 
$('#guidenavschalter').click(function(){ 
    switcher++; 
    if (switcher % 2 == 0) { 
     ... 
    } 
    return false; 
});