-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
✨ 이슈 설명
마감 앞두고 요구사항 이것저것 급하게 반영하다보니 굉장한 스파게티를 구웠어요
코드 쓰면서도 좀 언짢았는데 다시금 읽어보니 가슴이 무너져내립니다 😭
processOperation() 아래를 수정하려고 합니다
- Operation 처리하는 부분은 documentService로 분리
- OperationQueueProcessor는 documentService를 주입 받아서 작업을 맡기기만 함
- 실제 작업 처리, 문서에 변경점 적용, 문서 DB 저장 등의 책임은 documentService가 맡도록 변경
- SYNC 작업 처리하는 부분이랑 INSERT&DELETE 작업 처리하는 부분 분리
- 웬만하면 두 작업 엔드포인트랑 dto도 전부 분리
- Operation DTO 유효성 검사 강화 (OperationType 별로)
- Member 메모리에 캐싱 및 유효성 검사
- Member가 해당 classroom에 속한 게 맞는지 확인
- 메모리에 없는 memberId인 경우 DB에서 캐시로 fetch해옴 -> 텍스트 중계 처리시간 감소
- STOMP 에러 처리 핸들러 추가
✨ 작업 예상 시간
확정 짓긴 애매한데 널널하게 잡으려구요
데모데이 끝나고 어떻게 바꿀지 찬찬히 고민해보겠음
Metadata
Metadata
Assignees
Labels
No labels