テスト()
メソッドではファイルが存在するかどうかの情報を返し、呼び出し元ではその情報を元にif
で分岐すればよいのでは?
private void button1_Click(object sender, EventArgs e)
{
if (テスト()) {
System.IO.StreamReader sr = new System.IO.StreamReader("test.txt");
string contents = sr.ReadToEnd();
sr.Close();
}
}
public bool テスト()
{
...
return System.IO.File.Exists("test.txt");
}
呼び出し元のコード量が増えてしまうので出来ればメソッド中で終了したいと考えています。
無理です。そもそもbutton1_Click()
とテスト()
の両メソッドで"test.txt"
を扱っています。各メソッドの役割が適切に分離できていないためにそもそも記述量が増えてしまっています。その他にもC#言語には記述量を減らせる機能がいくつも用意されています。if
文だけを減らす努力をしても仕方がありません。
System.IO.StreamReader sr = new System.IO.StreamReader("test.txt");
string contents = sr.ReadToEnd();
sr.Close();
はvar
キーワードを使用することで型名を明示する必要はなくなります。
var sr = new System.IO.StreamReader("test.txt");
var contents = sr.ReadToEnd();
sr.Close();
とできます。using System.IO;
ディレクティブで名前空間の記述も省略できます。
var sr = new StreamReader("test.txt");
var contents = sr.ReadToEnd();
sr.Close();
using
ステートメントで後処理を確実に行えます。
using (var sr = new StreamReader("test.txt")) {
var contents = sr.ReadToEnd();
}
そもそもFile.ReadAllText()
メソッドで簡単に読み出せます。
var contents = File.ReadAllText("test.txt");
using static System.IO.File;
ディレクティブで更にクラス名も省略できます。
var contents = ReadAllText("test.txt");
さてテスト()
メソッドはファイルの有無の判定結果を返すのではなく、ファイルコンテンツを返す方法もあります。
private void button1_Click(object sender, EventArgs e)
{
var contents = テスト();
if (contents != null) {
...
}
}
public string テスト()
{
...
return Exists("test.txt") ? ReadAllText("test.txt") : null;
}
以上はあくまで一例ですが、このようにメソッドの役割分担を再検討されることをお勧めします。
button1_Click
を終了したいという意味ですか?