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.

Sign up for GitHub

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

Jump to bottom

그룹 상태 변경 커맨드 핸들러를 각 상태에 대한 커맨드 핸들러로 분리 #44

Open

Coalery opened this issue Dec 16, 2023 · 0 comments

Assignees

Labels

Member

Coalery commented Dec 16, 2023

feat: 그룹 변경 커맨드 핸들러 구현 #43

위 PR에서 추가된 그룹 상태 변경 커맨드 핸들러를 각 상태에 대한 커맨드로 분리시킵니다.

그 이유는 다음과 같습니다.

모든 상태 전이를 해당 커맨드 핸들러에서 처리하다보니 분기문이 많음
이 때문에 그 흐름을 이해하기 어려워짐
또, 각 상황에 맞는 유효성 검사를 처리하기가 어려워짐

따라서, 이들을 각각의 커맨드로 분리시켜야 합니다.

예를 들면, PROGRESS -> END_SUCCESS 상황은, SucceedGroupCommand 등의 커맨드로 옮깁니다.

The text was updated successfully, but these errors were encountered:

Coalery added the refactoring label

Coalery self-assigned this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment