クエリを取得するメソッドのreturn文の構文が改善されました。

Question

この単純なメソッドを使用して、Intを返すページからクエリ文字列を取得します。有効なIDが見つからない場合は、デフォルト値のint 0を返します。

私は詳細には、この方法を書くためのより良い方法があるかどうかを知りたい：

• 2だけではなく、単一のreturn文活用でしょうか？はいの場合は、それを行うための最善の方法は何ですか？
• クエリ文字列を取得するときに他のチェックを行いますか？

ありがとうございます。

protected int GetPageIndex() 
{ 
    int output; 
    bool result = Int32.TryParse(Request.QueryString["page"], out output); 
    if(result) 
     return output; 
    return output = 0; // Return Default value 
} 

Answer 1

でいいよ...変換が失敗した場合とにかく、Int32.TryParseは0に値を設定します。 .. :)私はいけないこれはどのようにクエリ文字列をチェックに関して、以下のコードを使用するhttp://msdn.microsoft.com/en-us/library/f02979c7.aspx

protected int GetPageIndex() 
    { 
     int output = 0; 
     Int32.TryParse(Request.QueryString["page"], out output); 
     return output; 
    } 

Answer 2

私はあなたがそれを解析しようとする前に、ちょうどそれを初期化し

protected int GetPageIndex() 
{ 
    int output = 0; 
    Int32.TryParse(Request.QueryString["page"], out output); 
    return output; 
} 

Answer 3

int output; 
return Int32.TryParse(Request.QueryString["page"], out output) ? output : 0; 

Answer 4

あなたがNEと思いますtryパースがnull値を処理する必要があるため、nullをチェックするためにedを使用します。

protected int GetPageIndex() 
{ 
    int output; 
    return Int32.TryParse(Request.QueryString[ "page" ], out output) ? output : 0; 
} 

Answer 5

あなたは一見TryParseが成功したか、と最初はoutputに0の値を割り当てることによって、明確にするため、そして、それがどうなるかであるかどうかの心配じゃないので、あなたは、単にoutputを返却しても安全ですとにかくTryParseが失敗した場合に返されます。

protected int GetPageIndex() 
{ 
    int output = 0; 
    int.TryParse(Request.QueryString["page"], out output); 
    return output; 
}