不良代碼展示-不要用數組迴圈判斷等於的方式，來判斷一個對象是否在一個集合內

廢話不說了。看以下代碼：

    /**     * 判斷是否在黑名單裡面     * @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去判斷是否存在，會比迴圈判斷更快捷方便。

當然，代碼更好的修改，是把黑名單放在設定檔裡，在系統啟動的時候調入記憶體。這個就不多說了。

文章最主要的目的，是告訴新手們，不要用數組迴圈判斷等於的方式，來判斷一個對象是否在一個集合內。