Comment

Comment by twic

Is the final code equivalent to:

  var getIncompleteTaskSummaries = function(membername) {
    return fetchData().then(function(data) {
      var incompleteTasks = [];
      for (i = 0; i < data.tasks.length; ++i) {
        var task = data.tasks[i];
        if (task.username == membername && !task.complete) {
          incompleteTasks.push({id: task.id, dueDate: task.dueDate, title: task.title, priority: task.priority});
        }
      }
      incompleteTasks.sort(function(a, b) {
        return a.getTime() - b.getTime();
      });
      return incompleteTasks;
    });
  }
?

Apologies if that's wrong, i barely know JavaScript.

That code is not so great, and a bit '90s, but it's not appalling. The horrifically verbose non-currying code in the article isn't something i see any need to write. It looks like an outrageous strawman to me.

My code here could definitely improved with some functional idioms for the looping, picking, and sorting, but i wouldn't reach for currying here.

parent twic 12 years ago

Discussion

Live from the official Hacker News API

A modern reader for Hacker News · SvelteKit + Skeleton