爲什麼CodeRush會在代碼中提醒我一個未使用的聲明？

Question

我有這樣的代碼，我用遍了我的應用程序將數據保存到數據庫中。

public bool SaveDemo() 
    { 
     bool success = false; 

     try 
     { 
      using (DataTable dt = DataAccess.ExecuteDataTable("[dbo].[udp_Customers_ups]", 
       DataAccess.Parameter(CustomerIdColumn, CustomerId), 
       DataAccess.Parameter(CodeColumn, Code), 
       DataAccess.Parameter(CompanyColumn, Company), 
       DataAccess.Parameter(IsDeletedColumn, IsDeleted), 
       DataAccess.Parameter(LastUpdatedColumn, LastUpdated), 
       DataAccess.Parameter(UpdatedByColumn, UpdatedBy))) 

       success = true; 
     } 
     catch 
     { 
      success = false; 
     } 

     return success; 
    } 

該代碼的工作原理是這樣的，我的意思是它將數據保存回數據庫。然而CodeRush抱怨dt是一個未使用的聲明。而且由於使用是（我認爲）使用dt我會認爲警告是誤報。所以我還想知道CodeRush是錯誤的還是我錯過了一些東西？

Answer 1

什麼CR想說的是，在：

using (DataTable dt = DataAccess.ExecuteDataTable ... 

你是使用的dt聲明不;變量保持未變。

的重構按鈕將改變這

using (DataAccess.ExecuteDataTable ... 

即它仍然是一個using聲明，但你不會有一個變量來引用它。

當你這樣做，你可以做一些內嵌結果轉換，產生：

try 
    { 
     using (DataAccess.ExecuteDataTable("[dbo].[udp_Customers_ups]", 
      DataAccess.Parameter(CustomerIdColumn, CustomerId), 
      DataAccess.Parameter(CodeColumn, Code), 
      DataAccess.Parameter(CompanyColumn, Company), 
      DataAccess.Parameter(IsDeletedColumn, IsDeleted), 
      DataAccess.Parameter(LastUpdatedColumn, LastUpdated), 
      DataAccess.Parameter(UpdatedByColumn, UpdatedBy))) 
      return true; 
    } 
    catch 
    { 
     return false; 
    } 

我會讓別人質疑是否在catch塊包裝這樣的電話是個好主意......