search

planetarium / NineChronicles

Unity client application for Nine Chronicles, a fully decentralized idle RPG powered by the community.
https://nine-chronicles.com/
GNU Affero General Public License v3.0
370 stars 156 forks source link

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

Closed eugene-doobu closed 3 months ago

eugene-doobu commented 4 months ago

Discussed in https://github.com/planetarium/NineChronicles/discussions/5118

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

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

관련이슈

https://github.com/planetarium/NineChronicles/issues/5159

eugene-doobu commented 3 months ago

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

eugene-doobu commented 3 months ago

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

eugene-doobu commented 3 months ago

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

eugene-doobu commented 3 months ago

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

eugene-doobu commented 3 months ago

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

eugene-doobu commented 3 months ago

AmazonCloudWatchLogsClient?? -> 제거 803074d7b248376ca57127968c1a4862038fb05f