提交代码时,详细准确的 comments,能不能让一个项目在多次换人之后依然不烂,可控. - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
请不要在回答技术问题时复制粘贴 AI 生成的内容
sunnysign
V2EX    程序员

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

  •  
  •   sunnysign
    wuzhefang 2015-03-11 18:48:18 +08:00 3166 次点击
    这是一个创建于 3868 天前的主题,其中的信息可能已经有所发展或是发生改变。
    我在想一个问题:
    如果我们提交代码的时候把所涉及的 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     Solana     2460 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 23ms UTC 15:31 PVG 23:31 LAX 08:31 JFK 11:31
    Do have faith in what you're doing.
    ubao snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86