代码覆盖的15种典型情景-1

上一篇 / 下一篇 2012-10-26 14:17:07 / 个人分类：杂谈

)R9S*VJ LU0　　代码覆盖（Code Coverage）为何物？相信程序员特别是测试人员不陌生，很多人都喜欢用代码覆盖来驱动测试的开展和完善。确实代码覆盖可以找出测试疏漏和代码问题，但是单纯的代码覆盖率高低并不能直接反映代码质量的好坏。大多我们的努力方向都是找出那些没有覆盖到的代码，然后补充用例，完善测试。而摆在我们面前的问题是：是否我们已经充分认识到哪些不需要、不能、必须被覆盖？只有对代码覆盖的各种情景了然于胸，才能不盲目乐观于代码覆盖率之高，悲观于代码覆盖率之低。在实践中（本文面向主要Java语言，基于 emma工具），梳理可知，对于代码覆盖我们可能都会遇到以下15种典型情景：51Testing软件测试网J1I}Fa dl

　　1、代码覆盖

　　即代码所有路径被经过，这种需要注意的是：不应该覆盖而被覆盖的情况。例如某种特殊异常就是不期望遇到的，但是遇到了，异常处理的代码也覆盖了，这时，我们应该追溯异常产生的根本原因，而不因覆盖了就直接忽略。

_DE*yfn\w0　　提示：不仅要关注未覆盖的代码，也要关注覆盖的，特别是偶然覆盖的代码。

　　2、废弃的功能

　　一些功能点随着产品版本不断更新，可能会被取消，这部分功能可以直接移除，保留只会让代码看起来越冗余。如果某天需要参考或找回删除的那些代码，CVS/SVN工具就搞定了。

　　提示：不用的功能不需要覆盖，要及时删除，不通过保留或者注释的方式残留在代码中。51Testing软件测试网|.z7e;k5THg

　　3、工具类（助手类）、常量类等的私有构造器

　　工具类和常量类共有的特征是对外开放的都是静态方法，调用方法的时候，无需创建实例，所以推荐实践是创建一个private的构造器方法。这导致类的构造器代码无法覆盖（不考虑反射等方式）。

　　相反，如果某天发现对于这样的类覆盖率为100%，那检查下是否代码写的不规范：用默认构造器，然后通过实例来调用静态方法。

AH4B%Q2O0　　例1：工具类51Testing软件测试网,} U-Re,_W

V!b4z y,T_0public final class StringUtil {

    public static String concatWithSpace(String... strings) {
s n*M;pe nSf0        return concat(MarkConstants.SPACE, strings);51Testing软件测试网Of1U8AmYTb
    }51Testing软件测试网Kb/i+Cm

4W~^qXG gE051Testing软件测试网p%Q^v?+j
    public static String concatWithSemicolon(String... strings) {51Testing软件测试网5?{fDRt
        return concat(MarkConstants.SEMICOLON, strings);51Testing软件测试网8Cya*_$|
    }

.?:~R k} M9S3U ep!}0 private StringUtil() {
}

:|U)q|!K1X8J0}51Testing软件测试网E,]r8cn vz

　　例2：常量类

public final class MarkConstants {
A}V6yX%Gr-B0    /**51Testing软件测试网kB4Dba#n'Du
     *{@value}
'wf%AT^tL!r0     */51Testing软件测试网&^3R_J7S/E
    public static final String SEMICOLON = ";";

private MarkConstants() {
~3sS0vq1Z;A(J d ZK0 }

}

dEQ1M2U1\MP0　　提示：工具类（助手类）、常量类等的私有构造器不能被覆盖

Q3a(Sog,_0　　4、日志级别配置51Testing软件测试网u6s5B'bSt0p'P

Pk p;eH\b @0　　日志级别不同，覆盖率高低也不同。在产品部署中，很少将日志的级别设成debug，因为日志占用磁盘空间会增长很快。只在做一些问题跟踪、调试时才会调高日志级别。

　　所以环境使用不同的日志级别，也会导致一些日志代码没有覆盖。如以下示例程序，不打开debug级别无法覆盖部分代码：

public static String formatPath(String path) {51Testing软件测试网V+@3t6xK
     ValidationUtil.checkString(path);
%I)T DaV$Zy]0     String returnPath = path.trim();
,R'Ad(Z!P)bV@0     if (!returnPath.startsWith(SPLIT))
~6ehAI O(Y}Hq0         returnPath = SPLIT + returnPath;
     if (returnPath.endsWith(SPLIT))
