Windowをもつクラスから、ロジックのクラスを参照して、ロジックのクラスでasyncで処理終了後に、Windowをもつクラスを終了して閉じたいです。

こんにちは。
Windows10でWPFのアプリケーションを開発しています。
Visual Studio 2015 Communityを使っています。

###前提・実現したいこと
Windowをもつクラスから、ロジックのクラスを参照して、ロジックのクラスでasyncで処理をします。
終了後に、Windowをもつクラスを終了して閉じたいです。
どのように書けばよいでしょう?
現在は、Windowのクラスのプロパティを参照してみましたが、Close();に失敗しています。

###疑問点
そもそもこういうときは、クラスを分ける意味があるでしょうか?
クラスを分けた場合、ロジッククラスとウィンドウクラスでなにを受け持つべきでしょう?
具体的に、Close()処理はウィンドウクラスで行うとしたら、ロジックのasync処理の終了はなにで伝えるものですか?

###発生している問題
Close();に失敗しています。

###該当のソースコード

C#
1using System;
2using System.IO;
3using System.Text;
4using System.Windows;
5
6namespace windowManager {
7	public class MainWindow : Window {
8		static bool  IsCloseOk = false;
9		public bool IsCloseOK {
10			get {
11				return IsCloseOk;
12			}
13
14			set {
15				IsCloseOk = value;
16				if (IsCloseOk)
17					Close();<-ここにはくるが閉じない。
18			}
19		}
20
21
22		public MainWindow() {
23			InitializeComponent();
24
25		}
26
27		private void Window_Loaded(object sender, RoutedEventArgs e) {
28				var logical = new logical();
29				logical.Operation();
30			}
31		}

C#
1namespace logical {
2	public class logical {
3		public async void Operation() {
4                   //処理
5			var response = await operationAsync();
6			var resultString = await response.Content.ReadAsStringAsync();
7
8			MainWindow mw = new MainWindow();
9			mw.IsCloseOK = true;
10			
11		}

###補足情報(言語/FW/ツール等のバージョンなど)
Microsoft Visual Studio Community 2015
Version 14.0.25424.00 Update 3
Microsoft .NET Framework
Version 4.6.01038

インストールしているバージョン:Community

Visual C# 2015 00322-20000-00000-AA575
Microsoft Visual C# 2015

です。
よろしくお願いします。

行動規範の内容に同意します

回答1件

ベストアンサー

ご提示いただいたソースの構造は以下のようになっています。

MainWindow(1).Window_Loaded内

logicalオブジェクト生成、Operation()呼び出し
↓

logical.Operation内

MainWindow(2)オブジェクト生成、IsCloseOK = true
↓

MainWindow(2).IsClosed内

自身をClose()

上記(1)と(2)は別のWindowオブジェクトなので、(2)に対してCloseを呼び出しても、表示中の(1)はクローズされていない、という状態です。
（フィールドのIsCloseOkだけstaticにしているのは、何か勘違いされているかも？
プロパティがstaticでないので、呼び出しは当然インスタンスに対して実行されます）

MainWindow側でロジックの終了を検知してCloseしたい場合、たとえば非同期呼び出しをawaitで待機する方法があります。

C#
1namespace logical {
2    public class logical {
3        public async Task/*Taskを戻り値にすることでawait可能になる*/ Operation() {
4                   //処理
5            var response = await operationAsync();
6            var resultString = await response.Content.ReadAsStringAsync();
7
8            // MainWindow mw = new MainWindow();
9            // mw.IsCloseOK = true;
10
11        }

C#
1namespace windowManager {
2    public class MainWindow : Window {
3
4        public MainWindow() {
5            InitializeComponent();
6
7        }
8
9        private async void Window_Loaded(object sender, RoutedEventArgs e) {
10                var logical = new logical();
11
12                //※awaitをつけることで、呼び出しの終了を待ってから以下が実行されます。
13                await logical.Operation();
14
15                Close();
16            }
17        }

そもそもこういうときは、クラスを分ける意味があるでしょうか?
クラスを分けた場合、ロジッククラスとウィンドウクラスでなにを受け持つべきでしょう?

すみません。上記のご質問については、ご提示のソースだけでは一概に答えかねます。
どのクラスに何を受け持たせるか、という点こそがクラス設計の肝だと思いますので、
クラスを分ける意味が見出せるまでは分けない、というのも選択肢のひとつかもしれません。
（ただ、Closeのような画面処理は画面クラスで、という考え方は間違っていないと思いますよ）

投稿2017/01/01 07:29

oika

総合スコア425

cancat

2017/01/01 14:09

なるほど。クラスの分割、static、プロパティ、インスタンスの生成、async/awaitの使い方など、ばらばらだった知識がぜんぶすんなり氷解しました。インスタンスが別であっても、Close()すれば閉じると思っていたのが思い違いでした。結論的には、クラスは分割し、staticとプロパティとインスタンスはやめて、async/awaitを使いました。クラスの分割は、ロジックは他のアプリケーションでも使用するために必要で、そのためにはロジックが他から独立していることが必要なので、このようになりました。たいへん感謝します。

行動規範の内容に同意します