HttpContextとデータベース接続を使用した静的メソッド

Question

静的メソッドを使用するいくつかのコアクラスを持つプロジェクトを取得しました。

JavaScriptはいくつかのメソッドを反映したインターフェイスを持つC＃プロジェクトです。このインタフェースで は、このようなのHttpContextを使用して、データベース（実際にはただ一つのアクセスデータベース）からデータを取得する静的メソッドへの呼び出しの多くは（いくつかのメソッドが内部のいくつかのロジックを持っている）があります。

public static string UsrId { 
    get{ 
     try { 
      return HttpContext.Current.Session["usrid"].ToString(); 
     } catch(NullReferenceException) { 
      return ServiceVars.GuestId; 
     } 
    } 
} 

ので、私の質問は：

これは悪いですか？これをすることの短所は何ですか？

データベース接続を使用する静的メソッドは、リクエストをキューに入れるため、高トラフィックのサイトでは問題があります。これは本当ですか？または、静的メソッドのデータベース接続について心配する必要はありませんか？

静的メソッド内のHttpContextはどうですか、それはなぜですか？

Answer 1

ここで悪いのはキャッチですNullReferenceExceptionです。それをしないでください。

代わりに、最初はnullをチェック：

public static string UsrId { 
    get{ 
     var userId = HttpContext.Current.Session["usrid"]; 
     if (userId == null) { return ServiceVars.GuestId;} 
     return userId .ToString(); 
    } 
} 

はまた、あなたはを参照することができる問題では、静的メソッドの問題ではありません、またはHttpContextで。 Session状態を使用すると問題です。