三步改善Java码质量

一、充分利用单元测试、代码覆盖

        单元测试、代码覆盖是最容易被接受和实现的方式。事实上，大多数开发人员都知道单元测试对他们很重要。在我们开始讨论这些东西之前，先看一下Google研究院主管Peter Norvig的一段话：“如果你认为你们不需要对自己的代码进行单元测试，那么就写在纸上写一所有的原因，并且仔细研究这张纸，然后扔了它，继续测试自己的程序吧”。看来Google也是非常推崇进行单元测试的。那么谁又来测试那么测试者呢？也就是说，我们怎么能验证对程序做了足够的测试呢？这是一个非常有价值的问题，因为那些未通过测试的程序才是我们更应该关注的地方。这个问题的一个解决方案就是使用代码覆盖工具，这种工具将告诉我们我们到底测试了多少代码（也就是被测试代码的百分比），然后使用一般的综合处理来合并覆盖核对结果。如果覆盖核对失败，那么我们建立应用程序的过程也就失败了。
        对了本文所讨论的递增代码策略选择了代码覆盖工具Cobertura，这是由于它非常容易使用，而且拥有良好的定义格式，以及四个Ant任务接口。这些任务之一就是cobertura-check，当代码不能完成我们要求的覆盖率时，它就会失败。如下面的代码显示如果覆盖率未达到80%，Ant在建立工程时就会失败：

<targetname="coverage_check"><cobertura-checktotallinerate="80"/></target name="coverage_check">

        除了使用硬编码来指定这个覆盖率外，我们还可能以使用一个更容易建立的结果作为当前核对的覆盖率。我们可以通过使用两个核心Ant任务连接一对Cobertura任务来完成这个任务。并不不用担心各种覆盖率的具体的值。我们的目标是完成可测量的代码改善，而不是设置一个绝对的代码覆盖率。
        在建立用于测试和运行我们的代码的targets后，我们可以将用于核对我们的建立脚本的增量覆盖率。第一步是使用一个cobertura报吿任务建立一个XML格式的覆盖报表。代码如下：

<cobertura-report format="xml"/>

    下面是由一个cobertura报表任务产生的
coverage.xml

     

<?xml version="1.0"?><!DOCTYPE coverage SYSTEM "http://cobertura.sourceforge.net/xml/coverage-02.dtd"><coverageline-rate="0.43612334801762115"branch-rate="0.48344370860927155"version="1.8"timestamp="1181043899853"><sources><source>./src/java</source></sources><packages>...</packages></coverage>

<xsl:stylesheetxmlns:xsl="http://www.w3.org/1999/XSL/Transform"version="1.0"><xsl:outputmethod="text"omit-xml-declaration="yes"/><xsl:templatematch="coverage">total.line-rate=<xsl:value-ofselect="floor(@line-rate*100)"/></xsl:template></xsl:stylesheet>

<targetname="coverage_check"depends="check_against_previous_rate"><antcalltarget="coverage_report"/></target><targetname="coverage_report"><cobertura-reportformat="xml"destdir="."/></target><targetname="check_against_previous_rate"depends="coverage_xml_to_properties"><propertyfile="build/coverage.properties"/><cobertura-checktotallinerate="${coverage.line-rate}"/></target><targetname="coverage_xml_to_properties"><xsltin="coverage.xml"out="build/coverage.properties"style="src/xsl/coverage.xsl"/></target>

<targetname="time"><tstamp><formatproperty="date.time"pattern="yyyy-MM-dd HH:mm"/></tstamp></target><targetname="log"depends="time"><echofile="${history.txt}"append="true">${date.time};total.line-rate;${total.line-rate}</echo></target>