on each button click store value in array

Question

<input type="button" class="sal" value="0">
<input type="button" class="sal" value="0">
<input type="button" class="sal" value="0">
<input type="button" class="sal" value="0">

On each button click store clicked value in a array I have tried this

$( document ).ready(function() {
            $('.sal').each(function() {
            $(this).click(function(i) {                    
                 var i[]=$(this).val();
                console.log(i);
            }); 
          });
});

What I am missing here !!!!!

Vladimir Drenovski · Accepted Answer · 2016-05-18 10:13:54Z

3

You can try this code with your html:

var i = [];

    $( document ).ready(function() {        
        $('.sal').each(function() {
                $(this).click(function(e) {        
                     i.push($(this).val());
                    console.log(i);
                }); 
         });
    });

JSFiddle

What is wrong with your code? first of all you are not initializing the i array properly. also you need to define it outside of the scope of the click function because you are going to loose it after the function is done. If you do not want to pollute the global scope you can define the array inside the function of the ready() method.

edited May 18, 2016 at 10:13

answered May 18, 2016 at 10:07

Vladimir Drenovski

3002 silver badges10 bronze badges

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

2 Comments

arpit Pathak Over a year ago

I want that array result outside the function

Vladimir Drenovski Over a year ago

It is outside of the function as you can see from the code its self. Like i said in other comments, this array can be accessed from anywhere. Therefore this is exactly what you want.

Jitendra Tiwari · Accepted Answer · 2016-05-18 10:02:46Z

1

Try this

$( document ).ready(function() {
    var arr=new Array();
    $('.sal').each(function() {
        $(this).click(function(i) {
            arr.push($(this).val());
        }); //missing ); here!
    });
});

answered May 18, 2016 at 10:02

Jitendra Tiwari

1,6913 gold badges16 silver badges28 bronze badges

Comments

Dhara Parmar · Accepted Answer · 2016-05-18 10:15:23Z

1

You can create array to store values of all clicked buttons:

Fiddle: https://jsfiddle.net/1tbLbmkg/

    var values = []; // declare it outside document.ready
    $( document ).ready(function() {   
        $('.sal').click(function(i) {
             values.push($(this).val()); // add value to array
        }); 
    });

edited May 18, 2016 at 10:15

answered May 18, 2016 at 10:02

Dhara Parmar

8,1231 gold badge19 silver badges27 bronze badges

3 Comments

arpit Pathak Over a year ago

but then I am not able to access values in values array when I finished with clicks (console.log(values);)

Vladimir Drenovski Over a year ago

Yes you can access them, the values array defined in this code is global it can be accessed from anywhere.

Dhara Parmar Over a year ago

check fiddle...in that you can get array

gavgrif · Accepted Answer · 2016-05-18 10:36:24Z

1

You can simplifyy this function. note that since all inputs contain 0 as their value- you will end up with a bunch of zeroes in the aray. You also have to declare the empty array before pushing the value into it. Fiddle:https://jsfiddle.net/gavgrif/xL7qsbsn/

$( document ).ready(function() {
            var newArray= new Array();
            $('.sal').click(function() {
                 newArray.push($(this).val());
                  alert("newArray contents = "+ newArray)
            });
       });

<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<input type="button" class="sal" value="0">
<input type="button" class="sal" value="0">
<input type="button" class="sal" value="0">
<input type="button" class="sal" value="0">

edited May 18, 2016 at 10:36

answered May 18, 2016 at 10:02

gavgrif

15.5k2 gold badges30 silver badges31 bronze badges

5 Comments

Audrius Lubys Over a year ago

newArray[] is not valid JavaScript.

gavgrif Over a year ago

I have been taught that it is - that [] the equivalent of new array(), but I will change it for you.

Dhara Parmar Over a year ago

Your code is not working at all..throwing jquery error: jsfiddle.net/03qjqags

Audrius Lubys Over a year ago

I meant newArray[] = $(this).val();. As far as I'm aware it works on php, but on JavaScript it should be newArray.push($(this).val());

gavgrif Over a year ago

sorry - answer amended and test it works now. Thanks for the fiddle @Dhara Parmar

Collectives™ on Stack Overflow

on each button click store value in array

4 Answers 4

2 Comments

Comments

3 Comments

5 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

2 Comments

Comments

3 Comments

5 Comments

Your Answer

Sign up or log in

Post as a guest

Related