10년 이상 시니어가 주니어의 PR 리뷰시 자주 피드백하는 것들

제가 주니어의 PR에 자주 피드백하는 것들을 생각난김에 정리해보았습니다

 

 

1. null이 꼭 필요한가요?

* string 또는 boolean타입으로도 충분할것 같은데 null이 꼭 필요한가요?

* false 또는"" (empty string)으로 처리되지 않을까요?

* optional로 처리되어야 하는 이유가 있나요?

 

 

2. 더 좋은 이름이 있지 않을까요?

* 다소 긴 이름이어도 괜찮습니다

* 함수 또는 변수의 목적이 좀 더 분명하면 좋겠네요

* 이것은 이름이 지나치게 구체적입니다 

 

 

3. 순수성을 유지해주세요

부수효과가 발생하였습니다

다른곳에서 예상치 못한 오류로 발전될수 있습니다

 

 

4. 함수가 하나의 일만 하는것이 좋겠네요

해당 함수가 너무 많은 일을 하는것 같습니다

 

 

5. 타입이 추론되지 않습니다

타입이 추론될수 있도록 해주세요

 

 

6. 해당 함수를 이해하기 위한 테스트코드가 필요합니다

 

함수의 동작을 이해하기 쉽도록 테스트코드가 있으면 좋겠네요

 

 

7. 모순된 상태가 발생할수 있습니다

 

상태를 나타내는 값이 2개 이상이라 서로 모순된 상태가 발생할수 있습니다

신뢰할수 있는 소스는 1개가 되도록 해주세요