Hacker News Reader

Comment by nkkollaw | Hacker News Reader

How would you have written that?

For point 1 I would use "reduce":

  lines.reduce((longest, line) => (longest.length > line.length) ? longest : line, "")

EDIT: sorry, initial code returned length, not line itself

This returns the length, not the line itself.

Edit: `lines.reduce((line, line_) => line_.length > line.length ? line_ : line, '')`

Thanks, edited.

It is a good use of reduce (accumulate as I call it), but maybe slightly overkill for this case compared to a simple iteration or a max function.