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

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

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

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

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