Zenject Example ~さよならManager編~

Transcript

1. 1.

   Zenject Example ～さよならManager ～ いも(@adarapata) 2019/08/26 Roppongi.unity #04 1

2. 2.

   今日話すこと Manager をZenject 使って滅ぼす話 Zenject を多少知ってる人向け 2019/08/26 Roppongi.unity #04 2

3. 3.

   どこのご家庭にもあるMasterDataManager public class MasterDataManager : SingletonMonoBehaviour<MasterDataManager> { public List<PlayerData> Players

   { get; private set; } public List<EnemyData> Enemies { get; private set; } public List<ItemData> Items { get; private set; } private CsvLoader _provider; public PlayerData FindById(int id) => Players.Find(p => p.id == id); void Start() { Players = _provider.LoadAll<PlayerData>("path/to/players"); Enemies = _provider.LoadAll<EnemyData>("path/to/enemies"); Items = _provider.LoadAll<ItemData>("path/to/items"); } } 2019/08/26 Roppongi.unity #04 3
4. 4.

   MasterDataManager の気になるポイント 責務が多い インスタンスを持つだけでなく便利メソッド生え始めた 今後あらゆるデータが追加されて肥大する香りがする テスタビリティが低い 1 メソッドテストするのに複数の要素が必要になる static なのでMasterDataManager

   に依存しているコード側もテス トしにくい 2019/08/26 Roppongi.unity #04 4
5. 5.

   MasterDataManaer の立ち位置を考える なぜMasterDataManager という名前がついてたのか？ 複数の種類のデータクラスを纏めて持っているから なぜ複数の種類のデータを持ってたのか？ 全部ユニークなインスタンスなのでSingleton なインスタンスに持 たせた方が楽 なぜSingleton

   なのか いろんなところから簡単にアクセスできるようにしたい 各依存先に渡すのが大変だから これらを解決できるならばManager は不要になる 2019/08/26 Roppongi.unity #04 5
6. 6.

   やっていくぞ 2019/08/26 Roppongi.unity #04 6

7. 7.

   まずはテスト namespace Tests { public class MasterDataManagerTest { private MasterDataManager

   _manager; [UnityTest] public IEnumerator FindById_Success() { _manager = new GameObject().AddComponent<MasterDataManager>(); yield return new WaitForEndOfFrame(); var playerData = _manager.FindById(1); Assert.AreEqual("Foo", playerData.name); } } } 2019/08/26 Roppongi.unity #04 7
8. 8.

   コードのお気持ちを探る なぜSingleton なの？ ← イマココ なぜMonoBehaviour なの？ 2019/08/26 Roppongi.unity #04

   8
9. 9.

   なぜSingleton なの？ いろんなところから簡単にアクセスできるようにしたい 各依存先に渡すのが大変だから これを解決できるならばSingleton である必要はない。 じゃあDI しましょう。 2019/08/26 Roppongi.unity

   #04 9
10. 10.

    MasterDataManager をBind する まだMonoBehaviour なのでZenject Binding でサッとやる 2019/08/26 Roppongi.unity #04

    10
11. 11.

    Project Context を作って子にする 2019/08/26 Roppongi.unity #04 11

12. 12.

    呼ぶ側が書き直せる public class FooBehaviour { public void Foo(){ var player

    = MasterDataManager.Instance.FindById(1); } } ↓ public class FooBehaviour { [Inject] private MasterDataManager manager; public void Foo(){ var player = manager.FindById(1); } } 2019/08/26 Roppongi.unity #04 12
13. 13.

    全箇所移行完了したらMonoBehaviour に書き直す public class MasterDataManager : MonoBehaviour { public List<PlayerData>

    Players { get; private set; } public List<EnemyData> Enemies { get; private set; } public List<ItemData> Items { get; private set; } private CsvLoader _provider; public PlayerData FindById(int id) => Players.Find(p => p.id == id); void Start() { Players = _provider.LoadAll<PlayerData>("path/to/players"); Enemies = _provider.LoadAll<EnemyData>("path/to/enemies"); Items = _provider.LoadAll<ItemData>("path/to/items"); } } 2019/08/26 Roppongi.unity #04 13
14. 14.

    コードのお気持ちを探る なぜSingleton なの？ :done: なぜMonoBehaviour なの？ ← イマココ 2019/08/26 Roppongi.unity

    #04 14
15. 15.

    なぜMonoBehaviour なの？ 初期化処理( ロード部分) を呼びたい 誰も初期化を呼ぶ人がいないのでStart() を使いたい これを解決できるならばMonoBehaviour である必要はない。 2019/08/26

    Roppongi.unity #04 15
16. 16.

    IInitializable を実装してみよう public class MasterDataManager : MonoBehaviour, IInitializable { public

    List<PlayerData> Players { get; private set; } public List<EnemyData> Enemies { get; private set; } public List<ItemData> Items { get; private set; } private CsvLoader _provider; public PlayerData FindById(int id) => Players.Find(p => p.id == id); public void Initialize(){ Players = _provider.LoadAll<PlayerData>("path/to/players"); Enemies = _provider.LoadAll<EnemyData>("path/to/enemies"); Items = _provider.LoadAll<ItemData>("path/to/items"); } } 2019/08/26 Roppongi.unity #04 16
17. 17.

    IInitializable Zenject が提供するinterface やることは初期化のみ 実装すると、依存関係解決後にInitialize() をコールしてくれる 裏側でInitializableManager が動いてる。 2019/08/26 Roppongi.unity

    #04 17
