代码优化4-看似不同却是重复的代码

新的一年了，最近也不知道忙什么，杂七杂八的事情太多了。

今天再发一个代码级别的优化点，就是业务意义不一样但功能一样，却有着两份代码：

    public boolean hasExcludeProperty(String[] excludeProperties,
            String[] properties) {
        if (excludeProperties != null) {
            for (String exPro : excludeProperties) {
                for (String pro : properties) {
                    if ((exPro.trim()).equals(pro.trim())) {
                        return true;
                    }
                }
            }
        }
        return false;
    }

    private boolean isExcludeCats(String[] excludeCats, String[] categoryIds) {
        if (excludeCats != null && categoryIds != null)
        {
            for (String cat : excludeCats) {
            	if (StringUtil.isEmpty(cat)) {
            		continue;
            	}
                for (String str : categoryIds) {
                    if (cat.equals(str)) {
                        return true;
                    }
                }
            }
        }
        return false;
    }

上面两段代码中同一个类中出现，其实实现的功能都是检查第二个数组中的字符串是否在第一个数组中出现，只要写一份代码即可。
优化后结果如下

    /**
     * 判断目标数组中的字符串是否在源数组中出现
     * @param src 源数组
     * @param dest 目标数组
     * @return  true 存在着相同的字符串
     */
    private boolean isContain(String[] src,String[] dest) {
    	if (src == null || dest == null || src.length<1 || dest.length<1)
        {
        	return false;
        }
    	for(String s : src) {
    		if (StringUtil.isEmpty(s)) {
        		continue;
        	}
    		s = s.trim();
    		for (String d : dest) {
    			if (StringUtil.isEmpty(d)) {
    				continue;
    			}
                if (s.equals(d.trim())) {
                    return true;
                }
            }
    	}
    	return false;
    }

评论

4 楼 laodizhuq 2010-03-11  

代码优化本来就是一个循序渐进的过程，又不可能一步到位的，只能是你在不断的熟悉系统不断的发现问题，然后不断的去完善它。

3 楼 zkgale 2010-03-02  

这种简单的,能一眼看出来的还好...稍微与业务或复杂点的,你不对整体有个把握的话,你能看出来?

2 楼 chris_in 2010-02-26  

[/quote

1 楼 ren 2010-02-26  

common-lang中没有相应util-methods吗？？