不要的功能的代码是注释掉还是删掉呢? - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
请不要在回答技术问题时复制粘贴 AI 生成的内容
judezhan
V2EX    程序员

不要的功能的代码是注释掉还是删掉呢?

  •  
  •   judezhan
    judezhan 2014-05-23 11:38:35 +08:00 9435 次点击
    这是一个创建于 4235 天前的主题,其中的信息可能已经有所发展或是发生改变。
    在开发的过程中,如果有些功能不需要了,那你会简单的把代码注释掉还是删掉?

    以前我比较倾向于注释掉,因为在这个需求多变的时代,说不定什么时候PM抽风就把不要的功能又放上去了,但是时间长了之后,代码就千疮百孔了,像我这种有轻微代码洁癖的用户,看到各种乱七八糟的注释,直接就整个人都不好了。更有甚者,一些喜欢用IDE的选手,批量注释的时候,所有的注释符都是在一行的最前端,这个时候我用Vim打开,直接代码折叠都不管用了。。。所以我现在会要求团队在开发的时候,把不要的功能就直接删掉代码。这样至少代码整洁很多。
    23 条回复    2014-05-25 17:20:47 +08:00
    inroading
        1
    inroading  
       2014-05-23 11:47:50 +08:00
    Git
    wy315700
        2
    wy315700  
       2014-05-23 11:50:20 +08:00
    Git++
    mercurylanded
        3
    mercurylanded  
       2014-05-23 11:50:57 +08:00
    有用 cvs 的话就删掉吧
    hellov22ex
        4
    hellov22ex  
       2014-05-23 11:51:30 +08:00
    有创造性的代码会注释掉留下
    bingyingu
        5
    bingyingu  
       2014-05-23 11:55:13 +08:00
    整两个项目呗!
    一个测试各种可能性,代码什么的可以比较乱!
    另一个干净整洁,用于发布、交付!
    我都这样做的,可能是我的项目都比较小吧,都没突破10W级
    oldman
        6
    oldman  
       2014-05-23 12:29:48 +08:00
    版本控制才是最根本的解决方案。妄图留下注释代码以防止之后再用的思路是错的,代码就应该让版本控制来做。这方面我推荐git。
    wklken
        7
    wklken  
       2014-05-23 12:32:31 +08:00
    git,打tag,删....
    akfish
        8
    akfish  
       2014-05-23 12:32:44 +08:00
    用Git,多commit或者多branch。
    merge到master和develop分支时要求注释不能有代码,每人自己的branch爱怎么折腾怎么折腾。

    感觉你们管理略混乱,代码风格都不规定么。。。
    jsonline
        9
    jsonline  
       2014-05-23 12:35:40 +08:00
    有 Git 还留着不删是几个意思?
    judezhan
        10
    judezhan  
    OP
       2014-05-23 13:26:39 +08:00
    @akfish 小公司,很多东西还在规范中。。。我们开发模式也是每个dev有自己的branch,然后master作为主分支,只有测试过的代码才能放到master,另外,我们只用rebase,不用merge。
    akfish
        11
    akfish  
       2014-05-23 13:48:21 +08:00
    @judezhan 既然都做分支管理了,那完全没必要注释代码,直接删。
    规范明确了,能保证执行,一段时间后自然就好了。
    不怕麻烦的可以继续注释掉代码,反正进主分支前要删干净,人天生都懒,多几次知道麻烦就自觉了。
    cougar
        12
    cougar  
       2014-05-23 14:11:06 +08:00
    压缩工具自动删除注释
    krafttuc
        13
    krafttuc  
       2014-05-23 14:18:50 +08:00
    如果有版本控制,就删干净。
    lazyphp
        14
    lazyphp  
       2014-05-23 14:22:06 +08:00
    得看情况的。如果以后会改进的话,我还是会保留一点 注释的代码在里面。但是整个项目,我不会保留太多。因为这样 会影响别人。
    而且,有些东西交给GIT(或者其他软件)处理才是王道。
    thinkif
        15
    thinkif  
       2014-05-23 14:35:47 +08:00
    业务反复多变,很多时候不是代码不仅仅是“用”或“不用”两种状态。

    如果用版本控制,当需要将很久某次干掉的代码和其他很多次被干掉的代码重新组织在一起用,也可以很方便的实现么?

    另外,当提交代码的次数很多之后,如何快速的找到需要的某个版本中的代码,重要的更新当然会有分支,但是操蛋的业务部门的要求可能实在太诡异,不可能都以里程碑式方式管理代码。

    这时有没有什么好的方法?
    zhibin
        16
    zhibin  
       2014-05-23 15:04:25 +08:00
    GIT, 是我的话就删掉,一个功能就用一个 commit .

    在自己的local 分支做完, merge 到主分支。 然后保留自己的local 分支。

    一般每个功能修改/bug修改都会新建一个分支,merge 到主分支之后 , 我一般都不删掉它。

    这样每次修改都异常的清楚 ~
    lightening
        17
    lightening  
       2014-05-23 16:59:24 +08:00
    @thinkif 原子提交,写看得懂的 commit message。
    TankyWoo
        18
    TankyWoo  
       2014-05-23 17:15:00 +08:00
    删掉

    可以看看<Clean Code>
    tonghuashuai
        19
    tonghuashuai  
       2014-05-23 17:33:36 +08:00
    版本控制工具,一个变更一个 commit
    Jacky001
        20
    Jacky001  
       2014-05-23 18:41:28 +08:00
    @TankyWoo +1
    发布版本,一定要删除掉。
    leavic
        21
    leavic  
       2014-05-23 19:01:16 +08:00
    确定不要了我一般就删掉,留着干啥
    niko
        22
    niko  
       2014-05-24 21:07:26 +08:00
    删掉!!!!!!
    kehr
        23
    kehr  
       2014-05-25 17:20:47 +08:00
    git是个好东西,发现我用的还是太浅了,此贴受教,感谢!
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     1180 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 27ms UTC 17:36 PVG 01:36 LAX 09:36 JFK 12:36
    Do have faith in what you're doing.
    ubao msn 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