我应该扩展ArrayList以添加非空的属性吗？

Question

我想添加一个对象的集合到一个arrayList，只有当这个特定的属性不是null的时候。

我在考虑扩展ArrayList并在子类中执行检查。

另一种方法是在将属性放入Arraylist之前检查属性，但这意味着，如果需要将对象添加到基于逻辑的阵列列表中，我将不得不分散if检查每个位置。

我想知道你的想法...第二个想法是矫枉过正？

Answer 1

Decorator模式

我真的建议使用证据充分的Decorator图案包裹ArrayList。你只需换你ArrayList与另一List实现，它代表大多数的方法，但增加了验证逻辑：

public class ValidatingListDecorator extends AbstractList<MyBusinessObject> 
{ 

    private final List<MyBusinessObject> target; 

    public ValidatingListDecorator(List<MyBusinessObject> target) { 
     this.target = target; 
    } 

    @Override 
    public MyBusinessObject set(int index, MyBusinessObject element) 
    { 
     validate(element); 
     return target.set(index, element); 
    } 

    @Override 
    public boolean add(MyBusinessObject o) 
    { 
     validate(o); 
     return target.add(o); 
    } 

    //few more to implement 

} 

优点：

• 您仍然可以访问未经验证的原始列表，如果你想要的（但你可以限制这个）
• 更容易堆叠不同的验证，有选择地打开和关闭它们。
• 促进composition over inheritance由@helios
• 注意提高可测试性
• 不配合你到一个特定的List实现，可以添加验证LinkedList或Hibernate -backed持续名单。你甚至可以考虑通用Collection修饰器来验证任何集合。

实现说明

尽管实施记得有相当多的方法，你必须要记住，同时覆盖：（？）add()，addAll()，set()，subList()等

而且你对象必须是不可变的，否则用户可以添加/设置有效的对象并修改它以后违反合同。

良好的面向对象设计

Finaly我写道：

validate(element) 

但考虑：

element.validate() 

这是一个更好的设计。

堆叠验证

如前所述之前，如果你想堆验证，在一个单一的，独立的类验证每个proprty/apsect，考虑下面的语句：

public abstract class ValidatingListDecorator extends AbstractList<MyBusinessObject> 
{ 

    private final List<MyBusinessObject> target; 

    public ValidatingListDecorator(List<MyBusinessObject> target) { 
     this.target = target; 
    } 

    @Override 
    public MyBusinessObject set(int index, MyBusinessObject element) 
    { 
     validate(element); 
     return target.set(index, element); 
    } 

    protected abstract void validate(MyBusinessObject element); 

} 

...和少数实现：

class FooValidatingDecorator extends ValidatingListDecorator { 

    public FooValidatingDecorator(List<MyBusinessObject> target) 
    { 
     super(target); 
    } 

    @Override 
    protected void validate(MyBusinessObject element) 
    { 
     //throw if "foo" not met 
    } 
} 

class BarValidatingDecorator extends ValidatingListDecorator { 

    public BarValidatingDecorator(List<MyBusinessObject> target) 
    { 
     super(target); 
    } 

    @Override 
    protected void validate(MyBusinessObject element) 
    { 
     //throw if "bar" not met 
    } 
} 

想，只确认富？

List<MyBusinessObject> list = new FooValidatingDecorator(rawArrayList); 

想验证这两个富和酒吧？

List<MyBusinessObject> list = 
    new BarValidatingDecorator(new FooValidatingDecorator(rawArrayList)); 

Answer 2

如果您想强制执行此操作，我不明白为什么不这样做（尽管您应该在添加以确保成功时检查add方法的返回值）。

这是摆脱冗余逻辑的一种好方法，它可能会或可能不会在后面的软件迭代中出现。

Answer 3

唯一的问题是如果你去重用这段代码，你不记得你已经覆盖了ArrayList类，请确保彻底地评论。

Answer 4

我不认为这是一个很好的做法。请考虑在具有两个参数的Util-Class中编写一个Util-Method：您想要添加的数组列表和对象。在那里你可以检查你想要的任何东西，并且可以在你的代码中重复使用逻辑。