Wie vermeidet man den folgenden wiederholten Code?

Question

Ich habe zwei Funktionen geschrieben, die ähnlich aussehen, wie kann ich sie optimieren?

Hinweis:

1. AsyncCompletedEventArgs ist die Basisklasse von DownloadStringCompletedEventArg und UploadStringCompletedEventArgs.

2. Result Eigentum ist nicht in der AsyncCompletedEventArgs.

3. DownloadStringCompletedEventArgs eine Error Eigenschaft hat, wenn Errornull ist, dann versuchen Result Eigenschaft zuzugreifen, tritt die Ausnahme.

void fun1(DownloadStringCompletedEventArgs e) 
{ 
    try 
    { 
     string s = e.Result; 
    } 
    catch (WebException eX) 
    { 
     HandleWebException(); 
    } 
} 

void fun2(UploadStringCompletedEventArgs e) 
{ 
    try 
    { 
     string s = e.Result; 
    } 
    catch (WebException eX) 
    { 
     HandleWebException(); 
    } 
} 

Answer 1

UploadStringCompletedEventArgs und DownloadCompletedEventArgs beide AsyncCompletedEventArgs verlängern aber leider ist die Basisklasse die Result Eigenschaft nicht definieren ist.

A Tryx Muster mit einem Ergebnis Accessor delegieren könnten hier sinnvoll sein:

public bool TryGetResult(Func<string> resultAccessor, out string result) 
{ 
    try 
    { 
     result = resultAccessor(); 
     return true; 
    } 
    catch(WebException) 
    { 
     HandleWebException(); 

     result = null; 
     return false; 
    } 
} 

void fun1(DownloadStringCompletedEventArgs e)  
{ 
    string result; 
    if (TryGetResult(() => e.Result, out result)) 
    { 
     // Success 
    } 
}  

void fun2(UploadStringCompletedEventArgs e)  
{  
    string result; 
    if (TryGetResult(() => e.Result, out result)) 
    { 
     // Success 
    } 
} 

ich empfehlen würde versuchen, einen Scheck zu AsyncCompletedEventArgs.Error zu arbeiten, obwohl, wie Ausnahmen recht teuer sind.

Answer 2

Sie Code kann wie unten etwas geändert werden:

void fun1(DownloadStringCompletedEventArgs e) { Process(e); } 

    void fun2(UploadStringCompletedEventArgs e) { Process(e); } 

    private void Process(dynamic eventArgs) 
    { 
     try 
     { 
      string s = eventArgs.Result; 
     } 
     catch (WebException e) 
     { 
      HandleWebException(e); 
     } 
    } 

Answer 3

Vielleicht könnten Sie eine Funktion schreiben, die einen Parameter vom Typ AsyncCompletedEventArgs nimmt (von denen beide eventArg Klassen, die Sie verwenden, erben) und dann versuchen, um es in Ihrem Code in den richtigen Typ zu konvertieren. Das würde es Ihnen ermöglichen, beide mit der gleichen Methode zu vervollständigen, aber wenn Sie Ihren Code betrachten, würde er wahrscheinlich nicht viel Nutzen für Sie haben. Viel Glück!

Answer 4

Etwas wie folgt aus:

void fun1(DownloadStringCompletedEventArgs e) 
{ 
    var result = Process<string>(e); 
    if (result != null) 
    { 
     // TODO your logic here 
    } 
} 

void fun2(UploadStringCompletedEventArgs e) 
{ 
    var result = Process<string>(e); 
    if (result != null) 
    { 
     // TODO your logic here 
    } 
} 

private T Process<T>(AsyncCompletedEventArgs result) 
{ 
    if (result.Error != null) 
     HandleWebException(result.Error); 
    else if (!result.Cancelled) 
    { 
     //var prop = result.GetType().GetProperty("Result"); 
     //return (T) prop.GetValue(result, null); 
     return (T) ((dynamic)result).Result; 
    } 
    //else // TODO handle cancelled 
    return default(T); 
}