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

더미데이터 삽입에 대한 코드 분리 #37

Open

2 of 6 tasks

kwonsye opened this issue Aug 3, 2020 · 0 comments

Open

2 of 6 tasks

더미데이터 삽입에 대한 코드 분리 #37

kwonsye opened this issue Aug 3, 2020 · 0 comments

Assignees

Labels

Collaborator

kwonsye commented Aug 3, 2020

더미데이터 삽입에 대한 코드를 분리한다.

기존에 컨트롤러에 있었던 더미데이터 삽입 코드를 별도의 컨트롤러와 서비스로 분리한다.

완료 조건

더미데이터 삽입 코드를 별도의 클래스로 분리한다.
필요없어진 코드는 삭제한다.
매직넘버에 대한 코드를 상수로 치환해서 리팩토링한다.
매직넘버에 대한 코드를 어떤 방법으로 리팩토링 해야 더욱 유지보수하기 쉬울지 고민해본다.

relate to issue

초대장 더미데이터 수정 및 초대장 해쉬코드 관련 버그 해결 #34

Reference

리팩토링:매직 넘버를 기호 상수로 치환하기

Check List

관련 issue가 있다면 추가했나요?
유의미한 Label을 추가했나요?

kwonsye added the ⛑refactor label

kwonsye self-assigned this

JaeHyeonKim19 self-assigned this

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