随着 PHP 从一种简单的脚本语言转变为一种成熟的编程语言,一个典型的 PHP 应用程序的代码库的复杂性也随之增大。为了控制对这些应用程序的支持和维护,我们可以使用各种测试工具来自动化该流程。其中一种是单元测试,它允许您直接测试所编写代码的正确性。然而,通常遗留代码库是不适合进行这种测试的。本文将介绍对包含常见问题的 PHP 代码的重构策略,以便简化使用流行的单元测试工具进行测试的过程,同时减少改进代码库的依赖性。
简介
回顾 PHP 的发展历程,我们发现它已经从一个简单的用来替代当时流行的 CGI 脚本的动态脚本语言变成一种成熟的现代编程语言。 随着代码库的增长,手动测试已经变成不可能完成的任务,无论是大是小,所有代码的变化都会对整个应用程序产生影响。这些影响可能小到只是影响某个页面的加 载或表单保存,也可能是产生难以检测的问题,或者产生只在特定条件下才会出现的错误。甚至,它可能会使以前修复的问题重新出现在应用程序中。为此开发了许 多测试工具来解决这些问题。
其中一种流行的方法是所谓的功能或验收测试,它会通过应用程序的典型用户交互来测试这个应用程序。这是一种 很适合测试应用程序中各个进程的方法,但是测试过程可能非常慢,而且一般无法测试底层的类和方法是否按要求正常工作。这时,我们需要使用另一种测试方法, 那就是单元测试。单元测试的目标是测试应用程序底层代码的功能,保证它们执行后产生正确的结果。通常,这些 “不断增大” 的 Web 应用程序会慢慢出现越来越多久而久之难以测试的遗留代码,这使开发团队很难保证应用程序测试的覆盖率。这通常被称为 “不可测试代码”。现在让我们看看如何识别应用程序中的不可测试代码,以及修复这些代码的方法。
识别不可测试的代码
关于代码库不可测试性的问题域通常在编写代码时是不明显的。当编写 PHP 应用程序代码时,人们倾向于按照 Web 请求的流程来编写代码,这通常就是在应用程序设计时采用一种更加流程化的方法。急于完成项目或快速修复应用程序都可能促使开发人员 “走捷径”,以便快速完成编码。以前,编写不当或者混乱的代码可能会加重应用程序中的不可测试性问题,因为开发人员通常会进行风险最小的修复,即使它可能产生后续的支持问题。这些问题域都是无法通过一般的单元测试发现的。
依赖全局状态的函数
全局变量在 PHP 应用程序中很方便。它们允许您在应用程序中初始化一些变量或对象,然后在应用程序的其他位置使用。然而,这种灵活性是有代价的,过度使用全局变量是不可测试代码的一个通病。我们可以在 清单 1中看到这种情况。
清单 1. 依赖于全局状态的函数
<?php function formatNumber($number) { global $decimal_precision, $decimal_separator, $thousands_separator; if ( !isset($decimal_precision) ) $decimal_precision = 2; if ( !isset($decimal_separator) ) $decimal_separator = '.'; if ( !isset($thousands_separator) ) $thousands_separator = ','; return number_format($number, $decimal_precision, $decimal_separator, $thousands_separator); }
这些全局变量带来了两个不同的问题。第一个问题是您需要在测试中考虑所有这些全局变量,保证给它们设置了函数可接受的有效值。第二个问题更为严重, 那就是您无法修改后续测试的状态并使它们的结果无效,您需要保证将全局状态重置为测试运行之前的状态。PHPUnit 有一些工具可以帮您备份全局变量并在测试运行后恢复它们的值,这些工具能够帮助解决这个问题。然而,更好的方法是使测试类能够直接给方法传入这些全局变量的值。清单 2显示了采用这种方法的一个例子。
清单 2. 修改这个函数以支持重写全局变量
<?php function formatNumber($number, $decimal_precision = null, $decimal_separator = null, $thousands_separator = null) { if ( is_null($decimal_precision) ) global $decimal_precision; if ( is_null($decimal_separator) ) global $decimal_separator; if ( is_null($thousands_separator) ) global $thousands_separator; if ( !isset($decimal_precision) ) $decimal_precision = 2; if ( !isset($decimal_separator) ) $decimal_separator = '.'; if ( !isset($thousands_separator) ) $thousands_separator = ','; return number_format($number, $decimal_precision, $decimal_separator, $thousands_separator); }
这样做不仅使代码变得更具可测试性,而且也使它不依赖于方法的全局变量。这使得我们能够对代码进行重构,不再使用全局变量。
无法重置的单一实例