피자집 창업 준비 1일차 #6

greeng00se · 2023-05-05T15:30:11Z

서오릉 피자의 창업 비용은 8,812만원입니다.

* docs: 요구사항 정의 Co-authored-by: greeng00se <[email protected]> * feat: 상품 도메인 추가 Co-authored-by: greeng00se <[email protected]> * feat: db 테이블 추가 Co-authored-by: greeng00se <[email protected]> * feat: Dao 상품 저장 및 전체 조회 기능 추가 Co-authored-by: greeng00se <[email protected]> * feat: 서비스 상품 저장 기능 추가 Co-authored-by: greeng00se <[email protected]> * feat: 페이지 접근 기능 추가 Co-authored-by: greeng00se <[email protected]> * feat: 데이터베이스 설정 추가 Co-authored-by: greeng00se <[email protected]> * feat: 상품 저장 기능 추가 Co-authored-by: greeng00se <[email protected]> * docs: 요구사항 체크 Co-authored-by: greeng00se <[email protected]> * feat: 서비스 상품 조회 기능 추가 Co-authored-by: greeng00se <[email protected]> * feat: 상품 조회 기능 추가 Co-authored-by: greeng00se <[email protected]> * feat: 상품 수정 기능 추가 Co-authored-by: greeng00se <[email protected]> * feat: 상품 삭제 기능 추가 Co-authored-by: greeng00se <[email protected]> * refactor: RestController, Controller 분리 Co-authored-by: greeng00se <[email protected]> * feat: Thymeleaf 단일 상품 조회 페이지 추가 Co-authored-by: greeng00se <[email protected]> * feat: 단일 조회 기능 구현 Co-authored-by: greeng00se <[email protected]> * test: 테스트 중복 제거 및 위치 변경 Co-authored-by: greeng00se <[email protected]> * feat: 저장시 location header 반환기능 추가 Co-authored-by: greeng00se <[email protected]> * feat: 예외처리 기능 추가 Co-authored-by: greeng00se <[email protected]> * refactor: 오타 수정 Co-authored-by: greeng00se <[email protected]> * refactor: 사용하지 않는 DTO 클래스 삭제 Co-authored-by: greeng00se <[email protected]> * refactor: 예외메시지 수정 Co-authored-by: greeng00se <[email protected]> * feat: 이미지 검증 기능 추가 * refactor: 모든 RequestDto -> Request 클래스명 변경 * refactor: 필요없는 Optional 제거 * feat: 커스텀 예외 적용 * feat: 엔티티에 audit 기능 추가 * refactor: 컨트롤러 메서드명 변경 * feat: Service에 Transactional 추가 * test: Long.MAX_VALUE 적용! * test: expect 빠진 부분 수정 * refactor: 예약어 대문자로 변경 * test: test 설정 파일 수정 * test: Controller 테스트에서 id를 이용하여 검증하는 방향으로 수정 --------- Co-authored-by: songusika <[email protected]>

- AuthMemberDao -> CredentialDao - 추가로 Credential을 검증하는 과정에서 에러메시지를 조금 더 자세하게 출력하도록 변경

- 올바른 Basic 유형의 헤더(파싱 불가능한 경우) InvalidBasicCredentialException 예외 던지도록 변경

…속 받도록 수정

drunkenhw

태코챗의 아버지 허브 덕분에 잘 쓰고 있습니다.

drunkenhw · 2023-05-07T11:27:12Z

src/main/java/cart/controller/ProductController.java

