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

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

   
全局:
那你就跟他不断强调和productivity的tradeoff,反正他就拿高标准来怼你你也没法说他是错的。

然后等下次他 pr 的时候你就也来要求要求他呗,有时候代码好坏不是绝对的总有利弊,你就盯着弊端讲,他说自己这样写好 maintain 那你就看是否不 readable 或者 performance 有一点影响,这东西也没法量化你就只管展开说弊端吧。
回复

使用道具 举报

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

这种人就是吃饱了没事做。。拿着鸡毛当令箭
回复

使用道具 举报

全局:
我觉得这是好事啊,尤其是对于新人来说,组里有这么个人能有效避免屎山代码
回复

使用道具 举报

全局:
我们组的code分成两类,一类是核心的,如果弄不好就会把整个产品弄崩的。另一种是边角的,就是不管谁搞砸了只会影响ta自己的那一小块项目。

不到两年前以前我们组来了两个ng. 开始时两个人我都给分配第二类工作,但review都严格要求。结果一个认真的改一个不认真。慢慢的我就把重要的工作交给那个认真的。他的代码也越写越好。另一个人因为工作不重要我也懒得再严格要求了。把他推给了一个l4带.

认真的一年刚出头就帮他升职了。另一个这么久仍然只能打杂。ta说想做重要项目升值,我直接了当地说你要先build trust. 心里想想升职我估计还得再两年吧。如果年景好我会建议他换组。
. 1point 3 acres
补充内容 (2023-03-18 09:40 +08:00):. Χ
认真的是印度人,不认真的是中国人。我也很想帮中国人但使不上劲啊。
回复

使用道具 举报

🔗
wangbo1987 2023-3-18 09:33:49 | 只看该作者
全局:
有这样的大哥是你职场的幸运,好好跟大哥学,把现在的代码好好写,延误了发布有大哥兜着怕什么。然后跟大哥一起花时间把现有的屎山再重构一下,以后如果这个产品很有前途很重要,那么这个组前景会非常好的。
回复

使用道具 举报

🔗
cswords 2023-3-18 09:40:04 | 只看该作者
全局:
哎我都是急事直接找人review,不急的话,让我改我就改,慢慢来呗。只要对方也是为了产品好,我觉得没啥。怕就怕那种根本就不是为了产品,就是要刷脸的那种同事,在需求部门等着hotfix的时候为这种事情插手让fix delay。。。事后你还要说感谢他主动帮忙。真的不爽这种。
回复

使用道具 举报

全局:
比起那种没要求,也不认真看就直接approve 的我喜欢这种。
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-RLI3M  | 添加认证 | 2023-3-18 09:45:45 来自APP
我刚毕业的时候自己做一块东西,组里没有人能review 我的代码都说看不懂,让我直接交,导致我代码质量一坨屎,后来自己重构几次都要吐了。. 1point3acres.com
后来跳槽了以后遇到了超级picky的老哥review 我的代码,我真是感恩戴德,代码质量飞速提高。那时候组里没有任何pip人的记录,默认学习周期一年,我在那一年里就把代码发给各种人Review,能把三个人拉进来喷我就绝对不只拉两个人。
不过现在时代变了有pip了,还是要从产出速度上再优化一下这个问题:.
.--
大体的框架/实现方法有问题:这个最好是在一开始prototype 里就达成一致的,也就是最好在你还没开始怎么做的时候多拉人进来。. .и
实现的方法有个人偏好:这点我也没有什么好的方法,有的人的偏好在我看来就是胡说八道,把其他语言的偏好拿来强加给另一个语言。你可以让他陈述原因,如果他聊什么可扩展性你就是嗯对有道理留个TODO那到时候再改,我们初期就baby step。
💩上堆💩:💩山的出现每个人都有责任,不能说让你自己一个人来重构。发个邮件给全组,大为赞叹那位老哥发现了这个问题,提出这是个tech debt,希望放进下一次的fix it/ code health的活动里修。这样你这次就可以继续堆💩了。
命名:没什么办法,我超讨厌命名,别人想帮我起名我开心还来不及。. 1point 3acres
改无关的东西要重新开pr: 这是在这种裁员的时候给你刷pr 数啊朋友。这种pr通常approve超级爽快,改得快交得快,我狂交。

评分

参与人数 1大米 +1 收起 理由
james051199 + 1 赞一个

查看全部评分

回复

使用道具 举报

全局:
我貌似也有楼主相似的经历。
刚开始一个CR要改五六版才能通过,. Χ
一个东西我实现了,但同事常会提出让我用另外一种方法实现。一个unit test的命名也会讨论一番。还有一些小的东西。.google  и
刚开始我也特别痛苦,总是在想我都实现了为啥非要用你推荐的方法?为啥你就认为你推荐的方法是最优的?但是人家资历深,我就沉住气一个一个的改。。。
慢慢的,你知道我发现了啥?. From 1point 3acres bbs
我发现通过别人指导之后修改的代码,看着很漂亮很顺眼,拿我之前写的代码一对比,自己之前写的就是屎。。。
过程虽然很痛苦,但我确确实实学到了特别多的东西. ----
神奇的是,我的代码质量也越来越高,现在基本上1版或者2版就能通过,comment也越来越少。。。真的是超开心。
我之后发的代码,如果和那个同事有关,我就尽量发给他让他来帮我看看,因为我感觉有一个特别负责人的人帮忙看代码,是提高写代码能力的一个特别好的方法。

我最近还发现我们组里有个人,他总是能找到一个更好更酷的方法实现同一个东西,有时候为了方便理解,他直接把那个更优的代码写在下面,或者给个example的链接,有时候还会把说明为什么要这么做的链接写出来。。。我真的是,第一次看到他的comment我都惊呆了!然后我就告诉他特别喜欢他的comment,让他以后多多给我提,因为通过他的comment,我可以学到很多新的东西。
总之,我个人感觉能遇见一个认真的reviewer,是很难得,他们也要花很多时间去看一堆屎代码(我感觉自己刚开始写的代码就很屎)。
所以如果对方提出的comment比较make sense,楼主不妨沉住气,以一种学习的状态去对待,可能会好受些。等熬过了这个过程,可能就柳暗花明了~
P.s.:我刚开始对我的reviewer是又爱又恨。。。现在就只剩下爱了,非常感谢他的认真,让我学到了很多。
回复

使用道具 举报

🔗
Bobliu2017 2023-3-18 10:27:49 | 只看该作者
全局:
很希望有一个这样的mentor training我,再说入职半年承担的业务需要估计不会特别复杂,应该不会特别忙碌,如果是我的话会多花点时间思考
回复

使用道具 举报

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

本版积分规则

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