组里的人代码质量差怎么委婉的说

junhezfm 发表于 2021-10-4 02:08
这种完全不写comment或者禁止写comment组一般会大言不惭的说，the code explains itself~然并卵

补充内容  ...

感觉不写注释就是为了让其他人看不明白，这样他显得有用，在组里能稳定些lz别戳破teammate，可以多安排会议让他讲解

小公司这种事情最好别说。完成自己的实习就好。The point is: you aren't his manager. Not responsible for it. People don't like to hear such things from peers unless it's on-going code reviews.

工作了, 就不要想着怎么指责别人. 就想好怎么做好自己, 高一层就怎么让整个 team/code base 变得更好:

- 自己带头写好代码
- 当你涉及到不好的代码的时候, 主动 clean up 代码
- 组织起 fixit campaign, 并且吸引大家参与进来
- tech talk 来讲好的代码规范, 写出来 best practice / code style doc. 并推动 team follow
- 推动更好的 code review process 和 CI automation. 写出更好的 lint rule, presubmit check, auto-fixer 来用代码推动 best practice

做这些, 你可能拿到 return offer, promo, 至少 recognition. 去找人对峙, 或者打小报告, 和同事抱怨, 只会让你被认为是个 "事多" 的人. 而且 "凭啥你一张嘴一说, 我就要 follow 你的想法". 要 take initiative, 而不是就张嘴抱怨.

----

退一步讲:
- codebase 参与的人多了, 时间久了. tech debt 和各种小问题必然出现. 不会像学生代码一样 - 一张白纸开始一个人画, 啥啥都是自己觉得对的. 你让 100 个人同时在 1 张大纸上画画 5 年, 肯定好多没画好的地方, 画糊了的.
- 不能要求别人怎么样 (除非你是 team lead, 并且你负责那个人的 performance). 有的人就是上班来划水的, 来养老的. 不是每个人都像 new grad 一样充满干劲, 全心都是投入在工作中. 也不会每个人都有非常好的 code style, 有很强的学习和保持 clean 的能力. 学会和各种人合作, 而不是想把各种人都变成你想要的人.

Hi bro, I think there’s more potential for you to knock out the stings before requesting to code reviews. When I was a child, my dad always criticize me not having a good grade comparing with my neighborhoods boys. Although I think it’s not humanity to compare with others since we have different level of intelligence, I still feel appreciated since dad’s push really made me more excellent than before. So I hope you too can be competent at code since I saw ur potential comparing to xxx. Thanks for trusting me and my dad.

lxxxxxxx 发表于 2021-10-4 12:18
那你说说代码质量不重要为什么这些“成功产品”牺牲代码质量换时间抢占了市场之后花几倍的时间重构提升代 ...

成功了以后想怎么玩都行，成功只是小概率事件

每只猫都是天使 发表于 2021-10-4 12:35
感觉不写注释就是为了让其他人看不明白，这样他显得有用，在组里能稳定些lz别戳破teammate，可以多安排会 ...

路过问一下。。。从来没见过谁专门安排讲解代码的会，真有人这样嘛？document 不足需要大家读代码理解逻辑确实是个pain point，但是读不懂别人的代码难道不是说明自己很弱嘛。。。
之前有人离职，我们tech lead说读他的代码想cry，也就这样了。那个人确实经验少，代码很弱。但是有的人的代码你刚开始看觉得bullsh*t，多看多研究才发现原来小丑竟然是自己。。。

lga2011_3 发表于 2021-10-4 08:39
听取大家意见，不纠结了，后面的comment也不写了

.. 感觉是“懂了，但是没完全懂”。前面的意思是，1. 让你别去管别人，自己做好自己的，2. 有余力就去optimize别人的代码，不要只动嘴皮子。不是让你自己就不去写comment，就降低代码质量了。
Ship product要考虑的东西太多了，一般的PM连refactor和clean up都不会觉得是什么重要的事，很多时候根本顾不上什么comment和document。

lulunini 发表于 2021-10-04 10:20:10
路过问一下。。。从来没见过谁专门安排讲解代码的会，真有人这样嘛？document 不足需要大家读代码理解逻辑确实是个pain point，但是读不懂别人的代码难道不是说明自己很弱嘛。。。
之前有人离

参加过这种讲代码的会，代码逻辑很复杂，新兵自己看多少遍还是会懵，单步调试调着调着就迷了，此时只能求助同事了

可以考虑：1.先私下跟同事交流一下，确认这是不是这就是公司的工作方式？2. 如果大家都这样，你最好还是别尝试去改变别人，可以自己按照自己的标准写代码  3. 代码风格是一个比较tricky的东西，每个人可能都有自己的标准

还是别说吧，如果真的是问题，那可能一开始code review就不应该merge进去啊，既然merge了肯定就已经批了，可能代码整体要求不高，deadline紧