전남대 BE_김보민 5주차 과제 (2단계) #373

KBM05haruin30 · 2024-07-28T10:39:55Z

2단계

코드 작성하면서 어려줬던 점

카카오 로그인과 기존의 로그인을 합치고 싶어 액세스 토큰을 이용해 이메일을 추출했습니다. 그 후 Member DB에 저장한 다음 jwtToken을 생성하여 이걸 사용했습니다. 주문을 통한 상품 수량 차감은 원활하게 되었지만 주문 버튼을 누를 때 카카오톡 메시지를 보내는 것이 어려웠습니다. 액세스 토큰도 같이 넘겨주어야 하는데 처음에는 jwt 토큰처럼 post가 발생하면 같이 보내는 형식으로 하다가 실패했습니다. 이것에 관하여 팀원에게 조언을 구했었습니다. 팀원이 Order에 관한 DTO를 Request와 Response로 나누고 Request에 String accessToken 작성한 후 html에서 localStorage의 accessToken을 RequestDTO에 같이 넣어 보내주면 된다고 알려주었습니다. 이 방식대로 하니 accessToken도 같이 넘어오는 것을 확인할 수 있었습니다.

궁금한 점

카카오 액세스 토큰을 발급 받을 때 refresh_token과 refresh_token_expires_in도 함께 있었습니다. 이것에 관하여 1단계 리뷰에서 찾아보면 도움이 될 것 같다고 하셨는데 아직 어떻게 써야 하는지 감을 못 잡아서 사용하지 못했습니다. 대신 기존 로그인과 연동하면서 jwtToken을 사용했습니다. 이 때 jwtToken에도 만료 시간이 존재하는데 이것으로 refresh_token이 대체가 되는지 궁금합니다.
jwtToken으로 만들기 전에 accessToken에서 email을 추출해 온 뒤 Member로 저장을 했습니다. 이 때 password를 숫자, 영어를 랜덤하게 만들어 저장을 했습니다. 이렇게 해도 되는지 궁금합니다.

로그인 시 발급받은 accessToken을 활용하여 email 추출합니다. 추출한 email을 가지고 Member에 저장하고, jwtTokne을 발급받습니다. 그 후 localStorage에 저장하기 위해 redirectAttributes를 사용하여 보내줍니다.

KakaoLoginController에서 넘긴 값들을 저장하기 위해 수정

Order를 통해 수량을 감소시키므로 불필요하다 생각하여 제거했습니다.

주문을 하면 해당 상품 옵션의 수량이 차감되고 위시 리스트에 있는 경우 위시 리스트에서 삭제됩니다.

버튼을 누르면 주문 폼으로 이동합니다.

주문 버튼을 누르면 jwt토큰을 이용하여 누구의 주문인지 알 수 있습니다. localStorage에 있는 accessToken을 같이 넘겨 카카오톡 메시지 보내기에 대비합니다.

Hongdonggeon

안녕하세요 보민님
미션 구현하느라 수고많으셨습니다!
코멘트 남겨놓았으니 확인 한번 해주세요!

Hongdonggeon · 2024-07-29T16:17:41Z

src/main/java/gift/controller/OrderController.java

+        OrderResponseDTO orderResponseDTO = orderService.createOrder(orderRequestDTO,
+            member.getEmail());
+        String accessToken = orderRequestDTO.accessToken();
+        kakaoService.sendKakaoMessage(accessToken, orderResponseDTO);


카카오톡 메시지 전송 로직에서 에러가 발생하게 된다면 정상 응답이 안내려갈 것 같아요. 주문은 정상적으로 완료되었으나, 카카오톡 전송 로직이 실패했다고 해서 사용자가 실패응답을 받는게 맞을까요?

맞지 않은 것 같습니다. 이것도 마찬가지로 예외가 발생하면 로그를 남기는 형식으로 처리해보도록 하겠습니다.

처음에는 model.addAttribute를 사용하여 "message"를 만들고 에러를 처리하려고 했었습니다. 하지만 생각대로 출력이 되지 않아서 로그를 남기는 형식으로 처리했습니다.

Hongdonggeon · 2024-07-29T16:20:29Z

src/main/java/gift/service/OrderService.java

