源代码管理的十条戒律
openkk 12年前
<div id="OSChina_News_26427" class="NewsContent TextContent NewsType2"> <p>源代码管理是我们工作中很重的一部分，是很多开发组的生命。但是我们往往在这方面犯错，不理解很多基本的，核心的版本控制的概念。<br /> <br /> 我在这里列出了十条建议，可以说是戒律。虽然我会用 Subversion 和 .NET 来做示例，但这些戒律和你用的编程语言还有源码管理工具无关。<br /> <br /> <strong>1. 彻底抛弃 VSS！</strong><br /> VSS 已死，就让它离去吧。它曾经很有用，但是现在其他 VCS（Version Control System）已经远远超越了它。微软也决定从明年开始<a href="/misc/goto?guid=4958331827612810923" target="_blank">不再支持 VSS</a>了。<br /> <img alt="源代码管理的十条戒律" src="https://simg.open-open.com/show/d85c5e5ecb2864ff8e7c0e7d60f11cfe.jpg" width="175" height="174" /><img alt="源代码管理的十条戒律" src="https://simg.open-open.com/show/67cf49fd37039fbe7a61499ccf4ceeb2.jpg" width="236" height="173" /><br /> 老实说，在1995年，VSS是一个伟大的工具，但是它的光环早已被它的晚辈，Subversion，Git 和 Mercurial 夺去。<br /> <br /> <strong>2. 没有进入版本库，它就不存在</strong><br /> 你应该每天把这条念一遍 - “工作进展的唯一标准就是代码进了版本库”。在代码进入版本库之前，它相当于不存在。<br /> <br /> 我承认你的代码藏在你机器的某个角落，但是对于其他人来讲，这有何意义？他们不能拿到你的最新版本，他们不能和你的版本合并，你也不能部署你的代码，一旦你的硬盘坏了，一切将烟消云散。<br /> <br /> 如果你坚持的执行这一条的话，你会发现其他的好习惯会随之而来。你会自觉的把任务分成小块所以你可以经常提交代码。你会更加频繁的更新，集成代码。最重要的是，经常提交代码说明了你正在做东西。<br /> <br /> <strong>3. 尽早提交，尽快提交，经常提交</strong><br /> 紧接上面一点，防止“幽灵代码”（只在你本地机器上能看到的代码）的唯一的方式就是把代码尽快的提交到版本库。这么做还有以下好处：</p> <ul> <li>每次提交都会生成一个新的版本，给你回滚的机会。假如你把代码搞乱了，你是希望回滚到一小时前还是一个星期前？</li> <li>间隔的时间越长，代码合并越困难。合并代码从来不是一件好玩的事情。当你好几天都不提交代码，你会突然发现，你已经累计了50个冲突要解决，你大概会疯掉。</li> </ul> <p>当你坚持经常提交代码的习惯以后，你会发现你的工作、代码提交会呈现出一定的规律。这个规律可以用来指导你的开发工作。当你发现你的团队好几天都没有代码提交的时候，你就会意识到 “something is wrong”。<br /> <br /> <strong>4. 在提交前检查你的更改</strong><br /> 提交代码到版本库看起来太简单了。反正在项目的根目录往下有东西更改了，提交吧！这样会导致你提交了一堆垃圾。很多人看到下面的对话框的时候，往往是选择所有，然后搞定！于是你的代码库就被污染了，例如debug文件夹之类的。<br /> <img style="width:568px;height:455px;" alt="源代码管理的十条戒律" src="https://simg.open-open.com/show/2d9d1e53dc4d742e15660499e0b887d9.png" /><br /> <br /> 还 有一些情况是人们提交代码前不检查自己到底更改了什么。例如一个项目配置文件，你会记得你改了什么吗？也许这次修改就不应该提交呢？如下图所示，这个 Web.config文件为什么被修改了？你可以通过SVN工具对比来查看更改。对于一些可能被更改但是不需要进入版本库的文件，例如 Thumbs.db，你可以用 SVN 的“ignore”功能来排除它。<br /> <img style="width:566px;height:452px;" alt="源代码管理的十条戒律" src="https://simg.open-open.com/show/98acbb0e9cd44a9745bd0a16203b8bdd.png" /><br /> <img style="width:571px;height:332px;" alt="源代码管理的十条戒律" src="https://simg.open-open.com/show/7c5de61cc624952a8ae0b23f8bc352ec.png" /><br /> <br /> <strong>5. 认真填写“commit messages”</strong><br /> 填写 commit message（提交注释）是有一点枯燥，但是想象一下别人看你的提交注释的时候是拿着一把斧子，你把他逼疯了他就来砍你！所以不要写“更新了一段代码”这种提交注释，你会把别人逼疯掉。<br /> <br /> “commit message”的目的解释你提交代码到底修改了什么。你也许不记得你一年前提交的一段代码是为什么，但是 SVN 的注释会让你想起来。<br /> <img style="width:544px;height:179px;" alt="源代码管理的十条戒律" src="https://simg.open-open.com/show/576aa82230d3078e60c1075fa2b68fa3.png" /><br /> 所以，请不要写以下这些注释：</p> <ul> <li>修复了一个bug</li> <li>有一个拼写错误</li> <li>更新1024</li> <li>这就是一坨屎</li> <li>提交了</li> </ul> <p>你还有见过比这个更烂的注释吗？还有，任何一个人的注释应该是不一样的，而且从逻辑上来讲永远不可能一样，因为你每次提交代码的基础一定是不一样的，所以你做的事情不可能一样。<br /> <br /> <strong>6. 你必须自己提交代码，而不是让别人代劳</strong><br /> 这个听起来有些奇怪。但这种事情确实存在，而且我还遇到过不止一次。有一些团队为了保证代码库的干净，让一个人专门负责审核和提交代码。这并不是一个好习惯。<br /> <br /> 首先，源代码管理并不是为了保持代码的纯净，起码在开发过程中不是这样。它的目的是让团队更频繁的集成各自的工作，当有问题的时候可以回退。在这个过程中，并不需要每一步都很完美。只有在版本发布的时候，我们才需要，或者尝试去达到“干净的代码”。<br /> <br /> 还有，从开发者的角度来看，这么做等于没有源代码管理。因为它意味着没有代码集成，没有回退，没有blame log，什么都没有。你只是坐在那里写代码，然后在你也无法确定的时间把代码交给你的老板。<br /> <br /> <strong>7. 数据库的版本控制是必须的</strong><br /> <img style="margin:5px;float:left;" alt="源代码管理的十条戒律" src="https://simg.open-open.com/show/1a66e0e10f02e4baaa729ba7b07bb2f9.png" width="172" height="220" />这是有很多人想做，但是觉得很困难而做不到的一点。这里的问题是，很多应用没有数据库根本无法运行。所以如果你不把数据库加入版本控制的话，你的应用是不完整的。<br /> <br /> 大部分的版本控制系统只针对文件系统工作，例如 HTML，CSS，图片，配置文件等等任何保存在文件系统中的东西。但是对于数据库中的数据却无能为力。<br /> <br /> 不过现在也有一些数据库版本控制工具，例如 Red Gate 出品的 <a href="/misc/goto?guid=4958331828421367977" target="_blank">SQL Source Control</a>。关于这个工具我曾经写过很详细的文章 <a href="/misc/goto?guid=4958331829224398840" target="_blank">Rocking your SQL Sourc Control world with Red Gate</a>，我在这里就不赘述了。总之，数据库的版本控制也很容易了！<br /> <br /> <strong>8. 编译出来的文件不应该加入版本控制</strong><br /> 简单的说，就是任何自动生成的东西都不应该放在版本库里面。以 .NET 为例，所有“bin”，“obj”文件夹下面的东西（.dll，.pdb等等）都不应该加入版本库。<br /> <br /> 为什么？因为如果这么做了，你团队的其他人会恨死你的。每次他们更新代码都会用你的编译输出来替代他们的编译输出，会导致他们本地的很多东西不能用。另外一个原因是，你完全没有必要把这些编译输出放在版本控制里面，它们只是在浪费服务器的硬盘和带宽。<br /> <br /> <strong>9. 别人不在乎你的个人配置</strong><br /> 很多时候，人们没有意识到它们正在提交它们自己的个人配置到版本库。开发工具往往会生成一些配置文件，记录你的文件路径，字体设置，快捷键设置等等，这些东西只对你有用。以 .NET 项目为例：<br /> <img alt="源代码管理的十条戒律" src="https://simg.open-open.com/show/54d4ebc5494e21091039aa2fec3d690d.png" width="605" height="170" /><br /> <br /> <strong>10. 依赖项也需要添加到版本库</strong><br /> 虽 然这是最后一条，但也是很重要的。如果你的代码需要依赖第三方的类库或者文件，你需要把这些文件也添加到版本库。你不能认为程序在你的机器可以跑，就在别 人的机器上也能跑。你团队的其他成员也许没有这些依赖的文件，他们更新了你的代码以后，会导致项目无法编译或者程序无法运行。<br /> <br /> 我今天还遇到一个这样的问题。我拉下了很久以前的一个项目，然后出现下面的编译错误：<br /> <img style="width:553px;height:224px;" alt="源代码管理的十条戒律" src="https://simg.open-open.com/show/18bbfe478034dfc1536207d96fc79ad9.png" /><br /> 这个项目也是我开发的，以前的工作环境总是有 NUnit 的，但是现在没有了，所以出现了这些错误。<br /> <br /> <strong>总结</strong><br /> 这十条的每一条都很简单，老实说都是很基础的东西：及时，尽早的提交代码，充分了解你提交的东西，并确认它们确实需要被提交到代码库，解释你的提交，自己提交自己的代码，不要忘记数据库版本控制，不要忘记依赖项。但请你忘记 VSS ：）</p> <p> </p> <p><a href="/misc/goto?guid=4958331830011886133" target="_blank">原文链接</a>，OSChina原创编译</p> </div>