Code Review 란
소프트웨어를 실행하지 않고 사람이 직접 검토하는 과정을 통해서, 잠재된 결함을 찾아내고 이를 개선해나가면서 전반적인 소프트웨어의 품질을 높이고자 하는 활동
Programmer’s Ego
개발자는 자신의 코드를 자신의 것으로 여기고 비판을 수용하지 않는 경우가 있다. 이를 Programmer's Ego라고 한다.
The Ten Commandments of Egoless Programming ref.
- Understand and accept that you will make mistakes. (당신이 실수했다는 것을 이해하고 받아들여라)
- You are not your code. (당신은 당신의 코드가 아니다)
- No matter how much “karate” you know, someone else will always know more. (‘Karate’를 당신이 얼마나 알던지간에, 누군가는 당신보다 더 잘 알 수 있다. )
- Don’t rewrite code without consultation (협의없이 코드를 다시 작성하지마라)
- Treat people who know less than you with respect, deference, and patience. (당신보다 많이 알지 못하는 사람이라 해도 존중과 인내로 대해라)
- The only constant in the world is change. (이 세상의 유일한 상수는 세계가 변화한다는 것이다.)
- The only true authority stems from knowledge, not from position. (권위는 지위에서 오는 것이아니라 오직 지식으로부터 나온다. )
- Fight for what you believe, but gracefully accept defeat. (당신이 믿는 것에 대해서 투쟁하되, 우아하게 패배를 받아들여라)
- Don’t be “the guy in the room.” (방 안에 혼자 박혀있는 사람이 되지 마라)
- Critique code instead of people – be kind to the coder, not to the code. (사람이 아니라 코드를 비판해라 - 코드에게는 친절하지 안되, 코더에게는 친절해라 )
Code review에 대한 공부
최근 코드리뷰에 대해서 엄청난 필요성을 느끼고 관심을 가져보고 있다. (늦었다고 생각할 때가 제일 빠를때.. ㅎㅎ)
이전부터 팀원 분들과의 협업 문화에 많은 갈증을 느끼고 있던 나였다. 하지만 협업문화를 누가 나에게 떠먹여주지 않는다는 것을 다시 한번 깨닫게 된 요즘.
내가 열심히 공부해서 팀원분들과 문화를 만들어가고 함께 배워야한다고 생각했다.
아래는 한번씩 보면 좋을 내용들을 정리해봤다.
읽을 서적
읽기 좋은 코드가 좋은 코드다좋은 코드, 나쁜 코드(Good Code, Bad Code)구글 엔지니어는 이렇게 일한다(9장, 코드리뷰)클린 코드