jQuery: Remplacer les chaînes par .each()

Question

Je veux une fonction qui remplace chaque li par une image. Ceci est mon code:

$(document).ready(function(){ 
    var tmphref; 
    var tmpname; 
    var str = '<a href="' + tmphref + '"><img src="http://www.somesite.com/a/' + tmpname[1] + '/avatar-small.jpg /></a>'; 
    $('#somediv li a').each(function(){ 
     tmphref = $(this).attr("href"); 
     tmpname = /http\:\/\/(\w+)\.somesite\.com\//.exec(tmphref); 
     $(this).parent().replaceWith(str); 
    }); 
}); 

L'image est dans ce chemin spécifique: www.somesite.com/a/username/avatar-small.jpg
Le code ne fonctionne pas ci-dessus. Des idées?

Merci d'avance.

Answer 1

Déplacer

var str = '<a href="' + tmphref + '"><img src="http://www.somesite.com/a/' + tmpname[1] + '/avatar-small.jpg /></a>'; 

après tmphref et tmpname, comme celui-ci

$('#somediv li a').each(function(){ 
    tmphref = $(this).attr("href"); 
    tmpname = /http\:\/\/(\w+)\.somesite\.com\//.exec(tmphref); 
    var str = '<a href="' + tmphref + '"><img src="http://www.somesite.com/a/' + tmpname[1] + '/avatar-small.jpg /></a>'; 
    $(this).parent().replaceWith(str); 
}); 

car la variable str est des valeurs déjà attribuées avec undefinedtmphref et tmpname, changeant ainsi les valeurs de tmphref et tmpname après , n'affecterait pas la variable str

Et, pour ce cas, vous n'avez pas besoin de déclarer la variable pour tmphref et tmpname en dehors de la fonction .each.