C# 既に定義されている引数無しActionに引数をつけたい

Question

困っている事、相談したい事

あらかじめ作られている、引数無しActionのコールバックに対して、
引数を渡したいが、渡し方が分からず困っています。

コールバックの作りが、添え字以外中身が同じため、
コールバックを一つにまとめてすっきりできないか相談するため投稿しました。

あらすじ

Unityでランキングを扱いたく、PlayFabというBaasを使いました。

ランキングを取得するために下記メソッドを使っています。

PlayFabClientAPI.GetLeaderboardAroundPlayer(
    request, 
    OnGetCountRanking, 
    OnGetLeaderboardAroundPlayerFailure
);

このメソッドの引数を調べたところ、下記のようになっていました。

void PlayFabClientAPI.GetLeaderboardAroundPlayer(
        GetLeaderboardAroundPlayerRequest request, 
        System.Action<GetLeaderboardAroundPlayerResult> resultCallback, 
        System.Action<PlayFabError> errorCallback, 
        [object customData = null], [Dictionary<string, string> extraHeaders = null]
)

実装で定義したコールバックはこのような形にしてあります。
(エラーのコールバックはログを出すだけなので割愛します)

    private void OnGetCountRanking(GetLeaderboardAroundPlayerResult result)
    {
        foreach (var entry in result.Leaderboard)
        {
            RankingsText[0].text = $"Rank:{entry.Position + 1}";
            ParamsText[0].text = $"{entry.StatValue}";
        }
    }

動きとしては、ウェブ上のランキングに対してリクエストを渡し、
順位を取得した後、指定のUnityのUI.Textに文字を表示する、といったものになります。

今回の実装では、複数のランキングが存在しており、
ランキング毎に別名のコールバックを定義し、作りは次のようになっています。

    private void OnGetStrRanking(GetLeaderboardAroundPlayerResult result)
    {
        foreach (var entry in result.Leaderboard)
        {
            RankingsText[1].text = $"Rank:{entry.Position + 1}";
            ParamsText[1].text = $"{entry.StatValue}";
        }
    }

御覧の通り、OnGetCountRankingとOnGetStrRankingの
実装の中身は添え字の数字が違うだけなのです。
(写してない他のランキングも同様の作りです)

最初はコールバックの関数に、新たに引数を追加して
添え字を渡すという風にしたかったのですが、
私自身の理解が及ばず、苦肉の策で新しく別名のコールバックを作ることでごまかしてました。

ただ、これだとコードの可読性も保守のしやすさも悪くなってしまいますし、
どうにかまとめたいと考えています。

そのような解決方法があるのでしょうか？

利用環境

• Unity2020.3.3f1
• PlayFab

Comments

[退会済みユーザー]

PlayFabClientAPI.GetLeaderboardAroundPlayerの呼出部分のコードはどうなっていますか？
customDataって引数がありますけど、これで任意のオブジェクト渡したり出来たりしないでしょうか。

[cacapon]

呼び出し部分のコードは下記のようになっています。

```
switch (targetStatisticName)
{
case "Countranking":
PlayFabClientAPI.GetLeaderboardAroundPlayer(request, OnGetCountRanking, OnGetLeaderboardAroundPlayerFailure);
break;
case "STRranking":
PlayFabClientAPI.GetLeaderboardAroundPlayer(request, OnGetStrRanking, OnGetLeaderboardAroundPlayerFailure);
break;
case "DEXranking":
PlayFabClientAPI.GetLeaderboardAroundPlayer(request, OnGetDexRanking, OnGetLeaderboardAroundPlayerFailure);
...// 以下似たような並び //
```

customDataは確かに渡せそうですね…
こちらに関してはもう少し調べてみます。調べ不足での質問で失礼いたしました。

Answer 1

目的が出力対象のUIを任意にコントロールしたいということであって、受け渡し方法が引数に拘らないのであれば、コールバックをラムダ式にして、外部変数で出力対象を渡してやればいいのではないかと。

using System;

class Program
{
    public class HogeAPI
    {
        public static void Get(object dummyRequest, Action<Result> resultCallBack)
        {
            resultCallBack(new Result() { Position = 100});
        }
    }

    public class Result
    {
        public int Position;
    }

    public class UI
    {
        public string Text
        {
            get => _Text;
            set
            {
                Console.WriteLine(value);
                _Text = value;
            }
        }
        private string _Text;
    }

    static void Main(string[] args)
    {
        UI rankingsText = new UI();

        HogeAPI.Get(
            new object(),
            (Result result) =>
            {
                rankingsText.Text = $"Rank:{result.Position + 1}";
            }
        );

        Console.ReadKey();
    }
}

Comments

[cacapon]

回答ありがとうございます。

なるほど、そんなやり方が！

私自身、外部変数が理解できていなそうなので、
一度理解してから問題のコードの修正に手を掛けてみようと思います。

[cacapon]

外部変数として扱う形にしたところ、
無事一つのメソッドにまとめることが出来ました。

今回は、こちらのやり方で解決できましたのでベストアンサーとさせていただきました。

Answer 2

このような話でしょうか．

class TheArgument
{
	public int[] Array;
}

class Program
{
	public static void CallTheCallback( Action<TheArgument> Callback )
	{
		var arg = new TheArgument();
		arg.Array = new int[]{ 10,20,30 };
		Callback( arg );
	}

	//This method has second argument
	public static void Callback_with_IndexArg( TheArgument vals, int index )
	{
		Console.WriteLine( vals.Array[index] );
	}

	//Main
	public static void Main(string[] args)
	{
		CallTheCallback(	(vals)=>{ Callback_with_IndexArg( vals, 0 ); }	);
		CallTheCallback(	(vals)=>{ Callback_with_IndexArg( vals, 2 ); }	);

		Console.WriteLine( "==終了==" );
		Console.ReadLine();
	}
}

Comments

[cacapon]

ええと…
引数無しActionとは別に引数有りActionを作って、
引数無しActionのコールバックを引数有りコールバックにするって感じですかね…？

提示されたコードは動かしたりして流れはちょっとわかったのですが、
問題が起きているコードに応用できるほど、理解できてないので、
参考にしながら色々試行錯誤してみようと思います。

[fana]

この例は……

void CallTheCallback( Action<TheArgument> Callback )
にコールバックを与える必要があるが，
そのコールバックの引数は TheArgument 型ということになっている．

それに対して，コールバックとして用いたいのは
void Callback_with_IndexArg( TheArgument vals, int index )
というメソッドであり，これの引数 index の値を指定した状態のものをコールバックとして用いたい．

……という感じ．
で，それを実現する手段として，Mainではラムダを用いている．

[fana]

ラムダが嫌(?)なら，単純に，

//共通実装たる Callback_with_IndexArg を，第2引数に特定の値を用いて呼ぶだけのメソッド群
void Callback_with_Index0( TheArgument vals ){ Callback_with_IndexArg( vals, 0 ); }
void Callback_with_Index1( TheArgument vals ){ Callback_with_IndexArg( vals, 1 ); }
void Callback_with_Index2( TheArgument vals ){ Callback_with_IndexArg( vals, 2 ); }

みたいなのを用意して，これをコールバックに使えばよい．

[cacapon]

なるほどです！解説有り難うございます！