public void DoDisappearStar(CatchGameModel sender) { if (DisappearStarEvent != null) { DisappearStarEvent(sender); } }
public void DoAppearStar(CatchGameModel sender) { if (AppearStarEvent != null) { AppearStarEvent(sender); } }
/// <summary> /// 필드아이템인 별이 필드에서 사라질때 호출되는 함수 /// </summary> /// <param name="sender"></param> void OnDisappearStar(CatchGameModel sender) { _playerController.OnDisappearStar(_model.GetStarOnField()); _view.ShowInfo("별이 사라졌습니다."); AudioMgr.instance.PlaySoundEffect("InfoFail"); }
/* [보완점] 하드코딩을 피하자. * 하드코딩을 하면, 디버깅이 어렵고 유연성이 떨어진다. * 하지만 내 코드에서 하드코딩이 쓰인 부분이 보인다. * 이 부분은 보완되어야 하는 부분 중에 하나로, info message와 bgm name을 따로 관리하는 클래스를 만들어서 * 그 클래스에서 이름을 받아오는 방향으로 짜면 좋을 것 같다. */ /// <summary> /// 필드아이템인 별이 필드에 등장할때 호출되는 함수 /// </summary> /// <param name="sender"></param> void OnAppearStar(CatchGameModel sender) { _playerController.OnAppearStar(_model.GetStarOnField()); _view.ShowInfo("별이 나타났습니다. 별이 사라지기 전에 얼른 가져가야 합니다."); AudioMgr.instance.PlaySoundEffect("Info"); }