Как сделать правильно установки href?

Question

[EarthFM]

Есть src в картинке, вытаскиваю оттуда id и сохраняю в переменную, потом ссылку с id пихаю в href, но эта ссылка с id является для всех одинаковой, а надо сделать чтобы для каждой картинки был свой link

$(function() {
		$('.left-img-thumb').each(function() {
			$('.left-img-thumb').unwrap().wrap('<a class="rewrd" target="_blank"></a>');
				var urlImg = $(this).attr('src');
				var nameImg = /\/(\d+)\.(?:\w{1,5})$/.exec(urlImg);
				var j = nameImg[1];
				var link = 'http://www.anastasiadate.com/pages/lady/profile/profilepreview.aspx?LadyID=' + j;
				$('.rewrd').attr('href', link);
		});
});

<a class="rewrd" target="_blank" href="http://www.anastasiadate.com/pages/lady/profile/profilepreview.aspx?LadyID=17274311">
<img alt="karina" src="http://www.anastasiahalloween.com/wp-content/uploads/2014/10/1693884.jpg" width="235" height="240" class="left-img-thumb">
</a>

Answer 1

[Иван Беленков]

$(function() {
		$('.left-img-thumb').each(function() {
                        // ошибка в следующей строке
			$('.left-img-thumb').unwrap().wrap('<a class="rewrd" target="_blank"></a>');
				var urlImg = $(this).attr('src');
				var nameImg = /\/(\d+)\.(?:\w{1,5})$/.exec(urlImg);
				var j = nameImg[1];
				var link = 'http://www.anastasiadate.com/pages/lady/profile/profilepreview.aspx?LadyID=' + j;
                                // и в этой
				$('.rewrd').attr('href', link);
		});
});

Вот так верно:

$(function() {
		$('.left-img-thumb').each(function() {
                          // так верно
			$(this).unwrap().wrap('<a class="rewrd" target="_blank"></a>');
				var urlImg = $(this).attr('src');
				var nameImg = /\/(\d+)\.(?:\w{1,5})$/.exec(urlImg);
				var j = nameImg[1];
				var link = 'http://www.anastasiadate.com/pages/lady/profile/profilepreview.aspx?LadyID=' + j;
                                // так будет работать, что-то я это пропустил:
				$(this).closest('.rewrd').attr('href', link);
		});
});

В вашем варианте последний атрибут применяется ко всем элементам с классом 'left-img-thumb', а ссылка в каждой итерации применяется ко всем элементам '.rewrd', а не к конкретной ссылке.

Comments

[EarthFM]

Всё равно одна одинаковая ссылка на всех(

[Иван Беленков]

ааа, виноват. щас подправлю.

[EarthFM]

@IoannGrozny , спасибо огромное))) Мой наставник мне через цикл for говорил делать, я подумал это бред если в jquery уже всё есть.

[Иван Беленков]

ну вообще, for работает порядком быстрее, чем .each(). так что, если не нужны коллбэки, лучше использовать встроенные циклы JS. habrahabr.ru/post/103296

[EarthFM]

@IoannGrozny, понятно, что быстрее) просто надо чисто на jquery сделать) js выучил более менее, теперь jquery осваиваю) а то некоторые jquery знают, а js только синтаксис)

[Виталий Киренков]

@IoannGrozny недоглядел я первую строку =)

[Иван Беленков]

@DeLaVega вот, а я последнюю)

Answer 2

[Виталий Киренков]

$('.rewrd').attr('href', link);
заменить на
$(this).closest(".rewrd").attr('href', link);

Comments

[EarthFM]

Ссылка пропала полностью, у меня короче перебирается src, и берёт последний id, и каждый раз перезаписывает j, поэтому получается всегда один id на всех ссылках(

[Виталий Киренков]

@EarthFM, а если так
codepen.io/vitaliy_kirenkov/pen/yjvrH

[EarthFM]

@DeLaVega , Выводит undefined(. var j = nameImg[index]; index должен быть всегда 1, для того чтобы вытащить из первой скобки РегВыр.