Это код javascript правильно?

Question

Я пытаюсь удалить класс при нажатии на элемент с определенным классом. Я сделал этот javascript, и он работает. Но является ли этот правильный синтаксис таким способом. Может ли это быть более эффективным?

// Clear notifications alerts 
$(document).on('click', '.js-clear-notifications', function() { 
    $('.AlertNotifications').remove(); 
}); 

// clear inbox alerts 
$(document).on('click', '.js-clear-inbox', function() { 
    $('.AlertInbox').remove(); 
}); 

Answer 1

Ваш javascript-код верен при условии, что вы загружаете jQuery.

Кроме того, у вас есть наиболее эффективное решение, в котором вы используете один обработчик событий для обработки событий, происходящих из нескольких элементов.

Альтернативой будет:

$('.js-clear-notifications').on('click', function() { 
    $('.AlertNotifications').remove(); 
}); 

который привязывается так много обработчиков событий, сколько элементов в объекте JQuery. Чуть менее эффективен, хотя, вероятно, вы никогда не заметите, кроме крайних случаев.

Answer 2

Я редактирую это в ответ на отзывы в комментариях.

Если вы хотите удалить все элементы с классом AlertNotifications, что и делает ваш код, то то, что у вас есть, является правильным.

Если то, что вы хотите, чтобы удалить только класс, который является то, что текст поста сказал, вы хотите removeClass, не удалять:

$('.js-clear-notifications').on('click',function() { 
     $(this).removeClass('AlertNotificitions'); 
    } 

Answer 3

мне более правильный способ сделать это что-то как это:

... 
$('.js-clear-inbox').on('click', function() { 
    $('.AlertInbox').remove(); 
}); 
... 

Я также предлагаю, чтобы иметь более специфические селекторы т.е.

$('div .js-clear-inbox') 

Я надеюсь, что это помогает.

Answer 4

Новый способ, если вы уже определили переменную, правильный способ удалить его из DOM будет:

var elem = document.getElementById("myDiv"); 
elem.remove(); 

Но если вы только начинаете вне, .remove будет лучшим мнение ,