Правильное ли удаление из localStorage и сохранение?

Question

[Genri_Rus]

Здравствуйте!

Как удалять из localStorage, если я убираю галочки с чекбоксов ?

Вот пример кода: ссылка

Код

const shoppingCart = (function() {
     let cart = [];
     const localName = 'local';

     function Item(id, p) {
        this.id = id;
        this.p = p;
        // console.log(p);
      }

      function saveCart() {
        localStorage.setItem(localName, JSON.stringify(cart));
      }

      function loadCart() {
        cart = JSON.parse(localStorage.getItem(localName));
        if (cart !== null) {
          for (var i in cart) {
            $('#' + cart[i].id).prop('checked', true);
           }
        } else if (cart === null) {
          cart = []
        }
      }
      
      loadCart();
  
      var obj = {};

			obj.addItemToCart = function(id, p) {
        var item = new Item(id, p);
        cart.push(item);
        saveCart();
			};
      return obj;
})();
    $('body').on('change', '.check', function(){
    let modal = $('.modal');

    if ($('input').is(':checked')) {
        if (!modal.children().is('.item')) modal.html('');
    } else {
        modal.removeClass('modal-show').text('Здесь ничего нет');
    }
    
    if ($(this).prop('checked')) {
        let id = $(this).attr('id');
        let p  = $(this).parent().siblings('.copy, .copy-2').text();
        shoppingCart.addItemToCart(id, p);
      
        modal.addClass('modal-show');
     }
});

В этом месте я вывожу сохраненные в localStorage id и если они сохраняются, то проставляю галочки чекбоксам

if (cart !== null) {
  for (var i in cart) {
	$('#' + cart[i].id).prop('checked', true);
   }
}

Answer 1

[Роман Дворянов]

Вы перезаписываете значение итема в localStorage на пустой массив, в следующий раз при проверке есть ли такой итем в хранилище вы получите true, и его значение пустой массив, чтоб удалить из хранилища используйте метод removeItem

localStorage.removeItem(localName)

Comments

[Genri_Rus]

Спасибо за позднее решение, а куда именно эту команду прописать *

[Роман Дворянов]

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

if ($(this).prop('checked')) {
        let id = $(this).attr('id');
        let p  = $(this).parent().siblings('.copy, .copy-2').text();
        shoppingCart.addItemToCart(id, p);
      
        modal.addClass('modal-show');
     } else {
       localStorage.removeItem(localName)
     }

[Genri_Rus]

Роман Дворянов, спасибо, как раз по снятию чекбокса и нужно было !