-
-
Save mikeal/2504336 to your computer and use it in GitHub Desktop.
| JSON._dateReviver = function (k,v) { | |
| if (v.length !== 24 || typeof v !== 'string') return v | |
| try {return new Date(v)} | |
| catch(e) {return v} | |
| } | |
| JSON.parseWithDates = function (obj) { | |
| return JSON.parse(obj, JSON._dateReviver); | |
| } |
i wouldn't override the global one, other people are going to assume certain properties are strings when they aren't.
also, this is always going to be slower than parsing without a reviver, so i wouldn't override it globally.
Sure, it's a judgment call. But if most of the json you're parsing has dates and you use dates a lot and you want them to be date objects, this is way better than always wondering which one you should use.
yeah, I wouldn't either. I'd just use it on json I'd suspect has dates in it so that I wouldn't have to individually Date.parse them afterwards.
@polotek, good point though.
what i was trying to say was, if you override it globally and then use someone else's library which internally uses JSON.parse they will get results they are not expecting, which would be very bad. if you want it to be this way in your code just always use this JSON function but don't override the global.
How's this?
JSON._dateReviver = function (k,v) {
if (v.length !== 24 || typeof v !== 'string') return v
try {return new Date(v)}
catch(e) {return v}
}
JSON.parseWithDates = function (obj) {
return JSON.parse(obj, JSON._dateReviver);
}
stealing.
i'd use JSON.dateyParse or JSON.timeyParse or something fun like that. think rachel ray: yummo! sammies!
JSON.dateParsely fits standard naming convention
How about dateify? I've always thought stringify was weird so why not keep up the tradition?
the above is off the cuff and totally untested.