본문 바로가기

분류 전체보기268

레거시 코드에서 테스트 코드 시작하기 무진장 긴 메서드, 알 수 없는 변수 이름, 텅 빈 테스트 코드 디렉터리...ㅠ 대다수의 기업의 경우, 기존에 개발되어 있는 프로젝트를 인계받거나 처음으로 유지 보수 업무를 맡게 되면 보게 되는 코드의 특징이다. 이런 코드를 이제 수정하거나 변경해야 한다. 이 코드들을 건드리기 시작하면 무수히 많은 오류와 버그에 부딪히게 될 것을 우리는 알고 있다. 손대지 않으면 안 되냐고? 안 된다. 기업이나 고객은 늘 새로운 요구사항을 가지고 온다. 요구사항에 맞추어 수정이 일어나지 않는다면 우리는 더 이상 돈을 받을 수 없다. 테스트 코드가 없는 상황에서 수정은 항상 잠재적인 문제를 가지고 있고 문제가 발생하면 쉽게 해결되지 않아 야근으로 이어진다. 즉 테스트 코드 개발 환경에서 테스트와 배포는 항상 불안하기만 하.. 2023. 12. 1.
우테코 프리코스 공통 피드백 정리 커밋 메시지를 의미 있게 작성한다. git을 통해 관리할 자원에 대해서도 고려 .idea, .metadata, .class 등의 파일을 굳이 관리하지 않아도 됨 PR을 한 번 작성했다면 닫지 말고 추가 커밋 이름을 통해 의도를 드러내기 연속된 숫자나 불용어(info, data, a, an, the) 사용 지양, 축약 금지 클래스와 메서드 이름을 한 두 단어로 유지하려고 노력하기 문맥을 중복하는 이름 자제 order.shipOrder()보다 order.ship이 더 적절 공백도 코딩 컨밴션 if, for, while문 사이의 공백도 코딩 컨벤션 공백 라인을 의미 있게 사용 문맥을 분리하는 부분에 사용 space와 tab을 혼용하지 않는다. 의미 없는 주석을 달지 않는다 주석 대신 변수명, 메서드 명을 통해.. 2023. 11. 17.
[우테코 프리코스] 2주차 회고 이번주에는 자동차 경주 게임 미션을 수행하였다. 카트라이더나 마리오 카트 같은 복잡한 게임은 아니지만 확률적으로 자동차를 이동시키고 먼저 끝에 도달한 사람이 이기는 게임이었다. 경주 게임 미션을 수행하면서 저번주 리뷰에서 알게 된 사실과 개선 방향들을 참고해서 코드를 작성하였다. 이번에는 애플리케이션의 구조를 명확하게 해 보기 위해 MVC 패턴을 사용해 보았다. 뷰와 컨트롤러, 모델을 분리해 놓고 비즈니스 로직은 최대한 모델에서 수행하도록 하였다. 이렇게 하니 각각의 레이어가 가지는 책임이 좀 더 명확해진 것 같다. 다만, 이번에는 모델들이 갖는 책임들이 적절하게 분배가 되었는지 각 계층 간 의존도는 없거나 낮은지 좀 더 생각해보아야 할 것 같다. 이번 주 리뷰에서 나온 내용 1. 람다식 매개변수에도 불.. 2023. 11. 4.
[우테코 프리코스] 1주차 회고 이번 주에는 우아한테크코스의 첫 관문인 프리코스의 1주 차를 수행하였다. 총평을 해보자면, 좋은 코드를 만들기 위해 스스로 고민해 볼 수 있는 좋은 기회였고, 다수의 동료들로부터 코드 리뷰를 받는 엄청난 경험을 할 수 있어서 뜻깊었다. 프리코스의 주제 자체가 함께 성장하기이다보니 지원자들이 모인 디스코드에서는 1주 차임에도 불구하고 정보 공유와 토론이 매우 활발하게 이뤄졌다. 이런 정도면 주차가 지날수록 더 활발해질 것으로 보인다. 여기서 좋은 코드 작성을 위한 인사이트와 여러 꿀팁들을 얻을 수 있었다. 프리코스가 지나면 이런 기회가 사라질 것이라고 생각하니 아쉽기도 하면서 더 많이 가져가고 싶다는 욕심이 생겼다. 프리코스에서 기대했던 것 중 하나가 코드리뷰였다. 서로의 경험과 실력의 차이에 상관없이 서.. 2023. 10. 28.
728x90