javascript variable scoping of anonymous functions

Question

I have this javascript/jquery function in which I store each elements inside a div to an array. The problem is that it doesn't save the results outside of the anonymous function. How do you setup a global variable?

This is my Javascript/jquery

function store_options(){
    var stored_options = new Array;
    $('[id^="tagboxfv-"]').each(function(){
        var last_index = slice_last_index(this);
        $('[id="form-'+last_index+'"] > select[id="v-'+last_index+'"] > option').each(function(){
            stored_options[last_index] = [];
            stored_options[last_index][$(this).val()]=$(this);
        });
    });
}

possible duplicate of What is the scope of variables in JavaScript? — Brian Dillingham
– Brian Dillingham, Commented Sep 7, 2014 at 6:50

Alex Art. · Accepted Answer · 2014-09-07 06:51:29Z

2

In general using global variables in JavaScript is not recommended. Bu if you still want to use it just define it out of the function scope:

var stored_options = new Array;

function store_options(){    
    $('[id^="tagboxfv-"]').each(function(){
        var last_index = slice_last_index(this);
        $('[id="form-'+last_index+'"] > select[id="v-'+last_index+'"] > option').each(function(){
            stored_options[last_index] = [];
            stored_options[last_index][$(this).val()]=$(this);
        });
    });
}

As an alternative your function can have a return statement so you could use it anywhere you want (this way no global variables are introduced):

function store_options(){  
    var stored_options = new Array;  
    $('[id^="tagboxfv-"]').each(function(){
        var last_index = slice_last_index(this);
        $('[id="form-'+last_index+'"] > select[id="v-'+last_index+'"] > option').each(function(){
            stored_options[last_index] = [];
            stored_options[last_index][$(this).val()]=$(this);
        });
    return stored_options;
    });
}

answered Sep 7, 2014 at 6:51

Alex Art.

8,7813 gold badges32 silver badges50 bronze badges

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

3 Comments

taptipblard Over a year ago

Tried the first one but it only records the last option for the 2nd array i also tried the second one and it outputs undefined function fv_tags(){ var stored_options = store_options(); console.log(stored_options); });

Alex Art. Over a year ago

For the firstone - i think the problem is in slice_last_index(this) function - i assume that your indexes a overridden. Regarding the second one - you need to insure that store_options() defined in outer scope ( not inside $(document).ready ) and check out definition your fv_tags function as well

taptipblard Over a year ago

your answer is right. my issue is with setting up the multidimensional arrays

Jivanysh Sohoni · Accepted Answer · 2014-09-07 07:02:36Z

0

You can use a variable outside all functions

    <script>
      var someVar;
      function foo(){

      }
    </script>

You could also set a property of the window and use it as you like i.e

       window.propName = "asd";

answered Sep 7, 2014 at 7:02

Jivanysh Sohoni

573 bronze badges

Collectives™ on Stack Overflow

javascript variable scoping of anonymous functions

2 Answers 2

3 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related