从多种策略

Question

删除重复的代码，最有效的方法我们有3种在我们的项目属性：的CategoryAttribute，ProductAttribute和ProductTypeAttribute。这些不在我们的控制范围之内，因为它们来自自动生成的类，并且可能包含不同类型的属性值，例如text，number or image。现在，每个属性都有自己的策略来检索attributeValue。为简单起见，我们假设他们全部3个都有TextStrategy,NumberStrategy和ImageStrategy。

实施例的策略：

@Component 
public class CategoryImageAttributeStrategy implements CategoryAttributeStrategy { 

    @Override 
    public boolean isApplicable(CategoryAttribute attribute) { 
     return attribute.getImage() != null; 
    } 

    @Override 
    public Object getAttributeValue(CategoryAttribute attribute) { 
     //return attribute value here 
     //may be different or may be the same 
     //for ProductImageAttributeStrategy and ProductTypeImageAttributeStrategy 
    } 

}

尽管取得图像值可以是用于所有这些不同的，获取文本值是相同的，我们最终3类的几乎相同的代码和我真的真的不喜欢复制代码。

我想过为每个策略类型创建一个抽象类/默认界面，例如DefaultTextStrategy所有3种文本策略都会继承，并使用提供的默认代码或使用自己的实现覆盖它，但是我对此方法并不满意，因为它需要为如此简单的任务创建更多的类。

也许是有可能将相同类型的策略（如图像）合并为一个？

我真的很想听听更多有经验的人在这个问题上有什么要说的，因为我想学习和改进。

提前感谢您的时间。

Answer 1

这里就是我所做的：

首先，我创建了一个接口名为“AttributeValueStrategy”所有类型的策略。然后添加3个回调（类型特定，例如NumberValueCallback等）。现在，每个策略都实现了其类型的回调接口和AttributeValueStrategy接口。然后有DefaultStrategyMethods类包含每个类型的默认“getAtrribute”，实际的策略调用defaultStrategyMethods（如下所示）或只是实现自己的代码。

@Override 
public Object getAttributeValue(Object attribute) { 
    return defaultStrategyMethods.getNumberValue(attribute, this); 
} 

回调创建，因为只有实际的战略，知道应该将它转换为哪一类（并有这样做的方法），并DefaultStrategyMethods需要所以这就是为什么我传递“这个”作为第二个参数（使用它这是回调本身）。

没有更多的重复，一切都清晰干净。

Answer 2

应该只有3个策略。 TextStrategy，NumberStrategy和ImageStrategy，它们扩展了基本策略。将属性和策略混合在一起会让人感到困惑，因为两者实际上都是独立的，并且彼此之间有很多关系。

让3个属性扩展一个Attribute类：CategoryAttribute，ProductAttribute和ProductTypeAttribute。

让策略根据传递给它的Attribute类对象决定需要做什么。对于文本策略，将会有单个实现。对于Image策略，您可能需要针对一个课程进行特殊处理。