DAY 7

🔖 오늘 읽은 범위 : 3. 함수 (54~65p)

---

🤓 책에서 기억하고 싶은 내용

부수 효과를 일으키지 마라!

• 함수에서 한 가지 하기로 약속해놓고 몰래 다른 부수 효과를 일으키지마라
• 시간적인 결합 이나 순서 종속성을 초래한다

public class UserValidator {
	private Cryptographer cryptographer;
	
	public boolean checkPassword(String userName, String password) {
		User user = UserGateway.findByName(userName);
		if (user != User.NULL) {
			String codedPhrase = user.getPhraseEncodedByPassword(); 
			String phrase = cryptographer.decrypt(codedPhrase, password); 
			if ("Valid Password".equals(phrase)) {
				Session.initialize();
				return true; 
		}
	}
	return false; 
	}
}

Session.initialize() 는 의도치 않게 세션 정보를 지워버린다

→ 적어도 이름을 checkPasswordAndInitializeSession 으로 바꾸자

출력 인수

• appendFooter(s); 는 무언가에 s를 바닥글로 첨부하는지 s에 바닥글을 첨부할 지 알 수 없다
• 객체지향에서는 출력인수 사용할 필요가 없다 → this를 사용하라 report.appendFooter()
  

명령과 조회를 분리하라!

• 함수는 뭔가를 수행하거나 뭔가에 답하거나 둘 중 하나만!
• 객체 상태를 변경하거나 객체 정보를 반환 하거나
• 둘 다 하면 혼란을 초래

  public boolean set(String attribute, String value);
  if (set("username", "unclebob"))...

  set이 형용사(p.p정해진-조회)인지 동사(정하다-명령)인지 모르므로 의미가 모호 → 명령과 조회를 분리해 혼란을 뿌리뽑자

  if (attributeExists("username")) { 
  	setAttribute("username", "unclebob"); 
  	...
  }

오류 코드보다 예외를 사용하라!

• 명령함수에서 오류 코드를 반환하는 방식은 명령/조회 분리 규칙을 미묘하게 위반 if문에서 명령을 표현식으로 사용하는 위반 사례 많다

  if (deletePage(page) == E_OK) {
  	if (registry.deleteReference(page.name) == E_OK) {
  		if (configKeys.deleteKey(page.name.makeKey()) == E_OK) { 
  			logger.log("page deleted");
  		} else {
  			logger.log("configKey not deleted");
  		}
  	} else {
  			logger.log("deleteReference from registry failed"); 
  		}
  	} else {
  		logger.log("delete failed"); 
  		return E_ERROR;
  }

  → 오류코드 반환하면 호출자는 곧바로 오류코드 처리해야 함

  try {
  	deletePage(page); 
  	registry.deleteReference(page.name); 
  	configKeys.deleteKey(page.name.makeKey());
  }
  catch (Exception e) {
  	logger.log(e.getMessage()); 
  }

  → 오류 처리코드가 원래 코드에서 분리되어 코드가 깔끔
  

Try/Catch 블록 뽑아내기

• try/catch 블록은 코드 구조 혼란 야기, 정상과 오류 처리동작을 뒤섞음
• try/catch 블록을 별도 함수로 분리하는것이 좋다

  public void delete(Page page) { 
  	try {
  		deletePageAndAllReferences(page); 
  	}
  	catch (Exception e) { 
  		logError(e);
  	} 
  }
  
  private void deletePageAndAllReferences(Page page) throws Exception { 
  	deletePage(page);
  	registry.deleteReference(page.name); 
  	configKeys.deleteKey(page.name.makeKey());
  }
  
  private void logError(Exception e) { 
  	logger.log(e.getMessage());
  }

오류 처리도 한가지 작업이다

• 함수는 ‘한 가지’ 작업만 → 오류 처리도 ‘한 가지’ 작업 → 함수에 try 가 있다면 try로 시작해 catch/finally 문으로 끝나야한다
  

Error.java 의존성 자석

• 오류코드반환? → 어디선가 오류코드를 정의했다(클래스, enum...)

public enum Error { 
	OK,
	INVALID,
	NO_SUCH,
	LOCKED, OUT_OF_RESOURCES, WAITING_FOR_EVENT;
}

의존성 자석: 다른 클래스에서 Error enum을 import 해 사용하므로 Error enum 이 변하면 해당 enum 사용하는 class전부를 다시 컴파일하고 배치해야한다.

→ 오류 코드 대신 예외를 사용하면 새 예외는 Exception 클래스에서 파생

→ 재컴파일/배치 없이도 새 예외 클래스 추가 가능

반복하지 마라

• include 방법으로 중복 없애기
• RDMBS 는 정규화로 중복 제거
• 객체지향은 코드를 부모 클래스로 몰아 중복 제거

구조적 프로그래밍

• Edsger Dijkstra의 구조적 프로그래밍 원칙
  • 모든 함수와 함수 내 모든 블록에 entry 와 exit이 하나만 존재해야 한다.
  • 함수에 return문은 하나여야한다 → loop 내 break, continue, goto ❌
• 함수가 아주 클 때만 이익을 제공하는 원칙 → 함수를 작게 만들면 return, break, continue를 여러 차례 사용해도 괜찮다 → 그래도 goto(큰함수전용)는 피하라
  

함수를 어떻게 짜죠?

• 처음에는 길고 복잡하게
• 들여쓰기, 중복, 인수 여러개, 이름 즉흥적
• 하지만 단위 테스트 케이스는 필수
• 코드 리팩터링, 이름 변경, 중복 제거, 메서드 줄이기, 순서 변경, 클래스 분리 → 단위 테스트는 통과 유지함으로서 이상적인 함수 완성
  

결론

• 모든 시스템은 도메인 특화 언어로 만들어지며 함수는 동사, 클래스는 명사
• 시스템은 구현할 프로그램이 아니라 풀어갈 이야기

🤔 떠오르는 생각

• 니꼬샘이 그 동안 강의에서 함수들을 잘게 쪼개셨던 내용들이 조금씩 이해가 가기 시작한다!

🔎 질문

• 출력인수가 뭔가요? 함수에게 작업거리를 주기 위해 전달되는 것이 아니라 함수가 작업한 결과를 돌려 받기 위해서 사용된다 http://soen.kr/lecture/ccpp/cpp1/6-3-3.htm
  

📝 소감 3줄 요약

• 부수 효과를 일으키지 마라!
• 명령과 조회를 분리하라!
• 오류 코드보다 예외를 사용하라!