的FxCop鑄造警告

Question

運行的FxCop時，我得到了以下錯誤：

CA1800：Microsoft.Performance： 'OBJ'，一個變量，是投在方法 鍵入 '工作' 多次「的ProductsController 。詳細信息（int， int）'。緩存的結果「爲」 運營商或直接投以 消除冗餘castclass 指令

代碼：

 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); 
      } 
     } 

如何處理此問題？我只能看到一個演員：Job j =（Job）obj。

Answer 1

只有一個鑄但也有測試。所以，你可以替換第一塊：

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

這意味着執行時間測試只需要進行一次，而不是兩次。這是一個有點微優化 - 在你的情況下，它會使代碼有點混亂，因爲你需要：

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); 
    } 
} 

（或聲明並初始化p更早，這浪費了測試，如果obj是實際上一個Job ...）