То, что у вас есть, выглядит нормально для меня, за исключением того, что вы хотите объявить и инициализировать i
где-нибудь. Но это может быть проще. Вы можете использовать индекс jQuery, передаваемый в обратный вызов $.each
, и просто назначить непосредственно свойству id
данного элемента, а не создавать объект jQuery вокруг него, просто чтобы установить атрибут с помощью attr
. E.g.:
$.each(o, function(index) {
this.id = 'listItem' + index;
});
Но если вы хотите построить объект jQuery вокруг каждого элемента по другим причинам:
$.each(o, function(index) {
$(this).attr("id", 'listItem' + index);
});
(Также нет необходимости объявлять и использовать эту переменную f
, если вы ее не используете, поэтому я оставил ее вне приведенного выше.)