不良代码展示-不要用数组循环判断等于的方式，来判断一个对象是否在一个集合内

废话不说了。看以下代码：

/**
* 判断是否在黑名单里面
* @param name 需要判断的名字
* @return true:在黑名单
*/
private boolean inBlackName(String name) {
String[] blackName = {
"black name 1","黑名单2","no name","kaokao",
};

for(int i = 0; i < blackName.length ; i++) {
if (blackName[i].equals(name)) {
return true;
}
}

return false;
}

其实呢，应该把黑名单记录在一个集合里面，判断集合是否包含需要判断的名字即可。

private static final Set blackNames = new HashSet();
static {
blackNames.add("black name 1");
blackNames.add("黑名单2");
blackNames.add("no name");
blackNames.add("kaokao");
}

/**
* 判断是否在黑名单里面
* @param name 需要判断的名字
* @return true:在黑名单
*/
private boolean inBlackName(String name) {
return blackNames.contains(name);
}

使用Set，是因为黑名单不需要有顺序记录，这样会更节省存储空间。

Set去判断是否存在，会比循环判断更快捷方便。

当然，代码更好的修改，是把黑名单放在配置文件里，在系统启动的时候调入内存。这个就不多说了。

文章最主要的目的，是告诉新手们，不要用数组循环判断等于的方式，来判断一个对象是否在一个集合内。