私は悪いプログラミングの練習をキャッチしよう

Question

私はストアドプロシージャを実行していて、文字列を返しています。文字列は、条件によっては1、0または"USER DOES NOT EXISTS"を返すように設定されています。

次が悪いプログラミングの実践であるかどうかを知りたかっただけです。

string result = _db.GetParameterValue(cmdObj, "@strMessage").ToString(); 
try 
{ 
    int a = int.Parse(result); 
    if (a == 1) 
     Console.WriteLine("A"); 
    else 
     Console.WriteLine("B"); 
} 
catch 
{ 
    Console.WriteLine(result); 
} 

Console.WriteLine(result); 

Answer 1

あなたは、try catchブロックで、それを含まないtryParseとを使用する必要があります。

int outValue = -1; 
int.TryParse(result, out outValue); 

Answer 2

失敗したint解析をキャッチすることに基づいて、「USER NOT EXISTS」と推定するのではなく、常に一致させる方が良いです。

いつも/ catch/swallowを試すのは悪い習慣です。例外をキャッチする場合は、ログに記録するか、スローします。

あなたは言語を指定していないので、C＃と仮定するとint.TryParse()はint.Parseよりもはるかにきれいで、try/catchです。