Skip to content

Commit

Permalink
발표자료(5)
Browse files Browse the repository at this point in the history
  • Loading branch information
devnproyj22 committed Oct 15, 2024
1 parent 10dcd50 commit 5f41716
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 13 deletions.
13 changes: 0 additions & 13 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -1004,18 +1004,6 @@ class GetObjectOr404Mixin:
원래 결제 처리 로직은 중간중간 예외 처리를 추가하여 중복 결제를 방지하려 했습니다. 그러나 온라인 강의라는 상품의 특성상 상품 수량 제한으로 인해 오류가 발생할 가능성이 적었고, 이러한 예외 처리 로직이 서버의 최적화를 저해할 것이라는 판단이 들었습니다. 이에 따라 예외 처리를 제거하였으나, 코드 리뷰 과정에서 예외 처리 구문이 결제 처리 코드의 길이를 증가시키고 실제로 결제 과중 상황에서 중복 결제를 방지하지 못한다는 피드백을 받았습니다.


### [ 총평 ]

- 독립적인 앱 연결:
이름 수정 => 나머지 코드 수정 방법?
=> 자주 수정할 때 생기는 에러

한 앱에서 수정했을 때 다른 앱에서도 변경
- BE 문서화: schema 를 상세히 작성하지 못했고
- BE 테스트: 구체적인 에러 유형별 확인 못함, 독립적인 앱 연결 -> TDD
- FE 빨리 시작: 늦은 시작, 시간 촉박


## 겪었던 에러
#### 필드값 공백으로 일어난 문제
**예시**:
Expand Down Expand Up @@ -1075,7 +1063,6 @@ CORS_ALLOWED_ORIGINS = [




## 개별 회고(느낀점과 배운점)

유정님
Expand Down
File renamed without changes
File renamed without changes

0 comments on commit 5f41716

Please sign in to comment.