jQuery on check, add or update array object item

Question

I have a series of checkboxs and on change event like so:

$(".checkbox").on('change', function () {
          UpdateHolder($(this).attr('id'), customerID, this.checked);
});


function UpdateHolder(questionid, customerid, checked) {
        holder.push({ questionid: questionid, customerid: customerid, checked: checked });
        console.log(holder);
    }

with this, I am able to add an item to the array, but my question is how would I update an item ? I only want to update if a checkbox is selected or not.

Rory McCrossan · Accepted Answer · 2015-09-02 15:09:28Z

1

The easiest way to achieve this is to not update the list at all, but instead to create a new one on each action. Try this:

$('.checkbox').on('change', function() {  
    holder = $('.checkbox').map(function() {
        return { 
            questionid: this.id,
            customerid: customerID, // I presume this is a global variable?
            checked: this.checked
        };
    }).get();
    console.log(holder);
});

edited Sep 2, 2015 at 15:09

answered Sep 2, 2015 at 14:24

Rory McCrossan

338k41 gold badges321 silver badges353 bronze badges

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

2 Comments

Rory McCrossan Over a year ago

True, it doesn't match the topic of your question, but it does meet the goal and is, in practice, a far better pattern to use.

Rory McCrossan Over a year ago

@GuruprasadRao thanks - although I missed get() from my code. Here's an updated example: jsfiddle.net/1Lw5y2c7/1

PeterKA · Accepted Answer · 2015-09-02 14:39:40Z

To be able to update existing entries, your function should look something like this:

function UpdateHolder(questionid, customerid, checked) {
    var index = holder.indexOf({
        questionid: questionid,
        customerid: customerid,
        checked: checked
    });
    index = index > -1 ? index : holder.indexOf({
        questionid: questionid,
        customerid: customerid,
        checked: !checked
    });
    //item was not found, insert it
    if( index < 0 ) {
        holder.push({ 
            questionid: questionid, 
            customerid: customerid, 
            checked: checked 
        });
    //item was found, update it
    } else {
        holder[index] = {
            questionid: questionid,
            customerid: customerid,
            checked: checked
        };
    }
    console.log(holder);
}

Collectives™ on Stack Overflow

jQuery on check, add or update array object item

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related