WIL | 12.4.일 [11.28.월 - 12.2.금]

(B2 | 5_team_project)

팀프로젝트 주간

 

- KPT 회고

1. 프로젝트 개요

• 업로드한 사진을 다양한 스타일의 유화로 변환해주는 웹 서비스
• 유화 변환 웹서비스 - 사용자가 업로드한 사진을 선택한 유화 화풍으로 변경해주는 시스템
• 프로젝트 명 : 유화 변환 웹서비스 프로젝트
• 필수 작업 : DRF CBV 사용, serializer, custom user model, CRUD, CORS, 회원가입/로그인, 유화 변환 시스템, 백엔드 AWS EC2 배포

 

K(keep)

• 팀원이 서로 격려하고, 모르는 것에 대해 질문에도 사려 깊게 알려주며 서로를 이끌어가는 협력의 자세
• 각자의 의견을 최대한 존중하고 배려하는 모습
• 끝까지 포기하지 않고 시도해서 할 수 있는 것을 완성해내는 노력
• 기한 내에 안정적으로 프로젝트를 마무리하는 습관
• 슬랙/git을 통한 활발한 의사소통과 트러블 슈팅 기록
• Github를 체계적으로 관리하기README.md에 프로젝트 설명 자세히 남기기(erd,api명세서 추가)
• 구성원 각자의 메인/서브 파트를 설정하고 효율적으로 분업 하는 것

p(problem)

• 코드 리뷰를 진행 하긴 했지만 원활히 진행 되지 않은 느낌
• 사용하지 않는 코드를 정리하지 않았음
• 쿼리셋 남발 등 깔끔하지 못한 코드를 짬
• 통일성이 부족한 함수 또는 변수
  • 예) profile페이지에 사용할 Serialize 함수를 보면, UserprofileSerializer, UserprofileImageCreateSerializer등 통일해서 사용할 수 있는 부분도 너무 세분화하여, 복잡하게 사용되고 있음.

T(try)

• 프로젝트를 실제로 활용할 수 있는 방안(간단한 기능이어도 실제 유저가 사용할 수 있는 것을 개발)에 대해서 진행할 것
• 우리가 만들 프로젝트에 대한 조금 더 상세하고 명확한 이해와 소통을 목표로 초기 프로젝트 설계하기
• 프로젝트 기간이 끝나고 나서 코드 리뷰를 하고 코드 리팩토링을 하면서 더 발전시켜보기
• 프로젝트 피드백 참고하여 다음 프로젝트에 적용하기
  • Clean code 지향
    • Legacy code및 불필요 코드 정리
    • 중복 쿼리 날리지 않기
    • 프론트에서 UX 상 필요없는 기능 정리하기
    • 시리얼라이즈를 줄이기
    • 피드백불필요 코드 정리특정 모델 시리얼라이저 writeonly, readonly, required 속성 이용해 하나의 시리얼라이저로 처리해보기
    • 수정,삭제 권한 없으면 보이지 않게 설정하기
    • 로그인 검사 중복 쿼리 정리