testable function

最新推荐文章于 2024-11-23 17:02:20 发布

原创最新推荐文章于 2024-11-23 17:02:20 发布 · 376 阅读

0 ·

CC 4.0 BY-SA版权

文章标签：

#function #object #each #string #c

flex 专栏收录该内容

41 篇文章

订阅专栏

本文介绍了一种简化数组比较的方法，通过排序和字符串连接操作，高效地判断两个数组是否相等，以此来优化原有的复杂代码实现。

public function countrySelections(e:Event):Boolean {		
	var countryChanged:Boolean = false;
	var countries:ArrayCollection = e.currentTarget.selectedItems;
	trace("countrySelections:"+countries);
	var tmp:String = '';
  	for each(var i:Object in countries) {
        	    if(i.countryCode == 'All'){
        	    	tmp = 'All';
        	    	break;
        	    } else {
        	    	tmp+=i.countryCode+'|';
        	    }    
        }
        //Alert.show(myStrings.countryList + '___' + tmp);
        if(tmp != 'All'){
        	tmp = tmp.substring(0, tmp.lastIndexOf('|'));
        	if(myStrings.countryList.split('|').length > countries.length){
        		countryChanged = true;
        	} else {
        		for each(var j:Object in countries){
		        	if(myStrings.countryList.indexOf(j.countryCode) < 0 ){
		        		countryChanged = true;
		               }
        	        }
        	}        	
        } else {
        	if(myStrings.countryList != 'All') {countryChanged = true;}
        }        
        myStrings.countryList = tmp;        
        return countryChanged;
}

上面的函数是同事写的，问题很多，其实主要的功能是myStrings.countryList 和e.currentTarget.selectedItems的比较。

1）首先可以把事件去掉

2）引用和改变了全局变量myStrings.countryList

3)两个for,很多的if-else

其实这个问题可以简化为两个排好序数组的比较,可见排序很重要

private function countrySelections2(c1:Array,c2:Array){
	return c1.sort().join("|")!=c2.sort().join("|")
}