Stikkord: extjs

Roll-your-own date parser in Javascript – please don’t!

 I was debugging some code the other day where there was a problem with the date being set incorrectly. A date picker was used in the GUI to set a date that was sent to the server. For some reason, when a date in September was picked, the date saved in the database ended up being in December! Well, it turns out that since the date format was using leading zeroes, and Javascript has this magic feature (apparently deprecated, but still in use) where if the number passed to the parseInt function starts with 0 it is interpreted as an octal value! Now this works fine with values from 01 – 07 (since they are the same anyway), but obviously not with 08 and 09. If the function had been used previously we might have spotted the pattern, but it was only starting to be used now.

Morale of the story? Don’t reinvent the wheel, there are many good already existing javascript date parsers: Moment.js, Datejs or as a part of ExtJS (if you’re already using that) or as a a JQuery plugin (if you’re using that). Secondly; using an IDE with proper Javascript support with quick documentation lookup might have helped spotting the problem in the first place.

At last, if you really, really,  have no option but to roll your own, then please test it with all (possible or feasible) permutations!

Reklamer