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

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

ただいまの
回答率

91.36%

  • C#

    4756questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

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

解決済

回答 4

投稿 2017/11/30 16:36

  • 評価
  • クリップ 0
  • VIEW 150

almond

score 1

前提・実現したいこと

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

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 4

+5

こんにちは。

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

従って、

こうするとtry-catchは不要に見えます。
なくした場合、return nullはどこに書くものでしょう?

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

投稿 2017/11/30 16:55

編集 2017/11/30 16:57

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/12/01 13:41

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

    キャンセル

checkベストアンサー

+2

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

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

投稿 2017/11/30 17:53

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/12/01 13:37

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

    キャンセル

+1

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

投稿 2017/11/30 16:45

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/12/01 13:42

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

    キャンセル

+1

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 は自動的に行ってくれますので、上で言うリソースリークの心配はないのですが。

投稿 2017/11/30 17:42

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/12/01 13:41

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

    キャンセル

  • 2017/12/02 12:17

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

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

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

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

    キャンセル

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

ただいまの回答率

91.36%

関連した質問

同じタグがついた質問を見る

  • C#

    4756questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。