Create an array from an each loop

Question

How is an array done from an each loop and then pass it on in a post with ajax? Now it is just making one array for each element, and doesn't post anything to the ajax.

$('a#export').on('click',function(){

    $('[id^="imgHolder"], [id^="textHolder"]').each(function(){

        TextElementID = $(this,'[id^="textHolder"]').attr('id');
        TextElementContent = $(this,'[id^="textHolder"]').text();   

    TextObjects = new Array(TextElementID, TextElementContent);   

        ImgSrc = $('img',this,'[id^="imgHolder"]').attr('src');
        ImgHolderID = $(this,'[id^="imgHolder"]').attr('id');
        ImgHolderClass = $(this,'[id^="imgHolder"]').attr('class');            

    ImgObjects = new Array(ImgHolderID, ImgHolderClass, ImgSrc);

        });

        $.ajax({
            url: "post.php",
            type: "post",
            data: { 
                ExportObjects: ImgObjects
            },
            success: function(){
               alert("success");
            },
            error: function(){
                alert("failure");
            }   
              });

    });

sriniris · Accepted Answer · 2013-05-12 20:00:24Z

1

There are a couple of problems with this. You are initializing the array in the each and will end up overwriting the values. Also you are doing an each on the text and images are are collecting the values of both text and images in the loop and will end up with duplicates. Here's a working version of the fiddle.

var ImgObjects = new Array();
$('[id^="imgHolder"]').each(function(){
    ImgSrc = $('img',this,'[id^="imgHolder"]').attr('src');
    ImgHolderID = $(this,'[id^="imgHolder"]').attr('id');
    ImgHolderClass = $(this,'[id^="imgHolder"]').attr('class');            
ImgObjects.push(ImgHolderID, ImgHolderClass, ImgSrc);
    });

var TextObjects = new Array();
$('[id^="textHolder"]').each(function(){
    TextElementID = $(this,'[id^="textHolder"]').attr('id');
    TextElementContent = $(this,'[id^="textHolder"]').text();   
    TextObjects.push(TextElementID); 
    TextObjects.push(TextElementContent);
});

answered May 12, 2013 at 20:00

sriniris

1334 bronze badges

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

1 Comment

Kim Over a year ago

Thanks a lot! Worked exactly as I would hope!! + for you!

Collectives™ on Stack Overflow

Create an array from an each loop

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related