Sollte ich auf ReadAsStringAsync() warten, wenn ich auf die Antwort wartete, die ich ReadAsStringAsync() ausführe?

Question

Sollte ich erwartenReadAsStringAsync() wenn ich erwartet die Antwort, auf der ich ReadAsStringAsync() durchführen? Um weiter zu klären, was ist der Unterschied oder der richtige Weg zwischen den folgenden? Sind sie tatsächlich gleich?

var response = await httpClient.GetAsync("something"); 
var content = await response.Content.ReadAsStringAsync(); 
return new AvailableViewingTimesMapper().Map(content); 

ODER

var response = await httpClient.GetAsync("something"); 
var content = response.Content.ReadAsStringAsync(); 
return new AvailableViewingTimesMapper().Map(content.Result); 

Answer 1

Ihr erstes Beispiel ist das richtige. Das zweite Beispiel gibt während der asynchronen Operation nicht nach. Wenn Sie stattdessen den Wert der Eigenschaft content.Result abrufen, erzwingen Sie, dass der aktuelle Thread wartet, bis der asynchrone Vorgang abgeschlossen ist.

Darüber hinaus, wie Kommentator Scott Chamberlain hervorhebt, durch Blockieren des aktuellen Threads ist es möglich, dass Sie die Möglichkeit eines Deadlocks einführen könnten. Das hängt vom Kontext ab, aber ein gängiges Szenario für await besteht darin, diese Anweisung im UI-Thread zu verwenden, und der UI-Thread muss für eine Vielzahl von Anforderungen reaktionsfähig bleiben, aber auch, um den Abschluss einer erwarteten Operation tatsächlich verarbeiten zu können .

Wenn Sie das zweite Muster vermeiden, dh den Wert der Result Eigenschaft aus einer Task Abrufen Sie wissen nicht abgeschlossen ist, können Sie nicht nur die effiziente Nutzung der Threads gewährleisten, können Sie auch gegen diese gemeinsame Deadlock-Falle gewährleisten .

Answer 2

Der Grund, warum ReadAsString ist ein Asynchron-Methode ist, die tatsächlich das Lesen von Daten ist ein IO-Betrieb. Der Inhalt ist möglicherweise nicht vollständig geladen, selbst wenn Sie bereits das http-Ergebnis haben. Es gibt keine zusätzlichen Threads oder große Rechenlasten.

HttpClient.GetAsync können Sie eine HttpCompletionOption hinzufügen, damit das GetAsync erst zurückkehrt, wenn das gesamte HttpResult geladen wurde. In diesem Fall wird HttpContent.ReadAsStringAsync synchron ausgeführt (ein sogenannter Fastpath), da der Inhalt bereits vorhanden ist.

So sollten Sie es definitiv erwarten.

Auch: Da dies wahrscheinlich Bibliothekscode ist, der nicht davon abhängig ist, dass der UI-Thread zurückgegeben wird, sollten Sie .ConfigureAwait (false) allen erwarteten Methodenaufrufen hinzufügen.

Answer 3

Hier ist .NET-Quellcode für ReadAsStringAsync. Wenn Sie tiefer in LoadIntoBufferAsync() -Methode schauen, werden Sie sehen, dies wird lesen Puffer aus HttpResponse und führt zu einem möglichen weiteren Netzwerkaufruf. Dies bedeutet, dass es eine gute Übung ist, anstelle des Ergebnisses zu verwenden.

[__DynamicallyInvokable] 
    public Task<string> ReadAsStringAsync() 
    { 
     this.CheckDisposed(); 
     TaskCompletionSource<string> tcs = new TaskCompletionSource<string>(); 
     HttpUtilities.ContinueWithStandard(this.LoadIntoBufferAsync(), (Action<Task>) (task => 
     { 
     if (HttpUtilities.HandleFaultsAndCancelation<string>(task, tcs)) 
      return; 
     if (this.bufferedContent.Length == 0L) 
     { 
      tcs.TrySetResult(string.Empty); 
     } 
     else 
     { 
      Encoding encoding1 = (Encoding) null; 
      int index = -1; 
      byte[] buffer = this.bufferedContent.GetBuffer(); 
      int dataLength = (int) this.bufferedContent.Length; 
      if (this.Headers.ContentType != null) 
      { 
      if (this.Headers.ContentType.CharSet != null) 
      { 
       try 
       { 
       encoding1 = Encoding.GetEncoding(this.Headers.ContentType.CharSet); 
       } 
       catch (ArgumentException ex) 
       { 
       tcs.TrySetException((Exception) new InvalidOperationException(SR.net_http_content_invalid_charset, (Exception) ex)); 
       return; 
       } 
      } 
      } 
      if (encoding1 == null) 
      { 
      foreach (Encoding encoding2 in HttpContent.EncodingsWithBom) 
      { 
       byte[] preamble = encoding2.GetPreamble(); 
       if (HttpContent.ByteArrayHasPrefix(buffer, dataLength, preamble)) 
       { 
       encoding1 = encoding2; 
       index = preamble.Length; 
       break; 
       } 
      } 
      } 
      Encoding encoding3 = encoding1 ?? HttpContent.DefaultStringEncoding; 
      if (index == -1) 
      { 
      byte[] preamble = encoding3.GetPreamble(); 
      index = !HttpContent.ByteArrayHasPrefix(buffer, dataLength, preamble) ? 0 : preamble.Length; 
      } 
      try 
      { 
      tcs.TrySetResult(encoding3.GetString(buffer, index, dataLength - index)); 
      } 
      catch (Exception ex) 
      { 
      tcs.TrySetException(ex); 
      } 
     } 
     })); 
     return tcs.Task; 
    }