查看: 6565| 回复: 23
跳转到指定楼层
上一主题 下一主题
收起左侧

[自我提升] 为什么好多人code review不会一次把comment都写出来

   
🔗
匿名用户-NSBZQ  | 添加认证 | 2023-2-8 14:06:05 来自APP |倒序浏览

注册一亩三分地论坛,查看更多干货!

您需要 登录 才可以下载或查看附件。没有帐号?注册账号

x
有些人一开始写一个comment,我改完以为可以ship了又去写一个comment(不是新改的代码),我再改完了又去写一个comment(还是老代码),本来以为一天结束的拖了很多天。。。而每改一次我后面的diff就要重新rebase一次很痛苦。。

这种应该怎么加快进度

上一篇:中小公司或机构的manager能否直接跳到大厂做manager
下一篇:公司今天刚刚fire了一个印度小哥
全局:
😭 因为每看一遍写出的读后感都是不同的

评分

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

查看全部评分

回复

使用道具 举报

推荐
licro 2023-2-8 14:17:22 | 只看该作者
全局:
people block others to justify their own impact
回复

使用道具 举报

地里匿名用户
推荐
匿名用户-QLKQF  | 添加认证 | 2023-2-8 16:21:40
严谨的review程序是每个iteration都要当作新的一样重新过一遍。
回复

使用道具 举报

全局:
如果这些comment都make sense的话也还说得过去
回复

使用道具 举报

全局:
本帖最后由 蝎子莱莱_ 于 2023-2-7 22:26 编辑

喜欢把流处理那一套用在PR Review上。。
回复

使用道具 举报

🔗
sleepychien 2023-2-8 14:17:46 | 只看该作者
全局:
我一般给人code review,都是一次性给完comment,除非是新加的部分,不然很讨厌这样。 聊一下呗,让他不要着急publish comments, 看完一起发,或者下次问他一下,是不是没别的comment。 或者就是你把review再split成更小的reviews,方便他一次性看完
回复

使用道具 举报

全局:
就是第一次没有认真看。你可以写些说明,让code review更容易理解,减少review难度,会有些帮助。
回复

使用道具 举报

🔗
invisibili 2023-2-8 14:27:17 | 只看该作者
全局:
本帖最后由 invisibili 于 2023-2-7 22:33 编辑

我有时候会在大概成形的时候就开始给别人看,同时自己也会再看看哪里可以改进。
回复

使用道具 举报

🔗
ZZC. 2023-2-8 14:27:19 来自APP | 只看该作者
全局:
我有些队友也这样,我和他们聊过了以后得到以下的几个解释:
- 有些重复的错误,但是只有其中一个被评论了
- 要注意有side-effect的情况,你修改了一个地方可能会导致其他地方也需要修改
- 个别的错误不易察觉,可能在第二次review的时候才看到

总结:pr在提交之前自己先做一次review,队友review完之后作出修改,并且再查漏补缺一下。我没办法要求我的队友做出改变,但是我可以自己多做一点来避免重复冗余交流。
回复

使用道具 举报

🔗
invisibili 2023-2-8 14:28:32 | 只看该作者
全局:
过了一段时间有了新想法。或者改动太多,看了一半就先发出review也是有可能的。
回复

使用道具 举报

全局:
如果你遇到同事有些行为你觉得不理解,可以直接问当事人。
.. 如果希望对方改变,可以直接跟当事人提。
如果希望公司/小区/社会做出改变,可以群发邮件提出呼吁,或者给领导提提案。
回复

使用道具 举报

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

本版积分规则

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