Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
전남대 BE_안원모_3주차 과제(0,1,2단계) #251
전남대 BE_안원모_3주차 과제(0,1,2단계) #251
Changes from 25 commits
8753eee
0eb427a
5f26e8f
1b6f069
eb87ec1
16400e6
e4514c0
e620367
625a5c4
b8a2981
39c50eb
2b7e729
f991f62
f7aec38
6cd3537
4389367
b1c90bb
0831c87
d9ce007
448076c
f576700
008ad15
75edcc2
56ba257
923e2e3
b5fd6ef
ab9dc63
e897194
2282ccc
cacfaae
2e67bfd
6d0f590
ad48400
98c2893
c5d7da7
b1184c2
d7982bd
e3cdc64
0b5f982
6608fcf
4a1ea83
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
data.sql 과 해당 방식을 함께 사용하게 된 이유가 궁금해요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
음 .. 다음 step인 step3 에서 페이지네이션 작성하여 확인할 때 편의성을 위해 작성하였습니다.
data.sql에는 현재 product2 까지 2개의 상품으로 초기 데이터가 작성 되어 있는데,
이 경우, 상품 여러개를 추가하여 페이지네이션 작동을 확인 할 때 data.sql파일을 업데이트를 계속해서 해줘야 할 것 같아 해당 방식을 사용하였습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
하나의 방식으로 통일하는게 좋을 것 같네요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
음.. 위시 리스트 구현방법에 대해 질문 드리고 다음 step에 반영 하겠습니다.
현재 구현된 방법은 상품에 대한 구체적인 정보가 단순히 인덱스만 증가되어 추가되는 형태입니다.
하지만 실제 상품의 경우는 이름도 다 다르고 imageurl, description같은 경우에도 인덱싱뿐만 아니라 설명하는 내용이 다를텐데 이런 요구사항을 어떻게 반영해야 할까요?
기존의 방법으로 data.sql에 상품의 구성요소들을 등록하는 방법으로 통일하는 편이 좋을까요?
아니면, JSON, CSV와 같은 상품 정보를 담은 외부 파일에서 데이터를 읽어와서 초기화 하는 방법으로 통일하는 편이 좋을가요?
추후에 위시 리스트가 추가적으로 어떻게 구현 될진 모르겠지만 어떤 방법이 앞으로의 작업을 진행하는데 있어서 더 나은 방향일까 고민이 됩니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
4주차 step1과제를 좀 진행 해보면서
위와 같은 초기데이터 설정은 페이지네이션 작동이 잘 되는지 확인할 때
여러 상품이 잘 등록,조회가 될수 있나? 테스트할 때 사용되기 좋은 방식이지 실제로 초기 데이터를 등록 할 때는 좋은 방법이 아닌 것 같다라고 생각이 들었습니다.
제가 이해한 4주차 step1은 상품등록에 있어 등록 할 수 있는 카테고리의 범주가 한정 되어있다고 생각했습니다.
(교환권, 상품권, 뷰티, 패션, 식품, 리빙/도서, 레저/스포츠, 아티스트/캐릭터, 유아동/반려, 디지털/가전, 카카오프렌즈, 트렌드 선물, 백화점)
저 로직은 여러 상품을 만들어 잘 작동 되는 지 확인하는 용으로 사용하고
앞으로 진행할 프로젝트에서는 data.sql 방식을 사용할까 합니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
예외를 잡아서 다시 runtime exception 으로 바꿔주시는 이유가 궁금합니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
음..
exception
을 잡아서 다시RuntimeException
으로 예외를 잡아서 다시 던지면서예외 메시지를 추가하거나 로깅을 통해 디버깅 정보를 더 자세하게 나타내보려 했습니다.
변경하는 편이 코드 가독성에 더 도움이 된다면 변경하겠습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 방식을 적용하는 경우도 있긴하지만, 지금은 말해주신
의미있는
예외 메시지를 추가둘 다 하지 않는 상황인 것 같아요. 가독성도 그렇지만, 불필요한 것 같다는 생각이 가장 크게 들어서
저는 변경하면 좋을 것 같네요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GlobalExceptionHandler 에서 에외처리 할 수 있도록
기존의 예외를 다시 던지는 코드 수정했습니다.