๐ย ์ฝ๋ ๋ฆฌ๋ทฐ ์ ์ค์ ์ ์ผ๋ก ๋ด์ผ ํ ๋ค์ฏ ๊ฐ์ง
์ฝ๋ ๋ฆฌ๋ทฐ๋ฅผ ํ ๋๋ ์ด๋ค ๊ฒ๋ค์ ์ค์ ์ ์ผ๋ก ๋ด์ผ ํ ๊น์? ์๋์ด ์คํํ ์์ง๋์ด Gunnar Morling์ ์ฝ๋ ๋ฆฌ๋ทฐ์์ ์ค์ ์ ์ผ๋ก ๋ด์ผ ํ ํญ๋ชฉ์ ๋ค์ฏ ๊ฐ์ง๋ก ๋๋ ์ฝ๋ ๋ฆฌ๋ทฐ ํผ๋ผ๋ฏธ๋๋ฅผ ๋ง๋ค์์ต๋๋ค. ํผ๋ผ๋ฏธ๋์ ์์ ์ถํ์ ์ฝ๊ฒ ๋ณ๊ฒฝ ๊ฐ๋ฅํ ํญ๋ชฉ, ๋ฐ์ ์์์๋ก ๋ณ๊ฒฝ์ด ์ด๋ ค์ด ํญ๋ชฉ์ผ๋ก ๊ตฌ๋ถํ์ด์. ๊ทธ๋ ๊ธฐ ๋๋ฌธ์ ์์ ์๋ ํญ๋ชฉ์ ๊ฐ๋ฅํ๋ฉด ํด์ ์ด์ฉํ ์๋ํ๋ฅผ ํ๊ณ , ๋ฐ์ ์๋ ํญ๋ชฉ์ ๋ ์ค์ ์ ์ผ๋ก ๋ฆฌ๋ทฐํ ๊ฒ์ ๊ถํฉ๋๋ค. ์๋๋ ์ฝ๋ ๋ฆฌ๋ทฐ ํผ๋ผ๋ฏธ๋์ ๋ค์ฏ ํญ๋ชฉ๊ณผ ๊ฐ๊ฐ์ ํญ๋ชฉ์ ๋ฆฌ๋ทฐํ ๋ ๋ฌผ์ด๋ด์ผ ํ ์ง๋ฌธ์ ๋๋ค. 1๏ธโฃย ์ฝ๋ ์คํ์ผ - ํ๋ก์ ํธ์ ์ฝ๋ ์คํ์ผ ํฌ๋งทํ ์ด ์ ์ฉ ๋์๋๊ฐ? - ๋ค์ด๋ฐ ๊ท์น์ ๋ง์กฑํ๋๊ฐ? - DRY(Don't Repeat Yourself) ์์น์ ์ ์ง์ผฐ๋๊ฐ? - ์ถฉ๋ถํ ์ฝ๊ธฐ ์ฌ์ด ์ฝ๋์ธ๊ฐ? (ํจ์ ๊ธธ์ด ๋ฑ) 2๏ธโฃย ํ ์คํธ - ๋ชจ๋ ํ ์คํธ๊ฐ ํต๊ณผ๋์๋๊ฐ? - ์๋ก์ด ๊ธฐ๋ฅ๋ค์ ์ ์ ํ๊ฒ ํ ์คํธ๋์๋๊ฐ? - ์ฃ์ง ์ผ์ด์ค๋ ํ ์คํธ๋์๋๊ฐ? - ์ ์ ํ ๊ฒฝ์ฐ์ ์ ๋ ํ ์คํธ์ ํตํฉ ํ ์คํธ๋ฅผ ์ฌ์ฉํ๋๊ฐ? - ๋น๊ธฐ๋ฅ์ ์๊ตฌ์ฌํญ์ ๋ํ ํ ์คํธ๊ฐ ์๋๊ฐ? (์ฑ๋ฅ ๊ด๋ จ ํ ์คํธ ๋ฑ) 3๏ธโฃย ๋ฌธ์ํ - ์๋ก์ด ๊ธฐ๋ฅ๋ค์ ์ ์ ํ๊ฒ ๋ฌธ์ํ๋์๋? - ํ์ํ ๋ฌธ์๊ฐ ์ถ๊ฐ๋์๋๊ฐ? (README, API ๋ฌธ์, ์ฌ์ฉ์ ๊ฐ์ด๋, ์ฐธ์กฐ ๋ฌธ์ ๋ฑ) - ๋ฌธ์๋ ์ดํดํ๊ธฐ ์ฝ๊ฒ ์ฐ์๋๊ฐ? ๋์ ๋๋ ์ค๋ฅ๋ ์๋๊ฐ? 4๏ธโฃย ๊ธฐ๋ฅ ๊ตฌํ ์คํ์ผ ๋ฐ ์ ํ๋ - ์๋์ ์๊ตฌ์ฌํญ์ ๋ง์กฑํ๋๊ฐ? - ๋ ผ๋ฆฌ์ ์ผ๋ก ์ ํํ๊ฐ? - ๋ถํ์ํ ๋ณต์ก๋๋ ์๋๊ฐ? - ์์คํ ์ ์์ ์ ์ธ๊ฐ? (๋์์ฑ ๋ฌธ์ , ์ ์ ํ ์ค๋ฅ ์ฒ๋ฆฌ ๋ฑ) - ๋ณด์์ ์ธ ์ธก๋ฉด์์ ์์ ํ๊ฐ? (SQL ์ธ์ ์ ๋ฑ) - ๋ชจ๋ํฐ๋ง์ ์ ์ ํ ์ถ๊ฐ๋์๋๊ฐ? (๋ชจ๋ํฐ๋ง ๋ฉํธ๋ฆญ, ๋ก๊น , ํธ๋ ์ด์ฑ ๋ฑ) - ์๋ก ์ถ๊ฐ๋ ๋ผ์ด๋ธ๋ฌ๋ฆฌ๋ ์ ์ ํ ์ญํ ์ ํ๊ณ ์๋๊ฐ? ๋ผ์ด์ ์ค๋ ์ ์ ํ๊ฐ? 5๏ธโฃย API ๊ตฌํ ์คํ์ผ ๋ฐ ์ ํ๋ - API๋ ๊ฐ๋ฅํ ์๊ฒ, ๊ทธ๋ฌ๋ ํ์ํ ๋งํผ ํฌ๊ฒ ์ค๊ณ๋์๋๊ฐ? - API๊ฐ ํ ๊ฐ์ง ์ผ๋ง ์ํํ๋๊ฐ? - ์ผ๊ด์ฑ์ด ์์ผ๋ฉฐ ์ฌ์ด๋ ์ดํํธ๋ ์ต์ํ ๋์๋๊ฐ? - API์ ๋ด๋ถ ๊ตฌํ์ด ๊น๋ํ๊ฒ ๊ตฌ๋ถ๋์๋๊ฐ? - ์ฌ์ฉ์์๊ฒ ๋ ธ์ถ๋๋ ๋ถ๋ถ์ ์ค๋ฅ๋ฅผ ์ผ์ผํฌ ๋งํ ๋ณ๊ฒฝ ์ฌํญ์ ์๋๊ฐ? (API ํด๋์ค, ์ปจํผ๊ทธ ์ค์ , ๋ฉํธ๋ฆญ, ๋ก๊ทธ ํ์ ๋ฑ) - ์๋ก์ด API๊ฐ ๋ณดํธ์ ์ผ๋ก ์ ์ฉํ๊ฐ? ๊ณผ๋ํ๊ฒ ํน์ ๋ชฉ์ ์ผ๋ก๋ง ์ฐ์ด์ง ์์๋๊ฐ? ์ถ์ฒ: https://www.morling.dev/blog/the-code-review-pyramid ๐ ํจ๊ป ์ฝ์ด๋ณด๋ฉด ์ข์ ๊ธ ์ฝ๋ ๋ฆฌ๋ทฐ์์ ํํ ๋ณด์ด๋ ์ธ ๊ฐ์ง ์ค์: https://careerly.co.kr/comments/86864