C# Task.Run()のラムダ式じゃない書き方を教えてください。

Question

string[] task = await Task.Run(async () => {
Task.Delay(1);
//   (
//      )
//複数行の処理
//      (
//       )
string[] 返り値 = { 処理結果1,処理結果2 ,処理結果3 };
return 返り値;//結果をかえす
});

これにより非同期の処理が出来ていますがコードが長すぎて辛いです。
ラムダ式部分を、名前のある引数付のdelegate。(引数付のメソッド名)に置き換えたいです
Task.Run()をwebで調べるとラムダ式のモノしか見つからず…
素人の為書き方が分からず困っています。

また、delegateの場合以下の使用法で非同期出来る様ですが

delegate void d();
IAsyncResult a = d.BeginInvoke(new AsyncCallback(cl), null);
//非同期処理の終了を待ち、結果を取得
d.EndInvoke(a);

この方法はあまりよくない?古い?とか… 詳しく分からないのでその辺りも教えてください。

Answer 1

まずTask.Run()のラムダ以外の書き方ですが、Taskクラスのヘルプを参照するとRunメソッドも色々とバリエーションがあります。
質問のコードに合わせてRun<TResult>(Func<TResult>)を使って書くとこのような感じでしょうか。

        private string[] Foo()
        {
            string[] result = { "a", "b", "c" };
            return result;
        }

        private async void Bar()
        {
            Func<string[]> FooFunc = Foo;
            string[] ar = await Task.Run<string[]>(FooFunc);
        }

async/awaitが使えるバージョンではTask.Run()の出番は少なくなり、次のようにより簡潔に記述することができます。

        private async Task<string[]> FooAsync()
        {
            //await Task.Delay(1);
            string[] result = { "a", "b", "c" };
            return result;
        }

        private async void Bar()
        {
            string[] ar = await FooAsync();
        }

asyncキーワードの付いたメソッドは戻り値をTaskで包んでくれるため、Task.Run()の記述を無くすことができます。

Comments

[hamaa]

なるほど～もうこれで非同期出来ちゃうんですね。ありがとうごさいます。
すごい勉強になります。

Answer 2

名前のある引数付のdelegate。(引数付のメソッド名)に置き換えたい

の意味がわからない（そもそも引数ないラムダ式書いてるやん…？）んですが、とりあえず例示のコードを単純にdelegateに書き換えれば以下のようになります。

元コード（ちょっと手直し）

            var results1 = await Task.Run(async () =>
            {
                await Task.Delay(1);
                //複数行の処理
                string[] 返り値 = { "処理結果1", "処理結果2", "処理結果3" };
                return 返り値;
            });

delegateコード

            var results2 = await Task.Run(async delegate
            {
                await Task.Delay(1);
                //複数行の処理
                string[] 返り値 = { "処理結果1", "処理結果2", "処理結果3" };
                return 返り値;
            });

で、たぶんやりたいのはこーゆーこと？
実ロジック切り出しコード

            //ラムダ式
            Func<Task<string[]>> func1 = async () =>
            {
                await Task.Delay(1);
                //なんか色々処理
                return new string[] { "処理結果1", "処理結果2", "処理結果3" };
            };
            var result3 = await Task.Run(func1);

            //delegate
            Func<Task<string[]>> func2 = async delegate
            {
                await Task.Delay(1);
                //なんか色々処理
                return new string[] { "処理結果1", "処理結果2", "処理結果3" };
            };
            var result4 = await Task.Run(func2);

ラムダ式の中のロジックを別に切り出したい、と読み取りました。いかがでしょう。

そもそもコードが長いのは適切なクラス設計・メソッド分割がなされていないだけで、例示したコードのようにfuncを定義して切り出すような書き方はあんまり見かけません。

やりたいことが非同期コードなのか、ラムダ式を除去したいのか、delegateを使ってみたいだけなのか、単にコードのリファクタリングの一環なのか、質問からはちょっと特定できません…。

Comments

[hamaa]

ShikaTech さん
>>ラムダ式の中のロジックを別に切り出したい、と読み取りました。
ありがとうございます。その通りです。
>>そもそもコードが長いのは適切なクラス設計・メソッド分割がなされていないだけで、例示したコードのようにfuncを定義して切り出すような書き方はあんまり見かけません。

はい。おっしゃる通りです。まだまだ素人でクラス設計・メソッド分割が経験浅いです。とりあえず動かす所から始めてます。

>>質問からはちょっと特定できません…。
日本語が下手ですみません。でも、読み取って頂き助かりました。
コードをみて勉強させていただきます。

[ShikaTech]

意図通りの例示ができたようで何よりです。
楽しく学んでいきましょう！C#はサイコーですぞ！
補足ですが、引数がほしければ例示のコードを以下のように書き換えてみてくださいな。
→Func<string,Task<string[]>> func1 = async (string arg) =>
これで、Inにstring型、OutにTask<string[]>型のFunctionが定義できます。
さらに呼び出し側を引数対応にして実行！
→var result3 = await Task.Run(() => func1("param"));

[hamaa]

ありがとうございます。C#に度ハマり中です。
ご親切にありがとうございます。

Answer 3

結局ラムダ式を使ってるやん！
っていう回答ですがやりたいことは実現できてると思うのでこれでどうでしょうか
（ラムダ式の中で関数を呼ぶイメージです）

class Program
{
    static void Main(string[] args)
    {
        Console.WriteLine("タスク開始");

        //パラメータ
        var param = true;

        //呼び出し
        var task = Task.Run(() => HeavyTask(param));

        Console.WriteLine("タスク待機");

        //待機
        task.Wait();
        Console.WriteLine($"Result = {task.Result}");
        Console.WriteLine("終了");
        Console.ReadKey();
    }

    static bool HeavyTask(bool param)
    {
        //なんか重たい処理
        System.Threading.Thread.Sleep(2000);
        return param;
    }
}

Comments

[hamaa]

azio_groupさん
ありがとうございます。目的は長いコードの切だしです。
どちらの方々もありがとうございます。