使用泛型避免 instanceof

Question

我需要添加到子属性列表（ProductOption 和 ProductAttribute），这些子属性是名为 Product 的父对象的属性。这三个类都扩展了一个抽象类 CMS。

我想一般地调用方法“attachChildToParent”，但我通过推迟instanceof 并强制转换为产品来延迟不可避免的事情。

有没有办法可以通用地写这个，这样我就可以避免演员表？

测试：

package puzzler;

import java.util.ArrayList;
import java.util.List;

public class Tester {
    public static void main(String[] args) {
        Product p = new Product();
        ProductAttribute pa = new ProductAttribute();
        ProductOffering po = new ProductOffering();

        List<ProductAttribute> lpa = new ArrayList<ProductAttribute>();
        List<ProductOffering> lpo = new ArrayList<ProductOffering>();

        attachChildToParent(lpa, p);
    }

    static void attachChildToParent(List<? extends CMS> listChild, Product parent) {
        for (CMS cmsItem : listChild) {
            parent.attach(cmsItem);         
        }
    }
}

产品类（父级）

package puzzler;

import java.util.List;

abstract class CMS {
    String node;
}
public class Product extends CMS {
    List<ProductAttribute> lpa;
    List<ProductOffering> lpo;

    public List<ProductAttribute> getLpa() {
        return lpa;
    }

    public void setLpa(List<ProductAttribute> lpa) {
        this.lpa = lpa;
    }

    public List<ProductOffering> getLpo() {
        return lpo;
    }

    public void setLpo(List<ProductOffering> lpo) {
        this.lpo = lpo;
    }

    public void attach(ProductAttribute childNode) {
        this.getLpa().add(childNode);
    }

    public void attach(ProductOffering childNode) {
        this.getLpo().add(childNode);
    }

    //  I want to avoid this.  Defeats the purpose of generics.
    public void attach(CMS cms) {
        if (cms instanceof ProductOffering) {
            this.getLpo().add((ProductOffering) cms);
        } else         {
            if (cms instanceof ProductAttribute) {
                this.getLpa().add((ProductAttribute) cms);
            }
        }
    }
}

儿童班 1

package puzzler;

import puzzler.CMS;

public class ProductAttribute extends CMS {
    String node;

    public String getNode() {
        return node;
    }

    public void setNode(String node) {
        this.node = node;
    }
}

儿童班 2

package puzzler;

import puzzler.CMS;

public class ProductOffering extends CMS {
    String node;

    public String getNode() {
        return node;
    }

    public void setNode(String node) {
        this.node = node;
    }
}

Answer 1

您可以使用访问者模式来解决强制转换问题：在您的 CMS 类中，添加一个新的（抽象）方法 attachTo(Product parent)。在每个子类中，都可以实现这个方法调用父类的attach，正确的函数就会被解析。

Answer 2

覆盖？

public void attach(ProductOffering cms) {
   this.getLpo().add(cms);
}

public void attach(ProductAttribute cms) {
   this.getLpa().add(cms);
}

Answer 3

委托给子类：

public void attach(CMS cms) {
    cms.callAdd(this);

}

在 CMS 上添加：

public abstract void callAdd(Product product);

在 ProductOffering 上添加：

public void callAdd(Product product) {
   getLpo().add(this)       
}

ProductAttribute 类似...

Answer 4

有一些方法可以避免这种情况，但不能使用泛型。泛型允许您编写允许避免强制转换的代码，但是当您需要避免 instanceof 时它无济于事。主要原因是所有具有泛型类型的变量在内部都被视为Object。

这就是为什么当您从attachChildToParent() 调用它时编译器使用通用的attach(CMS) 方法：要附加的子项的类型将是CMS - 编译器无法保留该类型你打电话时使用attachChildToParent()。

唯一的解决办法是在children中写一个attach方法：

public class ProductAttribute extends CMS {
    @Override
    public void attachTo( Product p ) {
        p.getLpo().add( this );
    }
}

Answer 5

泛型无法解决您的问题。由于您在 <? extends CMS> 之后丢失了实例特定信息，因此 List 中的所有元素都是 CMS 的子级。

为什么不添加一个简单的方法，将两个列表作为参数，既适合您的设计，又能解决您的问题，它也很有效

public void attach(List<ProductAttribute> lpa, List<ProductOffering> lpo) {
    this.lpa.addAll(lpa);
    this.lpo.addAll(lpo);
}