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

2014-05-23 11:38:35 +08:00
 judezhan
在开发的过程中,如果有些功能不需要了,那你会简单的把代码注释掉还是删掉?

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

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

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

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

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

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

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

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

可以看看<Clean Code>
tonghuashuai
2014-05-23 17:33:36 +08:00
版本控制工具,一个变更一个 commit
Jacky001
2014-05-23 18:41:28 +08:00
@TankyWoo +1
发布版本,一定要删除掉。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/113908

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX