I think your approach is fine, but just to be that guy I'll condense it some more (could be output as a hash but, if we're going to condense, well...):
```js const minMax = arr => arr.reduce( ([ min, max ], curr) => [ Math.min(curr, min), Math.max(curr, max) ], [ Infinity, -Infinity ] ) On Mon, 2 Oct 2017 at 16:34 Naveen Chawla <[email protected]> wrote: > I would just use reduce for this. Reason: I think any multi var result > format is a little messy. I find it better to let the dev decide on the > result format, e.g.: > > ```js > const minMax = > array.reduce( > (accumulator, currentValue)=>{ > return { > min: Math.min(currentValue, accumulator.min), > max: Math.max(currentValue, accumulator.max) > } > }, > { > min: Infinity, > max: -Infinity > } > ) > ``` > > Good thing is, this can easily be refactored to accept arrays with objects > that contain the values, instead of just an array of numbers, as well as > the ability to calculator other accumulated values (e.g. mean average etc.) > in the same call. > > Do let me know if you think I'm missing the point > > On Mon, 2 Oct 2017 at 19:16 Boris Zbarsky <[email protected]> wrote: > >> On 10/2/17 7:10 AM, Xavier Stouder wrote: >> > Don't know what Boris mean when he talks about recreation bits >> >> Fwiw, it looks like the code at >> https://esbench.com/bench/595c1b1899634800a03488b9 does not have the >> array recreation bits (function whatever(...args)) that earlier >> benchmarks for this had. >> >> -Boris >> _______________________________________________ >> es-discuss mailing list >> [email protected] >> https://mail.mozilla.org/listinfo/es-discuss >> > _______________________________________________ > es-discuss mailing list > [email protected] > https://mail.mozilla.org/listinfo/es-discuss >
_______________________________________________ es-discuss mailing list [email protected] https://mail.mozilla.org/listinfo/es-discuss
