Files
OnJava8/docs/book/16-Validating-Your-Code.md
2019-08-27 15:24:54 +08:00

25 KiB
Raw Blame History

[TOC]

第十六章 代码校验

你永远不能保证你的代码是正确的,你只能证明它是错的。

让我们先暂停学习编程语言的知识,看看一些代码基础知识。特别是能让你的代码更加健壮的。

测试

如果没有测试过,它就是不能工作的。

Java是一个静态类型的语言程序员经常对一种编程语言明显的安全性过于感到舒适“能通过编译器那就是没问题的”。但静态类型检查是一种非常局限性的测试。这说明编译器接受你的语法和基本类型规则但不意味着你的代码满足程序安全的目标。随着你代码经验的丰富你逐渐了解到你的代码从来没有满足过安全性这个目标。迈向代码的第一步就是创建代码测试争对你的目标检查代码行为。

单元测试

这个过程是将集成测试构建到你创建的所有代码中,并在当你每次构建你的系统时运行这些测试。这样,构建过程检查不仅仅是检查语法的错误,同时你也教它检查语义错误。

“单元”指的是测试代码中的一小部分的想法。通常,每个类都有测试检查它所有方法的行为。“系统”测试则是不同的,它检查完成的程序是否满足要求。

C风格的语言特别是C++通常会认为性能比安全更重要。用Java编程比C++一般认为大概快两倍快的原因是Java的安全性网络这种特征类似于垃圾回收以及键入检查。通过将单元测试集成到构建过程中你扩大了这个安全网从而导致了更快的开发效率。当你发现设计或实现的缺陷时可以更容易、更大胆重构你的代码并更快地生成更好的产品。

