废话不说了。看以下代码:
/** * 判断是否在黑名单里面 * @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去判断是否存在,会比循环判断更快捷方便。
当然,代码更好的修改,是把黑名单放在配置文件里,在系统启动的时候调入内存。这个就不多说了。
文章最主要的目的,是告诉新手们,不要用数组循环判断等于的方式,来判断一个对象是否在一个集合内。