Comments on: Optimized Speedy Ajax Code

By: Mourner

Mourner — Thu, 04 Oct 2007 13:03:05 +0000

As others have already pointed out in the article comments, this code doesn’t work at all, because readyState is only checked once before sending the request (instead of polling or putting it on onreadystatechange event)

By: Anonymous

Anonymous — Thu, 04 Oct 2007 08:56:33 +0000

@deadcabbit What you say is true, but only if the effect is (near) transparent. In the case of try-catch it isn’t on the iPhone. It literally takes seconds to pass try catch blocks.

By: deadcabbit

deadcabbit — Thu, 04 Oct 2007 06:04:57 +0000

Am I the only one, who prefers higher level “complexity”? I know that try-catch is slow or that foreach-like syntax is bad for performance, but I prefer to let these things handled by frameworks (either inhouse, or for pet projects prototype.js) – hardware components and browsers are getting faster/better by the day (by the year, sry:)), but code reusability and readibility will not improve by itself.

By: Eric Pascarello

Eric Pascarello — Wed, 03 Oct 2007 22:24:53 +0000

@Rogier
Totally agree with you. Object detection is always the way to go.

My comment is basically why you probably would want to use the around the ActiveX. Talked about the error a long time ago on my blog here: http://radio.javaranch.com/pascarello/2006/02/07/1139345471027.html

Eric

By: Rogier

Rogier — Wed, 03 Oct 2007 21:02:46 +0000

@Eric

Considering your argument about disabling ActiveX, I’d still narrow the try/catch block down to the ActiveX control construction.

But you simply don’t need to try/catch anything, if ActiveX is disabled and window.XMLHttpRequest doesn’t exist you can perfectly verify that you got what you need without “trying”.

By: Eric Pascarello

Eric Pascarello — Wed, 03 Oct 2007 19:43:02 +0000

[quote] I donâ€™t understand why frameworks must use a try/catch (try/catchs are usually not very fast, and very annoying with Firebug).[/quote]

Disable ActiveX and see what happens.

By: Kris Kowal

Kris Kowal — Wed, 03 Oct 2007 17:54:45 +0000

Neglecting the try/catch and micro-optimization accusations for a moment, I started with a similar chunk of code copied from Dojo (dojo/src/hostenv_browser.js) and applied the lazy function pattern as well, except that I managed to refactor some of the logic by using an array of functions.

https://cixar.com/tracs/javascript/browser/trunk/modules.js?rev=136#L1433

Microsoft.XMLHTTP will always get versions up to 3. Checking for Msxml2.XMLHTTP is a waste of time since it always gets version 3. Msxml2.XMLHTTP.4.0 and 6.0 are not returned by Microsoft.XMLHTTP. 6.0 was released with Vista.

http://www.telerik.com/documents/AJAX%20Page/Ajax-Part1.pdf page 3

This is a great exercise for the Lazy Function Declaration pattern, if only for its educational value, or because try/catch is expensive.

This sample is pretty green though (in the arboreal sense). It doesn’t take into account status eccentricities of various browsers over various protocols. For example KHTML doesn’t mask the 304 cache hit status (other browsers handle it and change it to a 200). Also in some browsers, under various conditions, you’ll get “undefined” or “0″ for OK.

By: Anonymous

Anonymous — Wed, 03 Oct 2007 17:16:16 +0000

In fact, try catch blocks have been shown to be extremely slow on certain browsers (the iPhone’s version of Safari for example).

By: Kris Zyp

Kris Zyp — Wed, 03 Oct 2007 16:42:56 +0000

Ignoring the lazy pattern for a minute, why can’t the XHR object be retrieved with simply:

if (window.XMLHttpRequest)
return new window.XMLHttpRequest();
else if (window.ActiveXObject)
return new ActiveXObject(“Microsoft.XMLHTTP”);

This what I have used for a long time without any problems. I don’t understand why frameworks must use a try/catch (try/catchs are usually not very fast, and very annoying with Firebug). And why the different ActiveXObjects, the MS specs seem to make it clear that you don’t need to do that (the code above should always resolve to the latest version)? Is this just case of code that has been copied time and time again without question?

By: TimD

TimD — Wed, 03 Oct 2007 15:27:04 +0000

Useless there is little overhead in creating the correct XHR and loading the data over HTTP. There is much more effective result in optimizing the parsing of JSON / XML / etc.