Я ищу для создания этого в vanilla JS, предоставленный jQuery. Я хотел использовать постное JS. Приведенный ниже код быстро дал быстрые результаты. Однако для сборки я бы хотел, чтобы он был более компактным.Включение этой ванили JS
Несколько раздражающих факторов, несколько дочерних потомков, неизвестные классы все и должны находиться внутри цикла.
$('.nav__adv').each(function(){
\t var $el = $(this),
\t \t $no_advert = $('.main_ad_container div a div div', $el);
\t $el.filter(function() {
\t return $.trim($no_advert.text()) === "Advertise Here";
\t }).css("display", "none").remove();
});
Любое направление/помощь/руководство?
бы вы назвали преобразование рабочего кусок кода, который использует (хорошо известный, тяжело испытанный) библиотеку с тем же кодом без библиотеки с использованием 3х строк кода unvarified компактнее? (Я бы не стал). Лион должен научиться: не исправить то, что не сломано. Оптимизируйте только тогда, когда у вас проблемы с производительностью. – Amit
'$ el.filter' кажется бессмысленным, учитывая, что' $ el' только ссылается на один элемент. Если вы просто завернули его в проверку «если» для теста рекламы, он сделает то же самое – CodingIntrigue
@ Если у вас есть справедливая точка, в то время как мне не нравится много накладываться на библиотеки. jQuery является исключением. Я просто думаю, что это хорошо, если возможно, ваниль. Да, это 3 линии, но я хотел знать, что думает сообщество. Если я смогу вытащить среднего человека (jQuery), я сделаю это. Но несколько строк кода кажутся заманчивыми по длинному блоку кода. – Neil