重複の可能性:
空のキャッチブロックが悪い考えであるのはなぜですか?
キャッチされた例外を無視する正当な理由はありますか
空のキャッチブロックが絶対的な悪ではない状況を知っていますか?
try
{
...
// What and When?
...
}
catch { }
重複の可能性:
空のキャッチブロックが悪い考えであるのはなぜですか?
キャッチされた例外を無視する正当な理由はありますか
空のキャッチブロックが絶対的な悪ではない状況を知っていますか?
try
{
...
// What and When?
...
}
catch { }
There are a lot of questions on this, try to look at:
Why are empty catch blocks a bad idea?
From that post's accepted answer:
Usually empty try-catch is a bad idea because you are silently swallowing an error condition and then continuing execution. Occasionally this may be the right thing to do, but often it's a sign that a developer saw an exception, didn't know what to do about it, and so used an empty catch to silence the problem.
It's the programming equivalent of putting black tape over an engine warning light.
これを見てください。基本的に、発生する可能性のある例外の種類を4つのカテゴリに分類しますが、いずれも空のcatchブロックで処理する必要はありません。
少なくとも、try {} に入力したものが例外をスローしたことを示す何らかのコメントまたはログに記録されたメッセージを提供する必要があると思います。これが、何もしていない理由です。
ある種のbool TrySomething(out object)
関数が必要な、またはobject TrySomething()
基になる呼び出しが例外として他のメカニズムを提供しない、いくつかの自己記述ライブラリに使用しました。この場合、空の catch ブロックを使用してfalse
orを返しnull
ます (関数のシグネチャによって異なります)。
public bool TrySomething(out object destination)
{
try
{
destination = DoSomething();
return true;
}
catch
{}
return false;
}
公理:
空のキャッチブロックは絶対悪
これを回避する方法を見つけようとしないでください。絶対的な悪ではないケースを見つけようとするだけで、貴重な脳のサイクルを浪費していることになります。ここでパターンを見つけようとしないでください。「うーん、ここに空の catch ブロックを配置する必要がありますか?」
誰かのコードで空の catch ブロックに出くわした場合は、技術的負債に遭遇したことになります。修理する。空の catch ブロック内にロギング ステートメントを 1 つ追加するだけでも、この世界をより良い場所にすることができます。