reduce().map() ? ex20 about learnrx HOT 2 CLOSED

Yes there seems to be a bug in exercise 20. It passes if you run it in the website but if you run it externally it will give you an error where that last .map() is.

from learnrx.

CC: You can probably close this as a duplicate of #44 @morenoh149.

Good catch! It seems like a bug if you run reduce(...).map outside of the site but this is actually intentional. In Exercise 16 we write our own implementation of Array.prototype.reduce that returns the result in an Array. It has this little note:

Let's add a reduce() function to the Array type. Like map. Take note this is different from the reduce in ES5, which returns a value instead of an Array!

I found it confusing too. This was an intentional design choice that is discussed in #44. I'd summarize #44 by saying "we" decided to use the modified version of Array.prototype.reduce that returns an Array because it is similar to Observable.prototype.reduce returning an Observable. It's "more clear" to chain calls like reduce().map then it is to explain why the result types are different. (Personally I disagree with this approach. It'd be straightforward to explain we need to return an Array so we can chain the calls. 🤷‍♂️ )

from learnrx.