짧고 습관적인 코드로 패턴 만들기

2607794 짧고 습관적인 코드로 패턴 만들기 238137 2009-01-31T14:11:23Z http://heycalmdown.springnote.com/pages/2607794 2009-01-31T04:55:38Z 두 요소를 비교하거나 컨테이너에 들어있는 요소의 정렬할 때 키가 둘 이상인 경우가 종종 있지요. 먼저 하나를 비교하고 같을 때 두 번째 값을 비교하는 형태로 구현합니다. 그러면 아래처럼 생긴 코드가 되지요. <ol class="code"> <li>bool greater(const Element& source, const Element& target)</li> <li>{</li> <li> if (source.first < target.first) { </li> <li> return true; </li> <li> } else if (source.first == target.first) { </li> <li> return source.second < target.second; </li> <li> } </li> <li> return false; </li> <li>}</li> </ol> 이런 함수를 클래스 연산자로 재정의하기도 하고 유틸리티 함수로 만들기도 하는데요 오늘은 범용으로 사용할 수 있는 함수를 하나 만들어 볼까 합니다. 오늘은 하고 썼다고 해서 내일도 뭔가 만들지는 않습니다만. <h3>논리표현식 적용하기</h3> 제가 복잡한 조건문을 읽기 편하게 만들기 위해 사용하는 클래스가 있습니다. 이름은 아무래도 상관없지만 여기서는 이걸 LogicalExpression 이라 부르겠습니다. And(&&)와 Or(||)가 복잡하게 섞인 조건문은 아무래도 읽기 어렵기 때문에 괄호로 싸인 각각의 묶음을 역할에 따라 이름을 붙여주려 할 때 사용합니다. 예를 들어 다음과 같은 코드가... <ol class="code"> <li>// 버프가 취소됐거나 지속 시간이 지났으면 제거합니다</li> <li>if (</li> <li style="margin-left: 2em;">(buff.live == false) ||</li> <li style="margin-left: 2em;">(buff.live && (buff.tickInitialized + buff.tickDuration <= Time::tickNow)) {</li> <li> remove(buff); </li> <li>}</li> </ol> 아래처럼 바뀔 수 있습니다.  <ol class="code"> <li>LogicalExpression isDead(buff.live == false);</li> <li>LogicalExpression isExpired(buff.live);</li> <li>isExpired.and(buff.tickInitialized + buff.tickDuration <= Time::tickNow);</li> <li>if (isDead.or(isExpired)) {</li> <li> remove(buff); </li> <li>}</li> </ol> 의미가 더 명확해졌죠. 이제 주석이 필요없는, 의미를 말하는 코드가 되었습니다. 비록 예를 위한 예이긴 해도 어떤 용도로 쓰는지는 아실 거에요. 그럼 우리가 처음에 만든 비교문을 LogicalExpression을 써서 다시 만들되 범용성을 갖도록 해봅시다. <ol class="code"> <li style="line-height: 2em;">template< typename _Ty1, typename _Ty2 ></li> <li style="line-height: 2em;">bool greater(_Ty1 s1, _Ty1 t1, _Ty2 s2, _Ty2 t2)</li> <li style="line-height: 2em;">{</li> <li style="line-height: 2em; margin-left: 2em;">LogicalExpression greaterFirst(s1 < t1);</li> <li style="line-height: 2em; margin-left: 2em;">LogicalExpression greaterSecond(s1 == t1);</li> <li style="line-height: 2em; margin-left: 2em;">greaterSecond.and(s2 < t2);</li> <li style="line-height: 2em; margin-left: 2em;">return greaterFirst.or(greaterSecond);</li> <li style="line-height: 2em;">}</li> </ol> 코드는 더 짧고 명확해졌고, 의도한 건 아니지만 복잡성이 줄어드는 부작용도 일어났습니다. 에전 함수는 Element에 대해서만 사용할 수 있었지만 새로운 함수는 그렇지 않고요. 대신 예전보다 부르기가 번잡스러우니 일을 줄여주는 매크로를 만들 수도 있겠지요. <ol class="code"> <li>#define GREATER_2(s,t,a,b) greater(s.a, t.a, s.b, t.b)</li> <li>{</li> <li> if (GREATER_2(source, target, first, second)) { </li> <li> // target이 더 크다! </li> <li> } </li> <li>}</li> </ol> <h3>뭔가 신경쓰인다</h3> 우리가 만든 새로운 greater 함수에서 첫 번째 요소가 두 번 나오잖아요? 큰지 한 번 보고 같은지 한 번 보니까요. 신경쓰이네요. 비슷한 구문이 반복되면 실수할 여지도 생기고, 이유야 어쨌든 왠지 기분 나쁩니다. 비교하는 값은 동일하고 부등호만 바꿔볼 수 있는 객체가 있으면 좋겠군요. 이걸 PairEvaluate 라고 합시다. 이 객체에 값을 한 번만 넣고 greater()나 equal() 메소들를 부르면 될 것 같네요.   <ol class="code" style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; padding-top: 0px; padding-right: 0px; padding-bottom: 0px; padding-left: 38px; line-height: 1.8em; font-family: courier, monospace; background-image: ; background-color: rgb(239, 239, 239); background-position: 8px 7px;"> <li style="line-height: 2em;">#define GREATER_2(s,t,a,b) greater(PairEvaluate(s.a, t.a), PairEvaluate(s.b, t.b))</li> <li style="line-height: 2em;">{</li> <li style="line-height: 2em; margin-left: 2em;"> if (GREATER_2(source, target, first, second)) { </li> <li style="line-height: 2em; margin-left: 4em;"> // target이 더 크다! </li> <li style="line-height: 2em; margin-left: 2em;"> } </li> <li style="line-height: 2em;">} template< typename _Ty1, typename _Ty2 ></li> <li style="line-height: 2em;">bool greater(const PairEvaluate< _Ty1 >& first, const PairEvaluate< _Ty2 >& second)</li> <li style="line-height: 2em;">{</li> <li style="line-height: 2em; margin-left: 2em;">LogicalExpression greaterFirst(first.greater());</li> <li style="line-height: 2em; margin-left: 2em;">LogicalExpression greaterSecond(first.equal());</li> <li style="line-height: 2em; margin-left: 2em;">greaterSecond.and(second.greater());</li> <li style="line-height: 2em; margin-left: 2em;">return greaterFirst.or(greaterSecond);</li> <li style="line-height: 2em; margin-left: 2em;">//return first.greater() || (first.equal() && second.greater());</li> <li style="line-height: 2em;">}</li> </ol> 마지막 줄에는 LogicalExpression 없이 동일한 의미를 하는 코드를 적어봤는데요, 오히려 이편이 읽기 좋군요. http://heycalmdown.myid.net/ http://heycalmdown.myid.net/ 161