@rts1005410788 #2 这就是适合用 reduce 的场景, 这 review 的已经暴露水平了. js 写业务真用不上传统 for, 要用也是 for...of 啊, for 一出至少三行, 早些年还有些性能优势, 现在也就需要中断才用了.

review 的人 代码质量还不如你

你能就着他的 comments 争论赢他吗？比如他给的理由我觉得还是很充分的，你真的想推进，可以回复说 reduce 更优雅且更不容易出现越界问题，如果团队大家不了解我们是不是可以考虑做一下技术分享让大家更了解 ES 的新 feature 和 api ，提提高整体代码质量？ 如果你没这个本事在他给出合理理由的情况下，用更合理的方式驳倒，那就只能照着做了 个人不觉得这是什么很苦恼的事

兄弟 单说对 js 这门语言的掌握，你的水平高他一个层次。

reduce 都看不懂还干什么开发，你们同事 base 低于 1w ？

这个用 reduce 应该没问题，但应该尽量避免用 reduce 很容易让代码变的晦涩难懂。

第一眼代码一定要是整洁的，然后内部逻辑要干净，比如 while + flag 这种判断肯定就设计的不好，还有就是代码注释、命名合理、NPE 处理等等，好代码、差代码还是能体会到的

你的代码没问题，reduce 比单纯循环简洁，清晰一些

OP 示例的代码在 MDN 上不建议使用 reduce ，而是推荐使用 for 循环代替： https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/reduce#when_to_not_use_reduce

另外，新手能看懂的代码是好代码