Table
Search
팀원 각자의 코드 리뷰 스타일을 적어주세요 
송용호
코드 길이보단 가독성이 최우선으로 중요하다 생각해요(필요 이상의 코드 압축X , 복잡한 부분엔 주석이 있었으면 좋겠습니다) 네이밍도 중요하지만 나중에 고치면 된다고 생각해요.
홍승표
코드의 중복성을 확인을 하고, 변수나 함수 네이밍(대충 무엇을 위한 변수나 함수 네이밍인지)을 고려합니다. 그래도 네이밍보다 코드의 중복성을 제일 우선적으로 고려합니다.
김지훈
변수 네이밍이 주석으로 설명을 안해도 무슨 역할인지 제대로 설명이 되는지를 주로 봅니다. 각 메서드가 하나의 역할만을 담당하고 있는지 봅니다.(예를들어 중복판별 메서드면 중복 판별만 담당: 단일책임)
박다정
코드의 가독성(통일성, 네이밍)을 제일 먼저 고려하는 것 같습니다! 그 후에는 레이어(controller, repository, service) 의 특성에 맞게 함수들이 제대로 된 위치에 구현되어있는지 보는것도 중요하다고 생각합니다!
김은정
네이밍과 코드의 가독성, 중복성을 고려하여 코드를 읽어볼 것 같습니다! 네이밍은 빡빡하게 고려하기보다는 무슨 기능을 하는지 한눈에 알아볼 수 있을 정도로만 나와 있으면 된다고 생각합니다.
이승훈
전반적인 로직의 흐름을 따라 코드를 이해 할 예정이고 함수와 변수등의 네이밍, 불필요한 코드(중복 코드) 등을 위주로 리뷰를 진행하며 스스로 개선할 수 있도록 도움을 드리고 싶습니다.
홍준표
개발 표준을 잘 정하고 최대한 팀에서 정해진 약속대로 개발이 진행되었는가를 중심으로 리뷰하고싶습니다. 많은 사람이 같이 해도 어느정도 일관된 개발 스타일을 유지할 수 있도록.
유동안
코드 중복 여부와 해당 엔티티계층에 맞는 메서드가 적절히 쓰였는지가 중요하다고 생각합니다.
관계가 복잡해질수록 기존 엔티티의 서비스 단에서 비즈니스 로직을 구성할 수 있음에도 새로운 서비스 단을 만들거나, 중복되는 내용의 메서드를 만들기 쉽다고 생각합니다.
우리 팀의 코드 리뷰 가이드를 세워봅시다!
코드리뷰 체크리스트
구현한 로직 상에서 오류가 날 수 있는 부분은 없는지
변수, 클래스, 메소드 등에 명확한 이름을 선택했는지
주석이 없어도 네이밍이 명확한데 필요 없는 주석이 있는지
코드가 필요 이상으로 복잡하지 않은지
유닛 테스트가 적절한지
리뷰어는 모든 줄을 리뷰하고, 전체적인 문맥을 살피고, 코드 품질을 개선시키고, 작성자가 잘한 점은 꼭 칭찬한다.
스타일 가이드를 잘 지켰는지
코드 리뷰 방식
1.
2.
코드 컨벤션
1.
중괄호 규칙
•
줄의 마지막에서 시작 중괄호{를 쓰고 열고 새줄을 삽입한다. 블럭을 마친후에는 새줄 삽입 후 중괄호를 닫는다.
•
조건, 반복문이 한 줄로 끝더라도 중괄호를 활용한다.
•
예시
public class SearchConditionParser {
public boolean isValidExpression(String exp) {
if (exp == null) {
return false;
}
for (char ch : exp.toCharArray()) {
....
}
return true;
}
}
Java
복사
2.
변수명은 한글소리 그대로 적지 말기
•
나쁜 예시
moohyungJasan(무형자산)
•
좋은 예시
intangibleAssets(무형자산)
3.
규칙명 (네이밍)
•
클래스는 명사
•
메서드는 동사, 전치사로 시작
우리 팀의 1인당 코드 리뷰 최소 시간은?
팀원당 최소 15분, 많게는 30분을 투자해 코드리뷰 진행
코드 리뷰 완료 확인은 어떻게 진행할까요?
디스코드 스레드를 만들어서 진행
ex)
@멘션 OO님, N주차 코드리뷰 완료했습니다 ‼️
Plain Text
복사
