Azure関数でクラスを使用する

Question

コンストラクタを使用して初期化することによって設定される外部ログ用のクラスを作成したいとします。私は、このクラスをFunctionの全期間を通して複数回使用できるようにしたいと考えています。

using System.Net; 

private static Logger logger; 

public static async Task<HttpResponseMessage> Run(HttpRequestMessage req, 
TraceWriter log, ExecutionContext executionContext) 
{ 
    log.Info("C# HTTP trigger function processed a request."); 

    string invocationId = executionContext.InvocationId.ToString(); 
    logger = new Logger(invocationId); 

    logger.Log("Start"); 

    // parse query parameter 
    string name = req.GetQueryNameValuePairs() 
    .FirstOrDefault(q => string.Compare(q.Key, "name", true) == 0) 
    .Value; 

    // Get request body 
    dynamic data = await req.Content.ReadAsAsync<object>(); 

    // Set name to query string or body data 
    name = name ?? data?.name; 

    logger.Log("Finish"); 

    return name == null 
    ? req.CreateResponse(HttpStatusCode.BadRequest, "Please pass a name on the query string or in the request body") 
    : req.CreateResponse(HttpStatusCode.OK, "Hello " + name); 
} 

public class Logger 
{ 
    private string _invocationId; 

    public Logger(string invocationId) 
    { 
     _invocationId = invocationId; 
    } 

    public void Log(string message) 
    { 
     message = $"{_invocationId} | {message}"; 
     // log to Splunk 
    } 
} 

Loggerクラスを関数全体で「グローバルに」使用する正しい方法はありますか？

private static Logger logger; 

public static async Task<HttpResponseMessage> Run(HttpRequestMessage req, 
TraceWriter log, ExecutionContext executionContext) 
{ 
    log.Info("C# HTTP trigger function processed a request."); 

    string invocationId = executionContext.InvocationId.ToString(); 
    logger = new Logger(invocationId); 

新しくクラスアップしたことは何ですか？

これは私が達成しようとしているものの簡略化されたバージョンです。

Answer 1

コードはあまり意味がありません。静的なフィールドがありますが、呼び出すたびに割り当てます。 2番目の呼び出しは最初の呼び出しによって作成された値を上書きする可能性があるため、予期しない結果が得られる可能性があります。

フィールドは静的で、一度作成してから次のすべての呼び出しでインスタンスを使用するか、または、ローカル変数を作成して、それを今のように使用するだけです。

2つ目のオプションは、本当に何かリクエストを共有する必要がない限り、優先します。可能であれば共有状態を避けてください。

UPDATE：あなたのコメントからの質問に基づいて

は、ここにあなたが何ができるかです：

public static Task<HttpResponseMessage> Run(
    HttpRequestMessage req, ExecutionContext executionContext) 
{ 
    string invocationId = executionContext.InvocationId.ToString(); 
    var processor = new Processor(invocationId); 
    return processor.Process(req); 
} 

public class Processor 
{ 
    private Logger logger; 

    public Processor(string invocationId) 
    { 
     this.logger = new Logger(invocationId); 
    } 

    public async Task<HttpResponseMessage> Process(HttpRequestMessage req) 
    { 
     await this.logger.Log("Start"); 
     await this.DoStep1(); 
     await this.DoStep2(); 
     await this.logger.Log("Finish"); 
    } 

    private async Task DoStep1() 
    { 
     await this.logger.Log("Step 1"); 
     // ... 
    } 

    private async Task DoStep2() 
    { 
     await this.logger.Log("Step 2"); 
     // ... 
    } 
} 

私はこのスタイルの特定のファンではないんだけど、それはあなたが欲しいものだ場合は、それは正常に動作します。