遇上一个对Code Quality 超级执着的组员，快被逼疯了，经验

年轻人谦虚点耐心点，多了解多思考再去挑战权威。遇到我大概就pip你了。

匿名用户 发表于 2023-03-18 00:30:15
这个帖子看下来我都懵了
以前的组天天撕变量名，撕Optional如何好，撕不让写for循环必须写streaming，Enum怎么能写出花儿，恨不得互相拿clean code和effective ja

可以请教一下 变量名，尽量用optional，不用for loop用streaming，不用string用enum哪一个不应该吗😂 没看到你说的哪一个是槽点啊 明明都是很正常基本每个组都会有的review建议啊

每个组有这种人应该庆幸，轻松让你过得组肯定后面ops重，并且越写越屎。

打不过就加入，这种事也就前2个月痛苦一些。
这人做的事从目的来说是为了全组好。

实现方式也许你的更好，但是最好全组统一
不然以后人看代码很累。

其实这种情况很多，找老婆遇到170的白富美
各种要求各种不适应，要么别找要么适应

找工作进血汗工厂，各种pua各种奇葩
要么跳槽要么适应

每个人的生活其实都是自己选的 :)

tentenymy 发表于 2023-3-17 12:18
lz站在对方的角度想想，他跟你耗同样时间，去搞一个不是自己的项目，还不得不看新人的code，他不是更痛苦吗 ...

他是爲了People impact.

看最新的评论终于正常点了，只能说因为软件行业门槛越来越低，来钱又快又相对轻松，导致一大批本身对CS毫无兴趣，单纯只是为了赚钱轻松的人涌进这个行业。这点本身无可厚非，有钱为什么不赚，但这样的动机也从决定了这些人急功近利，一切以产出优先，对技术的钻研就更不可能了，肯定是怎么简单怎么快怎么来。程序能运行就好，我像刷leetcode那样去实现就好了，为什么要写得干净，为什么要方便其他人用，为什么要方便以后扩展。很多概念和设计模式他们根本不理解，就像他们不理解为什么要用optional，为什么要用enum，为什么要SOLID，什么是SOLID。我最近review过几次新人的c++ code，raw pointer到处都是，讲道理从第一次我就有好好科普过RAII，每次都有指出过这个问题，但直到最近的提交里，这个问题依旧还在。也许在他眼里，这不是问题而只是每个人“实现方式”的不同，而我就是个把他逼疯的nitpicker。
最后水平过硬的IC占比会越来越小，他们的话语权也会越来越小，也不会再有知识传承，屎山则会越来越多，也难怪很多“根正苗红”的程序员对转码会如此愤愤不平。

匿名用户 发表于 2023-03-17 10:13:30
的确很头疼，这种事情可以找老板聊聊（不是告状那种），问问有什么建议，如果老板和这个人意见不统一的话你老板也可以帮助influence一下

但是话说回来了，如果你还是职场新人的话，应该很庆幸能遇到

同意！职场新人应该专注于发展一个好的工作习惯而不是图一时的工作快
好的工作习惯会帮你走的更远！

lifesuckad 发表于 2023-3-18 11:23
他是爲了People impact.

哈哈哈 公司考核改革了 这种trivial的code review用处不大哦 除非老板特意算credit 尤其这个组的老板不会因为这种code review多算这个senior的credit的

且行且珍惜，遇到组里人任你自生自灭，或者平时小问题不指出来，出大问题了开始整你，是不是难多了

匿名用户 发表于 2023-3-18 10:00
可以请教一下 变量名，尽量用optional，不用for loop用streaming，不用string用enum哪一个不应该吗&#12851 ...

> 尽量用optional

Optional<T> 增进 null awareness. 因为 null check (in Java) 经常被遗忘. 如果 project 引入了 null checkers (e.g. checkers framework). Optional<T> 的作用就没那么显著了.  在 Kotlin 有 builtin null check. Optional 基本 discouraged, 除非要和 java interop.

我个人觉得大部分时候用 List<T> 来 wrap 这种情况更方便, 这时 list size either 0 or 1. 比如一个 method 的 return type 是 List<T>. 在它的 call site 可以直接 addAll. 用 optional 还是得做 presence check. 大部分 API 还是考率 collection compatibility 更好些.

> 不用for loop用streaming

第一遍写, 我还是用 for-loop 比较多, 因为 debug 比较方便, logic 经常变. 逐渐 self-iteration 后 stream 逐渐变多. 因为原来 for-loop 由干很多事变成干一件事, 这样用 stream 就变的简洁并可以 lockdown “single responsibility” (避免自己或他人不必要乱改).

> 不用string用enum哪一个不应该吗

区分 type/category 尤其 cross class boundary, 首选 enum. 比如你想 clean up one enum value. 要是 string 很难找全所有的 logic dependencies.