我正在编写一个方法来返回给定字符是否有效,如下所示:
private static boolean isValid(char c) {
return c == '.' || c == ',' || c == '+' || c == '/' || c == ';' || c == ':';
}
Run Code Online (Sandbox Code Playgroud)
检查样式标记了这一点,因为布尔复杂度太大(5当它应该不超过3).我的开发经理已经标记了一些替代实现,我将作为答案发布.就个人而言,我认为我的代码足够可读,并且希望关闭此方法的检查样式.
你怎么看?
Tar*_*ski 11
private static boolean isValid(char c) {
String validChars =".,+/;:";
return (validChars.indexOf(c) > -1);
}
Run Code Online (Sandbox Code Playgroud)
private static boolean isValid(char c) {
switch (c) {
case '.' : // FALLTHROUGH
case ',' : // FALLTHROUGH
case '+' : // FALLTHROUGH
case '/' : // FALLTHROUGH
case ';' : // FALLTHROUGH
case ':' :
return true;
default : return false;
}
}
Run Code Online (Sandbox Code Playgroud)
| 归档时间: |
|
| 查看次数: |
770 次 |
| 最近记录: |