+@RestController
+public class ProductController {
+
+    private final ProductService productService;
+
+    public ProductController(final ProductService productService) {
+        this.productService = productService;
+    }
+
+    @PostMapping("/products")


product controller에는 공통적인 url인 /products를 request mapping으로 안해놓으셨네용

drunkenhw · 2023-05-07T11:28:13Z

src/main/java/cart/dao/CartProductDao.java

+    public List<Product> findAllProductByMemberId(final Long memberId) {
+        final String sql = "SELECT product.id AS id, name, image, price "
+                + "FROM cart_product "
+                + "LEFT JOIN product "
+                + "ON cart_product.product_id = product.id "
+                + "WHERE member_id = ?";
+        return jdbcTemplate.query(sql, productRowMapper, memberId);
+    }


메소드 명이랑 쿼리랑 뭔가 매칭이 안되는 느낌? cart까지 조인하는데 좀 그렇네요 저만 그런가요?>

안그래도 저도 이렇게 짜놓고 좀 걸려서 리뷰어한테 여쭤봤는데, 잘 했다고 하셨삼요

drunkenhw · 2023-05-07T11:32:52Z

src/main/java/cart/service/ProductService.java

+    public List<ProductDto> findAll() {
+        return productDao.findAll().stream()
+                .map(ProductDto::from)
+                .collect(toUnmodifiableList());


왜 나는 레벨 1때는 불변 리스트를 열심히 썼는데 2단계와서는 안쓰고 있을까요? 다 까먹었네ㅐ

drunkenhw · 2023-05-07T11:34:18Z

src/test/java/cart/controller/PageControllerTest.java

+    private Matcher<Object> generateProductPropertiesMatcher(
+            final Long id,
+            final String name,
+            final String image,
+            final long price
+    ) {
+        return allOf(
+                hasProperty("id", is(id)),
+                hasProperty("name", is(name)),
+                hasProperty("image", is(image)),
+                hasProperty("price", is(price))
+        );
+    }
+
+    private Matcher<Object> generateMemberPropertiesMatcher(
+            final Long id,
+            final String email,
+            final String password
+    ) {
+        return allOf(
+                hasProperty("id", is(id)),
+                hasProperty("email", is(email)),
+                hasProperty("password", is(password))
+        );
+    }


당신 참 멋져요

drunkenhw · 2023-05-07T11:35:54Z

src/test/java/cart/dao/CredentialDaoTest.java

+    void 이메일을_입력받아_동일한_이메일을_가진_사용자를_조회한다() {
+        // given
+        final MapSqlParameterSource parameter = new MapSqlParameterSource();
+        parameter.addValue("email", "[email protected]");
+        parameter.addValue("password", "password");
+        memberJdbcInsert.executeAndReturnKey(parameter).longValue();


그냥 credentialDao.save() 하면 안되나용??

drunkenhw · 2023-05-07T11:37:16Z

src/main/java/cart/dao/CredentialDao.java

+import org.springframework.stereotype.Component;
+
+@Component
+public class CredentialDao {


credential dao와 member dao의 차이가 뭐라고 생각하고 나누신건가요?

woo-chang

허브신! 허브신! 🌿

woo-chang · 2023-05-07T12:04:56Z

src/main/java/cart/auth/AuthArgumentResolver.java

+    @Override
+    public boolean supportsParameter(final MethodParameter parameter) {
+        final boolean hasParameterAnnotation = parameter.hasParameterAnnotation(Auth.class);
+        final boolean hasCredentialType = Credential.class.isAssignableFrom(parameter.getParameterType());


상속까지 고려해서 isAssignableFrom을 사용한거야?

그러게요 이게 뭐죠?

woo-chang · 2023-05-07T12:07:05Z

src/main/java/cart/auth/AuthInterceptor.java

+        final Credential savedCredential = credentialDao.findByEmail(credential.getEmail())
+                .orElseThrow(() -> new AuthenticationException("올바르지 않은 이메일입니다. 입력값: " + credential.getEmail()));
+
+        if (credential.isNotSamePassword(savedCredential)) {


email, password로 바로 찾지 않은 이유는 무엇이죠?

그러게요 findByEmailAndPassword 왜 안하셨죠?

woo-chang · 2023-05-07T12:10:37Z

src/main/java/cart/auth/CredentialThreadLocal.java

+@Component
+public class CredentialThreadLocal {
+
+    private final ThreadLocal<Credential> local = new ThreadLocal<>();


ThreadLocal 👍 ThreadLocal를 사용했을 때 어떤 장점이 있었어?

woo-chang · 2023-05-07T12:16:39Z

src/main/java/cart/domain/Member.java

+    private void validate(final String email, final String password) {
+        validateEmail(email);
+        validatePassword(password);
+    }
+
+    private void validateEmail(final String email) {
+        if (email == null || email.isBlank()) {
+            throw new MemberNotValidException("이메일은 공백일 수 없습니다.");
+        }
+    }
+
+    private void validatePassword(final String password) {
+        if (password == null || password.isBlank()) {
+            throw new MemberNotValidException("비밀번호는 공백일 수 없습니다.");
+        }
+    }


VO를 만들어서 검증 로직을 넘기는건 어떠신가요 허브신 🌿

woo-chang · 2023-05-07T12:18:27Z

src/main/java/cart/controller/CartProductController.java

+
+    @PostMapping
+    public ResponseEntity<Void> save(@Auth final Credential credential,
+                                     @RequestBody final CartProductSaveRequest request) {


쿼리 파라미터를 활용하는거에 대해서는 어떻게 생각하나용?

woo-chang · 2023-05-07T12:19:50Z

src/main/java/cart/dto/ProductSaveRequest.java

+    @NotBlank(message = "이미지는 공백일 수 없습니다.")
+    private final String image;
+
+    @Range(message = "가격은 최소 {min}원 이상, {max}원 이하여야 합니다.")


min, max 속성이 없는 것 같은데 🤔

woo-chang · 2023-05-07T12:22:16Z

src/main/java/cart/service/MemberService.java

+import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
+
+@Transactional


사용되지 않는 어노테이션은 제거하면 어떨까?

woo-chang · 2023-05-07T12:22:51Z

src/main/resources/data.sql

@@ -0,0 +1,13 @@
+INSERT INTO PRODUCT (name, image, price)
+VALUES ('피자스쿨 치즈피자', 'https://t1.daumcdn.net/cfile/tistory/2647BE3754B7E8B733', 8900);


피즈스쿨 치즈피자 얼마전에 먹었는데 존맛 🍕

hanueleee

허브 피자 사줘

hanueleee · 2023-05-08T06:07:06Z

src/main/java/cart/auth/AuthArgumentResolver.java

+    @Override
+    public boolean supportsParameter(final MethodParameter parameter) {
+        final boolean hasParameterAnnotation = parameter.hasParameterAnnotation(Auth.class);
+        final boolean hasCredentialType = Credential.class.isAssignableFrom(parameter.getParameterType());


그러게요 이게 뭐죠?

hanueleee · 2023-05-08T06:07:41Z

src/main/java/cart/auth/AuthArgumentResolver.java

+            final WebDataBinderFactory binderFactory
+    ) {
+        final Credential credential = credentialThreadLocal.get();
+        credentialThreadLocal.clear();


threadLocal 신기하네요

hanueleee · 2023-05-08T06:08:51Z

src/main/java/cart/auth/AuthInterceptor.java

+        final Credential savedCredential = credentialDao.findByEmail(credential.getEmail())
+                .orElseThrow(() -> new AuthenticationException("올바르지 않은 이메일입니다. 입력값: " + credential.getEmail()));
+
+        if (credential.isNotSamePassword(savedCredential)) {


그러게요 findByEmailAndPassword 왜 안하셨죠?

hanueleee · 2023-05-08T06:09:34Z

src/main/java/cart/auth/AuthInterceptor.java

+                .orElseThrow(() -> new AuthenticationException("올바르지 않은 이메일입니다. 입력값: " + credential.getEmail()));
+
+        if (credential.isNotSamePassword(savedCredential)) {
+            throw new AuthenticationException();


얘도 예외 메시지 지정해주는 거 어떨까요?

hanueleee · 2023-05-08T06:11:39Z

src/main/java/cart/auth/BasicAuthorizationParser.java

+
+@Component
+public class BasicAuthorizationParser {
+


parser를 빈으로 등록한 이유가 뭐죠?

hanueleee · 2023-05-08T06:24:36Z

src/main/java/cart/dao/CartProductDao.java

+    public List<Product> findAllProductByMemberId(final Long memberId) {
+        final String sql = "SELECT product.id AS id, name, image, price "
+                + "FROM cart_product "
+                + "LEFT JOIN product "
+                + "ON cart_product.product_id = product.id "
+                + "WHERE member_id = ?";
+        return jdbcTemplate.query(sql, productRowMapper, memberId);
+    }


안그래도 저도 이렇게 짜놓고 좀 걸려서 리뷰어한테 여쭤봤는데, 잘 했다고 하셨삼요

hanueleee · 2023-05-08T06:25:00Z

src/main/java/cart/dao/CredentialDao.java

+import org.springframework.stereotype.Component;
+
+@Component
+public class CredentialDao {


hanueleee · 2023-05-08T06:25:49Z

src/main/java/cart/domain/Member.java

+
+import cart.exception.MemberNotValidException;
+
+public class Member {


이번에 entity를 따로 만들지 않은 이유는 뭐죠?

hanueleee · 2023-05-08T06:27:46Z

src/main/java/cart/exception/GlobalExceptionHandler.java

+        logger.warn(errorMessage);
+        return ResponseEntity.badRequest()
+                .body(new ExceptionResponse(errorMessage));
+    }


세세한 예외처리 멋져요

hanueleee · 2023-05-08T06:33:13Z

src/main/java/cart/service/ProductService.java

+    public void update(final Long id, final ProductUpdateRequest request) {
+        final Product savedProduct = new Product(id, request.getName(), request.getImage(), request.getPrice());
+        final int affectedCount = productDao.update(savedProduct);
+        if (affectedCount == 0) {


greeng00se and others added 30 commits April 29, 2023 16:56

docs: 2단계 요구사항 정리

2256b19

test: 잘못된 테스트명 수정

27e9089

feat: data.sql 수정 및 schema.sql 파일 추가

02be345

refactor: auditing sql의 default를 사용하도록 변경

1a0bd1b

refactor: 사용자 엔티티 추가

33355cf

feat: 사용자 설정 페이지 연동 기능 추가

bf96972

feat: 장바구니 페이지 연동 기능 추가

64925cc

feat: Basic Auth로 오는 값을 파싱하는 클래스 추가

2cb430b

feat: 사용자 정보에 대한 ArgumentResolver 추가

b5cf8e6

test: 잘못 작성된 테스트 수정

5d075c1

feat: 장바구니 등록 및 조회 기능 추가

2d16d45

feat: 장바구니 삭제 기능 추가

f13465b

refactor: Cart -> CartProduct로 수정

6e8d373

refactor: create URI 수정

6040e7e

feat: 인증을 인터셉터가 담당하도록 설정

bbf1884

docs: 요구사항 업데이트

3d81cf3

feat: argumentResolver에서 threadLocal clear하는 기능 추가

eebb50b

feat: Credential에서 비밀번호가 다른지 확인하는 기능 추가

b273000

refactor: 기존에 인증과정에서 Member를 반환하던 Dao를 Credential을 반환하도록 수정

cbb8bc6

- AuthMemberDao -> CredentialDao - 추가로 Credential을 검증하는 과정에서 에러메시지를 조금 더 자세하게 출력하도록 변경

feat: ExceptionHandler에 로깅 적용

3d3799c

refactor: Auditing을 위한 필드 전부 제거

705361a

refactor: BasicAuthorizationParser의 isNotValid를 제거하고 조금 더 응집력 있는 클래스로 변경

ac83592

- 올바른 Basic 유형의 헤더(파싱 불가능한 경우) InvalidBasicCredentialException 예외 던지도록 변경

refactor: Credential의 id를 memberId로 변경

eb44589

feat: getMemberId 메서드에 Nullable 애너테이션 추가

1ac95f0

refactor: 패키지 구조 변경

7b8de76

feat: 도미노 페페로니 피자 추가

aa41153

remove: todo 주석 제거

43eece1

refactor: InvalidBasicCredentialException이 AuthenticationException을 상…

fb42fa4

…속 받도록 수정

test: BaiscAuthorizationParser 테스트 출력 수정

dd5dee5

drunkenhw reviewed May 7, 2023

View reviewed changes

woo-chang reviewed May 7, 2023

View reviewed changes

This comment was marked as duplicate.

Sign in to view

hanueleee reviewed May 8, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

피자집 창업 준비 1일차 #6

피자집 창업 준비 1일차 #6

greeng00se commented May 5, 2023

drunkenhw left a comment

drunkenhw May 7, 2023

drunkenhw May 7, 2023

hanueleee May 8, 2023

drunkenhw May 7, 2023

drunkenhw May 7, 2023

drunkenhw May 7, 2023

drunkenhw May 7, 2023

hanueleee May 8, 2023

woo-chang left a comment

woo-chang May 7, 2023

hanueleee May 8, 2023

woo-chang May 7, 2023

hanueleee May 8, 2023

woo-chang May 7, 2023

woo-chang May 7, 2023

woo-chang May 7, 2023

woo-chang May 7, 2023

woo-chang May 7, 2023

woo-chang May 7, 2023

This comment was marked as duplicate.

hanueleee left a comment

hanueleee May 8, 2023

hanueleee May 8, 2023

hanueleee May 8, 2023

hanueleee May 8, 2023

hanueleee May 8, 2023

hanueleee May 8, 2023

hanueleee May 8, 2023

hanueleee May 8, 2023

hanueleee May 8, 2023

hanueleee May 8, 2023

		@@ -0,0 +1,13 @@
		INSERT INTO PRODUCT (name, image, price)
		VALUES ('피자스쿨 치즈피자', 'https://t1.daumcdn.net/cfile/tistory/2647BE3754B7E8B733', 8900);


		import cart.exception.MemberNotValidException;

		public class Member {

피자집 창업 준비 1일차 #6

Are you sure you want to change the base?

피자집 창업 준비 1일차 #6

Conversation

greeng00se commented May 5, 2023

drunkenhw 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

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

woo-chang 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

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

This comment was marked as duplicate.

hanueleee 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

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