This repository has been archived by the owner on Aug 13, 2022. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 54
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- ShopUpdateDTO를 추가하여 정보 업데이트시 가져오는 정보를 표시 - 업데이트 매장이 로그인한 사장님 소유인지 확인
- 오픈, 마감 시도시 각각 오픈과 마감이 가능한 상태인지 확인한다. - 오픈, 마감시 각각 로그를 기록한다. - 오픈시에는 로그를 insert, 마감시에는 로그를 update한다. * Work ==> 매장 영업 기록 - ShopUpdateDTO 세팅시 값을 복사하여 전달하도록 변경
addressCode 말고 상세 주소(xxx호) 정보가 필요하다고 판단하여 해당 컬럼 추가
- AOP 의존성 추가 - AOP 적용 어노테이션 생성 - 중복되는 로그인체크, 매장 주인 체크 로직 삭제 - 어노테이션을 적용하여 로그인체크, 매장 주인체크 진행
f-lab-dev
reviewed
Nov 1, 2019
1. 배달 지역 추가시 유효성 검증 - 배달지역 추가 데이터를 Set으로 받아와서 중복된 추가 값을 하나로 통일 - 이미 추가되어있는 지역을 추가하려고 시도할 때 유니크 키 익셉션이 발생하는 것을 랩핑하여 예외처리 진행 2. 코드 리팩토링 - 위치정보 관련된 기능이 ShopMapper에 존재해서 DeliveryLocationMapper로 이동시킴 - 그 밖에도 메서드 위치가 잘못된 것을 찾아서 이동시킴 - ResponseEntity에 담을 정보가 없는 컨트롤러 메서드의 반환 값을 void로 변경 - 요청 url을 좀 더 직관적으로 변경 - 자잘한 변수 이름 수정 및 컨밴션 적용 3. 주소 로직 - 자신의 배달 가능 지역을 불러오는 기능 추가 - 배달 가능 지역을 추가할 때 한번에 다중 insert를 진행하도록 변경 - DB에 shop_id와 town_code를 유니크 키로 만들어서 중복된 배달지역이 생기지 않도록 변경 - DB에서 유니크키 관력 Exception이 발생하면 그 Exception을 랩핑하도록 추가 - 자신의 매장 정보를 조회할 때 AdressDTO로 배달 주소 조회 -> DeliveryLocationDTO로 배달 주소조회하도록 변경
- 캐싱을 위한 설정및 의존성 추가 - 주소 검색, 배달 가능지역 조회에 시범적으로 캐싱 적용
f-lab-dev
reviewed
Nov 13, 2019
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.
점점 코드가 깔끔해지는게 보이네요. 좋습니다~
@Around("@annotation(com.delfood.aop.OwnerLoginCheck)") | ||
public Object ownerLoginCheck(ProceedingJoinPoint pjp) throws Throwable { | ||
@Before("@annotation(com.delfood.aop.OwnerLoginCheck)") | ||
public void ownerLoginCheck(JoinPoint jp) throws Throwable { |
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.
깔끔하군요~
@@ -155,7 +157,8 @@ | |||
|
|||
// 해당 매장이 영업중이 아닐시 | |||
if (shopService.notOpenCheck(id) == true) { | |||
return new ResponseEntity<CloseShopResponse>(CloseShopResponse.NOT_OPEN, HttpStatus.BAD_REQUEST); | |||
return new ResponseEntity<CloseShopResponse>(CloseShopResponse.NOT_OPEN, |
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.
모든 값이 상수이기때문에 ResponseEntity
객체를 상수화시키는건 어떨까요?
f-lab-dev
approved these changes
Nov 19, 2019
- 주소 검색 Request를 도로명, 지번검색으로 나눔 - 캐싱 키를 제작하는 로직 제작 - 캐싱을 서비스 레이어에서 진행하도록 변경 주소 검색 쿼리 수정 - 통합 검색을 도로명, 지번 검색에 따른 쿼리로 나눔 - NULL값이 들어올 경우 공백으로 검색하도록 변경
#30 Feature/12
feature/11
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
주소 검색 로직 개발