C# デリゲート関連

Question

前提・実現したいこと

「Action(C#デリゲート)」を使用してます。
現状、isUpdateMethod()の引数に関数を渡すだけでGC（GC.Alloc）が走るのを何とかしたいです。

試した事（以下）に記載しておりますが、
「nameof」を使用してGCを走らせない事は確認してます。
しかしながらcheckMethod()の使用箇所が多いので「修正箇所が多い事と不便である」ので
何かしらの方法で改善したいです。

該当のソースコード

UpdateClass instance;

void mainLoop()
{
    instance.update();    // loop駆動関係（詳細は長いので省略します）    

    if(instance.isUpdateMethod(instance.updateA)){
        // HIT
    }
}

class UpdateClass
{
    Loop loop;  // loop駆動関係（詳細は長いので省略します）    

    public void updateA(){処理A}
    public void updateB(){処理B}

    void update(){
        for(int i=0; i<length; ++i){  
            loop.update();  //処理
        }
    }

    bool isUpdateMethod( Action meth )
    {
        rerturn update.func == meth;
    }
}

試したこと

============================================================

試した事１：nameofを使用してstringで渡す
・利点　→　GCは走らない、関数をリネームしてもコード修正が要らない
・欠点　→　nameofを記載する手間が掛る

void mainLoop()
{
    if(isUpdateMethod(nameof(instance.updateA))){ }
}

bool isUpdateMethod( string meth ){}

============================================================

試した事2：インターフェースの使用
実装上インスタンスの関数を使用したい為、現実味が無い。
参考にした　→　https://baba-s.hatenablog.com/entry/2017/12/28/205800

============================================================

Comments

[Zuishin]

GC って何ですか？

[yama68]

質問文を修正しました。（GC.Alloc）

[Zuishin]

意味がわかりません。どういうことですか？

[vann_2921]

GCはガベージコレクションですよね？
どうしてGCを起動したくないんでしょうか？

[退会済みユーザー]

Unity の話なら Unity のタグをつけてください。

Unity に興味のある人の目を引いて回答を得られるチャンスが増えるでしょうし、Unity に興味のない人はスルー出来るということで、質問者・閲覧者両方にメリットがあります。

[Zuishin]

ガベージコレクションはそんなに規則的に起こるものではないので、まずどういう現象で何が問題なのかを確認しないと、斜め方向に突き進むことになると思います。

[pepperleaf]

> ガベージコレクションはそんなに規則的に起こるものではない
条件次第では、結構、規則的に発生しますね。
(直そうとしたら、別のエラーで断念したけど)

Answer 1

具体的には思い出せませんが、私が初学者の頃も「GCが走るせいで上手く動かない」と思ったことがありました。
そのときの原因はGCが走ることではなく、結局自身の勘違いと設計の悪さだったと記憶しています。

yama68さんのお悩みも、かつての私の場合と同様、違うところに原因がある可能性が高そうに思えます。
ブレークポイントを挿入して変数に値が正しく代入されているか確認するなどして、違う所に間違いが無いかもう一度探して下さい。
それでも分からなければ、最悪コード全てを貼り付けても良いですから、また新しく質問してみて下さい。

###その他気になったこと
メソッド名が命名規則に違反しています。
C#では、名前空間、クラス・インターフェイス、メソッド、プロパティがアッパーキャメル、変数がローアーキャメルです。

Answer 2

単純に毎回Actionを生成してるからアロケーションが発生しているだけだと思いますが、そういう話ではなく？
毎回生成したくないのであれば、コンストラクタでまとめてコレクションオブジェクトにキャッシュしておくなりすれば良いかと思います。

public class Test
{
    public int Count { get; set; } = 0;

    public void Increment()
    {
        Count += 1;
    }

    public static void DoAction(Action act)
    {
        act();
    }
}

class Program
{
    const int LoopCount = 10;

    static void Main(string[] args)
    {
        var test = new Test();
        Test.DoAction(test.Increment);

        Loop1(test);
        Loop2(test);

        //一応入れ替えてもやってみる
        Loop2(test);
        Loop1(test);

        Console.ReadLine();
    }

    //Actionを保持しない
    private static void Loop1(Test test)
    {
        long before = GC.GetAllocatedBytesForCurrentThread();
        for (int i = 0; i < LoopCount; i++)
        {
            Test.DoAction(test.Increment);
        }
        long after = GC.GetAllocatedBytesForCurrentThread();
        Console.WriteLine($"Allocated(Loop1 before):{before}");
        Console.WriteLine($"Allocated(Loop1 after):{after}");
        Console.WriteLine($"test.Count:{test.Count}");
    }

    //Actionを保持する
    private static void Loop2(Test test)
    {
        Action action = test.Increment;

        long before = GC.GetAllocatedBytesForCurrentThread();
        for (int i = 0; i < LoopCount; i++)
        {
            Test.DoAction(action);
        }
        long after = GC.GetAllocatedBytesForCurrentThread();
        Console.WriteLine($"Allocated(Loop2 before):{before}");
        Console.WriteLine($"Allocated(Loop2 after):{after}");
        Console.WriteLine($"test.Count:{test.Count}");
    }
}

(実行結果)
Allocated(Loop1 before):38516
Allocated(Loop1 after):38836
test.Count:11
Allocated(Loop2 before):48812
Allocated(Loop2 after):48812
test.Count:21
Allocated(Loop2 before):49072
Allocated(Loop2 after):49072
test.Count:31
Allocated(Loop1 before):49300
Allocated(Loop1 after):49620
test.Count:41