6

Could somebody explain to me the difference between if(obj.x == undefined) and if(typeof obj.x == 'undefined')

In some context the first one works fine, but in other I need to use the second way.

Questions

1 - What is the difference between the two condition?

2 - Is there a best practice?

6
  • see stackoverflow.com/questions/776950/… Commented Aug 29, 2011 at 14:59
  • Often it's good enough to just check the truthiness of the value: if (obj.x). Commented Aug 29, 2011 at 15:01
  • undefined is a variable, not a constant, and can be assigned a value. Because of this, one school of thought says the second path is safer, since you cannot be sure of the value of undefined. There is another school of thought that says those who redefine undefined deserve exactly what they get. Commented Aug 29, 2011 at 15:07
  • If you want to know if a property exists on an object, do this if ( 'x' in obj ) or this if ( obj.hasOwnProperty( 'x' ) ). (Use the second one, if you don't want to look up the prototype chain. Commented Aug 29, 2011 at 15:10
  • @Chris Nielson - as of 1.8.5 undefined is non-writable Commented Aug 29, 2011 at 15:15

6 Answers 6

10

The best practice is to not just check the truthiness but the strict equality

example

if (obj.x === undefined) {}

this use to be an issue because undefined (a global property) use to be writable, as of 1.8.5 is is non-writable, providing you with a secure comparison in ES5 spec environments.

per MDN

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

9 Comments

A secure comparison? In which environment? Afaik, you can assign to the undefined variable in all modern browsers...
It's still not reliable. Even in ES5, undefined can be used as a variable name within a function: (function() { var undefined = "foo"; alert(undefined); })()
Then why would the MDN choose this as the primary example of how to see if a variable is undefined?
@Šime: @jondavidjohn is right about the global undefined property being read-only in current browsers.
@jondavidjohn: MDN is not infallible.
|
1

The two would usually be equivalent if you replaced the equality operator == with the strict equality operator ===. So obj.x === undefined and typeof obj.x == "undefined" are usually equivalent.

However, in pre-ECMAScript 5 environments (which still acount for the majority of web requests, in general), undefined is a writable property of the global object, meaning that undefined may be used as variable name or the global property may be assigned a different value. ECMAScript 5 makes the global property read-only, but even then, undefined may still be used as variable name within a function, meaning that the typeof check is always safer.

One further point in favour of typeof is that it may be used to check for a variable that may not have been declared whereas a direct comparison will throw a ReferenceError if the variable has not been declared. For example:

typeof foo == "undefined" // true
foo === undefined // ReferenceError

However, this is an unusual and not generally helpful thing to be doing.

3 Comments

In what browsers is undefined (the global variable) not writable?
@Šime: Try it. I've tried it on my machine in Firefox 6, Chrome 13, Safari 5.1 and Opera 11.50. Safari and Firefox do not allow assignment to the undefined global object property, Chrome and Opera do. However, that's not very relevant: the point is that a compliant ES5 environment will not allow it, and such environments do exist.
And I just happened to only have Chrome and Opera on my machine :).
1

The two are not equivalent tests because of the quite convoluted handling of special values by javascript. In the specific

undefined == null

is true, but typeof undefined is "undefined" while typeof null is "object".

The rules for those special values are quite complex and IMO illogical, so I think there's no "general rule". What you may find are common forms, for example

var value = obj.x || default_value;

that can be used if you're sure that obj will never be undefined or null (because in that case an exception would be thrown) and assuming that 0, NaN or an empty string should be considered as if no value was provided (because they're all "logically false" values). An empty array or an empty javascript object instead are considered "logically true".

Why is it that way? Why does (null).x throw an exception when null according to typeof is apparently an object and searching for a non-existent field in an object normally returns undefined instead?

I've no idea.

I never tried to find a logic in all those strange rules. I'm not actually even 100% sure there's one.

My suggestion is just to study and experiment with them.

2 Comments

If you're unsure of why this admittedly unintuitive behaviour happens, check the ECMAScript specification. Your observations follow directly from well-defined rules you'll find there.
@Tim Down: I've actually tried to check the specs but I personally found that document close to impossible to read. I prefer experimenting with those side cases because anyway it wouldn't be that helpful writing programs that in theory should run but that in practice don't (unless you target is not writing software that runs but just being able to find someone to blame instead).
0

second is easier and faster than first. First requires additional setup, definition of undefined and check wheter obj contains x as a property or method. second makes the check whether obj.x has been whenever defined and assigned

PS.: undefined will be evaluated to null so obj.x == undefined is equivalent to obj.x == null

Comments

0

the best way to test for conditionality isusing obj.x this checks for both null-ability and undefined .

Thus

if(!obj.x) 
{
  alert(obj.x);
}
else   
{
  alert("obj.x is null or undefined");  //obj.x is null or undefined or any false value which is what you want. like obj.x is false or 0
}

1 Comment

This is not correct. The else-branch will be executed if obj.x is any falsy value, not just null or undefined.
0

The main difference of these two condition is typeof

The typeof operator is used to get the data type (returns a string) of its operand. The operand can be either a literal or a data structure such as a variable, a function, or an object. The operator returns the data type.

if (typeof obj.x === 'undefined') {
    txt = "x is undefined";
  } else {
    txt = "x is defined";
  }

Comments

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.