jQuery Array Undefined Error

Question

I have been wrecking my brains trying to figure out why I get an undefined error for the array statusIdValues when I hit the first push in the code below. Its initialised in a ready function and is pushed into when a checkbox state changes.

$(document).ready(function () {
  var statusIdValues = [];

      $(':checkbox').change(function () {

      if ($(this).is(":checked")) {
          statusIdValues.push($(this).attr("value"));
      }
      else {
           var index= statusIdValues.indexOf($(this).attr("value"));
           if (index > -1) {
           statusIdValues.splice(index, 1);
      }
    });
 });

Any help would be appreciated.

array.indexOf($(this).attr("value")); What is array ?

Suresh Atta
– Suresh Atta

2017-03-08 15:01:23 +00:00
Commented Mar 8, 2017 at 15:01 — Suresh Atta
– Suresh Atta, Commented Mar 8, 2017 at 15:01
typo, ill update

Bad Dub
– Bad Dub

2017-03-08 15:12:00 +00:00
Commented Mar 8, 2017 at 15:12 — Bad Dub
– Bad Dub, Commented Mar 8, 2017 at 15:12

JC Hernández · Accepted Answer · 2017-03-08 15:03:55Z

1

Try this, change in the else the variable "statusIdValues" for "index", and "array" for your actual array "statusIdValues", like this:

$(document).ready(function () {
  var statusIdValues = [];

      $(':checkbox').change(function () {

      if ($(this).is(":checked")) {
          statusIdValues.push($(this).attr("value"));
      }
      else {
           var index = statusIdValues.indexOf($(this).attr("value"));
           if (index > -1) {
           statusIdValues.splice(index, 1);
      }
    });
 });

answered Mar 8, 2017 at 15:03

JC Hernández

8074 silver badges14 bronze badges

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

3 Comments

Bad Dub Over a year ago

Thats it, would you believe I also had a second set of eyes on this we both didnt spot that typo! Thank you.

fdomn-m Over a year ago

So, this is the answer? Then why did you update your question to match this? Is there even a question now?

Bad Dub Over a year ago

I updated it because I realised there was a typo, which coincidentally was also the problem which JC also saw and fixed.

Collectives™ on Stack Overflow

jQuery Array Undefined Error

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related