V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX  ›  MrRongts  ›  全部回复第 1 页 / 共 1 页
回复总数  17
38 天前
回复了 MrRongts 创建的主题 职场话题 这些话算 PUA,还是真的为你好!
@MiguelTapia 你这个评价非常的到位, 就是控制欲望。
38 天前
回复了 MrRongts 创建的主题 职场话题 这些话算 PUA,还是真的为你好!
他的上限很高,你已经到头了
46 天前
回复了 MrRongts 创建的主题 职场话题 被 Code Review 折磨疯的组员
@encounter2017 你没有理解我的意思, 我不是反对好的写法我不认同,我反感点是,为什么组员用新的语法糖,review 时候你看不懂,你就让别人改,而你用的时候你却要大吹特吹,表现你的优越感。 角色调换一下位置如果我使用最直接的写法用 `a.entry("1").or_insert_with(Vec::new).push("1")` review 的那人又要说,这样写不容易看懂,换成上面的吧,这写法通用,每个语言都可以这么干。 这些语法糖只是举个例子,改一下一分钟的事情,说实话这根本不是我在意的点。
归根结底我讨厌的是个人主义,动不动就推翻别人的成果,还强行让别人承认你的方案就是比他的差。
46 天前
回复了 MrRongts 创建的主题 职场话题 被 Code 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");
48 天前
回复了 MrRongts 创建的主题 职场话题 被 Code Review 折磨疯的组员
@lianhuayu420 你这个评价太到位了,说到心坎了,个人主义,专政,觉着自己无敌,我们组员一致的评价。review 这件事情,跟每个组员都闹个矛盾。
51 天前
回复了 MrRongts 创建的主题 职场话题 被 Code Review 折磨疯的组员
@yuanxing008 不经他人苦, 莫劝他人善。
51 天前
回复了 MrRongts 创建的主题 职场话题 被 Code Review 折磨疯的组员
@dynastysea 裸辞太亏了,放开了,准备开怼了。混个 n+1
51 天前
回复了 MrRongts 创建的主题 职场话题 被 Code Review 折磨疯的组员
@dlmy 忙的时候,MR 就堆在哪里,堆个 10 几 20 个一点不夸张,交付的时候经常出现合并冲突,然后丢代码,然后又让我们去改。

闲的时候,开始 review 了,一行一行的看, 写的跟他想法不一样,就要按他的想法来,重写,你还得拿笔记,想想心里都堵的慌。
51 天前
回复了 MrRongts 创建的主题 职场话题 被 Code Review 折磨疯的组员
@JingXiao 一点都不轻松,经常性的推翻重来,谁能受得了,关键我特喵的也记不住他的哪些吊想法,问多了,还得挨批。提交个 mr 战战兢兢的。
51 天前
回复了 MrRongts 创建的主题 职场话题 被 Code Review 折磨疯的组员
@dynastysea 当然是领导啊,要组员不得怼死他。
52 天前
回复了 MrRongts 创建的主题 职场话题 被 Code Review 折磨疯的组员
@bbao 很好奇你所谓强到底他妈的是啥。 都特么写一下 CRUD 能特么强到哪里去。 还能写出个花来?,还是发明了个什么设计模式,或者什么牛皮的算法。有些所谓的代码好,都是自我感觉良好罢了。

都是 7 ,8 多年程序员,功能都能实现,保证自己的东西不会出问题,对自己的东西负责。老是把自己想法强加于别人只会适得其反。
52 天前
回复了 MrRongts 创建的主题 职场话题 被 Code Review 折磨疯的组员
有次写 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 链式处理方法,他说不好,给我的感觉就是只有他不知道,你就得改
52 天前
回复了 MrRongts 创建的主题 职场话题 被 Code Review 折磨疯的组员
@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;
}

```
228 天前
回复了 shaoht 创建的主题 酷工作 郑州招 Python , Node.js 开发岗
非要全日制本科?有软考证书好使吗,9 年开发经验
关于   ·   帮助文档   ·   自助推广系统   ·   博客   ·   API   ·   FAQ   ·   Solana   ·   2726 人在线   最高记录 6679   ·     Select Language
创意工作者们的社区
World is powered by solitude
VERSION: 3.9.8.5 · 16ms · UTC 13:22 · PVG 21:22 · LAX 05:22 · JFK 08:22
♥ Do have faith in what you're doing.