18. 18.

    不要になったのでMonoBehaviour をやめよう public class MasterDataManager : IInitializable { public List<PlayerData>

    Players { get; private set; } public List<EnemyData> Enemies { get; private set; } public List<ItemData> Items { get; private set; } private CsvLoader _provider; public PlayerData FindById(int id) => Players.Find(p => p.id == id); public void Initialize(){ Players = _provider.LoadAll<PlayerData>("path/to/players"); Enemies = _provider.LoadAll<EnemyData>("path/to/enemies"); Items = _provider.LoadAll<ItemData>("path/to/items"); } } 2019/08/26 Roppongi.unity #04 18
19. 19.

    MonoBehaviour じゃなくなったのでInstaller を書こう public class MasterDataInstaller : MonoInstaller { public

    override void InstallBindings() { Container.Bind<MasterDataManager>().AsSingle(); } } 2019/08/26 Roppongi.unity #04 19
20. 20.

    テストも少し書き直そう namespace Tests { public class MasterDataManagerTest { private MasterDataManager

    _manager; // MonoBehaviour じゃないのでEditTest で十分 [Test] public void FindById_Success() { _manager = new MasterDataManager(); _manager.Initialize(); var playerData = _manager.FindById(1); Assert.AreEqual("Foo", playerData.name); } } } 2019/08/26 Roppongi.unity #04 20
21. 21.

    今一度MasterDataManaer の立ち位置を考える なぜMasterDataManager という名前がついてたのか？ 複数の種類のデータクラスを纏めて持っているから なぜ複数の種類のデータを持ってたのか？ 全部ユニークなインスタンスなのでSingleton なインスタンスに持 たせた方が楽 なぜSingleton

    なのか いろんなところから簡単にアクセスできるようにしたい 各依存先に渡すのが大変だから 2019/08/26 Roppongi.unity #04 21
22. 22.

    今一度MasterDataManaer の立ち位置を考える なぜSingleton なのか => DI でOK なぜ複数の種類のデータを持ってたのか？ => 直接Inject

    できるなら 纏める必要はない なぜMasterDataManager という名前がついてたのか？ => 纏める必 要ないなら不要では？ 2019/08/26 Roppongi.unity #04 22
23. 23.

    データ持ってるクラスを独立 public class PlayerDataRepository { private readonly List<PlayerData> _resources; public

    PlayerDataRepository(IResourceLoadProvider _provider) { _resources = _provider.LoadAll<PlayerData>("path/to/players"); } public PlayerData FindById(int id) => _resources.Find(p => p.id == id); } public class MasterDataInstaller : MonoInstaller { public override void InstallBindings() { Container.BindInterfacesAndSelfTo<MasterDataManager>().AsSingle(); Container.BindInterfacesAndSelfTo<PlayerDataRepository>().AsSingle(); Container.Bind<IResourceLoadProvider>().To<CsvLoader>().AsSingle(); } } 2019/08/26 Roppongi.unity #04 23
24. 24.

    PlayerDataRepository だけ差し替えて動作は変えない public class MasterDataManager : IInitializable { public List<EnemyData>

    Enemies { get; private set; } public List<ItemData> Items { get; private set; } [Inject] private IResourceLoadProvider _provider; [Inject] private PlayerDataRepository _playerDataRepository; public PlayerData FindById(int id) => _playerDataRepository.FindById(id); public void Initialize() { Enemies = _provider.LoadAll<EnemyData>("path/to/enemies"); Items = _provider.LoadAll<ItemData>("path/to/items"); } } 2019/08/26 Roppongi.unity #04 24
25. 25.

    新規に書くときは直接Repository を渡してあげればよい public class FooBehaviour { [Inject] private PlayerDataRepository _repository;

    public void Foo(){ var player = repository.FindById(1); } } 2019/08/26 Roppongi.unity #04 25
26. 26.

    Test もZenject 対応 public class MasterDataManagerTest : ZenjectUnitTestFixture { public

    class MockLoader : IResourceLoadProvider { public List<T> LoadAll<T>(string path) => new List<T>(/* なんかいい感じ生成 */); } private MasterDataManager _manager; [Test] public void FindById_Success() { Container.Bind<MasterDataManager>().AsSingle(); Container.BindInterfacesTo<MockLoader>().AsSingle(); Container.Bind<PlayerDataRepository>().AsSingle(); _manager = Container.Resolve<MasterDataManager>(); _manager.Initialize(); var playerData = _manager.FindById(1); Assert.AreEqual("Foo", playerData.name); } } 2019/08/26 Roppongi.unity #04 26
27. 27.

    さよならManager 2019/08/26 Roppongi.unity #04 27

28. 28.

    まとめ Zenject 使えばインスタンスを集約するだけのクラスは不要になる Manager と呼ばれてるものの役割を見直してみよう もしかしたらインスタンスを集めてるだけのクラスかもしれない 漸進的に変更できる方法を選ぼう 旧コードと新コードが一時は同居できるようにする テストを書くと安心感ある テストを書くと安心感ある

    テストを書くと安心感ある 2019/08/26 Roppongi.unity #04 28
29. 29.

    宣伝 Zenject 本書いてます 技術書典7 こ20D 前後編になりました 〆切と印刷費に負けた ごめん 今回は前編を出します DI

    の話～Scene 跨ぎのBind くらいまで 2019/08/26 Roppongi.unity #04 29