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

Feature/refactor On Room Event #5883

Open
wants to merge 12 commits into
base: prepare/230.0.x
Choose a base branch
from

Conversation

eugene-doobu
Copy link
Member

@eugene-doobu eugene-doobu commented Sep 11, 2024

240 타겟 입니다. prepare/230에 머지하지 않습니다.

개요

#5777 이슈 처리중에 BattleResultPopup.GoToMarket에서 전투중 EnterRoom 이벤트를 호출하고 전투UI를 정리하고 OnRoomEnterEnd 이벤트 발생시 마켓으로 보내주는 부분과 관련하여 'EnterRoom'이벤트에 수정이 필요한듯 하여 작업을 시작하게 되었습니다.

수정내용

  1. 9c의 메인 화면을 `Lobby'라고 부르는 경우와 'Room'이라고 부르는 경우가 섞여있어 lobby로 네이밍을 통일하려고 했습니다.
  2. PromotionContainer.TryOpenBanner에서 이벤트 팝업을 띄우는데 51스테이지 클리어 조건을 추가하였습니다
  3. '룸으로 가기'기능 변경
  • 기존: Game.Event.OnLobbyEnter.Invoke() 를 호출하면 어찌저찌해서 기존 UI들이 모두 종료된 상태가 됬다고 가정하고 Menu(Lobby) UI를 활성화시켜주는 방법
  • 신규: Lobby.Enter() 호출하면 다른 UI다 끄고 Menu UI 활성화 시 켜준 후 이벤트 실행
  1. RoomEntering 클래스 제거, 여기서 수행하던 로직을 Stage와 Lobby클래스가 수행

사실 4번 5번이 핵심이고 이 변경사항이 Lobby.cs에 있어서 이 코드만 리뷰봐주셔도 충분할 것 같습니다.

사실 개요에 해당하는 EnterRoom이벤트의 정상화는 하지 못한 것 같습니다. 스테이지에서 나왔을때 스테이지에서 사용하던것들을 정리하는 이벤트를 만들어서 처리하고 싶었는데, '스테이지 들어갔다, 나갔다'를 정확히 판단할 기준이 모호하다고 느껴졌습니다. 그나마 비슷한게 BattleRenderer.IsOnBattle 이건데 뭔가 중복으로 호출되는 부분이 많기도 하고 이런 여기저기서 프로퍼티 Set하는 부분을 믿어도 되나? 싶은 생각이 들었습니다. (현재 스테이지에 있다를 판단하는데 사용됨)

How to test

  1. 로비로 가는 수많은 동작을 해봅니다.

@eugene-doobu eugene-doobu added this to the v240.0.0 milestone Sep 11, 2024
@eugene-doobu eugene-doobu self-assigned this Sep 11, 2024
@eugene-doobu eugene-doobu linked an issue Sep 11, 2024 that may be closed by this pull request
@eugene-doobu eugene-doobu marked this pull request as draft September 11, 2024 10:14
@eugene-doobu eugene-doobu marked this pull request as ready for review September 11, 2024 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Review
Development

Successfully merging this pull request may close these issues.

OnRoomEvent 정리
1 participant