質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.44%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Q&A

解決済

1回答

4108閲覧

コンソールアプリでprivate async Taskでリフレクションを使用して帰り値を返せない

kazuya_

総合スコア78

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

0グッド

2クリップ

投稿2017/06/23 13:33

コンソールアプリでFormを継承させた時、何故か返り値が戻りません。
デバッガで見ると、LedSwitchTask内でreturn値の受け渡しが失敗するようです。

using System; using System.Collections.Generic; using System.Linq; using System.Text; using System.Threading.Tasks; using System.Threading; using System.Windows.Forms; using System.Reflection; namespace PrivateAccess { class Program { static void Main(string[] args) { PrivateAccess pr = new PrivateAccess(); Task<string> res; Task.WhenAll( res = Task.Run(() => { Task<string> rt = pr.privateasynctest(); return rt; }) ).Wait(); Console.WriteLine("status:" + res.Result); } } public class PrivateAccess :Form { //Formを継承させると、無限LOOPになる // public class PrivateAccess { //Formを継承させなければ、正常にresが返る /// <summary> public PrivateAccess() { } public async Task<string> privateasynctest() { try { PrivateAccess theObject = new PrivateAccess(); Type t = theObject.GetType(); MethodInfo dynMethod = t.GetMethod("LedSwitchTask", BindingFlags.NonPublic | BindingFlags.Instance); var task = await (Task<string>)dynMethod.Invoke(theObject, new string[] { "1 0" }); // res = await task.ConfigureAwait(false); return task; } catch (Exception ex) { string s; s = ex.Message; return "NG"; } } private async Task<string> LedSwitchTask(string allArg) { string res; res = await Task.Run(new Func<string>(() => { string chan; chan = allArg.Substring(0, 1); Thread.Sleep(800); return "OK"; })); return res; } } }コード

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

なるほど!!!!!
これは面白いですね。原因が分かりました。

実は、async/awaitがデッドロックを引き起こす条件の一つに、「コンソールアプリケーションではないこと」というのがあります。
厳密に言うと、「SynchronizationContextという、プログラムの実行コンテキストを同期するためのstaticなプロパティに中身が存在する場合にデッドロックが発生する可能性がある」のですが、コンソールアプリケーションだとこれがデフォルトではnullになっています。そうすると、awaitした後に特定のコンテキストに復帰する動作が行われなくなるため、根本的にデッドロックが発生する原因が失われます。つまり、コンソールアプリケーションである限り、たとえWaitメソッドを使ってもスレッド同期によるデッドロックが発生しないということです。

しかし!!System.Windows.Forms.Formのインスタンスを生成するとき、どうやらコンストラクタがSynchronizationContextを生成するようです。つまり、質問のPrivateAccessクラスがFormを継承しているかどうかで、SynchronizationContextの有無が異なっていることになります。
すると、Mainメソッド内で呼び出しているWaitprivateasynctestメソッドのawaitがぶつかってデッドロックを引き起こすようになるわけです。

これは……ちょっと想定外のクソ挙動で戸惑っていますが、、とりあえず、awaitを呼び出している式に対してそれぞれConfigureAwait(false)を付けることでSynchronizationContextの伝播を断ち切ることができるので、async/awaitを使う際はこのコーディングを徹底することでデッドロックを完全に回避することができます。

以下に、最低限の書き換えで動作するようにしたコードを掲載します。こちらをコピペで試してみてください。

csharp

1public class PrivateAccess :Form { //Formを継承させてもちゃんと動作するはず! 2 3 /// <summary> 4 public PrivateAccess() { 5 } 6 7 public async Task<string> privateasynctest() { 8 try { 9 PrivateAccess theObject = new PrivateAccess(); 10 Type t = theObject.GetType(); 11 12 MethodInfo dynMethod = t.GetMethod("LedSwitchTask", BindingFlags.NonPublic | BindingFlags.Instance); 13 14 var task = (Task<string>)dynMethod.Invoke(theObject, new string[] { "1 0" }); // <- awaitを外す 15 var res = await task.ConfigureAwait(false); // ! 16 return res; 17 } 18 catch (Exception ex) { 19 string s; 20 s = ex.Message; 21 return "NG"; 22 } 23 } 24 private async Task<string> LedSwitchTask(string allArg) { 25 string res; 26 res = await Task.Run(new Func<string>(() => { 27 string chan; 28 chan = allArg.Substring(0, 1); 29 Thread.Sleep(800); 30 return "OK"; 31 })).ConfigureAwait(false); // ! 32 return res; 33 } 34}

そして、元々の質問にあった、「IntelliTestから実行したときデッドロックする」については、こちらは推測ですが、おそらく内部でWaitかそれに類する動作を行っていると考えられます。
こちらも同期コンテキストの破棄を徹底することで問題は発生しなくなると予想できますが……(もしかして、UI関係とかのコンテキストが重要な非同期メソッドはIntelliTestでは評価できないってことかしら……?)
ここを掘り下げるにはさらなる調査が必要ですが、なんだか根の深い問題な気がします。


以下はお節介ですが。
自分でコードを書く場面では、決してWaitメソッドやResultプロパティを使わないようにしてくださいね。非同期に値を取得するメソッドをどこかしらで同期するというのは、実行戦略が異なる二つのコンテキストを無理矢理同期することになるため、デッドロックが発生する可能性が非常に高いのです。今回の話については結果的には回避不可能なハマりどころであったと言えますが、同じ種類のハマりどころを自分で作ることになるのだと言い換えると、どれだけヤバイかは理解できると思います。

投稿2017/06/23 15:25

編集2017/06/23 15:37
tamoto

総合スコア4150

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

kazuya_

2017/06/24 14:56

ありがとうございました。良い勉強になりました。同期コンテキストとConfigureAwait等、調べていきたいです。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.44%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問