본문 바로가기

우아한테크코스/코드 리뷰

블랙잭 1 2020. 3. 18. 코로나 바이러스가 점점 심해지고 있어서 재택 교육이 계속해서 연장되고 있다. 그래서 이번 주는 페어 프로그래밍을 온라인으로 진행했다. 구글의 행아웃을 통해 화상으로 대화하고 각자 push & pull하는 방식으로 진행했다. 이번 주는 미션을 진행하면서 몰랐던 Java API 몇 가지를 페어에게서 배울 수 있었다. CQS원칙에 대해 공부하고 정리할 수 있었다. 객체 지향적인 설계, 유지보수하기 좋은 구조에 대해 깊은 고민을 할 수 있었다. 이번 미션은 프리코스때 해봤던 블랙잭게임 이다. 2단계에 걸쳐서 총 2주동안 진행되는 미션이다. 페어에게서 배운 점 Map result = playerResults.stream() .map(UserResult::getReverseResult) .collect(Collec..
로또 게임 2 2020. 3. 2. 코로나 바이러스 인해 2월 25일부터 재택 교육을 하게되었다. 계획은 2주지만 더 연장이 될지도 모른다고 하셨다. 우테코 생활을 한지 몇 주 지나지 않아 이런 일이 생겨서 타오르던 열정에 물을 붓는 느낌이었다. 그래도 2평짜리 고시원 생활을 잠시 접고, 집에서 편하게 잘 수 있다는 생각에 조금은 좋아하기도 했다. 재택 교육을 한다고해서 진행 중이던 미션은 멈추지 않았다. 이번 주는 저번 주에 했던 로또에서 요구사항이 조금 변한 로또 2단계 미션을 진행하였다. 이미 있는 코드를 깔끔하고 유지보수하게 좋게 리팩토링하는 것이 더 힘들다. 업무의 대부분이고 그 일을 잘해야 잘하는 개발자이다. 같은 미션을 반복적으로 설계해야 실력이 늘 수 있다. 캡틴의 강의 내용 중 일부분이다. 말씀대로 이미 있는 코드를 리팩토..
로또 게임 1 2020. 2. 26. 3주차 역시 새로운 짝과 새로운 미션을 시작하였다. 저번 주에 했던 TDD는 그대로 유지를 하면서 진행하였다. 이번 미션은 로또 게임이였다. 요구사항에 보면 모든 원시값과 문자열을 포장한다. 라는 규칙이 있다. 3주차에는 확실히 2주차에 했던 자동차 게임보다 객체의 수도 훨씬 많고 어려워져서 객체지향적인 코드를 짜기 힘들었다. 그래도 페어와 최대한 객체지향적인 코드를 작성하려고 노력했다. 3주차에는 미션을 진행하면서 Exception 컴포지션 패턴 팩토리 패턴 ENUM Stream API 등등 여러가지 새로 배우는 것들이 많았다. 굵직굵직한 주제들은 따로 블로그에 정리해서 올릴 예정이다. Stream API는 지난 미션을 통해 알고있었던 부분들이지만 겉핥기식으로 알고있었다는 걸 이번 미션을 통해 깨달았다..
자동차 경주 게임 2020. 2. 18. 2주차에는 TDD라는 개발 방법론에 대해 공부하고 직접 적용하는 실습을 진행했다. TDD라는 새로운 방식으로 프로그래밍을 하는 것이 익숙치않고 적응하기 힘들었다. 하지만 그 만큼 장점이 있는 방법론이기 때문에 힘들어도 익숙해지려고 노력하는 중이다. TDD에 대해 정리를 해서 글로 작성하기도 했다! 2주 차 미션은 TDD로 진행하는 자동차 경주 게임이었다. 프리코스때 한 번 해본 미션이라서 쉽고 빠르게 진행될 줄 알았는데 생각보다 오래걸렸다. 낯선 TDD로 진행하려다 보니 오래 걸린 것 같다. 익숙치 않고 어색했던 TDD지만, 앞으로 TDD로 개발하며 TDD의 장점을 흡수하기위해 부단히 노력했던 한 주였다. 무엇보다 좋았던 점은 미션 도중에 자세히 몰랐던 부분들을 발견하고 그것들에 대해 학습하고 알게 되었..
문자열 계산기 2020. 2. 10. 1주 차 대망의 첫 번째 미션은 문자열 계산기였다. 크루들과 친해지기 위해 했던 연극과 미션이 겹쳐서 미션에 많은 시간을 투자하지 못한 아쉬움이 있었다. 포비는 첫 주차부터 너무 잘하려고 하지말라고 하셨지만 그래도 첫 주차다보니 욕심이 나는 건 어쩔 수 없었다. 단위 테스트를 실습해보는 미션이었다. 우테코에서 처음으로 받는 미션이었고 처음으로 해보는 페어 프로그래밍이었다. 첫 미션이기 때문에 잘하고싶은 열정, 욕심만 가득했던 한 주였던 것 같다. 리뷰어님께 받은 리뷰내용과 연극 팀원들끼리 진행한 코드리뷰를 정리하고자 한다. 나누기를 할 때 0으로 나누는 것을 검사하고, 0으로 나눈다면 오류를 throw하는 내용인데 모호한 파라미터 명과 모호한 숫자로 인해 if 조건식이 무엇을 판별하는지 명시적으로 보이지..

티스토리툴바