feat(fdl-cmmn): RuntimeException 생성 메서드 추가#279
Open
LeeBaekHaeng wants to merge 1 commit into
Open
Conversation
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
수정 사유 Reason for modification
소스를 수정한 사유가 무엇인지 체크해 주세요. Please check the reason you modified the source. ([X] X는 대문자여야 합니다.)
수정된 소스 내용 Modified source
검토자를 위해 수정된 소스 내용을 설명해 주세요. Please describe the modified source for reviewers.
EgovAbstractServiceImpl에 메시지 키를 이용하여RuntimeException타입의EgovBizException을 생성하는processRuntimeException(String)메서드를 추가했습니다.기존
processException(String, String[])처리 흐름을 재사용하여 메시지 리소스와현재 로케일을 동일하게 적용합니다.
이를 통해 호출부에서 검사 예외 선언 없이 비즈니스 예외를 발생시켜야 하는 경우
RuntimeException반환 타입을 직접 사용할 수 있습니다.selectSample()메서드에서 Checked Exception을 상위 계층으로 전파하지 않도록 개선하는 것을 제안합니다.조회 결과가 없는 경우
processException()대신processRuntimeException()을 사용하여 Runtime Exception을 발생시키고, 이에 따라 메서드 선언부의throws Exception과 Javadoc의@exception Exception을 함께 제거합니다.이를 통해 서비스 계층의 불필요한 Checked Exception 전파를 줄이고, 예외 처리를 일관되게 유지할 수 있습니다.
https://github.com/eGovFramework/egovframe-web-sample/blob/main/src/main/java/egovframework/example/sample/service/impl/EgovSampleServiceImpl.java
JUnit 테스트 JUnit tests
테스트를 완료하셨으면 다음 항목에 [대문자X]로 표시해 주세요. When you're done testing, check the following items.
테스트 브라우저 Test Browser
테스트를 진행한 브라우저를 선택해 주세요. Please select the browser(s) you ran the test on. (다중 선택 가능 you can select multiple) [X] X는 대문자여야 합니다.
테스트 스크린샷 또는 캡처 영상 Test screenshots or captured video
테스트 전과 후의 스크린샷 또는 캡처 영상을 이곳에 첨부해 주세요. Please attach screenshots or video captures of your before and after tests here.