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

提交代码时,详细准确的 comments,能不能让一个项目在多次换人之后依然不烂,可控.

  •  
  •   sunnysign ·
    wuzhefang · 2015-03-11 18:48:18 +08:00 · 2672 次点击
    这是一个创建于 3306 天前的主题,其中的信息可能已经有所发展或是发生改变。
    我在想一个问题:
    如果我们提交代码的时候把所涉及的 bug 或需求的来龙去脉,修改情况,提交文件数量和说明.....等等很多信息都说的十分清楚.

    那么在这个项目换了几次人后,能否避免出现积重难返最后无法维护的状况.

    仅通过这种方式能不能有效避免日久项目维护困难的问题.

    大家谁还有更好的办法,欢迎讨论..
    10 条回复    2015-03-12 12:42:32 +08:00
    overflow
        1
    overflow  
       2015-03-11 18:52:25 +08:00
    从来没有听说能单单通过管理 commit message 达到项目管理目的的。
    ming2281
        2
    ming2281  
       2015-03-11 18:53:57 +08:00
    如果遇到猪队友,任你做得怎么好...
    joyeblue
        3
    joyeblue  
       2015-03-11 19:06:06 +08:00
    给几条可行建议(以前我们组就是这样实行的):
    1. 提交时必须写详细说明,长度少于50字不能提交
    2. 在发布之前必须做代码 review, review代码需附上需求或者bug单,另外要对修改代码做简短说明
    3. 发布前需要提交发布申请,包括需求,或者bug单,代码review通过邮件,测试报告等
    4. leader或者总监回复发布申请后,才可进行正式发布

    以上基本可以解决lz提出的问题。

    但lz真是要把流程做的这么长这么复杂么。
    sunnysign
        4
    sunnysign  
    OP
       2015-03-11 19:06:49 +08:00 via iPhone
    @overflow 我觉得comments对于coder来说最直观,反正我很不愿意去翻文档或jira去找修改历史。
    sunnysign
        5
    sunnysign  
    OP
       2015-03-11 19:07:58 +08:00 via iPhone
    @ming2281 假设队友中没有猪,或者说项目经理对所有comments都有审核,保证其清楚 正确
    overflow
        6
    overflow  
       2015-03-11 19:08:51 +08:00
    @sunnysign 那是你没做过项目管理的原因。完全是小白。
    sunnysign
        7
    sunnysign  
    OP
       2015-03-11 19:13:08 +08:00 via iPhone
    @joyeblue 很nice 我觉得这不长 也不麻烦。其实这就是规则,只是新加入的同学需要学习规则。还有个难点就是如何保证所有人都能认真的,富有工匠精神的完成每一次提交。
    randyzhao
        8
    randyzhao  
       2015-03-11 19:16:43 +08:00
    我们现在用的方法:
    git commit 里必须包含 reviewboard 的 reviewid 和 bugzillia 的 bugid.
    用 hook 去处理.
    比如: 缺少 reviewid 或 bugid
    又比如: reviewid 对应的 review 没有被 ship.

    所有的 commits 都能跟踪到 reviewboard 和 bugzillia.

    PS. feature branch 我们是没有 hook 的, 但是 merge 到主分支的时候, 会有要求.
    GuangXiN
        9
    GuangXiN  
       2015-03-12 11:01:14 +08:00
    解决代码可读性问题的方法基本上只能靠CodeReview。
    CodeReview常常被项目赶工而忽视。我以前公司的要求是CodeReview在代码提交测试前进行,没有Review的代码不能提测。程序员在估时间的时候就要估进去Review的时间,如果delay了就自己负责。
    merlinran
        10
    merlinran  
       2015-03-12 12:42:32 +08:00 via Android
    日清原则,保持当下干净,别管什么历史,除非某段代码是为了某个bug的workaround。

    code review顶顶重要,没有之一。
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   2853 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 27ms · UTC 11:41 · PVG 19:41 · LAX 04:41 · JFK 07:41
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.