freeCodeCamp / wiki

freeCodeCamp's deprecated wiki articles

http://www.freecodecamp.com/wiki

281 stars 307 forks source link

Added ES6 Alternative Intermediate Code Solution #1090

Closed bmorelli25 closed 8 years ago

bmorelli25 commented 8 years ago

FreeCodeCamp - Wiki Pull Request Template

Added in a new solution utilizing ES6's new spread operator. This is a new solution that works well and is easy to understand.

ghost commented 8 years ago

@Rafase282, Do we want another solution or this challenge?

ghost commented 8 years ago

LGTM :+1:

Rafase282 commented 8 years ago

Nope, only 3 solutions. @bmorelli25 @atjonathan

This needs to be fixed.

bmorelli25 commented 8 years ago

@atjonathan Do you want to leave it as-is or swap out one of the solutions? Let me know and I'll either close the pull request or make the changes

Rafase282 commented 8 years ago

We would have to decide in which category it falls then to either replace or merge the solution.

Rafase282 commented 8 years ago

After taking some time to look at the answer I think we gain nothing from adding it. It uses elements from the current intermediate and advance solutions, call it a hybrid if you will. Nothing wrong with the code but I would say why not use map already? and we have one that already does that. I'm closing this for now but feel free to discuss further if you do not agree. @bmorelli25