可以看到，由于每个目标只担负一种责任，清单 4 中的代码理解起来要容易得多。根据用途分离目标，不但可以减少复杂性，还为在不同上下文中使用目标创造了条件，必要时还可以重用。

　　庞大的构建文件也有一种很重的气味

　　Fowler 还将 庞大的类也看作一种代码气味。构建脚本而言，有这种类似气味的是庞大的构建文件，它相当难以读懂。很难知道哪个目标是做什么的，目标的依赖关系是什么。这同样会给维护带来问题。而且，庞大的构建文件通常有相当多的剪切-粘贴的痕迹。

　　为了缩小构建文件，可以从脚本中找出逻辑上相关的部分，将它们提取到更小的构建文件中，由主构建文件来执行这些较小的构建文件（例如，在 Ant 中，可以使用 ant 任务调用其他构建文件）。

　　通常，我喜欢根据核心功能拆分构建脚本，确保它们可以作为独立脚本来执行（想想构建组件化）。例如，在我的 Ant 构建中，我喜欢定义四种类型的开发者测试：单元、组件、系统和功能。而且，我还喜欢运行四种类型的自动检查工具：编码标准、依赖性分析、代码覆盖范围和代码复杂度。我不是将这些测试和检查工具的执行放在一个庞大的构建脚本中（还加上编译、数据库集成和部署），而是将测试和检查工具的执行目标提取到两个不同的构建文件中，如图 2 所示：

　　图 2. 提取构建文件

　　更小、更简洁的构建文件维护和理解起来要容易得多。实际上，这种模式对于代码而言同样适用。我们似乎在这里看到了模式的概念，不是吗？

　　没有清理

　　没有严格减少所有底层假设的构建无疑是一颗定时炸弹。例如，如果构建没有避免一些简单的假设，例如会去掉用陈旧的数据生成的二进制文件，那么前一次构建遗留下来的文件会引起错误。或者，正是由于前一次构建留下的文件，构建竟然得以"成功"，这种情况更糟糕。

　　幸运的是，这个问题的解决办法很直观：只需删除任何之前的构建留下的所有目录和文件，可以很容易地消除假设。这个简单的动作可以减少假设，保证构建的成功或失败都是正确的。清单 5 演示了通过使用 delete Ant 任务删除之前的构建所使用的所有文件或目录，从而清理构建环境的一个例子：

　　清单 5. 事先清理

双击代码全选
 
<target name="clean">
　<delete dir="${logs.dir}" quiet="true" failonerror="false"/>　　
　<delete dir="${build.dir}" quiet="true" failonerror="false"/>　　
　<delete dir="${reports.dir}" quiet="true" failonerror="false"/>　　
　<delete file="cobertura.ser" quiet="true" failonerror="false"/>　　
</target>

　　众所周知，旧的构建遗留下来的文件会导致很多不必要的麻烦。为了自己的方便，在运行一个构建之前，务必先删除构建所创建的任何工件。

　　硬编码的臭味

　　复制-粘贴式的编程有碍重用，将值进行硬编码又何尝不是呢。当构建脚本包含硬编码的值时，如果某个方面需要修改，那么需要在多个地方修改那个值。更糟糕的是，很可能会忽略了某个地方而没有改那个值，从而引起与不匹配的值相关的错误，这种错误是很隐蔽的。而且，如果相信我的建议，选择使用多个构建脚本，那么硬编码的值将可能会成为构建维护中终的挑战。在这一点上也请相信我！

　　例如，在清单 6 中，run-simian 任务有很多硬编码的路径和值，即 _reports 目录：

　　清单 6. 硬编码的值

双击代码全选
 
<target name="run-simian">
　<taskdef resource="simiantask.properties"
　　classpath="simian.classpath" classpathref="simian.classpath" />
　<delete dir="./_reports" quiet="true" />
　<mkdir dir="./_reports" />
　<simian threshold="2" language="java"
　　ignoreCurlyBraces="true" ignoreIdentifierCase="true" ignoreStrings="true"
　　ignoreStringCase="true" ignoreNumbers="true"　ignoreCharacters="true">
　　<fileset dir="${src.dir}"/>
　　<formatter type="xml" toFile="./_reports/simian-log.xml" />
　</simian>
　<xslt taskname="simian"
　　in="./_reports/simian-log.xml"
　　out="./_reports/Simian-Report.html"
　　style="./_config/simian.xsl" />
</target>