+        Order order = new Order(null, option, quantity, LocalDateTime.now(),
+            orderRequestDTO.message(), member);
+        orderRepository.save(order);
+        wishlistService.removeWishlist(member.getEmail(), option.getProduct().getId());


장바구니에서 상품삭제 로직에서 에러가 발생한다면 주문까지 실패하게 될것 같은데요. 어떻게 생각하실까요?

장바구니에서 삭제 로직에서 에러가 발생하더라도 주문은 생성되어야 합니다. 하나의 트랜잭션에서 작성을 해서 수량 차감 후 상품 삭제 로직에서 에러가 발생하면 롤백되어 주문까지 실패하게 된다고 생각합니다. 이를 해결하기 위해 트랜잭션을 분리하려고 합니다.

트랜잭션을 분리하기 위해서는 @Transactional(propagation = Propagation.REQUIRES_NEW)를 사용하여 분리하면 되는 것 같습니다.
그래서 새로운 메서드를 만들고 그 위에 트랜잭션을 붙여주려고 합니다.

트랜잭션을 분리했습니다. 에러가 발생했을 때 어떻게 처리해주어야 할 지 고민하다가 log를 남기기로 했습니다.

Hongdonggeon · 2024-07-29T16:21:25Z

src/main/java/gift/service/KakaoService.java

+            + "\"text\":\"주문 정보:\\n주문 ID: " + id + "\\n옵션 ID: " + optionId + "\\n수량: " + quantity
+            + "\\n주문 시간: " + orderDateTime + "\\n메시지: " + message + "\","
+            + "\"link\":{\"web_url\":\"http://localhost:8080/admin/products\",\"mobile_web_url\":\"http://localhost:8080/admin/products\"}"
+            + "}";


이렇게 문자열로 다 입력하게 된다면, 관리하기 힘들지 않을까요? 전송할 내용이 많아진다면 문자열을 일일히 수정해야 될 것 같아요

확실히 전송할 내용이 많아지게 되면 문자열을 일일히 수정해야 한다는 불편한 점이 생길 것 같습니다. 하나의 TemplateObject DTO 클래스를 만들어 관리할 수 있도록 수정하겠습니다!

Hongdonggeon · 2024-07-29T16:21:57Z

src/main/java/gift/service/KakaoService.java

+        int rightLimit = 122;
+        int targetStringLength = 20;
+        Random random = new Random();
+        String generatedPassword = random.ints(leftLimit, rightLimit + 1)


범위를 이렇게 잡은 이유가 어떻게 될까요?

비밀 번호를 생성할 때 랜덤하게 만들고 싶었습니다. 그 중 영어 대문자, 소문자, 숫자를 사용해서 만들고 싶었습니다. 숫자 0은 아스키 코드에서 48이고 소문자 z는 아스키 코드에서 122이므로 범위를 위와 같이 잡았습니다. 비밀 번호의 길이는 몇 자가 적당할까 하다가 20자로 정했습니다.

Hongdonggeon · 2024-07-29T16:22:19Z

src/main/java/gift/service/KakaoService.java

+            .limit(targetStringLength)
+            .collect(StringBuilder::new, StringBuilder::appendCodePoint, StringBuilder::append)
+            .toString();
+        return generatedPassword;


작성하신 알고리즘의 원리가 궁금합니다

random을 사용하여 48에서 122까지의 난수를 생성합니다.
filter를 통해 숫자와 알파벳만 포함하도록 합니다.
limit를 통해 제가 원하는 길이만큼만 생성되도록 제한합니다.
collect를 통해 필터링된 난수를 문자열로 반환하여 StringBuilder에 저장합니다.
toString을 통해 StringBuilder를 문자열로 반환하여 generatedPasseord에 저장합니다.

StringBuilder에 관한 설명
StringBuilder::new는 새로운 StringBuilder 객체를 생성합니다.
StringBuilder::appendCodePoint는 주어진 숫자에 해당하는 아스키 코드를 StringBuilder에 추가합니다.
StringBuilder::append는 두 StringBuilder 객체를 병합합니다. 결과를 하나의 StringBuilder로 결합하는 역할을 합니다.

TemplateObjectDTO를 사용하고 ObjectMapper를 이용해 변환시켜줍니다.

