master v0.4.1 (REFACTOR) #79

chhs2131 · 2022-12-28T08:14:11Z

요약

전체 내용은 Refactoring이 중점으로 내부 로직만 개선되었습니다.
하위 버전에서 요청시 호환성을 보장합니다.
include

주요내용

시장 및 주식형태에 대한 Enum 클래스 추가 및 관리
ipo Service Layer에서 검증을 진행하고 예외를 발생시키도록 수정
- controller에서 customException으로 핸들링진행
IPO Domain Logic 개선
- IpoPaging 객체를 통해 페이징 관련 내용을 전달하도록 함
- Comment Service 및 Mapper(with xml)를 별도로 생성하여 책임 분리 및 가독성 증대
- 신규상장 CommentJson이 정상적으로 처리되지 못하던 문제 수정
- 이제 ipo/schedule 없이 ipo api 호출시 옵션값을 설정하여 SCHEDULE을 조회할 수 있음

…록 수정

Enum 객체 사용시 null인지 우선 확인하도록 수정

Refactor/#72 - Enum 적용 (stockKinds, stockExchange)

SQL Mapper를 통해 Optional 값을 반환하도록 하고 null일시 orElseThrow IllegalStateException 발생

List값이 null or empty일시 IllegalStateException 발생

Refactor/#74 - 예외 발생 구문을 컨트롤러 -> 도메인으로

…페이징 조건값을 전달하도록 기존 로직을 대체

… 통해 가져오던 잠재적 문제를 수정

… 에러에 대한 처리 진행 - IllegalArgumentException 발생가능 - 해당 에러를 IG000(BadRequest) CustomException으로 변환하여 사용자에게 노출

명절 및 최근과 같은 공모가 잘 없는 경우를 대비

중복기능제거: 해당기능은 이미 {ipo_index}/comment 에 존재함

- getter 추가 - 반환할 필요없는 값(haveTitle)에 JsonIgnore 추가

…로 분리 comment관련 기능들을 전부 commentService layer로 분리

…었던 문제 수정 - 신규등록일에는 '신규 등록 되었습니다.' 라는 title 문구만 표시하도록 수정

- ipo/schedule 기능을 해당으로 대체가능함 Closes #78

- 더 이상 targetDate를 sql에서 사용하지 않음 (startDate,endDate로 대체) Closes #78

Refactor/#67 - ipo domain 로직 개선

- TODAY와 동일한 쿼리로 동작함

chhs2131 added 30 commits December 16, 2022 20:37

feat(StockExchange,StockKinds): #72 - 시장 및 주식형태에 대한 Enum 뼈대 추가

9c03f24

feat(StockExchange): #72 - 시장 형태에 대해 내부적으로 Enum으로 관리하도록 수정

07638fa

feat(StockKinds): #72 - 종목 종류(공모주,실권주,,,)에 따른 Enum을 내부적으로 적용

c76a759

refactor(IpoSequence): #72 - 공모 진행 단계에 대한 검증을 IpoSequence Enum에서 진행하도…

e44ff8e

…록 수정

style(*): 자동 정렬 적용

780911d

style(StockKinds): setter getter 순서 정리

ccad445

fix(StockKinds,StockExchange): null check 진행

8ff9a3e

Enum 객체 사용시 null인지 우선 확인하도록 수정

docs: README 및 Gradle 수정

cfbf59c

Update feature_request.md

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

GPG key ID: 4AEE18F83AFDEB23
Expired

Verified
Learn about vigilant mode

07b1b63

Update refactoring-request.md

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

GPG key ID: 4AEE18F83AFDEB23
Expired

Verified
Learn about vigilant mode

d2a5016

Update bug_report.md

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

GPG key ID: 4AEE18F83AFDEB23
Expired

Verified
Learn about vigilant mode

072ee8d

docs: spring doc ver 수정

4beae5f

feat(Ipo): 단일 인덱스 조회 항목들에 대해 Service에서 예외를 발생시키도록 지정

2ba802d

SQL Mapper를 통해 Optional 값을 반환하도록 하고 null일시 orElseThrow IllegalStateException 발생

feat(Ipo): List값에 대한 검증을 Service에서 진행하고 예외를 발생시키도록 수정

723774b

List값이 null or empty일시 IllegalStateException 발생

feat(IpoMappper): 포괄적인 에러 책임전가구문 제거 throw exception

b10b179

refactor(IpoController,IpoService): 포괄적인 에러 전가문을 제거하고 실제 사용되는 것으로 개선

