[BE] 7.05/7.06 매수/매도 예약 취소 API 구현 #51,#52 #73
Conversation
🟢 확실히 맞는 말씀이시네요. 매수/매도는 돈이 추가되거나 줄어들거나 하면서 로직이 바뀌니까 따로 구현하는게 맞는데, 취소하는 것 자체는 그냥 롤백한다는 느낌이니까 하나로 만들어도 괜찮겠군용 |
| if (!order) throw new NotFoundError('주문을 찾을 수 없습니다.'); | ||
|
|
||
| if (order.user_id !== userId) | ||
| throw new ForbiddenException('다른 사용자의 주문은 취소할 수 없습니다.'); |
There was a problem hiding this comment.
🟢 이게 가능한 시나리오에서의 에러인가요? 지난 번 학습 스프린트 때 멘토님이 불가능한 상황에 대한 예외처리는 굳이 할 필요가 없다고 하셨던게 생각나서 여쭤봅니다.
로그인한 사용자만 사용할 수 있는 api인 만큼, 토큰으로 해당 사용자의 정보를 구분해서 취소를 할 수 있는데 이게 가능한 시나리오일까..? 하는 생각이 드네요!
There was a problem hiding this comment.
음... 프론트엔드랑 연결된 실제 서비스에서는 일어날지 잘 모르겠는데, swagger 여러 명이 같이 사용할 때 혼동이 있을까봐 추가해두긴 했습니다.
There was a problem hiding this comment.
하긴 swagger 같은 환경에서는 충분히 혼동이 올 수도 있겠네요.. 제가 너무 실제 환경으로만 생각한 거 같아요! 좋은 의견 감사합니다
| if (order.status === StatusType.COMPLETE) | ||
| throw new ConflictException('이미 체결된 주문은 취소할 수 없습니다.'); | ||
|
|
||
| await this.stockOrderRepository.remove(order); |
There was a problem hiding this comment.
🟢 이거 그냥 hard delete 하는 건가요?
There was a problem hiding this comment.
넵 지금은 hard delete이긴 한뎅 내일 다른 백엔드 분들 이야기 들어보고 결정해보려고 합니당
+ 뭔가 제 생각으로는 주문 취소는 soft delete로 굳이 남겨둘 필요가 있나 시프네용
네넵 맞습니다! 아직 체결되는 로직은 구현을 안한 상태고, 매도/매수 예약? 등록 및 취소만 구현해두었습니당. |
| status: StatusType.PENDING, | ||
| }); | ||
|
|
||
| await this.stockOrderRepository.save(order); |
There was a problem hiding this comment.
🟢 여기 order 변수를 생성 부분이 반복되니 따로 메소드를 구성해보는건 어떨까요? userId, stockOrderRequest, trade_type을 매개변수로 받는 createOrder 같은 메소드로요
✅ 주요 작업
💭 고민과 해결과정
+ 추가로 soft/hard delete 논의 해보아야 할 듯 합니당