设计建议所需

Question

需要以下

public class SomeClass 
{ 
    public void AuthoriseSmartPhone(...) 
    { 
     try 
     { 
      Authorise(...) 
     } 
     catch(CustomException) 
     { 
      throw new CustomException("SmartPhone Message") 
     } 
    } 

    public void AuthoriseFeaturePhone(...) 
    { 
     try 
     { 
      Authorise(...) 
     } 
     catch(CustomException) 
     { 
      throw new CustomException("FeaturePhone Message") 
     } 
    } 

    private void Authorise(...) 
    { 
     if(manySuchConditionsCheckedHere)     
      throw new CustomException(""); 

     if(someOtherConditionCheckFails) 
      throw new YetAnotherException("Common Message to both phones"); 
    } 
} 

在代码设计建议授权是有很多业务规则检查

的私有方法

授权可以为智能手机或功能手机

虽然智能手机和功能电话的规则相同，但是在任何情况下返回的信息都不相同。

目前上面的代码抛出CustomException和同样是捕获并重新抛出。希望得到一些关于如何改善这些代码的设计建议。

Answer 1

一个想法是推出一些loose coupling到你的代码。

你有一个像AuthoriseSmartPhone和AuthoriseFeaturePhone方法。此代码不会在您的设计中留下任何扩展空间。如果你有正在使用的参数都实现通用接口的对象，你可以有一个方法：

public void AuthorisePhone(...) 
{ 
    try 
    { 
     if(parameter is a FeaturePhone) { 
      Authorise(...); 
     } 
     else if(...) 
    } 
    catch(CustomException ex) 
    { 
     throw new CustomException("Relevant Message", ex) 
    } 
} 

这样，你可以调整类中的代码，但从来没有改变代码在课外。这鼓励了类之间的松散耦合，这是一个理想且非常有用的特性。

编辑

我也注意到，如果你采用这种方法，这意味着你可以采用Strategy Pattern在你的代码。这有助于您有if...else if... else if结构。

Answer 2

我建议使用内部异常只是包装由Authorise抛出的异常：

public void AuthoriseSmartPhone(...) 
{ 
    try 
    { 
     Authorise(...) 
    } 
    catch(CustomException ex) 
    { 
     throw new CustomException("SmartPhone Message", ex) 
    } 
} 

public void AuthoriseFeaturePhone(...) 
{ 
    try 
    { 
     Authorise(...) 
    } 
    catch(CustomException ex) 
    { 
     throw new CustomException("FeaturePhone Message", ex) 
    } 
} 

private void Authorise(...) 
{ 
    if(condition1)     
     throw new CustomException("Condition 1 Failed"); 
    if(condition2)     
     throw new CustomException("Condition 2 Failed"); 
    if(condition3)     
     throw new CustomException("Condition 3 Failed"); 
} 

这样你仍然可以准确地确定哪些条件通过内部异常失败，但与自定义消息外例外。

如果你想避免两投在同一个班级，你可以简单地从Authorise返回例外：

public void AuthoriseFeaturePhone(...) 
{ 
    Exception ex = Authorise(...) 
    if (ex != null) 
     throw new CustomException("FeaturePhone Message", ex); 
} 

private Exception Authorise(...) 
{ 
    if(condition1)     
     return new CustomException("Condition 1 Failed"); 
    if(condition2)     
     return new CustomException("Condition 2 Failed"); 
    if(condition3)     
     return new CustomException("Condition 3 Failed"); 
} 

但是请注意，内部异常不会有StackTrace。这也使得创建一个安全的bool TryAuthoriseFeaturePhone(...)方法非常容易，该方法不会抛出异常，但如果手机无法授权，只需返回false。该弹簧想到