@rts1005410788 你们 review 通常不是你们 leader ？这水平还不如你，怎么敢 review 你的代码还大放厥词的。

@rts1005410788 就算要循环，也是这样比较简单吧： const obj = {}; arr.forEach(item => { obj[item.id] = item.name; });

可维护性好，可读性强，尽量大白话，少用抽象和注解，高度泛型抽象的拉过去打一顿问问你这破业务值不值这么玩，不过度依赖框架，命名规范，尽量不要命名方法和类高度相似的，禁止使用魔法值，统一管理全局静态类，全局变量尽可能放到一个类里，全局变量和静态类，枚举类必须有注释，禁止在正式项目里塞数据库代码生成器这些奇技淫巧，有足够接口文档，并且接口文档要有接口范例，前后端统一 post 请求，项目启动和部署尽可能提供一键脚本，最好有文档，哪怕主程序和组长突发意外死亡也能立刻找人接手

据我多年的工作经验来说，能满足老板要求能运行的就是好代码，其它都是扯淡

@youyouzi 他就是 forEach ，我纠结的 review 时候，真的有必要纠结这种细节代码，搞的被 review 那个人很没有自信。比如我，😂

补充一下，任何与跑项目没关系的代码生成工具都禁止放到项目里，这东西设计的人过于个性化，鬼知道今天把你裁了后面接手的能不能看明白你写的鬼画符

好的代码就是，1000 行的代码，看个三五十行就能理解做了什么事情，改起来不会瞻前顾后，能很快定位到要改动的地方。

@rts1005410788 #14 不必在意。你的写法已经很优雅了。 我是 review 的人的话，看到你这个代码是没问题的。 反而组内很多人喜欢写循环操作，我会让他尝试换个思路，提升一下技能。 当然，不是强制要求，毕竟每个人对工作态度不一样，不是太混的，写的太恶心的我一般都是直接 merge 了

能实现功能能赚钱的就是好代码

不同的写法，只要可读性，性能没问题就行，纠结这些真没啥意义。