代码删除大赛

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

这是一个专门讨论 idea 的地方。

每个人的时间，资源是有限的，有的时候你或许能够想到很多 idea，但是由于现实的限制，却并不是所有的 idea 都能够成为现实。

那这个时候，不妨可以把那些 idea 分享出来，启发别人。

这是一个创建于 3636 天前的主题，其中的信息可能已经有所发展或是发生改变。

如果你们公司有那种 N 年历史的大型 codebase，可以考虑定期组织代码删除大赛：在不影响功能的情况下，删掉多余或者不用的代码。

代码

codebase

大赛

54 条回复 • 2015-01-24 17:01:11 +08:00

dabaii

2015-01-06 20:00:06 +08:00

然后半年后发现，某个秘密角落的功能已经bug了半年。

liul85

2015-01-06 20:02:41 +08:00

@dabaii 同感哈哈

402645707

2015-01-06 20:03:36 +08:00

感觉是一场阴谋，像是在让程序员故意写出难维护的代码

luoweihua7sync

2015-01-06 20:07:12 +08:00

试过了，删完没2周，到处冒烟。。。

Livid

MOD

2015-01-06 20:10:34 +08:00 via Android

嗯，要靠谱实施的话，先让每个人去删自己写的。如果是删别人的，确实有可能会冒烟。

moonkiller

2015-01-06 20:22:36 +08:00

你昵称旁边的mod啥意思哈？
@Livid

lsylsy2

2015-01-06 20:24:16 +08:00

@moonkiller 网站管理员

yellowV2ex

2015-01-06 20:26:26 +08:00

于是有人把空格和回车都删了

cattail

2015-01-06 20:29:48 +08:00

@Livid 自己的代码也依赖别人的代码, 根本就没有自己的代码一说

xcatliu

2015-01-06 20:40:52 +08:00

@cattail 别人的代码也依赖了自己的代码，删了后火灾了

ls25145

2015-01-06 20:54:28 +08:00

现在公司产品里的垃圾代码一大堆，经常出诡异的bug，还有memory leak。
花了大量工数改bug，就是不肯花时间整理代码。
更恶心的是，代码结构好了，别人看不见，没功劳。但是改出问题来了，全是个人责任，大过！
哪里来的动力啊！体制太垃圾了

clowwindy

2015-01-06 22:02:58 +08:00

每次 code review 看着一堆红色的删掉的行就高兴。

paloalto

2015-01-06 22:32:19 +08:00

豌豆荚每隔几个月就有一个 polish week。

cvrock

2015-01-06 22:34:50 +08:00

说实话，别人的代码真心不敢删，谁知道原作者对这个函数有什么假设？
例如函数A看起来不需要做某件奇葩的无关操作，但其实A调用了B，B调用了C，C却依赖A的这个操作，我真心见过，而且依赖关系更复杂，更可怕的删了之后对于普遍情况没影响，但遇到某些特例时……

Mutoo

2015-01-06 22:38:07 +08:00

没做测试驱动的话，有多少人有信心改

spacewander

2015-01-06 22:40:44 +08:00

一场紧张刺激的大赛结束后，程序猿们周末加班加点处理新出现的大量的bug。

Cee

2015-01-06 22:41:21 +08:00

注意不错!

Cee

2015-01-06 22:41:32 +08:00

@Cee 唔主意..

nicai000

2015-01-06 22:44:14 +08:00

If it ain't broke, don't fix it...

cattail

2015-01-06 22:57:32 +08:00 via iPhone

@Mutoo 测试是正解

rannnn

2015-01-06 23:24:49 +08:00

我以前一个公司好几个10万行一个文件的代码。生成的code + 十几年的不停加东西。没人愿意去重构，这玩意谁碰谁倒霉。

Showfom

2015-01-06 23:25:51 +08:00 via iPhone

别删除注释哈哈

lsmgeb89

2015-01-06 23:30:53 +08:00

@rannnn 打开眼界，10w 行。

kxxoling

2015-01-06 23:39:25 +08:00 via iPad

@yellowV2ex Pythoner表示做不到啊！

KentY

2015-01-07 00:36:40 +08:00

5年来一直开发，冗余代码很多，但是谁也不敢动

kent$ (master|…) git gll|head -1
* 13b2fb6 Merge branch 'master' of ssh://... (47 minutes ago by Kai ****) (HEAD, origin/master, origin/HEAD, master)

kent$ (master|…) git gll|tail -1
* 781a979 New remote folder for project. (5 years ago by Stephan ****)

branchzero

2015-01-07 03:04:36 +08:00

怕删了之后留下暗坑，不敢乱动，而且写东西的时候没留单元测试= =

ybh37

2015-01-07 08:14:28 +08:00

这个有点刺激~

knightluffy

2015-01-07 08:54:57 +08:00

删个毛线啊，js代码混淆一下基本就是免删的剧情了。。

wdlth

2015-01-07 09:11:48 +08:00

比如osName.startsWith("Windows 9")？

yinheli

2015-01-07 09:29:17 +08:00

@moonkiller 是 Moderator 的缩写

kaizixyz

2015-01-07 09:32:34 +08:00

想起一句话：技术不是一种优势~是一种债务。

sieon

2015-01-07 09:41:42 +08:00

专门组织，我觉得组织之前考虑清楚应对方法吧。应该没问题。

msg7086

2015-01-07 11:42:16 +08:00 via iPhone

测试驱动开发不已经是标配了么

iamshaynez

2015-01-07 12:56:08 +08:00

重构和代码删除本身都不会产生显性价值但带来风险，在任何一个公司都不会有人愿意冒险做这种事情。只要是存活超过3年的公司，宁愿立项目来做重构的新版本，也不会愿意花成本对已有的（别人的）遗产动手脚。

Elethom

2015-01-07 13:15:04 +08:00

直到入職第三天 contributions 中的綠色才高過紅色...

sitin

2015-01-07 13:15:49 +08:00

我觉得还是测试覆盖要到位啊，才能大胆做删除啊，不然真心不敢乱删。

fising

2015-01-07 13:18:37 +08:00

MOD Moderator

@moonkiller 版主，管理员

moonkiller

2015-01-07 13:35:01 +08:00

@fising
@yinheli 其实我是来秀头像的。。。

yuexiafeng510

2015-01-07 14:54:53 +08:00

在一个根本没有代码规范可言的公司这无疑是自寻死路。。

yuexiafeng510

2015-01-07 14:56:49 +08:00

@rannnn 感同身受目前就在维护一个这样的老不死的项目比我年龄都大

lizheming

2015-01-07 15:10:27 +08:00

写完代码几个月之后准备重构我的代码，一下是场景：
"这几行代码好像没干啥啊，我当时为啥要写啊？"
....
"算了想不太起来了，好像真的没有啥用，删掉吧"
....
"卧槽，有BUG诶，快修修修！"
...(拼命补bug中)
"总算搞定了！诶？！等一下！原来之前的代码是干这个的啊！"

很多时候都会处于以上这种作死的情况..2333