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

本文探讨了在程序中使用数组和Set来判断元素是否存在于黑名单中时的性能差异与空间效率，并建议使用Set进行改进。通过将黑名单记录在Set中，可以实现更快的查找速度和更高效的内存使用。

摘要生成于 C知道，由 DeepSeek-R1 满血版支持，前往体验 >

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

/** * 判断是否在黑名单里面 * @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去判断是否存在，会比循环判断更快捷方便。

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

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