fd460aa

fix(IpoService): selectIpoUnderwriter 누락되어 동일하게 적용

aa47d2a

feat(IpoPaging): #67 - IPO 페이징 처리를 위한 정보를 담는 객체 생성

b73a3ce

refactor(IpoPaging,IpoController,IpoService): #67 - IpoPaging 객체를 통해 …

8ce8776

…페이징 조건값을 전달하도록 기존 로직을 대체

refactor(IpoPaging,Sort): #67 - Sort가 여러항목에 대치될 수 있도록 수정

2af017e

refactor(IpoPaging,IpoSequence): #67 - IpoSequence 이름을 toString() 함수를…

3ee8668

… 통해 가져오던 잠재적 문제를 수정

feat(DatePeriod): #67 - 날짜 범위를 다르는 객체 추가

adbbdab

feat(IpoController,IpoPaging,ErrorCode): #67 - IpoPaging 사용시 발생할 수 있는…

aaa8691

… 에러에 대한 처리 진행 - IllegalArgumentException 발생가능 - 해당 에러를 IG000(BadRequest) CustomException으로 변환하여 사용자에게 노출

feat(IpoPaging,DatePeriod): #67 - IpoPaging시 DatePeriod를 통해 기간을 설정하도록 지정

d32af7e

feat(IpoController): comment 리스트 기본 조회 기간 최근 7일에서 14일로 수정

7e1e95e

명절 및 최근과 같은 공모가 잘 없는 경우를 대비

feat(IpoController): comment 를 통해 특정 ipo를 조회하는 기능 제거

d256a85

중복기능제거: 해당기능은 이미 {ipo_index}/comment 에 존재함

refactor(Ipo): ipo api 중 기간 관련 파라미터를 받는 요청들에 대해서 내부적으로 DatePeriod를 사용

d34f44f

fix(IpoComment): comment 내용이 json으로 제대로 반환되지 못하던 문제 수정

6b06ce6

- getter 추가 - 반환할 필요없는 값(haveTitle)에 JsonIgnore 추가

chhs2131 added 10 commits December 24, 2022 13:55

feat(CommentService): IpoService에서 comment 부분만 분리하여 별도의 Service Layer…

ee67f6f

…로 분리 comment관련 기능들을 전부 commentService layer로 분리

feat(CommentMapper): IpoMapper에서 comment 부분만을 분리하여 CommentMapper 기능으로 분리

5f55845

fix(IpoCommentDto,CommentServiceImpl): 신규등록된 종목의 comment가 의도하지 않게 반환되…

c447896

…었던 문제 수정 - 신규등록일에는 '신규 등록 되었습니다.' 라는 title 문구만 표시하도록 수정

refactor(CommentServiceImpl): 변수명 수정 commentList -> comments

e484d98

style(CommentServiceImpl): 주석 설명 추가

5ed3b9b

feat(sql-ipo.xml): ipo list TODAY 조회시 기간을 기준으로 조회되도록 로직 수정

d01277e

- ipo/schedule 기능을 해당으로 대체가능함 Closes #78

feat(sql-ipo.xml): ipo list 조회시 기간을 기준으로 조회되도록 로직 수정

66f7322

- 더 이상 targetDate를 sql에서 사용하지 않음 (startDate,endDate로 대체) Closes #78

Merge pull request #77 from chhs2131/refactor/#67

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

GPG key ID: 4AEE18F83AFDEB23
Expired

Verified
Learn about vigilant mode

f4ebd43

Refactor/#67 - ipo domain 로직 개선

style(IpoApiController): swagger 문구 수정

75cc2eb

feat(IpoSequence,sql-ipo.xml): ipo list 조회시 SCHEDULE 조회도 가능하게 추가

f30e777

- TODAY와 동일한 쿼리로 동작함

chhs2131 changed the title ~~master v0.4.1~~ master v0.4.1 (REFACTOR) Dec 28, 2022

chhs2131 merged commit a62ffa9 into master Dec 28, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

master v0.4.1 (REFACTOR) #79

master v0.4.1 (REFACTOR) #79

chhs2131 commented Dec 28, 2022 •

edited

Loading

master v0.4.1 (REFACTOR) #79

master v0.4.1 (REFACTOR) #79

Conversation

chhs2131 commented Dec 28, 2022 • edited Loading

요약

주요내용

chhs2131 commented Dec 28, 2022 •

edited

Loading