我应该扩展 ArrayList 以添加不为 null 的属性吗？

Question

我想将对象集合添加到 arrayList ，前提是特定属性不为空。

我正在考虑扩展 ArrayList 并在子类中实现检查。

一种替代方法是在将属性放入 Arraylist 之前检查该属性，但这意味着，如果我需要根据逻辑将对象添加到 arraylist，我将不得不将 if 检查分散在每个位置。

我想知道你对此的想法……再想一想，这是不是太过分了？

Answer 1

装饰器模式

我实际上建议使用记录良好的装饰器来包装ArrayList em> 模式。您只需用另一个 List 实现包装您的 ArrayList，该实现委托大多数方法，但添加验证逻辑：

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
    
}

优点：

• 如果需要，您仍然可以访问原始列表而无需验证（但您可以限制这一点）
• 更容易堆叠不同的验证，有选择地打开和关闭它们。
• 促进组合优于继承，如@helios所述
• 提高可测试性
• 不捆绑如果您要使用特定的 List 实现，您可以向 LinkedList 或 Hibernate 支持的持久列表。您甚至可以考虑使用通用 Collection 装饰器来验证任何集合。

实现注意事项

尽管实现了，但请记住，在重写时您必须记住很多方法：add()、addAll()、set()< /code>、subList() (?) 等。

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

良好的 OO 设计

最后我写道：

validate(element)

但考虑一下：

element.validate()

这是一个更好的设计。

堆叠验证

如前所述，如果您想要堆叠验证，在单个单独的类中验证每个属性/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
    }
}

只想验证 foo？

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

想要验证 foo 和 bar 吗？

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

Answer 2

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

这是消除冗余逻辑的好方法，这些逻辑在以后的软件迭代中可能会存在，也可能不会存在。

Answer 3

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

Answer 4

唯一的问题是，如果您要重用此代码并且您不记得您已经重写了 ArrayList 类，请确保彻底注释。