Как задать условие в js вместо each?

Question

[arle]

Всем привет!

Есть код%:

$(document).ready(function(){
    var i = 0;
    var css = "";
    $("div,a").each(function(){
        $(this).attr("id","st-"+i);
        css += "#"+$(this).attr("id")+"{"+$(this).attr("style")+"}";
        $(this).removeAttr("style");
        i++;
    });
    $("style").html(css);
});

Он выделяет стили из html и задает id.
То что он делает меня вполне устраивает, но есть одно НО. Он задает id каждому элементу div и a. Подскажите, возможно ли вместо each(function() прописать правило, при котором скрипт будет задавать id не каждому, а только тем div и a, для которых прописаны стили(style="").

Comments

[Егор Живагин]

Уберите свою i и задайте встроенную)

.each(function(i){
// i - первый аргумент each - встроенный счётчик. При каждой итерации сам увеличится.
})

Answer 1

[Andrey Perov]

Проверяйте так:

$("div,a").each(function(index, element){
   if($(element).attr('style')) {
       // to do ...
   }
})

<div></div>
<div style="clear: both"></div>

в console выведет
1. undefined
2. clear: both

дальше сами разберетесь

Comments

[Negwereth]

тогда уже ...$("div[style], a[style]").each(...

[Andrey Perov]

Negwereth так еще лучше)

[arle]

Negwereth, Спасибо тебе дружище!!

[Negwereth]

arle, плюс у тебя там счётчик лишний. each свой предоставляет.

[arle]

Negwereth, ты про var i = 0?

Есть по нему такой вопрос, можно ли сделать присвоение id не по порядку 1,2,3, а присваивать случайное генерированное значение? Мне нужно много раз так обработать html и id дублируется каждый раз(1,2,3), я меняю var i = 0 на 100, 200 и т.д., а если бы была возможность присваивать случайный id, то дублей не было бы и каждый раз счетчик сдвигать не нужно.

[Andrey Perov]

arle https://developer.mozilla.org/ru/docs/Web/JavaScri...

var id = '_' + Math.random().toString(36).substr(2, 9);