V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
youngcong
V2EX  ›  程序员

如何看待 code review ?

  •  
  •   youngcong · 2022-12-22 21:02:54 +08:00 · 3032 次点击
    这是一个创建于 462 天前的主题,其中的信息可能已经有所发展或是发生改变。

    因为小组内所有代码需要通过 mr 后才会合到 master ,我会参与到 code review 并给予一些修改建议,但是有时候我提的一些建议其他小伙伴好像并不能做出针对性的调整。我理解这主要是认知的不同导致的,即使我说出了自己的看法,但是对方并不能 get 到。

    那么 review 代码时候,我是否还要继续无所保留的给出自己的意见呢,我要如何平衡这个落差感呢?

    20 条回复    2022-12-24 13:26:28 +08:00
    yikyo
        1
    yikyo  
       2022-12-22 21:09:03 +08:00 via iPhone
    找个第三者评估
    optional
        2
    optional  
       2022-12-22 21:11:17 +08:00 via iPhone
    CR 前是需要统一思想的,CR 我看起来是比单元测试更重要的东西
    tt67wq
        3
    tt67wq  
       2022-12-22 21:23:15 +08:00
    提还是要提,改不改就不归你管了
    liaojl
        4
    liaojl  
       2022-12-22 21:26:44 +08:00 via iPhone   ❤️ 2
    当这件事情已经失去本质意义,但组内还要求必须走 mr 的流程,那就做个样子走完流程就行了,不要做一些费力不讨好的事情。
    fkdog
        5
    fkdog  
       2022-12-22 21:35:49 +08:00
    如果组里没有这样的氛围的话,走个流程就完事了。
    代码风格一类的不是自己的不要多管,大部分人都不喜欢别人教自己做事。
    如果是有明显 bug 的话,你可以用委婉的方式提一下,比如“这里我刚 debug 下好像有点问题,你帮忙看下我这样是不是正常的。blah blah”
    god7d
        6
    god7d  
       2022-12-22 21:42:02 +08:00 via iPhone
    与其费时劳力 cr ,不如招聘的时候就把好关
    codexian
        7
    codexian  
       2022-12-22 21:42:20 +08:00
    看下来楼主是对技术有追求的,其他小伙伴是混口饭吃的(绝大多数人的状态)
    但后续怎么做,还是看楼主自己,但就我主观而言,改变别人的认知是一件非常吃力不讨好的事情
    youngcong
        8
    youngcong  
    OP
       2022-12-22 22:00:59 +08:00   ❤️ 1
    @codexian 是的,我已经受挫了,cr 的时候甚至不想给出主观意见了
    lazyfighter
        9
    lazyfighter  
       2022-12-23 09:27:36 +08:00
    提是你的事情 不改是他的事情, 能力不同,有时候就是对牛弹琴, 还觉得你挑刺
    marco330
        10
    marco330  
       2022-12-23 10:31:36 +08:00
    同感。现在我做 cr ,只要没有逻辑问题或者和我预期没有太大的偏差,我都忽略了。每个人都有自己的代码风格。
    lsnl8480
        11
    lsnl8480  
       2022-12-23 10:34:41 +08:00   ❤️ 1
    kubernetes 社区有一句话可以借鉴一下:
    教别人怎么做,而不是告诉他你改做什么。

    如果别人 get 不到你的 idea ,就直接给他方案,show code ,过几次水平就拉齐了
    brader
        12
    brader  
       2022-12-23 11:07:52 +08:00
    这方面几年下来我也经历了很多,现在我期盼我的合作伙伴就一个要求,代码格式符合基本要求就行了,不要那种缩进没有、缩进乱、注释乱,就 OK 了,你逻辑怎么写,一个方法多长,怎么封装,甚至你自认为没必要写注释的,这些我都不管你。
    怀着这个心态,我这几年不知道多省心
    Meltdown
        13
    Meltdown  
       2022-12-23 16:00:01 +08:00
    给出意见就好了,改不改是他自己的事
    Leviathann
        14
    Leviathann  
       2022-12-23 17:46:06 +08:00
    0. 别瞎改我代码
    1. 别太明目张胆的堆屎
    2. 逻辑对不对是测试的事,只看代码的写法
    Leviathann
        15
    Leviathann  
       2022-12-23 17:47:39 +08:00
    另外碰到代码写法的问题,我会自己写一遍看看是不是能按我想的那样改,写完贴上去一般他都没话说了
    cnin0770
        16
    cnin0770  
       2022-12-23 19:43:45 +08:00 via iPhone
    我 review 的时候:能不改别人的 code 就不改 除非功能性的 性能上的 因为很多时候改的冲动都是“看不惯他这么写”而已 再三思考 是不是一定改 改了比不改好
    IvanLi127
        17
    IvanLi127  
       2022-12-23 23:25:03 +08:00 via Android
    没用,除非组员憧憬你,否则人家可能还觉得你给的意见是扯淡呢。
    mysunshinedreams
        18
    mysunshinedreams  
       2022-12-24 01:03:06 +08:00
    "我知道我这块代码有问题,但是今天老板要求必须要发版,你让我改代码是需要时间的,耽误了发版是你来承担责任吗?"
    lixiang2017
        19
    lixiang2017  
       2022-12-24 13:24:39 +08:00 via Android
    目前所在公司 code review 极其严格。给出的 comments 基本都要修改,
    lixiang2017
        20
    lixiang2017  
       2022-12-24 13:26:28 +08:00 via Android
    不少时候要修改好多次才能通过,最后才让合进去。某些不想改的地方,除非自己能说出合理的理由
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   2833 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 40ms · UTC 15:24 · PVG 23:24 · LAX 08:24 · JFK 11:24
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.