javascript, если условие не работает?

Question

Я сижу как часы перед следующим фрагментом кода и не могу заставить его работать, как я хочу.

В основном этот код создает меню навигации по правому клику, но щелчок на коммутаторе должен отключить эту функцию, а при следующем нажатии он снова включается.

Все работает нормально (как и ожидалось), только инструкция if (строка) (строка (2)) работает не так, как ожидалось, что означает, что код внутри него всегда выполняется, независимо от того, или даже нет. Я также пробовал другие условия, такие как «> 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, которая отцепляет эти события, чтобы возобновить исходную функциональность? т.е. щелкните правой кнопкой мыши, создайте контекстное меню по умолчанию.

Update:

Чтобы возобновить первоначальную функциональность, вызовите

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; 
});