VContainerのHello Worldを実行するボタンをクリックした所ログに文字列が2回出るが一回にしたい

Question

Unityでゲームを作っていたんですが、クラスの依存関係がグチャグチャになったので、解決策はないかと探しました。そこでよく解らないままVContainerを使ってみようと思い、VContainerのHelloWorldを参考にして作ってみたのですが、ログに"Hello World"が二回出力されます。
![]
以下に画像とコードを記載します。心当たりがお有りの方、いらっしゃいましたら教えていただけないでしょうか？

＜ヒエラルキー＞

＜GameLifetimeオブジェクトのインスペクター＞

＜HelloScreenオブジェクトのインスペクター＞

public class HelloWorldService
{
    public void Hello ()
    {
        Debug.Log ("Hello world");
    }
}

public class GameLifetimeScope : LifetimeScope
{
    [SerializeField]
    HelloScreen helloScreen;

    protected override void Configure (IContainerBuilder builder)
    {
        builder.RegisterEntryPoint<GamePresenter> (Lifetime.Singleton);
        builder.Register<HelloWorldService> (Lifetime.Singleton);
        builder.RegisterComponent (helloScreen);

        builder.UseEntryPoints(Lifetime.Singleton, entryPoints =>
        {
            entryPoints.Add<GamePresenter> ();
        });
    }

}

public class HelloScreen : MonoBehaviour
{
    public Button HelloButton;
}

public class GamePresenter : IStartable
{
    readonly HelloWorldService helloWorldService;
    readonly HelloScreen helloScreen;

    public GamePresenter (HelloWorldService helloWorldService,
        HelloScreen helloScreen)
    {
        this.helloWorldService = helloWorldService;
        this.helloScreen = helloScreen;
    }

    void IStartable.Start ()
    {
        helloScreen.HelloButton.onClick.AddListener (() => helloWorldService.Hello ());
    }
}

unity2019.4.16f1

よろしくお願いします。

Comments

[fumu7]

どのような操作をした時に「ログに"Hello World"が二回出力される」という現象が起きるのですか？
helloScreen.HelloButtonをクリックすると、helloWorldService.Hello ()が呼び出され、ログに"Hello World"が記録されるプログラムになっていますが、このボタンを２回クリックしたという事はないですか？

[MujinSekai]

失礼しました。
ボタンをクリックすると、”HelloWorld”がログに出るプログラムです。
1回クリックすると、2つ出ます。

Answer 1

        builder.RegisterEntryPoint<GamePresenter> (Lifetime.Singleton);
        builder.Register<HelloWorldService> (Lifetime.Singleton);
        builder.RegisterComponent (helloScreen);

        builder.UseEntryPoints(Lifetime.Singleton, entryPoints =>
        {
            entryPoints.Add<GamePresenter> ();
        });
    }

上記、GamePresenter の登録を2回行なってしまっているのが原因だと思います。

UseEntryPoints〜 は、 RegisterEntryPoint のエイリアスですので、どちらか一方を消してみてください。

Comments

[MujinSekai]

おっしゃる通り、一方を消すと一度だけ表示されるようになりました。
回答ありがとうございます。