@MiguelTapia 你这个评价非常的到位， 就是控制欲望。

他的上限很高，你已经到头了

@encounter2017 你没有理解我的意思， 我不是反对好的写法我不认同，我反感点是，为什么组员用新的语法糖，review 时候你看不懂，你就让别人改，而你用的时候你却要大吹特吹，表现你的优越感。 角色调换一下位置如果我使用最直接的写法用 `a.entry("1").or_insert_with(Vec::new).push("1")` review 的那人又要说，这样写不容易看懂，换成上面的吧，这写法通用，每个语言都可以这么干。 这些语法糖只是举个例子，改一下一分钟的事情，说实话这根本不是我在意的点。
归根结底我讨厌的是个人主义，动不动就推翻别人的成果，还强行让别人承认你的方案就是比他的差。

@encounter2017 我的代码贴的有点问题， 我思想是如果 key 没有值，就插入数组， 因为上面已经确定了肯定肯定有值了，unwrap 就是一个安全的操作，就好比其他语言的 obj!.cc 强行解析一个意思的东西。

let mut a = HashMap::new();
// 如果没有值就插入数组
if !a.contains_key("1") {
a.insert("1", vec![]);
}

a.get_mut("1").unwrap().push("1");

@lianhuayu420 你这个评价太到位了，说到心坎了，个人主义，专政，觉着自己无敌，我们组员一致的评价。review 这件事情，跟每个组员都闹个矛盾。

@yuanxing008 不经他人苦, 莫劝他人善。

@dynastysea 裸辞太亏了，放开了，准备开怼了。混个 n+1

@dlmy 忙的时候，MR 就堆在哪里，堆个 10 几 20 个一点不夸张，交付的时候经常出现合并冲突，然后丢代码，然后又让我们去改。

闲的时候，开始 review 了，一行一行的看, 写的跟他想法不一样，就要按他的想法来，重写，你还得拿笔记，想想心里都堵的慌。

@JingXiao 一点都不轻松，经常性的推翻重来，谁能受得了，关键我特喵的也记不住他的哪些吊想法，问多了，还得挨批。提交个 mr 战战兢兢的。

@dynastysea 当然是领导啊，要组员不得怼死他。

@bbao 很好奇你所谓强到底他妈的是啥。 都特么写一下 CRUD 能特么强到哪里去。 还能写出个花来？，还是发明了个什么设计模式，或者什么牛皮的算法。有些所谓的代码好，都是自我感觉良好罢了。

都是 7 ，8 多年程序员，功能都能实现，保证自己的东西不会出问题，对自己的东西负责。老是把自己想法强加于别人只会适得其反。

有次写 js 的时候， 我用了一个 ?? 语法，他问这是什么屌语法改掉。 有次写 rust 的时候， 我使用 HashMap 的时候，

我这样
```
let a = HashMap::new();

if a.contains_key("1") {
a.insert("1", vec![]);
}

a.get("1").unwrap().push("1");
```

然后他跟我说有一个 or_insert, 让我改掉

最早的时候，我经常使用 rust 链式处理方法，他说不好，给我的感觉就是只有他不知道，你就得改

@vfs 你费劲巴力的写了 2 天, 业务功能也完成了， 功能也测试过了。Code Review 说你这样写不好，有更好的方法，好你在根据他的想法重写，来回 Review, 为了一个他所谓“好” 折腾别人，无非是显自己很优越，我见过组里有人，为这种问题搞了好几周，人都麻了。

@listenerri 哈哈，你这个说服力就很强了

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

写了好长时间的业务代码，功能都自己测试过了，review 后很多地方都要改，完了又要重新测试，想想都头大，你们是怎么解决的？


案例代码

```js
// 我的代码
// 把数组转成 key-value 对象
const arr = [
{ id: 1, name: 'a' },
{ id: 2, name: 'b' },
{ id: 3, name: 'c' },
];

const obj = arr.reduce((acc, item) => {
acc[item.id] = item.name;
return acc;
}, {});

console.log(obj); // { '1': 'a', '2': 'b', '3': 'c' }

// review 人员：
// 团队都这样么，你那么写，别人不好看的懂，reduce 通常用在 xxxx ，没听进去
// 改成这样
const obj = {}
for (let i = 0; i < arr.length; i++) {
const item = arr[i];
obj[item.id] = item.name;
}

```