avoid declaring global variables

If I understand correctly, this works because it assumes the array.length is < 1000. Would cause a nasty bug in production ;)

Cheeky bastard... I love it!

Please consider this is used in a real codebase:

const foo = [2,3,5];
const bar = distributionOf(foo);
console.log(foo);

The above outputs [] instead of [2,3,5], because the array foo was modified from inside the distributionOf function. Functions shouldn't mutate their parameters.

Please consider this is used in a real codebase:

const foo = [2,3,5];
const bar = distributionOf(foo);
console.log(foo);

The above outputs [] instead of [2,3,5], because the array foo was modified from inside the distributionOf function. Functions shouldn't mutate their parameters.

Please consider this is used in a real codebase:

const foo = [2,3,5];
const bar = distributionOf(foo);
console.log(foo);

The above outputs [] instead of [2,3,5], because the array foo was modified from inside the distributionOf function. Functions shouldn't mutate their parameters.

Please consider this is used in a real codebase:

const foo = [2,3,5];
const bar = distributionOf(foo);
console.log(foo);

The above outputs [] instead of [2,3,5], because the array foo was modified from inside the distributionOf function. Functions shouldn't mutate their parameters.

Thanks for the feedback! I'm surprised that a test case did not cover that.

A good practice is to always specify second parameter for parseInt(): radix, in this case it should be 10. Otherwise if the array contained 0 you could be in for a surprise ;)

A good practice is to always specify second parameter for parseInt(): radix, in this case it should be 10. Otherwise if the array contained 0 you could be in for a surprise ;)

I think you can either drop the filter part or change regexp to /(, )(\S+)$/ (or even /, (\S+)$/, 'and $1') to avoid redundancy.

I like how this solution doesn't count $100 bills, that some (indluding me) unnecessarily keep count of…