Jquery: Get each image src

Question

I have a series of images each with the class "photo";

I want to go through each of these and retrieve the photo source, for use later in an if statement. I have written the below code to do this, but have not been successful:

$.each($(".photo"), function() {
  var imgsrc = $(this).attr("src").length;
  console.log(imgsrc);
});

I am not sure where I have gone wrong here. It seems to make sense to me, but I dont get anything in the console.

Can anyone point me in the right direction?

@ubercooluk thanks but its seems to make no difference whether I use it or not — MeltingDog
– MeltingDog, Commented Feb 25, 2013 at 5:48
.photo is the class of the image..can you doublecheck that — coolguy
– coolguy, Commented Feb 25, 2013 at 5:49

sites · Accepted Answer · 2013-02-25 05:56:39Z

24

If you have given same class name for all img tag then try this ,

  $(".photo").each(function() {  
   imgsrc = this.src;
   console.log(imgsrc);
  });

edited Feb 25, 2013 at 5:56

sites

21.9k18 gold badges91 silver badges147 bronze badges

answered Feb 25, 2013 at 5:50

EnterJQ

1,0168 silver badges18 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Alexander Kim Over a year ago

Why we can't use imgsrc = this.attr('src')? it throws me an error

DriveItLikeYouStoleIt Over a year ago

this.attr('src') doesn't work because "this" isn't jQuerified yet: to fix that, do $(this).attr('src)

sites · Accepted Answer · 2013-02-25 05:55:47Z

4

$(document).ready(function(){
  $(".photo").each(function() {  
       imgsrc = this.src;
       console.log(imgsrc);
  });    
});

edited Feb 25, 2013 at 5:55

sites

21.9k18 gold badges91 silver badges147 bronze badges

answered Feb 25, 2013 at 5:48

coolguy

7,9549 gold badges48 silver badges72 bronze badges

Comments

Jai · Accepted Answer · 2013-02-25 06:06:32Z

1

May be you are missing doc ready handler:

$(function(){
  $.each($(".photo"), function() {
    var imgsrc = $(this).attr("src").length;
    console.log(imgsrc); // logging length it should now print the length
  });
});

make sure to load this script first then your $.each() function:

<script src="http://code.jquery.com/jquery-1.9.1.min.js"></script>

answered Feb 25, 2013 at 6:06

Jai

74.8k12 gold badges77 silver badges104 bronze badges

Comments

yoda · Accepted Answer · 2013-02-25 06:02:06Z

0

You are trying to read the lenght of an object / array out of context here :

var imgsrc = $(this).attr("src").length;

change to

var imgsrc = $(this).attr("src");

edit : in order to check if the attribute exists

if (imgsrc == undefined) { /* do something */ }

edited Feb 25, 2013 at 6:02

answered Feb 25, 2013 at 5:52

yoda

11k19 gold badges68 silver badges94 bronze badges

2 Comments

MeltingDog Over a year ago

surely that would still work though? It would just give me a number instead of a string. I have length there because I want to make and if statement later on eg: if(imgsrc < 1)

yoda Over a year ago

you can still make that statement, if (imgsrc == undefined), see updated answer

Collectives™ on Stack Overflow

Jquery: Get each image src

4 Answers 4

2 Comments

Comments

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

2 Comments

Comments

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related