Create Stored Array in Javascript, and push element ID on each click

Question

I am trying to put element ID's into an array upon clicking the element, and store this array in local storage. Then call this array, and push more element ID's every time an element is clicked. It is turning out to be trickier for me, than I thought it would be.

I am using javascript/jquery.

My code:

var reportArray = []
var storedArray = JSON.parse(localStorage.getItem("reportarray"));
if(storedArray == '') {
    reportArray.push("foo");
    localStorage.setItem("reportarray", JSON.stringify(reportArray));
} else {
    reportArray.push(storedArray);
    localStorage.setItem("reportarray", JSON.stringify(storedArray));
}

I realize this code is probably way off (it doesn't even work correctly), but I am at least giving it a try. Help please?

Post an working example including HTML and JavaScript snippet — Adam Azad
– Adam Azad, Commented Jan 22, 2016 at 8:33
Thats the problem. I am unsure how to make it work. That is why I am asking for help. — user3259138
– user3259138, Commented Jan 22, 2016 at 8:34
It's not clear what is i3 and also the functions GetC and SetC — elTomato
– elTomato, Commented Jan 22, 2016 at 8:35

Ken Clubok · Accepted Answer · 2016-01-22 08:40:17Z

2

It doesn't look that far off. The main issue I see is that in the else clause, if the storedArray already exists, then you don't push any new elements to it. It might also be safer to check reportarray for null before attempting to parse it.

answered Jan 22, 2016 at 8:40

Ken Clubok

1,2486 silver badges11 bronze badges

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

2 Comments

user3259138 Over a year ago

My thoughts were that if it does exist, then I may want to "update" it with new element ID's... Would this not mean that I would need to push new elements to it? I am willing to admit that I could be thinking about this the wrong way.

Ken Clubok Over a year ago

You said that every time an element is clicked, you want to push an element ID to the array. That's what this code isn't doing. The line after the else should read something like storedArray.push("bar");

Prashant · Accepted Answer · 2016-01-22 09:43:08Z

1

API:

function storeId(id) {
    var ids = JSON.parse(localStorage.getItem('reportArray')) || [];

    if (ids.indexOf(id) === -1) {
        ids.push(id);
        localStorage.setItem('reportArray', JSON.stringify(ids));
    }

    return id;
}

The function takes an ID to store in the local storage. If the local storage item is not yet present, it creates a fresh new array otherwise uses the array from the local storage. Also, makes sure the id does not already exist while pushing it to the storage.

USAGE:

$(function () {
    $('.btn').click(function () {
        var id = $(this).attr('id');
        storeId(id);
    });
});

EXAMPLE

edited Jan 22, 2016 at 9:43

answered Jan 22, 2016 at 8:53

Prashant

8,1503 gold badges27 silver badges25 bronze badges

3 Comments

user3259138 Over a year ago

I tried to implement this, but I get an error in the javascript console: Uncaught SyntaxError: Unexpected end of input

Prashant Over a year ago

Check out the example fiddle. It works there. May be a syntax error on your implementation.

user3259138 Over a year ago

Your fiddle works, but for whatever reason, I am totally unable to get your exact code working in my script. I had to use el_tamatoe's code to get it to work. Thank you for your help! It was so invaluable.

elTomato · Accepted Answer · 2016-01-22 08:47:26Z

0

If you need to put an ID inside the local storage, this could be the strategy:

var reportArray = localStorage.getItem("reportarray");
if(reportArray != null && reportArray != '') {
    var storedArray = JSON.parse(reportArray);
    storedArray.push("foo");
    localStorage.setItem("reportarray", JSON.stringify(storedArray));
} else {
    var storedArray = ["foo"];
    localStorage.setItem("reportarray", JSON.stringify(storedArray));
}

Obviously you need to put the ID instead of "foo".

answered Jan 22, 2016 at 8:47

elTomato

3333 silver badges11 bronze badges

1 Comment

user3259138 Over a year ago

Ended up using this exact code, except for replacing foo with the id to get it working. Thank you!

OxyDesign · Accepted Answer · 2016-01-22 09:14:29Z

0

I would say something like this :

HTML :

<button id="elt1">Elt 1</button>
<button id="elt2">Elt 2</button>
<button id="elt3">Elt 3</button>
<button id="elt4">Elt 4</button>
<button id="elt5">Elt 5</button>

JS :

$(document).ready(function(){
  $('button').click(function(){
    var id = this.id;
    var reportArray = JSON.parse(localStorage.getItem("reportarray")) || [];
    if (!~reportArray.indexOf(id)) reportArray.push(this.id);
    localStorage.setItem("reportarray", JSON.stringify(reportArray));
  });
});

JS (push id even if already existing) :

$(document).ready(function(){
  $('button').click(function(){
    var id = this.id;
    var reportArray = JSON.parse(localStorage.getItem("reportarray")) || [];
    reportArray.push(this.id);
    localStorage.setItem("reportarray", JSON.stringify(reportArray));
  });
});

edited Jan 22, 2016 at 9:14

answered Jan 22, 2016 at 8:55

OxyDesign

7545 silver badges6 bronze badges

4 Comments

user3259138 Over a year ago

This one works... except for it only saves one id, not multiple id's in an array. Seems so close!

OxyDesign Over a year ago

Ok. Then just remove if (!~reportArray.indexOf(id)) It was to make sure every id is pushed only once

user3259138 Over a year ago

It still didn't seem to want to work for me. Probably my fault. Thank you so much!

OxyDesign Over a year ago

Really ? That's weird ... Mine works perfectly fine : jsfiddle.net/OxyDesign/yvtkwr4t (I'm trying with Chrome)

Collectives™ on Stack Overflow

Create Stored Array in Javascript, and push element ID on each click

4 Answers 4

2 Comments

3 Comments

1 Comment

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

2 Comments

3 Comments

1 Comment

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related