Windows Formでエラーメッセージを表示するスタンダードな方法

Question

現在、Windows Formを使用したアプリケーションを作成しています。
Exceptionの内容に応じてメッセージボックスを表示させる良い方法をお教えください。

<やりたいこと>
「ファイルが既に開かれている場合にはエラーメッセージを表示する」
という処理を実装しようと考えています。

<問題点>
エラーのcatchをクラス内で呼び出しており、
それをどうやってForm1クラスに渡すべきなのか というところに悩んでいます。

<考えたここと>
・戻り値をstringにしてForm1クラスに渡す(下記コード)
・ExceptionのcatchをInputFileクラス内で行わずにForm1で行う
・InputFileクラスにFormをインクルードして直接メッセージボックスを表示させる

<聞きたいこと>
エラーメッセージを表示させるのはどのように実装するのが良いのでしょうか。
ご存知の方はお教えくださると嬉しいです。

    public partial class Form1 : Form
    {
        // ... 略 ...

        private void button1_Click(object sender, EventArgs e)
        {
            string msg = inputFile.ReadFile;
            if ( msg != "" )
            {
                MessageBox.Show(msg, "エラー", MessageBoxButtons.OK, MessageBoxIcon.Error);
            }
        }
    }

    class InputFile
    {
        // ... 略 ...

        public string ReadFile()
        {
            FileInfo fileInfo = new FileInfo(Path);

            try
            {
                using (var fs = new FileStream(Path, FileMode.Open)) { }
            }
            catch (IOException e)
            {
                return e.ToString();
            }

            using (var package = new ExcelPackage(fileInfo, password))
            {
                return "";
            }
        }
    }

Answer 1

そこらへんは個々それぞれのやり方があるでしょうね

私なんかはあらゆるエラーを自前の例外のmessageにエラーメッセージを入れてthrowして、あとの catchでエラーダイアログを出す、というズボラな方法をとってます

Comments

[eien_beginner]

やはり、メッセージダイアログそのものはForm1クラスで出力するという認識で正しいでしょうか？

[y_waiwai]

そうですね
例外を出しとけばどこでもだせますが、Formで出すほうがわかりやすいでしょうね

[eien_beginner]

ありがとうございます。

Answer 2

そういう場合はReadFileでは例外を捕まえずに呼び出し元(Form1)で処理させるほうがいいです。
「例外」はその関数で想定される、行われるべき処理が継続できない事態が発生して関数から予定外の脱出が行われる場合に使用します。
「ReadFile」という関数がその名の通り「ファイルを読む」という役割を担っているなら、正しくファイルを読めた場合の結果が戻り値として返され、それが失敗したら例外で呼び出し元に伝えます。例外の分類(クラスの種類)に問題がなければFileStreamの例外はReadFileでは処理せずForm1(呼び出し元)で捕まえてメッセージを出すなりしてください。
そうすることでcatchにエラー処理を任せて、try{}内は常に想定している(エラーの無い)処理の流れを記述できます。

private void button1_Click(object sender, EventArgs e)
{
	try{
		var readResult = inputFile.ReadFile;

		// ここではReadFileが成功した前提で処理を記述できる

    }
    catch(Exception ex){
        MessageBox.Show(ex.Message, "エラー", MessageBoxButtons.OK, MessageBoxIcon.Error);
    }
}

public string ReadFile()
{
	FileInfo fileInfo = new FileInfo(Path);

	try
	{
	    using (var fs = new FileStream(Path, FileMode.Open)) { }
	}
	catch (IOException e)
	{
	    // なんか必要な処理
	    // 処理が必要なければ try---catchも不要
	    throw;	// 例外を投げ直す
	}
	...
	rteturn readResult;  // 関数の戻り値は正しく読みだせた場合の内容を返す
}

Comments

[eien_beginner]

ご回答ありがとうございます。大変勉強になりました。またの機会が御座いましたらよろしくお願いいたします。