혼자 연습 후 CheckList 관련 코드이슈

[inspirit941]

@dididy 님이 디스코드로 요청하셨던 /api/checklist 관련 문제.

초기 기획안으로는 "혼자 연습한 영상을 다시 보면서 스스로 피드백하고, 체크리스트 형태로 스스로를 평가한 결과를 보내서 저장한다" 는 로직. 위와 같이 체크리스트도 종류별로 구분되어 있다.

그래서 현재 /api/checklist 요청의 경우 CheckListType이라는 객체를 리턴하도록 되어 있고, 각 Type마다 체크리스트를 반환하는 것으로 되어 있던 것으로 보임.

witherview_backend/src/main/java/com/witherview/selfPractice/CheckList/SelfCheckService.java

Line 43 in a409324

List<CheckListType> listTypes = checkListTypeRepository.findAll();

하지만 용재님에게 확인받은 바로는

• 현재 checkList는 Type별 구분 없이, 각 체크박스의 index 값을 checkListId에 넣어 요청하고 있음
• 체크리스트의 각 항목은 프론트에 하드코딩되어 있는 상태

---

맨 위 사진처럼, 원래 기획안대로 checkListType 객체를 살려보는 방법도 있지만,
프로젝트 데드라인과 프론트엔드 작업량을 감안하면.... 일단 프론트에서 보내고 있는 방식을 최대한 살리는 게 맞을 것 같습니다.

구조를 바꾸지 않는다고 할 때, 고민할 사안은

• 같은 영상을 여러 번 보고, 그때마다 평가를 다르게 해서 요청을 보낸다면 어떻게 할 것인가?
  • 허용할 것인가? 허용한다면 이전 데이터를 덮어쓸 것인가? 아니면 새로 추가할 것인가
  • 새로 추가한다면... createdAt 같은 타임스탬프를 추가해서 구분해줘야 함
  • 허용하지 않는다면..?

정도가 있을 것 같습니다. 개인적으로는

• 체크리스트에 코멘트를 따로 다는 형태가 아니라는 점 (check여부만 전송하고 있음)
• 동일한 영상을 혼자 여러 번 돌려보면서 평가를 바꿀 것 같지는 않음

이유 때문에라도 "허용하지 않는다" 혹은 "덮어쓴다" 정도가 괜찮을 것 같긴 합니다... 만

의견 어떠신지 자유로이 알려주세용

[dididy]

@inspirit941 checkListType 객체로 구분되게끔 기획되었다면 아래 swagger의 checkListId가 checkListType을 의미하는 것이었을까요?

그렇다고 한다면 checkListType를 살리고 순서는 불변하므로 순서에 따라 checkListType에 해당하는 id를 지정해줄 수 있을 것 같습니다.

checklist post 요청이 여러 번일 경우 저는 덮어쓴다는 선택지가 가장 좋을 것 같습니다!

[MoonYeeun]

오 일단 저는 애초에 생각을 처음 혼자 연습 - 셀프 평가 후에는 따로 다시 평가를 한다고 생각을 안했었어요,,!! ㅎㅎ

동건님이 말씀하신 "혼자 연습한 영상을 다시 보면서 스스로 피드백하고, 체크리스트 형태로 스스로를 평가한 결과를 보내서 저장한다" 는 부분은

이 화면에서 혼자 면접 종료 후 영상 보면서 평가하고 이 결과를 체크리스트 저장 버튼을 통해 서버로 보내서 저장한다. (처음 한번) 라고 생각했어요..!

그리고 위의 혼자연습내역 부분에서는 평가 내역을 보여주는데 따로 변경 사항을 저장할 버튼이나 그런게 없어가지고..!
그냥 기록 내역은 기록 확인 용도로 생각을 했었는데요..!

음 혼자연습내역 페이지에서 평가를 바꿀 수 있게 한다면 기존 결과를 덮어쓰는 방향으로 가는게 맞을 거 같습니다~!
이 경우 덮어쓰기 전 결과가 별로 중요할 것 같진 않아요 😀