コントローラのASP.NETと静的メソッド

Question

WebApi2コントローラの起動時にスコープライフスタイルの依存関係に設定されたSearchClientがあるとします。

public class SearchController : ApiController { 

    private readonly SearchClient _indexClient; 

    public SearchController(SearchClient client) { 
     _indexClient = client; // dependency injected 
    } 

    public IEnumerable<string> Get(string keyword){ 
     return SearchDocuments(_indexClient, keyword); 
    } 

    public static IEnumerable<string> SearchDocuments(SearchClient indexClient, string text) 
    { 
     return indexClient.Search(text); 
    } 
} 

私たちが見ることができるように、SearchDocuments方法はstaticキーワードを持っています。

私の質問は次のとおりです。

1. staticの方法が良いか悪いかをどのように判断できますか？
2. このような複数アクセスのWeb環境では、staticメソッドは安全か推奨ですか？
3. ウェブ環境でasync staticメソッドはどうなりますか？ asyncメソッドとは違いますか？

Answer 1

静的メソッドが良いか悪いかをどのように判断できますか？

Webアプリケーションの静的メソッドは、デスクトップアプリケーションの静的メソッドとまったく同じです。それらがWebアプリケーションで実行されると、処理や解釈の方法に違いはありません。だから彼らは悪くないか良いことではなく、あなたはインスタンス固有でないすべてのものに使います。

このような複数アクセスのWeb環境で静的メソッドが安全か推奨されていますか？

staticフィールドまたは特性がstatic変数はセッション間で共有されるので、ユーザまたはセッション固有のデータがそこに格納されている望ましくない副作用を有することができます。しかしこれはメソッドであり、メソッドは共有状態を持たない。したがって、マルチユーザー/マルチセッション環境での使用は安全です。

ウェブ環境での非同期スタティックメソッドはどうですか？非同期メソッドとは違いますか？

あなたの最初の質問に対する答えには既に説明されている以外の何もありません。

Answer 2

すでに提供されている回答に追加するだけです。

コントローラーで静的メソッドを使用しても実際に値が追加されるわけではなく、特定のシナリオでは実際には必要ありません。

コントローラの明示的な依存関係を抽象化することを検討してください。

public class SearchController : ApiController { 

    private readonly ISearchClient indexClient; 

    public SearchController(ISearchClient client) { 
     indexClient = client; // dependency injected 
    } 

    public IEnumerable<string> Get(string keyword){ 
     return indexClient.Search(keyword); 
    } 
} 

また、これは、疎結合とテストとも依存の実装としてリファクタリングするコントローラをタッチすることなく変更することができ、より多くの柔軟性を可能にします。