JQuery удалить/добавить класс скрипт не работает

Question

Вот скрипт JQuery У меня возникли проблемы с:

$(document).ready(function(){ 
    $("li").click(function(){ 
     $(".active").removeClass("active") 
     $(this).addClass("active") 
    }); 
}); 

сценарий должен сделать цвет фона ссылки/страницы вы в до оранжевого используя css, запрограммированный для класса (.) active , сценарий должен удалить активный класс из активной ссылки и добавить класс к ссылке, которая нажата.

о том, как предполагается, скрипт, чтобы сделать это: если Ли щелкнул вещь с активным классом будет иметь это класс удалено, чем щелкнутая вещь (это) Виль получить класс активен. я использовал это, но кажется, что это не работает

Answer 1

Ваш код должен выглядеть следующим образом:

$(function() { 
    $("li").click(function() { 
     $(".active").removeClass("active"); 
     $(this).addClass("active"); 
    }); 
}); 

Если я понимаю, что вы пытаетесь сделать, то это должно работать.

Я предлагаю сформулировать ваш вопрос немного лучше, так как это просто трудно понять с первого взгляда. Кроме того, предоставление jsFiddle часто является хорошим выбором, когда вы спрашиваете о javascript/jQuery.

Answer 2

Я не знаю, если это ошибка копирования пасты или нет, но вам не хватает запятой:

$(document).ready(function(){ 
    $("li").click(function(){ 
     $(".active").removeClass("active"); 
     $(this).addClass("active"); 
    }); 
}); 

Update: На основе комментариев, которые я получил на этот ответ, я чувствую, что я должен добавьте, что я сказал, что полуточка отсутствует, потому что код был все в одной строке. Если это не так, то точка с запятой не решит проблему. С учетом сказанного, нет ничего плохого в вышеуказанном коде, поэтому ошибка должна быть где-то еще на странице без достаточной информации, предоставленной нам для ее исправления.

Answer 3

Try this: 

JS

$(document).ready(function(){ 
     $("li").click(function(){ 
     $(this).parent("ul").find("li").removeClass('active'); 
     $(this).addClass("active"); 
     }); 
    }); 

Кроме того, не забыл добавить ; в конце линии.

Answer 4

Я не эксперт JQuery, но, если я хорошо понял ваш вопрос, это должно Worke ... он сделал для меня:

$('li').each(function(){ 

    $(this).click(function(){ 
     DeselectCurrent();   
     $(this).addClass("active"); 
    }); 
}); 

var DeselectCurrent = function() { 
    $('li').each(function(){ 
     var attr = $(this).attr('class');    

     if (typeof attr !== 'undefined' && attr !== false) { 
      $(this).removeClass("active"); 
     } 
    }); 
}; 

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

jsfiddle пример