high ncss method 方法有效代码行太高
某个代码块中代码行数过多(只统计有效的语句),查看代码块中代码是否能拆分,公共功能能否提供一个公共接口。空语句,空块,右括号或分号后的右括号会被忽略。(default value is 30)
void example() // 1
{
if (1) // 2
{
} else // 3
{
}
}
Long method 太长的方法
方法太长,影响阅读,应该实现单一职责。(default value is 50)
void example() {
cout << "hello world";
cout << "hello world";
// repeat 48 times
}
unnecessary default statement in covered switch statement
如果switch覆盖了所有的条件,default是不需要的应该被移除。如果不是default还是需要的
typedef enum {
value1 = 0,
value2 = 1
} eValues;
//
void aMethod(eValues a)
{
switch(a)
{
case value1:
break;
case value2:
break;
default: // this break is obsolete because all
break; // values of variable a are already covered.
}
}