try-catchをusingにした場合、catchのreturn nullはどこに書けばよいですか？

Question

###前提・実現したいこと
try-catchをusingにした場合、catchのreturn nullはどこに書けばよいですか？
try-catchは不要ですか？

###発生している問題・エラーメッセージ
try-catchをusingにしてみました。
こうするとtry-catchは不要に見えます。
なくした場合、return nullはどこに書くものでしょう？

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

using MySql.Data.MySqlClient;
using System;
using System.Data;

namespace MySQLAccess
{
	public class MySQL
	{
		public DataSet Select()
		
			string query = "select * From MyTable;";
			var command = new MySqlCommand(query, Connection);
			try
			{
				var dataadapter = new MySqlDataAdapter(query, Connection);
				var dataset = new DataSet();
				dataadapter.Fill(dataset);
				return dataset;
			}
			catch (Exception exception)
			{
				return null;
			}
		}
	}
}

using MySql.Data.MySqlClient;
using System;
using System.Data;

namespace MySQLAccess
{
	public class MySQL
	{
		public DataSet Select()
		
			string query = "select * From MyTable;";
			using (var command = new MySqlCommand(query, Connection, transaction))
			{
				try
				{
					var dataadapter = new MySqlDataAdapter(query, Connection);
					var dataset = new DataSet();
					dataadapter.Fill(dataset);
					transaction.Commit();
					return dataset;
				}
				catch (Exception exception)
				{
					return null;
				}
			}
		}
	}
}

###補足情報(言語/FW/ツール等のバージョンなど)
Visual Studio

Answer 1

こんにちは。

usingはtry-catchでできることの全てを行えうものではありません。
関数内で獲得したリソースの開放漏れを防ぐのが目的の機能です。例外を補足しないとできない処理（catchしてreturnはその一つです）をusingで行うことはできません。

従って、

こうするとtry-catchは不要に見えます。

なくした場合、return nullはどこに書くものでしょう？

catch内でreturn nullしたい場合はtry-catchは必要です。

Comments

[almond]

こんにちは。ありがとうございます。

Answer 2

try-catchをusingにした場合、catchのreturn nullはどこに書けばよいですか？

そもそもの認識に誤りがあります。tyr/catchとusingそれぞれが提供する機能は全く別の目的なので、どちらかがもう一方を代替するなどということはできません。
例外を処理したいのならtry/catchを使ってください。自動でDisposeさせたいのならusingを使ってください。例外を処理しつつ自動でDisposeさせたいのなら、try/catchとusing両方を使ってください。

Comments

[almond]

ありがとうございます。頭を整理できました。
例外の処理が必要なので、usingに加えてtry-catchします。

Answer 3

try-catchをusingにした場合、catchのreturn nullはどこに書けばよいですか？
try-catchは不要ですか？

using が try-catch の代わりにはならないことは他の回答者の方が答えられているので、別の問題についてレスします。

それは何かと言うと例外処置で、特に Exception をキャッチして全ての例外を握りつぶしているのは絶対やってはいけないことです。

例外処置について、自分的に一般的と思うことを書きますと：

(1) 予測可能で正しい業務フローに戻すことができる「業務エラー」（例：ユーザーの入力間違い）と、予測できないもしくは予測はできても何の対応もできない「例外」（例：DB サーバーダウン）を区別して対処。

(2) 「例外」はランタイムに拾わせてアプリケーションを停止させる。

(3) よほどのことがない限り try-catch は書かない。

(4) キャッチせざるを得ない場合でも Execption はキャッチしない。

(5) 間違って補足してしまった例外は throw する。（注：catch ブロックでキャッチした例外を throw するとスタックトレースが途切れるので単に throw と書く）

(6) ユーザーへの通知が必要なら、集約的例外処置を利用する。

詳しくは以下の記事を見てください。

NETの例外処理 Part.1
https://blogs.msdn.microsoft.com/nakama/2008/12/29/net-part-1/

.NETの例外処理 Part.2
https://blogs.msdn.microsoft.com/nakama/2009/01/02/net-part-2/

あと、.NET 4 からは破損状態例外は catch できなくなっているそうですが、「それでも Catch (Exception e) を使用するのはよくない」ということについては以下の記事を見てください。

破損状態例外を処理する
https://msdn.microsoft.com/ja-jp/magazine/dd419661.aspx

その他、

質問者さんが書いたコードの using の例は MySqlCommand を確実に Dispose するという意味しかありません。（IDisposable を継承しているクラスは原則使い終わったら Dispose すべきですが、Command の場合あまり意味はない）

通常、ADO.NET を使って DB にアクセスするケースで using を使うのは、接続（MySQL で Connector/NET を使った場合は MySqlConnection）が必ず Dispose（Close と同じ）されるようにし、リソースリーク（この場合は、接続プールにおける接続の枯渇）を防止するためです。そのあたりと混同してませんか？　詳しくは上に紹介した記事の Part 2 を見てください。

ただし、MySqlDataAdapter を利用した場合は、その内部で Open / Close は自動的に行ってくれますので、上で言うリソースリークの心配はないのですが。

Comments

[almond]

コメントありがとうございます。
開発中なので、try-catchあります。

[退会済みユーザー]

> 開発中なので、try-catchあります。

それは全く理屈が通りません。質問者さんは全くの初学者のようですので理解しがたいのかもしれませんが、重要のことなのでよく考えて見てください。

むしろ開発中だからこそ try - catch で Exception を catchすべきではないはずです。

特に質問に書いてあったようなコードですと、問題の原因究明と対策に必要な情報を自ら破棄しているようなものです。（null が返ってきたら何らかの問題で DataSet は生成できなかったことは分かるかもしれませんが、何がその原因になったのかを究明するのは困難。結局例外情報が必要で Exception は catch しないようコードを書き換えることになるのでは？）

Answer 4

不要とはどういうことでしょうか？
usingは確実にDisposeするだけで、例外処理をしてくれるわけではありません。
なくした場合、呼び元に例外が飛ぶかもしれません。

Comments

[almond]

コメントありがとうございます。