KBM05haruin30 added 25 commits July 28, 2024 17:09

docs : 2단계 주문하기 구현할 기능 목록 정리

60fe159

feat : 액세스 토큰으로부터 정보 받기 위한 DTO 생성

6997d12

feat : 액세스 토큰으로부터 이메일 추출

0640ac4

feat : 이메일 사용하여 멤버에 저장

b23b8ae

feat : jwtToken 만들기

cff38b6

feat : KakaoProperties에서 url 관리

fef7ea6

feat : kakao_access_token.html 수정

2b9840f

KakaoLoginController에서 넘긴 값들을 저장하기 위해 수정

feat : authToken에서 jwtToken으로 이름 변경으로 인한 수정

0795b1d

feat : Order 엔티티 생성

5ae4719

feat : orders 테이블 생성

c16fa52

feat : Order 관련 DTO 생성

c02d103

feat : OrderRepository 생성

01c1d33

feat : OptionSubtractQuantityDTO 제거

29f1c77

Order를 통해 수량을 감소시키므로 불필요하다 생각하여 제거했습니다.

feat : OrderService 생성

16dd27f

주문을 하면 해당 상품 옵션의 수량이 차감되고 위시 리스트에 있는 경우 위시 리스트에서 삭제됩니다.

feat : OrderController 생성

96645ac

feat : 주문하기 버튼 생성

493847c

버튼을 누르면 주문 폼으로 이동합니다.

feat : order_form 만들기

00c15b8

주문 버튼을 누르면 jwt토큰을 이용하여 누구의 주문인지 알 수 있습니다. localStorage에 있는 accessToken을 같이 넘겨 카카오톡 메시지 보내기에 대비합니다.

feat : 템플릿 만들기

ea2c247

feat : 카카오톡 메시지 보내는 메서드 추가

15f57aa

feat : url은 KakaoProperties에서 관리

1312eff

feat : OrderController 수정

60c8919

feat : 잘못된 코드 수정

9c13918

refactor : 코드 수정

c3ad5d1

feat : password 랜덤하게 생성

5408450

Hongdonggeon requested changes Jul 29, 2024

View reviewed changes

KBM05haruin30 added 4 commits July 30, 2024 09:33

feat : TemplateObjectDTO 생성

d8bb7dd

feat : TemplateObjectDTO 사용

c656659

TemplateObjectDTO를 사용하고 ObjectMapper를 이용해 변환시켜줍니다.

refactor : token 이름 수정

27a49dd

feat : 위시리스트에 해당 상품 있을 때만 위시리스트에서 상품 삭제

c0f87b3

KBM05haruin30 added 4 commits July 30, 2024 13:35

feat : 이전 상태로 되돌리기

f5d5d15

feat : 로그 남기기 위해 추가

09f1468

feat : 주문하기 트랜잭션 분리

0527adc

feat : 메시지 전송 수정

3f27c40

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

전남대 BE_김보민 5주차 과제 (2단계) #373

전남대 BE_김보민 5주차 과제 (2단계) #373

KBM05haruin30 commented Jul 28, 2024

Hongdonggeon left a comment

Hongdonggeon Jul 29, 2024

KBM05haruin30 Jul 30, 2024

KBM05haruin30 Jul 30, 2024

Hongdonggeon Jul 29, 2024

KBM05haruin30 Jul 30, 2024

KBM05haruin30 Jul 30, 2024 •

edited

Loading

KBM05haruin30 Jul 30, 2024

Hongdonggeon Jul 29, 2024

KBM05haruin30 Jul 30, 2024

Hongdonggeon Jul 29, 2024

KBM05haruin30 Jul 29, 2024

Hongdonggeon Jul 29, 2024

KBM05haruin30 Jul 30, 2024

전남대 BE_김보민 5주차 과제 (2단계) #373

Are you sure you want to change the base?

전남대 BE_김보민 5주차 과제 (2단계) #373

Conversation

KBM05haruin30 commented Jul 28, 2024

2단계

코드 작성하면서 어려줬던 점

궁금한 점

Hongdonggeon left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

KBM05haruin30 Jul 30, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

KBM05haruin30 Jul 30, 2024 •

edited

Loading