date fuckery

fuckingDates.js

var times = { "Schedule": {
      "2014-09-27T00:00:00": [
        16
      ],
      "2014-09-28T00:00:00": [
        8,
        9,
        10,
        11,
        14
      ]
    }
};


_.map(times.Schedule, function(val, key) { 

    return _.map(val, function(hour, date){
        var d = new Date(key);
        return { 
          label: hour > 12 ? [hour - 12, '00 PM'].join(':') : [hour, '00 AM'].join(':'),
          value: new Date(d.setHours(d.getHours() + hour)).toISOString()
        }
    });

});

Here's my first stab at refactoring this. The only real differences are that I've given names to various operations and I'm using _.partial for redundant function calls with the same leading argument.

Pulling things apart and giving them names may seem like overkill, but I'd argue that it helps readability, the different pieces become easier to test and if there's a performance bottleneck (unlikely in this scenario...), it'd be trivial to use _.memoize to cache return values.