1 问题描述

今天在使用switch语句时，Clang-Tidy提示了这样一个信息：

Clang-Tidy: Potential uncovered code path; add a default label

它表示，函数中可能存在未被覆盖的路径；

2 问题分析

我们使用函数的伪代码图如下：

Room *getEventRoom(int id) {
    switch (id) {
        case 0:
            return new Campsite;
        case 1:
            return new RegularRoom;
        case 2:
            return new TrapRoom;
    }
    // no returning value in other cases
}

分析可知，如果id不属于[0,1,2]的时候，就会出现没有传送返回值的情况；

3 解决方案

其实 Clang-Tidy 在提示中已经给出了解决方案，也就是加粗的部分：

Clang-Tidy: Potential uncovered code path; add a default label

也就是说，我们需要在switch语句中加入default分支，以覆盖所有可能的情况；