楼主: 匿名
跳转到指定楼层
上一主题 下一主题
收起左侧

[同事协作] 遇上一个对Code Quality 超级执着的组员,快被逼疯了,经验

   
🔗
pfwtm 2023-3-19 01:55:11 来自APP | 只看该作者
全局:
年轻人谦虚点耐心点,多了解多思考再去挑战权威。遇到我大概就pip你了。
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-FIRRE  | 添加认证 | 2023-3-19 02:00:24 来自APP
匿名用户 发表于 2023-03-18 00:30:15
这个帖子看下来我都懵了
以前的组天天撕变量名,撕Optional如何好,撕不让写for循环必须写streaming,Enum怎么能写出花儿,恨不得互相拿clean code和effective ja
可以请教一下 变量名,尽量用optional,不用for loop用streaming,不用string用enum哪一个不应该吗😂 没看到你说的哪一个是槽点啊 明明都是很正常基本每个组都会有的review建议啊
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-AQYGX  | 添加认证 | 2023-3-19 02:05:39 来自APP
每个组有这种人应该庆幸,轻松让你过得组肯定后面ops重,并且越写越屎。
回复

使用道具 举报

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

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

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

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

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

使用道具 举报

🔗
lifesuckad 2023-3-19 02:23:05 | 只看该作者
全局:
tentenymy 发表于 2023-3-17 12:18
lz站在对方的角度想想,他跟你耗同样时间,去搞一个不是自己的项目,还不得不看新人的code,他不是更痛苦吗 ...

他是爲了People impact.
回复

使用道具 举报

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

评分

参与人数 1大米 +1 收起 理由
spilt-over + 1 赞一个

查看全部评分

回复

使用道具 举报

全局:
匿名用户 发表于 2023-03-17 10:13:30
的确很头疼,这种事情可以找老板聊聊(不是告状那种),问问有什么建议,如果老板和这个人意见不统一的话你老板也可以帮助influence一下. From 1point 3acres bbs

但是话说回来了,如果你还是职场新人的话,应该很庆幸能遇到
同意!职场新人应该专注于发展一个好的工作习惯而不是图一时的工作快
好的工作习惯会帮你走的更远!
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-9WB6J  | 添加认证 | 2023-3-19 04:04:14
本帖最后由 匿名 于 2023-3-18 13:05 编辑 . ----
lifesuckad 发表于 2023-3-18 11:23
他是爲了People impact.

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

使用道具 举报

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

使用道具 举报

🔗
wisdompilot 2023-3-19 04:53:23 | 只看该作者
全局:
匿名用户 发表于 2023-3-18 10:00
可以请教一下 变量名,尽量用optional,不用for loop用streaming,不用string用enum哪一个不应该吗&#12851 ...
.--
> 尽量用optional
. From 1point 3acres bbs
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.

评分

参与人数 2大米 +2 收起 理由
zsgxyxzdsk + 1 很有用的信息!
james051199 + 1 赞一个

查看全部评分

回复

使用道具 举报

您需要登录后才可以回帖 登录 | 注册账号
职场达人
  • ↑ 本版用于讨论职场各种干货话题,闲聊请去🔗聊聊或者🔗匿名版
  • ❌ 本版严禁水贴,引战,发布广告,拉群,贴个人联系方式,扣分无警告
  • ☑ 求职、面经等去 🔗北美求职和 🔗回国求职大区,刷题和学习请去 🔗终身学习大区
  • ☑ 请去专版发布 🔗内推, 🔗招聘信息,和讨论 🔗创业内容
  • ☑ PIP / DevList/ Need Support 等话题也已开设 🔗专版

本版积分规则

>
快速回复 返回顶部 返回列表