Skip to content
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.

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

외부 서비스 초기화 코드 정리 #5333

Open
eugene-doobu opened this issue Jul 4, 2024 Discussed in #5118 · 6 comments · May be fixed by #5407
Open

외부 서비스 초기화 코드 정리 #5333

eugene-doobu opened this issue Jul 4, 2024 Discussed in #5118 · 6 comments · May be fixed by #5407
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@eugene-doobu
Copy link
Member

Discussed in #5118

Originally posted by eugene-doobu June 18, 2024
많은 의존도를 가진 Game 클래스 정리

Game.Start에서 수행하는 외부 서비스 초기화 코드 정리, 외부 서비스가 초기화 안 된 상태에서 요청시 케이스 정리
수행시 서비스 초기화 과정에서 생기는 예외처리들도 한꺼번에 진행할 수 있을듯 함

관련이슈

#5159

@eugene-doobu eugene-doobu added the enhancement New feature or request label Jul 4, 2024
@eugene-doobu eugene-doobu added this to the v210.0.0 milestone Jul 4, 2024
@eugene-doobu eugene-doobu self-assigned this Jul 4, 2024
@eugene-doobu
Copy link
Member Author

서비스 초기화 과정에서 생기는 예외처리들도 한꺼번에 진행할 수 있을듯 함
-> 서비스 초기화가 되었는지 안되었는지 판단을 하는 것 조차 어렵다. 이걸 지금 처리하기에는 일감이 커질거같아 후순위로 미루는게 좋을 것 같음 (@ipdae 님이 별도 이슈도 있다고 하시는 듯 하다)
#4353
요런거인듯

@eugene-doobu
Copy link
Member Author

불필요한 주소가 null이나 빈 문자열일때 체크하는 부분들 생성자에서 처리하도록 수정

@eugene-doobu
Copy link
Member Author

ApiClients라는 클래스를 만들어 기존 Game클래스에서 관리되던 이러한 초기화 로직들 및 객체들의 책임 이동

@eugene-doobu
Copy link
Member Author

이벤트 용으로 사용하던 길드 시스템 제거
a0a0840

@eugene-doobu
Copy link
Member Author

Game.URL은 공용으로 쓰는건가? DCC에서만 쓰는거 같긴 한데, 확인 필요

@eugene-doobu
Copy link
Member Author

eugene-doobu commented Jul 10, 2024

AmazonCloudWatchLogsClient??
-> 제거 803074d

@eugene-doobu eugene-doobu linked a pull request Jul 11, 2024 that will close this issue
@eugene-doobu eugene-doobu linked a pull request Jul 11, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

1 participant