Check if object is defined where object name is string

Question

Is there a way to do the following without using eval()?

The following function accepts an array of strings where the strings are object names. I iterate over them to ensure that none of them are undefined. I'd like to be able to do this without using eval()

function hasMissingDependencies(dependencies) {
    var missingDependencies = [];

    for (var i = 0; i < dependencies.length; i++) {
        if (eval('typeof ' + dependencies[i]) === 'undefined') {
            missingDependencies.push(dependencies[i]);
        }
    }

    if (missingDependencies.length > 0) {
        return true;
    }

    return false;
}

With this, I'm able to pass something like the following and get what I want:

alert(hasMissingDependencies(['App.FooObject', 'App.BarObject']));

I'd prefer to not use eval() but haven't found another way to check if an object is undefined when the object names are passed as strings.

I'm unaware of another way to pass object names that may be undefined to another function. For example, calling hasMissingDependencies(App.FooObject, App.BarObject) throws an error before ever making it into the function if either of those objects are not defined. Am I missing something? — Jared
– Jared, Commented Apr 9, 2013 at 19:11
where are you passing it from? You can see my answer for a possible solution, but it would be much simpler to check if they exist BEFORE you pass them, and then pass null or some other value if not. — Ben McCormick
– Ben McCormick, Commented Apr 9, 2013 at 19:13
If App is undefined or null, then yes, it would throw an error because you're trying to dereference a null reference. But if App is not undefined or null, then you can pass those two arguments and then just check if they are undefined in the function, which would be checking if the FooObject and BarObject properties exist on the App object. — ajp15243
– ajp15243, Commented Apr 9, 2013 at 19:14
@ben336 My use case is that I'm creating "modules" that can be used by other developers in our application. This function is used to ensure that the modules that the current one depends on are included on the page. So to make it "easy", each module defines a list of dependencies and calls this function. Ideally we'd be using something like RequireJS for this, but for reasons I won't get into it's unlikely this will change. — Jared
– Jared, Commented Apr 9, 2013 at 19:17

Ben McCormick · Accepted Answer · 2013-04-09 19:11:06Z

5

Assuming that these are global variables (IE not defined within an inner function scope), you could split on periods and use bracket notation.

something like

function isDefined(name, obj){
   if(!obj){
      obj = window;
   }
   var pathArr = name.split(".");
   obj = obj[pathArr.splice(0,1)];
   if(typeof obj !== undefined){

     return pathArr.length ? isDefined(pathArr.join("."),obj): true;
   }

   return false;


}

Note that unless you're trying to write a requireJS replacement, this is pretty much always a bad idea though. Check if the values exist in the context you're calling the function, and then pass in the values if they exist or null/undefined/some default value otherwise. If you don't control the passing context, ask for objects and then check to see if they're defined.

answered Apr 9, 2013 at 19:11

Ben McCormick

25.8k12 gold badges56 silver badges71 bronze badges

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

1 Comment

Jared Over a year ago

Noted. Thanks for the help. I'm going to mark this answer as accepted since it's the easiest to follow, but I encourage anyone using this thread to look at the answer @dystroy for a more concise version of this.

Denys Séguret · Accepted Answer · 2013-04-09 19:12:14Z

2

You seem to want to check if global variables whose names are in your array are defined.

You can build a simple function to get those variable :

function getVar(name) {
   var v = window, path=name.split('.');
   while (path.length>0 && v) v = v[path.shift()];
   return v;
}

which would allow you to test

if (typeof getVar(dependencies[i]) === 'undefined')

edited Apr 9, 2013 at 19:12

answered Apr 9, 2013 at 19:03

Denys Séguret

384k90 gold badges813 silver badges780 bronze badges

3 Comments

Ben McCormick Over a year ago

that was my thought too. That won't work for his example with namespaces though. window['App.FooObject'] will return undefined even if App.FooObject exists

Denys Séguret Over a year ago

@ben336 I hadn't understood the requirement. I think it's better now

Ben McCormick Over a year ago

yep, cleaner than my version. Upvoted. Though probably worth mentioning in the answer that this usually isn't a great idea :)

Collectives™ on Stack Overflow

Check if object is defined where object name is string

2 Answers 2

1 Comment

3 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

3 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related