&P!j+}uHq0         returnPath = returnPath.substring(0, returnPath.length() - 1);

     if (LOGGER.isDebugEnabled())51Testing软件测试网jf7eSd"Yl ^
         LOGGER.debug(String
F ur2X0S&K } vAtUP(N0                 .format("[util]convert [%s] to [%s]", path, returnPath));
     return returnPath;
}

　　那么这部分代码需要覆盖嘛？需要。假设代码误写成：

b6Jo"P0P9[0

LOGGER.debug(String.format("[util]convert [%] to [%s]", path, returnPath));

)U%n Zr:WT0　　某天日志级别设为debug，就会发现报错。类似的还有日志中经常输出某个对象信息，但是该对象可能是null，从而抛出空指针异常。51Testing软件测试网 g;s,H(q3x'mw

　　提示：日志也是代码的一部分，需要通过调整日志级别来覆盖。51Testing软件测试网'z3p?Y4z\

in&z MR0　　5、JVM等参数51Testing软件测试网7TBB3Q8fb)p%L

　　程序的配置参数会直接影响代码路径覆盖，不仅包括业务上的一些配置，也包括依赖平台的参数，例如JVM参数除了会影响性能，也会影响代码的覆盖情况，例如断言相关参数：

-ea[:...|:] 和-da[:...|:]

　　分别是启用和关闭用户断言（-esa/-eda，针对系统断言），在JAVA中断言是默认关闭的，所以涉及断言的代码默认无法覆盖。

　　提示：一些代码路径能否覆盖与JVM等参数有关，需要通过调整参数来覆盖

　　6、main()方法51Testing软件测试网4HI-Q(MDI

　　一些程序员喜欢临时写一个main()方法方便于测试，完成测试后寻思以后还能方便测试就留了下来。导致产品代码中的这些代码无法被覆盖。在产品代码中，应该删除这些，部署的毕竟是产品代码，不是测试代码。

^8D?&`rs(Sl0　　提示：main()方不需要被覆盖，产品代码不保留测试代码51Testing软件测试网7y:yntB~o

/p-KXg2_ch`Y0　　7、编码习惯写法

#RdL~;v&\-['[3B3L ]0　　在编码过程中，常常有一些习惯写法，最常见的比如：（1）覆盖toString()方法；（2）以意义配对形式写一些方法：比如数据连接中 Connect()搭配DisConnect()，枚举中常用的toString()搭配fromString()，这些惯用的写法告诉读者一些涵义，但是不见得所有的方法都必须被调用，例如在产品应用中，我们可能启动起一个周期性的job，但是本身尚未添加“取消“功能（或本来就不需要停止）。自然也就无法调用：但是它应该不应该存在？笔者认为作为完整的功能应该存在。类似的还有异常定义的时候，会定义很多重载的方法，虽然不见得每个都调用，但是不定某天就会被调用。51Testing软件测试网SE8G1a\

!n1hS9w:I*M0　　提示：编码习惯写法造成的未覆盖代码需要被覆盖，是代码的一部分。

!@mN6i+t0V Q-g4s.|0　　8、项目的使用方式

#Z!N.wf\b u$_ q O5^.~0　　下面两种使用方式会造成代码不能全部被覆盖：

　　1）客户端Jar方式：部分代码作为客户端Jar包形式提供给他人使用；

　　2）分布式系统交互：分布式系统之间存在交互时，例如从系统1复制文件到系统2，如果始终按照从1到2的顺序，又仅仅统计系统1的代码覆盖肯定不能覆盖全部。需要覆盖的代码虽出于一处，但是使用方式不同也会导致在不合并覆盖数据情况下代码未覆盖。51Testing软件测试网Ba.fkf9V

"T3cb)n5?B(NI8?7HN0　　提示：项目使用方式造成的代码覆盖统计数据分散需要通过合并数据来覆盖。51Testing软件测试网.lbu ^"K {3H

Q?"NH,\#XX9U0　　9、常用最佳实践

\M8_hSMX0　　一些很难覆盖的最佳实践：例如对于一些资源（IO,lock）的释放，可能直接try…catch然后记录异常，这些异常一般很难发生。51Testing软件测试网xQ#w/j_:u]

:J7V7ao'e,mh-]w(B0

public static void close(InputStream inputStream)
{
         try {
             inputStream.close();
         } catch (Exception e) {
             LOGGER.warn("fail to close inputstream");
         }
}

　　提示：常用最佳实践可以不覆盖。51Testing软件测试网 D$@9B&s1DtJ Q

收藏举报

TAG:

查看全部评论

bob123654

不要让那些真正对你好的人，慢慢的从你的生活中消失，无论爱情还是友情，都需要用心经营。

代码覆盖的15种典型情景-1

用户菜单

我的栏目

标题搜索

日历

我的存档

数据统计

RSS订阅