Wie evaluiert C# Logikblock?

Question

Ich habe die folgende Codezeile:

public bool dcpl_radar() 
    { 
     if (radar == null) 
      return false; 
     else 
     { 
      if (radar != null) 
      { 
       if (radar.InvokeRequired) 
        radar.BeginInvoke(new MethodInvoker(delegate() 
               { 
                radar.Visible = false; 
               })); 
       else 
        this.radar.Visible = false; 

       radar = null; 
      } 
      return true; 
     }//end of else statement 
    } 

aber VStudio hält auf der invoke Linie einen Fehler werfen. Ich habe den Debugger überprüft und wenn (Radar == null) ist wahr, doch VStudio versucht, einen Teil des Codes zu bewerten, in dem es nicht sein sollte. Kann jemand erklären, warum es das bitte tut?

Answer 1

Warten Sie eine Minute ... Ich denke, wir haben eine Wettlaufbedingung. Sagen wir Sie BeginInvoke, fast sofort setzen Sie Radar = Null. Es ist wirklich nicht zu sagen, wenn Ihr anonymer Delegat ausgeführt wird.

Ich könnte mir vorstellen, dass dies Ihr Problem lösen sollte.

public bool dcpl_radar() 
{ 
    if (radar != null) 
    { 
     if (radar.InvokeRequired) 
     { 
      radar.BeginInvoke(new MethodInvoker(HideRadar)); 
     } 
     else 
     { 
      HideRadar(); 
     } 

     return true; 
    } 

    return false; 
} 

private void HideRadar() 
{ 
    this.radar.Visible = false; 
    this.radar = null; 
} 

Answer 2

Was geschieht:

Der anonyme Delegaten wird, nachdem Sie setzen die radar aufgerufen wird auf null.

Wie beheben es

public bool dcpl_radar() 
{ 
    if (radar == null) 
     return false; 
    else 
    { 
     if (radar != null) 
     { 
      if (radar.InvokeRequired) 
       radar.BeginInvoke(new MethodInvoker(delegate() 
              { 
               radar.Visible = false; 
               radar = null; 
              })); 
      else { 
       this.radar.Visible = false; 
       radar = null; 
      } 


     } 
     return true; 
    }//end of else statement 
} 

(Beachten Sie, wo ich dein 'null' Zuweisungen verschoben haben).

Obwohl ich ein bisschen besorgt über den Punkt der Einstellung einer Variablen auf null bin, ist es im Allgemeinen ein Zeichen für ein schlechtes Design.