Javascript scope issue - variable is undefined

Question

I am slightly confused by this, as i am sure that all variables are taken to the 'top' of the javascript before run time and then processed from there.

So my error

 TypeError: hutber.portfolio.ko is undefined
 [Break On This Error]  
 items: ko.observableArray(hutber.portfolio.ko.data),

Object

(function ($) {
"use strict"; //For good development standards :)
hutber.portfolio = {
    init: function(){
        e(typeof(hutber));
        hutber.portfolio.changeOptionsBoxHeight();
        //Bind the height resize in window resize
        $(window).resize(function(){
            hutber.portfolio.changeOptionsBoxHeight();
        });
        //start KO
        hutber.portfolio.ko.init();
    },
    changeOptionsBoxHeight: function(){
        var height = $(window).height();
        $('.portfolio--options').height(height-400);
    }
};
hutber.portfolio.ko = {
    init: function(){
        ko.applyBindings(new hutber.portfolio.ko.portfolioViewModel());
    },
    data: [],
    items: ko.observableArray(hutber.portfolio.ko.data),
    portfolioViewModel: function(){

        hutber.portfolio.ko.items = ko.observableArray(hutber.portfolio.ko.data);

        $.getJSON('/js/pages/portfolio.json').done(function(info){
            hutber.portfolio.ko.data = info;
            hutber.portfolio.ko.items (hutber.portfolio.ko.data);
        });
    }
};
hutber.portfolio.init();
})(jQuery);

I really wanted to upload this to a fiddle but for some reason, i'm getting js errors on their site. I believe my firewall is blocking certain files from loading.

I'm getting a hutber is not defined. You can use this jsbin instead of jsfiddle — Pablo
– Pablo, Commented Jan 11, 2013 at 14:05
@PabloGonzálezAlba: hutber has not been defined. But that is not the question. — Amberlamps
– Amberlamps, Commented Jan 11, 2013 at 14:07
cheer for that :) add hutber = {}; above (function ($) { to declare it — Jamie Hutber
– Jamie Hutber, Commented Jan 11, 2013 at 14:08

Amberlamps · Accepted Answer · 2013-01-11 14:05:08Z

1

At the point ko.observableArray(hutber.portfolio.ko.data) is run, hutber.portfolio.ko has not be defined yet.

You can work around it like that:

hutber.portfolio.ko = {
    init: function(){
        ko.applyBindings(new hutber.portfolio.ko.portfolioViewModel());
    },
    data: [],
    portfolioViewModel: function(){

        hutber.portfolio.ko.items = ko.observableArray(hutber.portfolio.ko.data);

        $.getJSON('/js/pages/portfolio.json').done(function(info){
            hutber.portfolio.ko.data = info;
            hutber.portfolio.ko.items (hutber.portfolio.ko.data);
        });
    }
};

hutber.portfolio.ko.items = ko.observableArray(hutber.portfolio.ko.data);

But at this point hutber.portfolio.ko.data is always []. So you might as well put ko.observableArray([]) in your original code.

answered Jan 11, 2013 at 14:05

Amberlamps

40.7k6 gold badges46 silver badges54 bronze badges

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

3 Comments

Jamie Hutber Over a year ago

does indeed fix the error, another one pops up. but thats really the point now is it :) I hadn't thought about adding to the object on the fly like that. Not sure why i was so dead set on making sure it was defined in the objects chain as it were.

Amberlamps Over a year ago

Also hutber.portfolio.ko.items (hutber.portfolio.ko.data); this row might throw an error as long as ko.observableArray() does not return a function. But I do not know that much about Knockout.

Jamie Hutber Over a year ago

ye, i also thought that it had to be a function. Enough hunting round turns out it doesn't need to be a function. Just an object, array etc etc

Roman · Accepted Answer · 2013-01-11 13:59:33Z

0

Just guess: Because you accessing variable before you declare it?

answered Jan 11, 2013 at 13:59

Roman

5044 silver badges10 bronze badges

1 Comment

Jamie Hutber Over a year ago

pretty sure i have declared it. :/

Collectives™ on Stack Overflow

Javascript scope issue - variable is undefined

2 Answers 2

3 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related