設計決策：通過方法進行溝通

Question

這是一個簡單的設計決策，似乎對每一方都有重大的熱情。我正在努力去真正瞭解哪種設計的負面影響最小。

我要補充一個香蕉的方法：

public Banana AddBanana(string name) 
{ 
    // add the banana 
    var _Banana = new Banana { Name = name }; 
    this.Bananas.Add(_Banana); 
    return _Banana; 
} 

但在某些情況下，我不能添加一個香蕉，像這樣：

public Banana AddBanana(string name) 
{ 
    // test the request 
    if (this.Bananas.Count > 5) 
     return null; 
    if (this.Bananas.Where(x => x.Name == name).Any()) 
     return null; 
    // add the banana 
    var _Banana = new Banana { Name = name }; 
    this.Bananas.Add(_Banana); 
    return _Banana; 
} 

現在，我想回去傳達給呼叫者爲什麼他們不能。

哪種方式更好？

方法1：用一個異常

public Banana AddBanana(string name) 
{ 
    // test the request 
    if (this.Bananas.Count > 5) 
     throw new Exception("Already 5 Bananas"); 
    if (this.Bananas.Where(x => x.Name == name).Any()) 
     throw new Exception("Banana Already in List"); 
    // add the banana 
    var _Banana = new Banana { Name = name }; 
    this.Bananas.Add(_Banana); 
    return _Banana; 
} 

方法2進行通信：與測試

public Class CanAddBananaResult 
{ 
    public bool Allowed { get; set; } 
    public string Message { get; set; } 
} 

public CanAddBananaResult CanAddBanana(string name) 
{ 
    // test the request 
    if (this.Bananas.Count > 5) 
     return new CanAddBananaResult { 
      Allowed = false, 
      Message = "Already 5 Bananas" 
     }; 
    if (this.Bananas.Where(x => x.Name == name).Any()) 
     return new CanAddBananaResult { 
      Allowed = false, 
      Message = "Banana Already in List" 
     }; 
    return new CanAddBananaResult { Allowed = true }; 
} 

public Banana AddBanana(string name) 
{ 
    // test the request 
    if (!CanAddBanana(name).Allowed) 
     throw new Exception("Cannot Add Banana"); 
    // add the banana 
    var _Banana = new Banana { Name = name }; 
    this.Bananas.Add(_Banana); 
    return _Banana; 
} 

在方法1進行通信，消費者知道基於所述exception.Message的問題。

在方法2中，消費者可以防止發生異常而不是抓住異常。

哪種方法更好？

我讀過這樣的內容：設計類，以便在正常使用中不會拋出異常。例如，FileStream類公開了另一種確定是否已到達文件末尾的方法。這樣可以避免在讀取文件末尾時引發的異常。 http://msdn.microsoft.com/en-us/library/seyhszts(v=vs.71).aspx

但「特殊」的方法似乎是少代碼。這是否意味着更簡單/更好？

Answer 1

您應該同時使用兩者。

如果可以在方法被調用之前確定它是否會成功，那麼使用Can（）方法（或者屬性，如果失敗的條件不是特定於方法參數的話）是合適且有用的。框架中有很多例子：TypeConverter.CanConvertFrom /立即想到。

但是，在調用方法之前，您不能保證調用者將使用您提供給他們的方法。所以如果方法調用不正確，你仍然需要拋出適當的異常（TypeConverter.ConvertFrom將拋出與TypeConverter.CanConvertFrom返回false相同的條件）。

Answer 2

我想說這取決於情況。

如果班級的消費者知道如果不能添加香蕉或者不能添加香蕉是常規事件，那麼使用第二個選項。

另一方面，如果「不能添加香蕉」的處理邏輯是呼叫鏈上的幾層，或者如果不能處理香蕉是很少見的（只發生在網絡中斷的情況下，邏輯錯誤，磁盤錯誤等），然後更多地依賴異常 - 這就是他們的目標！

Answer 3

第二種方法似乎不必要的複雜，所以我傾向於選擇第一種方法。

爲了給調用者進行檢查自己的機會，我想補充以下成員：

public bool IsFull 
{ 
    get { return this.Bananas.Count > 5; } 
} 

public bool ContainsBanana(string name) 
{ 
    return this.Bananas.Any(b => b.Name == name); 
} 

我可能也返回現有香蕉如果名稱已經存在：

public Banana GetOrAddBanana(string name) 
{ 
    var banana = this.Bananas.FirstOrDefault(b => b.Name == name); 
    if (banana == null) 
    { 
     if (this.IsFull) throw new Exception("Collection is full"); 
     banana = new Banana { Name = name }; 
     this.Bananas.Add(banana); 
    } 
    return banana; 
} 

如果不能添加香蕉不是特例，但可以在正常操作過程中發生，我會使用Try ...模式：

public bool TryGetOrAddBanana(string name, out Banana banana) 
{ 
    banana = this.Bananas.FirstOrDefault(b => b.Name == name); 
    if (banana == null) 
    { 
     if (this.IsFull) return false; 
     banana = new Banana { Name = name }; 
     this.Bananas.Add(banana); 
    } 
    return true; 
}