2018년 8월 3일

📓 읽기 좋은 코드가 좋은 코드다. || part3. 코드 재작성하기

**[읽기 좋은 코드가 좋은 코드다.]**를 보고 메모할 부분을 정리했습니다.

PART 3 ) 코드 재작성하기

엔지니어링은 커다란 문제를 작은 문제들로 쪼갠 다음, 각각의 문제에 대한 해결책을 구하고, 다시 하나의 해결책으로 맞추는 일련의 작업을 한다. 이러한 원리를 코드에 적용하면 코드가 더 튼튼해지며 가독성도 좋아진다.

주어진 함수나 코드 블록을 보고, 스스로에게 질문하라 “상위수준에서 본 이 코드의 목적은 무엇인가?”
코드의 모든 줄에 질문을 던져라 “이 코드는 직접적으로 목적을 위해서 존재하는가? 혹은 목적을 위해서 필요하긴 하지만 목적 자체와 직접적으로 상관없는 하위문제를 해결하는가?”
상당히 원래의 목적과 직접적으로 관련되지 않은 하위문제를 해결하는 코드 분량이 많으면, 이를 추출해서 별도의 함수로 만든다.

10. 상관없는 하위문제 추출하기 ⇒ 관심사 분리

11. 한번에 하나씩 ⇒ 관심사 분리 방법 함수는 오직 한 가지 작업만 수행해야 한다. 1. 코드가 수행하는

모든 ‘작업’을 나열한다. 2. 이러한 작업을 분리하여 서로 다른 함수로 혹은 적어도 논리적으로 구분되는 영역에 놓을 수 잇는 코드로 만들면 됟나.

11-1. **작업은 작을 수 있다.

예시 ) 투표**


vote_changed(old_vote, new_vote); // 각 투표는 '추천','반대' 혹은 ''이다.
 
var vote_changed = function (old_vote, new_vote) {
  var scroe = get_score();
  if (new_vote !== old_vote) {
    if (new_vote == 'Up') {
      score += old_vote === 'Down' ? 2 : 1;
    } else if (new_vote == 'Down') {
      score -= old_vote === 'Up' ? 2 : 1;
    } else if (new_vote === '') {
      score += old_vote === 'Up' ? -1 : 1;
    }
  }
  set_score(score);
};

위 코드의 흐름

old_vote와 new_vote가 수치 값으로 ‘해석’된다.

점수가 변경된다. 투표를 수치값으로 해석한다.


  var vote_value = function(vote){
  	if(vote === 'Up'){
  		return +1;
  	}
  	if(vote === 'Down'){
  		return -1;
  	}
  	return 0;
  }
 
  var vote_changed = function(old_vote, new_vote){
  	var score = get_score();
  	score -= vote_value(old_vote);
  	socre += vote_value(new_vote);
  	set_score(score);
  }