3주차 개발 내용 코드 리뷰 - [closed] #99

Closed PromotezCitizen closed 3 weeks ago

In GitLab by @HanSanghyeon on Oct 20, 2024, 12:14

Merges develop -> master

In GitLab by @mumunuu on Oct 20, 2024, 21:25

Commented on buy-gurus-b/src/main/java/com/team04/buy_gurus/common/enums/CommonSuccess.java line 9

보통 성공인 경우는 코드로 따로 주는 경우는 없습니다. 뭘 활용할 것도 없고요.
http status code에서 2XX 번대로 오는 경우 모두 성공으로 확인하므로 굳이 코드를 넣을 필요가 없습니다.
그럼에도 여럿 사이트 들에서 저런 코드들이 많이 보이는 건, 레거시 때문에 그렇습니다.
restapi를 잘 이해하지 못하고 body 응답에 코드까지 넣어서 보낸거죠.
code: 200 이런것도 전부 안티패턴 입니다.

In GitLab by @mumunuu on Oct 20, 2024, 21:25

Commented on buy-gurus-b/src/main/java/com/team04/buy_gurus/order/controller/OrderController.java line 35

sout 코드 넣지 않기

In GitLab by @mumunuu on Oct 20, 2024, 21:25

Commented on buy-gurus-b/src/main/java/com/team04/buy_gurus/order/controller/OrderController.java line 43

한줄로 줄이기

return ResponseEntity.ok(new CommonResponseDTO<>(CommonSuccess.ORDER_FOUND, new OrderResponse(orderService.getOrder(id, userDetails))));

In GitLab by @mumunuu on Oct 20, 2024, 21:25

Commented on buy-gurus-b/src/main/java/com/team04/buy_gurus/order/controller/OrderController.java line 43

id라고 하지 않고, invoiceId 등으로 확실하게 id값이 어떤 id 값인지 명확하게 하기

In GitLab by @mumunuu on Oct 20, 2024, 21:25

Commented on buy-gurus-b/src/main/java/com/team04/buy_gurus/order/controller/OrderController.java line 63

역시 한줄로 줄이면 더 좋습니다 :)

In GitLab by @mumunuu on Oct 20, 2024, 21:25

Commented on buy-gurus-b/src/main/java/com/team04/buy_gurus/order/service/OrderService.java line 66

이렇게 하면 이 API 응답이 너무 느릴 수 있습니다.
쿼리 한방으로 가져올 프로덕트를 리스트로 먼저 다 가져오는건 어떨까요?
하나씩 for 문을 돌면서 상품을 1개씩 가져오는건 쿼리가 굉장히 느릴 수 있습니다.
10개면 10번의 select 문이 날라가니까요.
그리고 insert 문도 한번에 여러개를 넣는 방법을 알아봅시다.

In GitLab by @mumunuu on Oct 20, 2024, 21:25

Commented on buy-gurus-b/src/main/java/com/team04/buy_gurus/order/service/OrderService.java line 128

magic number 쓰지 않기

In GitLab by @mumunuu on Oct 20, 2024, 21:25

Commented on buy-gurus-b/src/main/java/com/team04/buy_gurus/orderitem/controller/OrderItemController.java line 30

잘 하셨어요

In GitLab by @mumunuu on Oct 20, 2024, 21:25

Commented on buy-gurus-b/src/main/java/com/team04/buy_gurus/orderitem/service/OrderItemService.java line 85

중괄호 컨벤션을 맞춰주시기 바랍니다.
요 스타일은 C 스타일이라 자바에서 안써요.

In GitLab by @mumunuu on Oct 20, 2024, 21:25

Commented on buy-gurus-b/src/main/java/com/team04/buy_gurus/review/service/ReviewService.java line 51

RunTimeException 일까요? 익셉션을 커스텀으로 만드시고, not found user exception 등으로 처리하시기 바랍니다.