Calling var array=[] within each function

Question

I'm working on a project where I have different sets of div like heading, text, image etc..

Upon clicking I want to append input boxes for that relevant numbers of divs.

You can see working fiddle here : http://jsfiddle.net/UN62E/

This script does a job for me but I need improvement with this script.

This is how part of my script look like :

var eltofind = "heading";

if($(this).find("." + eltofind).length != 0){

    var eltoget = $(this).find("." + eltofind);

    $(eltoget).each( function(index) {
         var item = $('input.' + eltofind + ':first').clone();
        var count = (index +1);
        var getting = eltofind + "-" + count;
        item.addClass(getting);
        item.appendTo('#input');
    });
}

Currently I'm repeating whole script for individual sets of div. Is there a way to call all sets of divs within array? for example :

var eltofind = ["heading", "text", "image"]

& call them within each function ?

My jQuery skills aren't that good & Any help would be appreciated.

Ortiga · Accepted Answer · 2012-06-26 17:21:07Z

3

You can make multiple selectors in jQuery

$(".heading, .text, .image").foo();

answered Jun 26, 2012 at 17:21

Ortiga

8,8466 gold badges45 silver badges74 bronze badges

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

Comments

mVChr · Accepted Answer · 2012-06-26 17:23:50Z

1

Just run $.each() on the array an replace eltofind with the el argument:

var elstofind = ["heading", "text", "image"];

$.each(elstofind, function(i, el){

    if($(this).find("." + el).length != 0){
        var eltoget = $(this).find("." + el);

        $(eltoget).each(function(index) {
            var item = $('input.' + el + ':first').clone();
            var count = (index + 1);
            var getting = el + "-" + count;
            item.addClass(getting);
            item.appendTo('#input');
        });
    }

});

answered Jun 26, 2012 at 17:23

mVChr

50.3k11 gold badges111 silver badges105 bronze badges

Comments

Robert · Accepted Answer · 2012-06-26 17:20:18Z

0

Create a loop that loops through each value of eltofind and performs the appropriate actions.

var eltofind = ["heading", "text", "image"];

for(var i=0; i < eltofind.length; i++){
     var thisElement = eltofind[i];

    if($(this).find("." + thisElement).length != 0){

        var eltoget = $(this).find("." + thisElement);

        $(eltoget).each( function(index) {
            var item = $('input.' + thisElement + ':first').clone();
            var count = (index +1);
            var getting = thisElement + "-" + count;
            item.addClass(getting);
            item.appendTo('#input');
        });
    }
}

answered Jun 26, 2012 at 17:20

Robert

8,7672 gold badges29 silver badges34 bronze badges

Collectives™ on Stack Overflow

Calling var array=[] within each function

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related