FXCop casting Attention

Question

je reçois l'erreur suivante lors de l'exécution FXCop:

CA1800: Microsoft.Performance: 'obj', une variable, est casté en type plusieurs fois 'travail' dans la méthode « ProduitsController .Détails (int, int) '. Mettre en cache le résultat de la 'comme' opérateur ou coulée directe pour éliminer le castclass redondant instruction

code:

 object obj = repository.GetJobOrPlace(jobId);//Returns (object) place or (object) product 

     if (obj != null) 
     { 
      if (obj is Job) 
      { 
       Job j = (Job) obj; 
       Debug.WriteLine(j.Title); 
      } 
      else if (obj is Place) 
      { 
       Place p = (Place) obj; 
       Debug.WriteLine(p.Title); 
      } 
     } 

Quel est le problème avec ça? Je ne peux voir qu'un seul casting: Job j = (Job) obj.

Answer 1

Il n'y a qu'un cast mais il y a aussi un test. Vous pouvez donc remplacer le premier bloc par:

Job j = obj as Job; 
if (j != null) 
{ 
    Debug.WriteLine(j.Title); 
} 

Cela signifie que le test du temps d'exécution ne doit être effectué qu'une seule fois au lieu de deux fois. Il est un peu d'un micro-optimisation - et dans votre cas, il serait un peu le code messier, comme vous auriez besoin:

Job j = obj as Job; 
if (j != null) 
{ 
    Debug.WriteLine(j.Title); 
} 
else 
{ 
    Place p = obj as Place; 
    if (p != null) 
    { 
     Debug.WriteLine(p.Title); 
    } 
} 

(Ou déclarer et initialiser p plus tôt, ce qui gaspille un test si obj est en fait Job ...)