Lee UI jin

[korwww]

요약

Pull Request 체크리스트

TODO

• PR 올리기 전 rebase 동기화를 하셨나요?
• 마지막 줄에 공백 처리를 하셨나요?
• 커밋 단위를 의미 단위로 나눴나요?
  • 예시
    • 코드 가독성을 위해 메서드를 추출하라
    • if-else 문을 if 문으로 분리하라
    • 불필요한 메서드를 인라인화하라
• 커밋 본문을 작성하셨나요?
  • 예시
    • 함수는 한 가지 일을 해야 한다는 원칙에 따라 메서드를 추출합니다.
    • if-else는 컴파일 시 처리가 되어 재컴파일 없이 수정 할 수 없습니다.
      이에 따라 코드가 실행되는 순간에 실행이 결정되는 if 문으로 수정합니다.

[korwww]

merge : draw function extract method

[jihwooon]

Suggested change

	}
	const car = new Car();
	export interface TrafficLight{
	}
	const car = new Car();
	export interface TrafficLight{

이 해당 문단은 띄어쓰기 필요합니다.

[jihwooon]

Suggested change

	const updateCarForLight = (current: TrafficLight) => {
	const updateCarForLight = (current: TrafficLight) => {
	return current.updateCar();
	}

매개 변수는 클래스와 함수가 아닌 인터페이스로 매개변수를 사용하고 있습니다. 왜 인터페이스를 사용 했을까요? 이 부분을 찾아보고 공부를 해보면 어떨까요?

[korwww]

이에 대해 공부해보니 매개변수로 사용하는 것은 그 인터페이스를 따르는 객체들만 매개변수로 할당할 수 있게 해주고
객체들은 인터페이스를 구현해야 하므로 객체마다 .updateCar()의 동작이 다른 경우에도 함수에서 모두 처리할 수 있다는 것을 알았습니다.