当我意识到要保证书中代码的正确性时我自己的测试经历就开始了。这本书通过Gradle构建系统 你需要安装JDK你可以通过输入gradlew compileJava编译本书的所有代码。自动提取和自动编译的效果对本书代码的质量是如此的直接和引人注目在我看来任何编程书籍的必备条件——你怎么能相信你没有编译的代码呢? 并且我发现我可以利用搜索和替换在整本书大范围的修改。如果我引入了一个错误,代码提取器和构建系统就会清除它。随着程序越来越复杂,我在系统发现了一个严重的漏洞。 编译程序是毫无疑问的第一步, 对于一本要出版的书而言,这看来是相当具有革命意义的发现(由于出版压力, 你经常打开一本程序设计的书并且发现了上面代码的缺陷。然而我收到了来自读者反馈的语法问题。我在实现一个自动化执行测试系统的时候使用了在早期能看到效果的步骤但这是迫于出版压力与此同时我明白我的程序绝对有问题这些都会变成bug让我自食其果。我也经常收到读者的抱怨说我没有显示足够的代码输出。我需要验证程序的输出并且在书中显示验证的输出。我以前的意见是读者应该一边看书一边运行代码许多读者就是这么做的并且从中受益。然而这种态度背后的原因是我无法保证书中的输出是正确的。从经验来看我知道随着时间的推移会发生一些事情使得输出不再正确或者我一开始就没有把它弄对。为了解决这个问题我利用Python创建了一个工具你将在下载的示例中找到此工具。本书中的大多数程序都产生控制台输出该工具将该输出与源代码清单末尾的注释中显示的预期输出进行比较所以读者可以看到预期的输出并且知道这个输出已经被构建程序验证的。

JUnit

最初的Junit发布于2000年大概是基于Java 1.0因此不能使用Java的反射工具。因此用旧的JUnit编写单元测试是一项相当繁忙和冗长的工作。我发现这个设计令人不爽并编写了自己的单元测试框架作为注解一章的示例。这个框架走向了另一个极端“尝试最简单可行的方法”极限编程中的一个关键短语。从那之后Junit通过反射和注解得到了极大的改进这大大简化了编写单元测试代码的过程。在Java8中他们甚至增加了对lambdas表达式的支持。本书使用当时最新的Junit5版本

在JUnit最简单的使用中使用 @Test 注解标记表示测试的每个方法。JUnit将这些方法标识为单独的测试并一次设置和运行一个测试采取措施避免测试之间的副作用。

让我们尝试一个简单的例子。CountedList 继承 ArrayList ,添加信息来追踪有多少个CountedLists被创建:

// validating/CountedList.java
// Keeps track of how many of itself are created.
package validating;
import java.util.*;	
public class CountedList extends ArrayList<String> {
    private static int counter = 0;
    private int id = counter++;
    public CountedList() {
    System.out.println("CountedList #" + id);
    }
	public int getId() { return id; }
}

标准实例是将测试放在它们自己的子目录中。测试还必须放在包中以便JUnit能够发现它们:

// validating/tests/CountedListTest.java
// Simple use of JUnit to test CountedList.
package validating;
import java.util.*;
import org.junit.jupiter.api.*;
import static org.junit.jupiter.api.Assertions.*;
public class CountedListTest {
private CountedList list;
	@BeforeAll
    static void beforeAllMsg() {
    System.out.println(">>> Starting CountedListTest");
    }
    
    @AfterAll
    static void afterAllMsg() {
    System.out.println(">>> Finished CountedListTest");
    }
    
    @BeforeEach
    public void initialize() {
    	list = new CountedList();
    	System.out.println("Set up for " + list.getId());
        for(int i = 0; i < 3; i++)
        list.add(Integer.toString(i));
    }
    
    @AfterEach
    public void cleanup() {
    	System.out.println("Cleaning up " + list.getId());
    }
    
    @Test
    public void insert() {
        System.out.println("Running testInsert()");
        assertEquals(list.size(), 3);
        list.add(1, "Insert");
        assertEquals(list.size(), 4);
        assertEquals(list.get(1), "Insert");
    }
    
    @Test
    public void replace() {
    	System.out.println("Running testReplace()");
    	assertEquals(list.size(), 3);
    	list.set(1, "Replace");
   		assertEquals(list.size(), 3);
    	assertEquals(list.get(1), "Replace");
    }
    	
    // A helper method to simplify the code. As
    // long as it's not annotated with @Test, it will
    // not be automatically executed by JUnit.
    private void compare(List<String> lst, String[] strs) {
        assertArrayEquals(lst.toArray(new String[0]), strs);
    }
    
    @Test
    public void order() {
    System.out.println("Running testOrder()");
    compare(list, new String[] { "0", "1", "2" });
    }
    
    @Test
    public void remove() {
    	System.out.println("Running testRemove()");
    	assertEquals(list.size(), 3);
    	list.remove(1);
    	assertEquals(list.size(), 2);
    	compare(list, new String[] { "0", "2" });
    }
    
    @Test
    public void addAll() {
    	System.out.println("Running testAddAll()");
    	list.addAll(Arrays.asList(new String[] {
    	"An", "African", "Swallow"}));
    	assertEquals(list.size(), 6);
    	compare(list, new String[] { "0", "1", "2",
    	"An", "African", "Swallow" });
    }
}

/* Output:
>>> Starting CountedListTest
CountedList #0
Set up for 0
Running testRemove()
Cleaning up 0
CountedList #1
Set up for 1
Running testReplace()
Cleaning up 1
CountedList #2
Set up for 2
Running testAddAll()
Cleaning up 2
CountedList #3
Set up for 3
Running testInsert()
Cleaning up 3
CountedList #4
Set up for 4
Running testOrder()
Cleaning up 4
>>> Finished CountedListTest
*/

@BeforeAll 注解是在任何其他测试操作之前运行一次的方法。 @AfterAll 是所有其他测试操作之后只运行一次的方法。两个方法都必须是静态的。

@BeforeEach注解是通常用于创建和初始化公共对象的方法并在每次测试前运行。或者您可以将所有这样的初始化放在test类的构造函数中尽管我认为 @BeforeEach 更加清晰。JUnit为每个测试创建一个对象以确保测试运行之间没有副作用。然而所有测试的所有对象都是同时创建的(而不是在测试之前创建对象),所以使用 @BeforeEach 和构造函数之间的唯一区别是 @BeforeEach 在测试前直接调用。在大多数情况下,这不是问题,如果您愿意,可以使用构造函数方法。

如果您必须在每次测试后执行清理如果修改了需要恢复的静态文件打开文件需要关闭打开数据库或者网络连接etc那就用注解 @AfterEach.

每个测试创建一个新的 CountedListTest 对象,任何非静态成员变量也会在同一时间创建。然后为每个测试调用 initialize() 于是list被分配了一个新的 CountedList 对象,然后用 String“0”、“1”“2” 初始化。观察 @BeforeEach@AfterEach 的行为,这些方法在初始化和清理测试时显示有关测试的信息。

insert()replace() 演示了典型的测试方法。JUnit使用 @Test 注解发现这些方法,并将每个方法作为测试运行。在方法内部,您可以执行任何所需的操作并使用 JUnit 断言方法(已"assert"开头)验证测试的正确性(更全面的"assert"说明可以在Junit文档里找到。如果断言失败将显示导致失败的表达式和值。这通常就足够了但是你也可以使用每个JUnit断言语句的重载版本它包含一个字符串以便在断言失败时显示。

断言语句不是必须的;你可以在没有断言的情况下运行测试,如果没有异常,则认为测试是成功的。

compare() 是“helper方法”的一个例子它不是由JUnit执行的而是被类中的其他测试使用。只要没有**@Test** 注解Junit就不会运行它也不需要特定的签名。在这 compare()private ,表示在测试类使用,但他同样可以是 public 。其余的测试方法通过将其重构为 compare() 方法来消除重复的代码。

本书使用build.gradle控制测试,运行本章节的测试,命令:

gradlew validating:test

Gradle不运行已经运行过的测试所以如果你没有得到测试结果先运行:

gradlew validating:clean

你可以用这个命令运行本书的所有测试:

gradlew test

尽管你可以用最简单的方法如CountedListTest.java所示

JUnit包含许多额外的测试业务您可以在其上了解这些结构

Junit是Java最流行的单元测试框架但也有其它可以替代的。你可以通过互联网发现更适合你的那一个。

测试覆盖率的幻觉

测试覆盖率,同样也称为代码覆盖率,度量代码的测试百分比。百分比越高,测试的覆盖率越大。这里有很多方法

计算覆盖率,还有有帮助的文章Java代码覆盖工具

对于没有知识但处于控制地位的人来说很容易在没有任何了解的情况下也有概念认为100%的测试覆盖是唯一可接受的值。这是一个问题因为100%并不意味着是对测试有效性的最佳测量。您可以测试所有需要它的东西但是只需要65%的覆盖率。如果需要100%的覆盖,您将浪费大量时间来生成剩余的代码,并且在向项目添加代码时浪费的时间更多。

当您分析一个未知的代码库时测试覆盖率作为一个粗略的度量是有用的。如果覆盖率工具报告的值特别低比如少于百分之40则说明覆盖不够充分。然而一个非常高的值也同样值得怀疑这表明对编程领域了解不足的人迫使团队做出了武断的决定。覆盖工具的最佳用途是发现代码库中未测试的部分。但是不要依赖覆盖率来告诉你关于测试质量的任何信息。

前置条件

前置条件的概念来自于契约式设计(Design By Contract, DbC), 利用断言机制实现。我们从Java的断言机制开始来介绍DBC最后使用谷歌Guava库作为前置条件。

断言Assertions

断言通过验证在程序执行期间满足某些条件而增加了程序的健壮性。举例假设在一个对象中有一个数值字段它表示日历上的月份。这个数字总是介于1-12之间。通过断言检查如果超出了该范围则报告错误。如果在方法的内部则可以使用断言检查参数的有效性。这些是确保程序正确的重要测试但是它们不能在编译时被检查并且它们不属于单元测试的范围。

Java断言语法

你可以通过其它程序设计架构来模拟断言的效果因此在Java中包含断言的意义在于它们易于编写。断言语句有两种形式 :

assert boolean-expression

assert boolean-expression: information-expression;

两者似乎告诉我们 “我断言这个布尔表达式会产生一个真正的值” 否则,将抛出AssertionError异常。

这是Throwable的派生类,因此不需要异常规范。

不幸的是,第一种断言形式的异常不会生成包含布尔表达式的任何信息(与大多数其他语言的断言机制相反)。

下面是第一种形式的例子:

// validating/Assert1.java

// Non-informative style of assert
// Must run using -ea flag:
// {java -ea Assert1}
// {ThrowsException}
public class Assert1 {
    public static void main(String[] args) {
    assert false;
    }
}

/* Output:
___[ Error Output ]___
Exception in thread "main" java.lang.AssertionError
at Assert1.main(Assert1.java:9)
*/

如果你正常运行程序,没有任何特殊的断言标志,则不会发生任何事情。你需要在运行程序时显式启用断言。一种简单的方法是使用 -ea flag 它也可以表示为: -enableassertion 这将运行程序并执行任何断言语句。

输出中并没有包含多少有用的信息。另一方面,如果你使用 information-expression 你将生成一条有用的消息作为异常堆栈跟踪的一部分。最有用的 information-expression 通常是一串针对程序员的文本:

// validating/Assert2.java
// Assert with an information-expression
// {java Assert2 -ea}
// {ThrowsException}

public class Assert2 {
    public static void main(String[] args) {
    assert false:
    "Here's a message saying what happened";
    }
}
/* Output:
___[ Error Output ]___
Exception in thread "main" java.lang.AssertionError:
Here's a message saying what happened
at Assert2.main(Assert2.java:8)
*/

information-expression 可以产生任何类型的对象,因此,通常你将构造一个包含对象值的更复杂的字符串,它是否与失败的断言有关。

还可以通过类名或包名打开或关闭断言也就是说您可以为整个包启用或禁用断言。实现这一点的详细信息在JDK的断言文档中。您想要打开或关闭某些断言时此特性对于使用断言进行工具化的大型项目非常有用。然而,日志记录(Logging)或者调试(Debugging,可能是捕获这类信息的更好工具。

这有另一种办法控制你的断言:编程方式,通过链接到类加载器对象(ClassLoader)。类加载器中有几种方法允许动态启用和禁用断言,其中 setDefaultAssertionStatus () ,它为之后加载的所有类设置断言状态。因此,你可以认为你像下面这样悄悄地开启了断言:

// validating/LoaderAssertions.java
// Using the class loader to enable assertions
// {ThrowsException}
public class LoaderAssertions {
public static void main(String[] args) {

	ClassLoader.getSystemClassLoader().
        setDefaultAssertionStatus(true);
		new Loaded().go();
	}
}

class Loaded {
    public void go() {
    assert false: "Loaded.go()";
    }
}
/* Output:
___[ Error Output ]___
Exception in thread "main" java.lang.AssertionError:
Loaded.go()
at Loaded.go(LoaderAssertions.java:15)
at
LoaderAssertions.main(LoaderAssertions.java:9)
*/

这消除了在运行程序时在命令行上使用 -ea 标志的需要,使用 -ea 标志启用断言可能同样简单。当交付独立产品时,您可能必须设置一个执行脚本让用户能够启动程序,配置其他启动参数。这是有道理的,然而,决定在程序运行时启用断言可以使用下面的 static 块来实现这一点,该语句位于系统的主类中:

static {
    boolean assertionsEnabled = false;
    // Note intentional side effect of assignment:
    assert assertionsEnabled = true;
    if(!assertionsEnabled)
    throw new RuntimeException("Assertions disabled");
}

如果启用断言,然后执行 assert 语句,assertionsEnabled 变为 true 。断言不会失败,因为分配的返回值是赋值的值。如果不启用断言,assert 语句不执行,assertionsEnabled 保持false将导致异常。

Guava断言

因为启用Java本地断言很麻烦Guava团队添加一个始终启用替换断言的 Verify 类。他们建议静态导入 Verify 方法:

// validating/GuavaAssertions.java
// Assertions that are always enabled.

import com.google.common.base.*;
import static com.google.common.base.Verify.*;
public class GuavaAssertions {
    public static void main(String[] args) {
    	verify(2 + 2 == 4);
    	try {
    		verify(1 + 2 == 4);
   	 	} catch(VerifyException e) {
    		System.out.println(e);
    	}
        
		try {
			verify(1 + 2 == 4, "Bad math");
		} catch(VerifyException e) {
			System.out.println(e.getMessage());
		}
        
		try {
			verify(1 + 2 == 4, "Bad math: %s", "not 4");
		} catch(VerifyException e) {
        	System.out.println(e.getMessage());
        }
        
        String s = "";
        s = verifyNotNull(s);
        s = null;
        try {
            verifyNotNull(s);
        } catch(VerifyException e) {
        	System.out.println(e.getMessage());
        }
        
        try {
        	verifyNotNull(
        		s, "Shouldn't be null: %s", "arg s");
        } catch(VerifyException e) {
        	System.out.println(e.getMessage());
        }
	}
}
/* Output:
com.google.common.base.VerifyException
Bad math
Bad math: not 4
expected a non-null reference
Shouldn't be null: arg s
*/

这里有两个方法,使用变量 verify()verifyNotNull() 来支持有用的错误消息。注意,verifyNotNull() 内置的错误消息通常就足够了,而 verify() 太一般,没有有用的默认错误消息。

使用断言进行契约式设计

*契约式设计(DbC)*是Bertrand Meyer提出的一个概念Eiffel语言的发明者通过确保对象遵循某些规则来帮助创建健壮的程序。这些规则是由正在解决的问题的性质决定的这超出了编译器可以验证的范围。虽然断言没有直接实现 DBCEiffel也是如此但是它们创建了一种非正式的DBC编程风格。DbC假定服务供应商与该服务的消费者或客户之间存在明确指定的契约。在面向对象编程中服务通常由对象提供对象的边界 — 供应商和消费者之间的划分 — 是对象类的接口。当客户端调用特定的公共方法时,它们希望该调用具有特定的行为:对象状态改变,以及一个可预测的返回值。

Meyer认为:

1.应该明确指定行为,就好像它是一个契约一样。

2.通过实现某些运行时检查来保证这种行为,他将这些检查称为前置条件、后置条件和不变项。

不管你是否同意第一条总是对的在足够多的情况下DbC确实是一种有用的方法。我认为与任何解决方案一样它的有用性也有界限。但如果你知道这些界限你就知道什么时候去尝试。尤其是设计过程中一个有价值的部分是特定类DbC约束的表达式如果无法指定约束则可能对要构建的内容了解得不够。

检查指令

详细研究DbC之前思考最简单使用断言的办法Meyer称它为检查指令。检查指令说明你确信代码中的某个特定属性此时已经得到满足。检查指令的思想是在代码中表达非明显性的结论,而不仅仅是为了验证测试,也同样为了将来能够满足阅读者而有一个文档。

在化学领域,你也许会用一种纯液体去滴定测量另一种液体,当达到一个特定的点时,液体变蓝了。从两个液体的颜色上并不能明显看出;这是作为其中复杂的一部分。滴定完成后一个有用的检查指令是能够断定液体变蓝了。

检查指令是对你的代码进行补充,当您可以测试并阐明对象或程序的状态时,应该使用它。

前置条件

前置条件确保客户端(调用此方法的代码)履行其部分契约。这意味着在方法调用开始时几乎总是会检查参数(在你用那个方法做任何操作之前)以此保证它们的调用在方法中是合适的。因为你永远无法知道客户端会传递给你什么,前置条件是确保检查的一个好做法。

后置条件

后置条件测试你在方法中所做的操作的结果。这段代码放在方法调用的末尾,在return语句之前(如果有的话)。对于长时间、复杂的方法,在返回计算结果之前需要对计算结果进行验证(也就是说,在某些情况下,由于某种原因,你不能总是相信结果),后置条件很重要,但是任何时候你可以描述方法结果上的约束时,最好将这些约束在代码中表示为后置条件。

不变性

不变性保证了必须在方法调用之间维护的对象的状态。但是,它并不会阻止方法在执行过程中暂时偏离这些保证,它只是在说对象的状态信息应该总是遵守状态规则:

1. 在进入该方法时。

2. 在离开方法之前。

此外,不变性是关于构造后对象状态的保证。

根据这个描述,一个有效的不变性被定义为一个方法,可能被命名为 invariant() ,它在构造之后以及每个方法的开始和结束时调用。方法可调用如下:

assert invariant();

这样,如果出于性能原因禁用断言,就不会产生开销。

放松DBC检查 或 非严格的DBC

尽管他强调了前置条件、后置条件和不变性的价值所在以及在开发过程中使用它们的重要性Meyer承认在一个产品中包含所有DbC代码并不总是实用的。您可以放松DbC检查它基于在特定的地方你可以对代码的信任程度。以下是放松检查的顺序最安全到最不安全

1. 不变性检查在每个方法一开始的时候是不能进行的,因为在每个方法结束的时候进行不变性检查能保证一开始的时候对象处于有效状态。也就是说,通常情况下,你可以相信对象的状态不会在方法调用之间发生变化。这是一个非常安全的假设,你可以只在代码末尾使用不变性检查来编写代码。

2. 接下来禁用后置条件检查,当你进行合理的单元测试以验证方法是否返回了适当的值时。因为不变性检查是观察对象的状态,后置条件检查仅在方法期间验证计算结果,因此可能会被丢弃,以便进行单元测试。单元测试不会像运行时后置条件检查那样安全,但是它可能已经足够了,特别是如果你对代码有信心的话。

3. 如果你确信方法主体没有把对象改成无效状态,则可以禁用方法调用末尾的不变性检查。可以通过白盒单元测试(通过访问私有字段的单元测试来验证对象状态)来验证这一点。尽管,它可能没有调用 invariant() 那么稳妥,可以将不变性检查从运行时测试 “迁移” 到构建时测试(通过单元测试),就像使用后置条件一样。

4. 最后,万不得已,禁用前置条件检查。这是最不安全、最不明智的选择,因为尽管你知道并且可以控制你自己的代码,但是你无法控制客户端可能会传递给方法的参数。然而,A 在迫切需要性能和概要分析的情况下,将前置条件检查作为瓶颈,(B) 并且你有某种合理的保证,即客户端不会违反前置条件(如你自己编写客户端代码的情况)。禁用前置条件检查是可以接受的。

不应该直接删除检查的代码,因为只需要禁用检查(添加注释)。这样如果发现错误,你可以轻松地恢复检查以快速发现问题。

DBC + 单元测试

测试驱动开发

日志

调试

基准测试

分析和优化

风格检测

静态错误分析

代码重审

结对编程

重构

持续集成

本章小结