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

[同事协作] 同事写的code很乱,需要指出来吗?

 
🔗
slaink2 2021-12-4 08:35:42 | 只看该作者
全局:
取决于你的目标。.1point3acres

* 你需要指出他错误的credit吗?如果需要,礼貌指出。
* 你希望提高组里代码质量 / 你跟这人还算熟想帮帮忙 / 这人需要好的PR来promo 《- offline sync,如果可以让manager/tech lead知道一下。或者看这个人下一次PR revision有没有写based on offline sync with you 如果没写,下次考虑直接留字面PR
* 如果就是不想背锅: 有别人也是owner吗?你提点小意见,然后不LGTM或者多tag几个其他的人问opinion?
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-ATSKF  | 添加认证 | 2021-12-4 10:04:47 来自APP
我一般逻辑上没问题会approve,然后到处comment。

楼主说的几点我会直接问。
1. 我们的代码库中已经有xxx了,在保持代码统一性的目标下我希望和他们尽可能有差不多的结构,但是有任何我没考虑到的地方不能让我们也用同样的方式实现吗?
2. 不clean,命名不好,都是一种很主观的说法。其中一部分能客观分析并且指出标准的,一般都能在code style里找到,直接引用说我们根据code style可以这么写。.
尤其是命名,很多组没有命名约定,更多的是凭经验,甚至需要很多领域知识。更有甚者好几个人给我cr,我根据A的说法把名字改成了A,B说不好要B,说服A改成B,C说C更好。我死了。
可以的话就自己写个文档分享,说自己命名一般是怎么考虑的,然后让大家指正完善,最后全组用一套方法论,有了共识才方便。
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-RF0EA  | 添加认证 | 2021-12-4 11:28:01
要指出来,最好带上评论的依据和来源,可以是style guide、内部document、general rule比如design pattern等等,最好严格把关,否则后面出了问题你作为reviewer也有责任
回复

使用道具 举报

🔗
ddmmtx 2021-12-4 15:39:04 来自APP | 只看该作者
全局:
你想过没有你告诉他代码质量不高他会怎么想?. 1point 3acres
有帮他的时间你都自己写完了。。。何必纠结呢,自己学到东西有机会了想办法换公司换队友得了。
你没有话语权的时候说啥都没用。
回复

使用道具 举报

全局:
pubu666 发表于 2021-12-03 15:13:31
我觉得需要指出来,否则这种PR被别人看到多了会觉得你们两个都是菜b……
附议!一对菜逼。
回复

使用道具 举报

全局:
大的pr 你可以要求他弄个review meeting,这样沟通起来效率更高,也不会因为comments 也太多,被对方误会你针对她
回复

使用道具 举报

🔗
老豆腐 2021-12-4 23:08:28 | 只看该作者
全局:
“总体上完全不搭但是能跑”

不是我吧 哈哈哈哈
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-MX55T  | 添加认证 | 2021-12-5 00:32:43 来自APP
ddmmtx 发表于 2021-12-03 23:39:04
你想过没有你告诉他代码质量不高他会怎么想?
有帮他的时间你都自己写完了。。。何必纠结呢,自己学到东西有机会了想办法换公司换队友得了。
你没有话语权的时候说啥都没用。
我本来也是这么想的,所以才会提出这个问题。特别是“我有时间教他还不如自己写了”这点深有同感。但是事实是工作中已经assign给别人的任务,人家写好了你不签收,自己重写一套是不可能的,比直接指出错误还差,要写也只能手把手教他写,更累。
昨天看了评论,我最后还是委婉指出我的疑虑,给了理由。改不改看他吧。他说觉得原本更好的话我也就approve了反正我该说的说了。
回复

使用道具 举报

全局:
摸鱼啊兄弟
回复

使用道具 举报

🔗
ddmmtx 2021-12-5 01:37:00 来自APP | 只看该作者
全局:
匿名用户 发表于 2021-12-04 08:32:43
我本来也是这么想的,所以才会提出这个问题。特别是“我有时间教他还不如自己写了”这点深有同感。但是事实是工作中已经assign给别人的任务,人家写好了你不签收,自己重写一套是不可能的,比直接指出错误还差
唉,我也是真的屡次失望过后,还碰上一个全是理由的人才这么说。. From 1point 3acres bbs
不过具体情况还是具体分析吧,能学到些东西总归是好的。
回复

使用道具 举报

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

本版积分规则

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