JQuery неправильно считает элементы

Question

Это много, что я уже собрал вместе (благодаря Stack Overflowers):

У меня есть серия модулей продукта, каждый из которых содержит название продукта и флажок. При установке флажка соответствующее имя продукта добавляется в список в другом месте на странице, при снятии флажка удаляется соответствующее название продукта из списка. Когда пользователь отправляет кнопку «Сравнить», мне нужно, чтобы jQuery проверил количество выборок, подтвердив минимум два и произвольный максимум.

Моя проблема: следующий jQuery, по-видимому, не в состоянии учитывать элементы, которые были удалены (снятием флажка). Например, если я установил флажок, затем снял флажок - и он действительно не был проверен, и он действительно удален из списка - и затем снова проверил его, jQuery сообщает, что нужно сравнить 2 элемента.

<div class="product-module">
<div class="product-pic">
    <div class="checkbox-wrapper">
        <label for="compare1">
            <input type="checkbox" class="checkbox" name="compare" id="compare1" />
            Compare
        </label>
    </div>
</div>

<div class="product-info">
    <p><a href="#" title="#"><span class="product-name">Product Name here</span></a></p>    
</div>
</div>


<div class="compare">
<ul>
</ul>
<p class="compare-button"><button type="submit">Compare</button></p>
<p class="clear-selections"><a class="button" id="clear-selections" href="#">Clear Selections</a></p>
</div>

Сценарий:

// clear all checkboxes on load
$(function(){
    $('input[type="checkbox"]').attr('checked', false);
});


    $(function(){
    $('.column-main input[type="checkbox"]').click(function()    {
        var title = $(this).closest('.product-module').find('.product-name').html();

        // if user checks the checkbox, add the item to the ul
        if    ($(this).attr('checked'))    {
        var html = '<li><a href="'+title+'">' + title + '</a></li>';
        $('.compare ul').append(html);

        // if user un-checks the checkbox, remove the item from the ul
        }    else if ($(this).attr('checked', false))    {
        $('li a[href="' + title + '"]').remove();
        }
    })
})


$(function(){
    $('.clear-selections').click(function(){
        $('.compare ul').empty();
        $('input[type="checkbox"]').attr('checked', false);
    })
});

    $(function(){
    $('.compare button').click(function(){
        minRequests = 2;
        maxRequests = 3;
        requested = $('.compare ul li').size();    // go figure: why not .length()?

        if(requested < 2)    {
        alert ('Compare ' + requested + ' products?');

        } else if((requested >= 2) && (requested <= 5 ))    {
        alert ('There are ' + requested + ' products to compare');

        } else {
        alert (requested + ' is too many');
        }
    });
});

Answer 1

Проблема здесь:

$('li a[href$="' + title + '"]').remove();

Вы удаляете тег «a» изнутри «li», но оставляете «li» - это то, что вы считаете позже.

Попробуйте:

$('li a[href$="' + title + '"]').parent().remove();