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

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

ただいまの
回答率

90.01%

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

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 1,042

cancat

score 247

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

前提・実現したいこと

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

疑問点

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

発生している問題

Close();に失敗しています。

該当のソースコード

using System;
using System.IO;
using System.Text;
using System.Windows;

namespace windowManager {
    public class MainWindow : Window {
        static bool  IsCloseOk = false;
        public bool IsCloseOK {
            get {
                return IsCloseOk;
            }

            set {
                IsCloseOk = value;
                if (IsCloseOk)
                    Close();<-ここにはくるが閉じない。
            }
        }


        public MainWindow() {
            InitializeComponent();

        }

        private void Window_Loaded(object sender, RoutedEventArgs e) {
                var logical = new logical();
                logical.Operation();
            }
        }
namespace logical {
    public class logical {
        public async void Operation() {
                   //処理
            var response = await operationAsync();
            var resultString = await response.Content.ReadAsStringAsync();

            MainWindow mw = new MainWindow();
            mw.IsCloseOK = true;

        }

補足情報(言語/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 

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

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

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

  • 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で待機する方法があります。

namespace logical {
    public class logical {
        public async Task/*Taskを戻り値にすることでawait可能になる*/ Operation() {
                   //処理
            var response = await operationAsync();
            var resultString = await response.Content.ReadAsStringAsync();

            // MainWindow mw = new MainWindow();
            // mw.IsCloseOK = true;

        }
namespace windowManager {
    public class MainWindow : Window {

        public MainWindow() {
            InitializeComponent();

        }

        private async void Window_Loaded(object sender, RoutedEventArgs e) {
                var logical = new logical();

                //※awaitをつけることで、呼び出しの終了を待ってから以下が実行されます。
                await logical.Operation();

                Close();
            }
        }

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/01/01 23:09

    なるほど。
    クラスの分割、static、プロパティ、インスタンスの生成、async/awaitの使い方など、ばらばらだった知識がぜんぶすんなり氷解しました。
    インスタンスが別であっても、Close()すれば閉じると思っていたのが思い違いでした。

    結論的には、クラスは分割し、staticとプロパティとインスタンスはやめて、async/awaitを使いました。
    クラスの分割は、ロジックは他のアプリケーションでも使用するために必要で、そのためにはロジックが他から独立していることが必要なので、このようになりました。
    たいへん感謝します。

    キャンセル

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

  • ただいまの回答率 90.01%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる
  • トップ
  • WPFに関する質問
  • Windowをもつクラスから、ロジックのクラスを参照して、ロジックのクラスでasyncで処理終了後に、Windowをもつクラスを終了して閉じたいです。