파이널 프로젝트 2차 중간 점검 (스파르타 클라우드 엔지니어 캠프 Day+85)

1. 작성일

• 2022-07-11

 

 

2. 회고

파이널 프로젝트 2차 중간 점검을 성공적으로 끝냈다.

발표를 맡아주신 형규님께서 너무 위트있게 발표해 주셔서 다들 재미있게 발표에 집중할 수 있었고

예지님이 준비해주신 시연영상은 완벽한 목소리와 딕션을 베이스로 우리 프로젝트의 구현 기능에 대한 설명이 완벽했다.

팀웍도 좋고 분위기도 좋고 이 흐름 타고 프로젝트 마무리까지 성공적으로 끝내보자!

 

 

 

3. 튜터님들 피드백

 

장형철 튜터님

• 기능의 우선순의를 정하여 남아있는 기능들을 효율적으로 구현할 수 있도록 하세요.
• 필수가 아닌 기능은 전략적으로 제외해도 좋습니다.

박서림 튜터님

• 정말 많은 기능들을 잘 구현하고 계시는것같고 기능별 & 논리적인 기능적 처리등 많은것을 신경쓰면서 구현하고 계신것 같습니다.
• 팀원간의 커뮤니케이션을 집중하며 에자일모형으로 계속 구현을 하시면 현재 정해진 기간안에 서비스흐름에 영향을 주지 않을 꼭 필요한 중요 요소들에 대해 좋은 팀웍을 유지해가며 계속 프로젝트 진행을 잘 하시리라 생각합니다.
• 집중할것과 얻어갈것을 생각해가며 남은 캠프기간동안 알차고 유익하게 화이팅입니다.

강민철 튜터님

• 화면 정의서 잘 보았습니다.
• 모든 팀원이 고르게 contributing 하고 계신 점이 인상적입니다. (팀웍이 좋은 팀 같아 좋았습니다)
• Spring Security에 대한 이해도가 높아 보입니다.
• form에 대한 UI 개선이 필요해 보입니다.
• 또한 README 업데이트가 필요해보이네요. (사용되지 않는 기술 스택이 있다면 제거해주시고, 주요 기능도 업데이트해주세요)
• [발표 중 추가] 깃허브로 Issue를 남기고 PR를 활용하고 계셨군요. 너무 멋지십니다. 한가지 팁을 드리자면 깃허브는 PR의 commit 내의 prefix를 이용하여 간편하게 issue와 커밋을 연결할 수 있습니다. 이와 관련하여 궁금하시다면 말씀드리겠습니다.

 

한창호 튜터님

• 깃헙을 활용한 협업이 잘 이루어지고 있는. 것 같다.
• 일감 관리도 잘 되고 있는 것 같음.
• 권한 없는 페이지 등록 시 단순히 오류화면 혹은 빈 화면을 보여주는 것이 아니라 적절한 화면을 띄워주면 좋을 것 같음.
• 어드민 기능이 구현 잘 된 것 같음.
• 커밋 메시지에 prefix로 feat, fix 등 달아주는거 아주 좋음!
• pr과 이슈를 연결 시켜 주면 좋을듯.
• pr에 템플릿 사용하는 것도 굿!!
• eventcafecloud/src/main/java/com/eventcafecloud/ 패키지 하위 패키지들이 종류 별로 분류가 되어있으면 좋겠음
• dto 객체 만들 때 ‘불변 객체’에 대해 검색 해보고 적용 해보면 좋을 것 같음.
• Builder 패턴과 함께

 

오준오 튜터님

• Dto 클래스가 toDto()와 같은 로직을 갖는 거 보다, service나 mapper 역할을 하는 클래스에서 toDto() 로직을 수행하는 게 나아 보임.
• AuthController와 같이 controller에서 repository 의존성을 가지는 부분 제거하고, controller는 service 레이어에 의존성을 가지게 수정
• return null 대신 return Optional.empty() 사용
• 인터페이스를 활용하여 service <-> controller에 DIP 적용
• 시간 여유가 된다면 테스트 코드 추가해보면 좋을 거 같음.