V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX 提问指南
Sponsored by
LinkedIn
2000 个不用坐班的远程好工作在召唤你 · 弹性上班不打卡,工作和生活都能拥有
2000 个不用坐班的全球远程工作,帮助 V2EX 的小伙伴开启全新的工作方式。
Promoted by LinkedIn
sqfphoenix
V2EX  ›  问与答

请问大家对于代码风格比较差的 PR 是如何合并的

  •  
  •   sqfphoenix · 47 天前 · 711 次点击
    这是一个创建于 47 天前的主题,其中的信息可能已经有所发展或是发生改变。

    目前有一个代码风格比较差(当然是我认为,能跑就行的老哥没必要看下去了)的 PR ,涉及到的代码行数有好几百行,新增的方法名称还都是模棱两可的名称,没有明确语义。现在人已经看麻了,根本看不懂改了啥,也没办法描述 PR 有啥问题,对于这种 PR ,大家是直接打回吗

    10 条回复    2022-08-09 10:05:08 +08:00
    CodeM
        1
    CodeM  
       47 天前
    风格好的代码大几百行估计也没几个人愿意每次看完就是了(
    ss098
        2
    ss098  
       47 天前
    编写 Review 意见打回呀
    sqfphoenix
        3
    sqfphoenix  
    OP
       47 天前
    @ss098 很多代码写的不太好,导致没法给出很明确的 Review ,这种情况是让作者先把代码优化一下吗
    anonymousar
        4
    anonymousar  
       47 天前
    @sqfphoenix 如果是你负责 review 那我觉得还是应该一行一行仔细 review 给出明确建议的。
    比如名称你觉得模棱两可 那就应该给出你认可的名称。
    sqfphoenix
        5
    sqfphoenix  
    OP
       47 天前
    @anonymousar 感谢,我现在是一行一行看的,但是疑问比较多,感觉可能是看多了不够耐心,我还是沉下心来继续看吧
    ss098
        6
    ss098  
       47 天前
    @sqfphoenix 是的,可以写逻辑不够清晰之类的。
    HuskyYellow
        7
    HuskyYellow  
       47 天前
    肯定打回, 但是最好说下哪里需要改进,如果着急上线, 那就先合并,以后提优化的板子.
    ragnaroks
        8
    ragnaroks  
       46 天前
    功能没有问题就先合并,然后在其基础上格式化
    bfdh
        9
    bfdh  
       46 天前
    @anonymousar #4 其实这个很难。有些代码真的是差到根本无法 review 。每一行好像都没问题,但加在一起就全是问题,逻辑不对、条理不清,要给建议那也只能建议推倒重来。遇到这种代码,有时候真的是无从下手。
    sqfphoenix
        10
    sqfphoenix  
    OP
       46 天前
    @bfdh 我现在就是这个情况哈哈,已经提了十几个点,先解决可读性和基本逻辑上的问题,再往下看具体功能实现
    关于   ·   帮助文档   ·   API   ·   FAQ   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   1043 人在线   最高记录 5497   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 93ms · UTC 21:47 · PVG 05:47 · LAX 14:47 · JFK 17:47
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.