[Feat] #212 - Objective, KeyResult Entity Validator 분리 #213

its-sky · 2024-02-24T16:07:55Z

🚀PullRequest🚀

ObjectiveService Validator 분리
KeyResultService Validator 분리
ObjectiveService와 KeyResultService에서 UserAuthrorization을 검증하는 로직을 기존에 UserService에 메시지를 보내서 검증하였는데 이 부분이 사실상 Long id값이 같은지 비교하는 부분이기 때문에 각 Validator에서 같은지 검증하는 것으로 변경함.
변경 이후 ObjectiveService와 KeyResultService에서 UserService에 대한 의존성이 사라졌음.
또한 Validator를 분리하면서 분기 처리에 대한 정확한 메시지를 표출할 수 있어서 코드의 직관성이 향상됨을 느꼈음.

기존 로직을 메소드로 옮기는 과정에서 메소드의 Validation이 정확하지 않거나 분기 처리 순서를 변경하면서 분기 처리가 제대로 되지 않을 것 같으면 정확히 지적해주세요 ~~

0lynny

아주 아름다운 코드군요... 저도 이렇게 반영하겠습니당~

0lynny · 2024-02-24T16:59:40Z

moonshot-api/src/main/java/org/moonshot/keyresult/service/validator/KeyResultValidator.java

+
+public class KeyResultValidator {
+
+ private static final int ACTIVE_KEY_RESULT_NUMBER = 3;


p5;
이런 Constant는 common 모듈 안의 constant파일 만들어서 해줘도 좋을 것 같아요!

its-sky added 4 commits February 24, 2024 23:26

[Chore] #212 - SpringBoot Application 실행 시 띄워질 배너 추가

cf72723

[Feat] #212 - ObjectiveService의 검증 로직을 Validator로 분리

0092264

[Chore] #212 - KRStateConverter 패키지 이동

b4236ee

[Feat] #212 - KeyResultService Validator 분리

a6db1e8

its-sky added 🚀SP1 Sprint 1 민철👃🏼 labels Feb 24, 2024

its-sky self-assigned this Feb 24, 2024

0lynny self-requested a review February 24, 2024 16:31

its-sky added 3 commits February 25, 2024 01:38

[Chore] #212 - Validator import 문 축소

5878e69

[Chore] #212 - SpringBoot Application banner 수정

58559c3

[Del] #212 - 불필요한 공백 삭제

7cf9b68

0lynny approved these changes Feb 24, 2024

View reviewed changes

its-sky merged commit aaedcdb into develop Feb 24, 2024
1 check passed

its-sky deleted the feature/#212 branch February 27, 2024 16:42


		public class KeyResultValidator {

		private static final int ACTIVE_KEY_RESULT_NUMBER = 3;