什麼名字2種方法用相同的簽名

Question

最初我曾在我們的DL的方法，將採取在它的更新，像這樣的對象：

internal void UpdateCash(Cash Cash) 
{ 
    using (OurCustomDbConnection conn = CreateConnection("UpdateCash")) 
    { 
     conn.CommandText = @"update Cash 
          set captureID = @captureID, 
            ac_code = @acCode, 
            captureDate = @captureDate, 
            errmsg = @errorMessage, 
            isDebit = @isDebit, 
            SourceInfoID = @sourceInfoID, 
            PayPalTransactionInfoID = @payPalTransactionInfoID, 
            CreditCardTransactionInfoID = @CreditCardTransactionInfoID 
           where id = @cashID"; 

     conn.AddParam("@captureID", cash.CaptureID); 
     conn.AddParam("@acCode", cash.ActionCode); 
     conn.AddParam("@captureDate", cash.CaptureDate); 
     conn.AddParam("@errorMessage", cash.ErrorMessage); 
     conn.AddParam("@isDebit", cyberCash.IsDebit); 
     conn.AddParam("@PayPalTransactionInfoID", cash.PayPalTransactionInfoID); 
     conn.AddParam("@CreditCardTransactionInfoID", cash.CreditCardTransactionInfoID); 
     conn.AddParam("@sourceInfoID", cash.SourceInfoID); 
     conn.AddParam("@cashID", cash.Id); 

     conn.ExecuteNonQuery(); 
    } 
} 

我的老闆認爲，每一次創建一個對象只是爲了更新一個或兩個領域是矯枉過正。但是我有一些代碼使用這個地方。他建議只使用UpdateCash併發送我想要更新的CAsh和字段的ID。那麼問題是我有我的原始方法在代碼中的2個地方。而這兩個地方正在更新Cash表中的兩個完全不同的字段。在我能夠獲取現有現金記錄並將其轉儲到Cash對象之前，然後更新我想要在數據庫中更新的屬性，然後將現金對象發送回我上面的方法。

我需要一些建議在這裏做什麼。我有2個方法，他們有相同的簽名。我不太確定，因爲兩者都在更新現金錶2個完全不同的領域如何重命名這些：

internal void UpdateCash(int cashID, int paypalCaptureID) 
{ 
    using (OurCustomDbConnection conn = CreateConnection("UpdateCash")) 
    { 
     conn.CommandText = @"update Cash 
          set CaptureID = @paypalCaptureID 
        where id = @cashID"; 

     conn.AddParam("@captureID", paypalCaptureID); 

     conn.ExecuteNonQuery(); 
    } 
} 

internal void UpdateCash(int cashID, int PayPalTransactionInfoID) 
{ 
    using (OurCustomDbConnection conn = CreateConnection("UpdateCash")) 
    { 
     conn.CommandText = @"update Cash 
          set PaymentSourceID = @PayPalTransactionInfoID 
        where id = @cashID"; 

     conn.AddParam("@PayPalTransactionInfoID", PayPalTransactionInfoID); 

     conn.ExecuteNonQuery(); 
    } 
} 

所以我想嗯，也許改變名稱，這些使他們現在唯一的，有點解釋它更新什麼領域：

UpdateCashOrderID 

UpdateCashTransactionInfoID 

好的，但這不是很好的名字。我不能去過於籠統，例如：

UpdateCashTransaction(int cashID, paypalTransactionID) 

如果我們有不同類型的transactionIDs的，現金持有記錄除了剛剛的paypalTransactionInfoID？如creditCardInfoID？那又怎麼樣？交易不告訴我什麼樣的。進而，如果你要更新2場，所以你有2個PARAMS旁邊cashID PARAM什麼：

UpdateCashTransaction(int cashID, paypalTransactionID, someOtherFieldIWantToUpdate) 

看到我的無奈？處理這個問題的最好辦法是我的老闆不喜歡我的第一條路線？

Answer 1

我的老闆覺得每次創建一個對象只是爲了更新一個或兩個字段是矯枉過正的。

他會是對的，如果您必須每次創建對象。對此的正確迴應是，您應該已經在整個應用程序中使用這些業務對象。您不需要創建新的Cash對象。您將Cash對象傳遞給您必須保存。

Answer 2

爲什麼不乾脆：

UpdateCashPaymentSource(int cashID, int PayPalTransactionInfoID) 

UpdateCashCapture(int cashID, int paypalCaptureID) 

Answer 3

「UpdateCashWithCapture」 和 「UpdateCashWithTransaction」？

Answer 4

如何UpdateCashByCaptureID和UpdateCashByTransactionInfoID？

Answer 5

UpdateCashByTransactionInfoID
UpdateCashByCaptureID（）

？

Answer 6

，添加字段更新的名稱，即

internal void UpdateCash_paypalCaptureID(...) 
internal void UpdateCash_PayPalTransactionInfoID(...) 

Answer 7

將一個方法和枚舉下調嗎？

internal void UpdateCash(int cashID, int id, FieldName field) 
{ 
    using (OurCustomDbConnection conn = CreateConnection("UpdateCash")) 
    { 
     conn.CommandText = string.format("update Cash set {0} = @id where id = @cashID", field.ToString()); 

     conn.AddParam("@id", id); 
     conn.AddParam("@cashId", cashId); 

     conn.ExecuteNonQuery(); 
    } 
} 

public enum FieldName 
{ 
    PayPalCaptureId, 
    PayPalTransactionInfoID 
} 

編輯：

在現在讀你的編輯，我同意你的原來的辦法是去，在我看來的方式 - 通過在一個對象，並更新所有相關的字段在數據庫中，與傳遞對象屬性值並更新數據庫中的值相比，最大的性能殺手將打開數據庫連接，而不是與一個數據庫記錄相關的字段數。

Answer 8

您可以封裝更新查詢邏輯類：

public abstract class CashUpdateQuery 
{ 
    public CashUpdateQuery(int cashId) 
    { 
     this.CashId = cashId; 
    } 

    protected int CashId { get; private set; } 

    public abstract void SetConnectionProperties(OurCustomDbConnection conn); 
} 

然後，你可以爲每個更新方案的具體子類。因此，對於你的PayPal查詢你有這樣的事情：

public PaypalTransactionCashUpdateQuery : CashUpdateQuery 
{ 
    private readonly int paypalCaptureId; 
    public PaypalTransationCashUpdateQuery(int cashId, int paypalCaptureId) 
    { 
     this.paypalCaptureId = paypalCaptureId; 
    } 

    public override void SetConnectionProperties(OurCustomDbConnection conn) 
    { 
     conn.CommandText = @"update Cash 
          set CaptureID = @paypalCaptureID 
          where id = @cashID"; 

     conn.AddParam("@captureID", this.paypalCaptureId); 
     conn.AddParam("@cashID", this.CashId); 
    } 
} 

那麼你的更新方法可以查詢對象，並用它來設置連接上的查詢屬性和執行：

internal void UpdateCash(CashUpdateQuery query) 
{ 
    using(OurCustomDbConnection conn = CreateConnection("UpdateCash")) 
    { 
     query.SetConnectionProperties(conn); 
     conn.ExecuteNonQuery(); 
    } 
} 

這意味着添加新查詢只是添加CashUpdateQuery的新子類的一個例子。