Calling named function from other named function in javascript

Question

I have run into a problem with a named function in Javascript.

I have this reload function

   SN.Reload = function(settings) {

        var _timer = null;
        var $grid = null;

        var init = function () {

            $grid = $(settings.wrapperSelector);

            if (_timer > 0 || _timer != null)
                _timer = settings.timer;
            else
                _timer = 600000;

            window.setInterval(function () {

                LoadData();

            }, _timer);

        };

        var LoadData = function () {

            $.ajax({
                url: '/data.json',
                type: 'GET',
                dataType: 'json',
                cache: false,
                success: UpdateData,
                error: DataErrorHandler
            });

        };
}

In the normal state this will run LoadData function at X minutes - this works as intended.

I now have another named function

SN.CreateJsonFromDate = function (settings) {

 .... 

  var SuccessLoad = function () {

        _dateLoader.hide();
        _wrapper.slideUp();



    }
}

Is it possible to use LoadData from SN.Reload Inside the SuccessLoad function in SN.CreateJsonFromDate ?

The LoadData function call UpdateData on success an updates the HTML from the json data and I want to call this function again in SN.CreateJsonFromDate as this will generate a new json file.

Hang on...can I clarify that the intent of SN.Reload is to declare some variables (some of which are functions) and then do nothing with them and abandon them? — Katana314
– Katana314, Commented Jan 6, 2016 at 15:36
@Katana314 - yes you can clarify that, but it makes no difference. LoadData is only accessible inside the scope of SN.Reload and, as the code is written, is not accessible outside of it. — Adam Jenkins
– Adam Jenkins, Commented Jan 6, 2016 at 23:37
@Adam I kinda just assumed Reload was used for a primary purpose, and OP wanted to now use it for a secondary purpose. As the code example is written right now, it does not even have a primary purpose as it is never ever called and some minifiers would simply drop the functions entirely; I was implying he may have left out important code. The situation would change if, for instance, he had left out code that did make it accessible from the outside in some way. — Katana314
– Katana314, Commented Jan 6, 2016 at 23:55

Adam Jenkins · Accepted Answer · 2016-01-06 14:56:24Z

2

No, because simply LoadData does not exist outside the scope of SN.Reload

If you do want to re-use the LoadData function, do not restrict it's scope to being inside SN.Reload and instead, perhaps (depending on what you want), attach it to the namespace as SN.LoadData

answered Jan 6, 2016 at 14:56

Adam Jenkins

56.3k13 gold badges85 silver badges113 bronze badges

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

1 Comment

Morten Hagh Over a year ago

Yeah.. The problem was the scope! Rewrote the entire script to make the reload function accessible and it is working now! Thanks!

Sachin · Accepted Answer · 2016-01-06 15:01:29Z

2

Yes, if you instantiate SN.Reload. for example:

var obj = new SN.Reload(settings);

then you can use LoadData from this object, like this:

obj.LoadData();

And yes you have to make LoadData public using this:

this.LoadData = function(){/*your code*/}

edited Jan 6, 2016 at 15:01

answered Jan 6, 2016 at 14:59

Sachin

1,2181 gold badge10 silver badges20 bronze badges

Collectives™ on Stack Overflow

Calling named function from other named function in javascript

2 Answers 2

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related