180bpm

13년 4월 10일 코드 리뷰 본문

Flash/AS2

13년 4월 10일 코드 리뷰

powdersnow 2013. 4. 10. 19:05

근 2달만에 한 코드리뷰.


1. array의 순서가 꼭 필요없다면 for in으로 할것.

- Object의 요소를 단체로 훑어버리는것이기 때문에.


2. 꼭 return null 할 필요 없음



function foo (){
}
trace(foo());
//output : undefined



3.invalidate() 에 대해

CoreList의 dataProvider에 값을 넣을때 가끔 렌더링이 갱신이 안될때가 있었다.

정확한 이유는 모르고 invalidate() 를 하니 되더라.. 해서 계속 이런식으로 하고 있었는데



var a:Array = buttonBar.dataProvider;

//값 수정

buttonBar.dataProvider = a;

이런 식으로 할 경우


public function set dataProvider(value:Object):Void {
		if (_dataProvider == value) { return; }
//중략

여기서 걸린다.

그래서 invalidate()



4. === (완전항등연산자)

if (true == 1)
이러면 1이 true로 변하고 뭐하고 뭐하고... 연산이 많아져서 완전항등연산자로 비교하는게 빠르다.

다음과 같은 형식에 적용하라고 하셨다


//code start
function f($value){
	if ($value.bools === true) {};
	if ($value.numbers === 1) {};

}
//code end


Comments