在 ArrayList 中存储对对象的引用

Question

大家好，我正在尝试将当前引用存储到数组列表“pl”。 例如pl.add(this); 出于某种原因，我只得到对最后一项的引用，而没有得到以前的引用。循环确实会遍历所有三个项目。

下面是我得到的代码和输出。谁能告诉我我做错了什么谢谢你提前的帮助。

       // variables
private String productType;
private String hyperLinkParam;
private ArrayList <ProductList> pl = new ArrayList<ProductList> ();

public ProductList() {

    try {

        DocumentBuilderFactory dbf = DocumentBuilderFactory.newInstance();
        DocumentBuilder db = dbf.newDocumentBuilder();

       InputStream url = null;
        url = getClass().getResourceAsStream("inventory.xml");


        Document doc = db.parse(url);
        doc.getDocumentElement().normalize();

        // loop through each item
        NodeList items = doc.getElementsByTagName("item"); //Returns a list of elements with the given tag name item
        for (int i = 0; i < items.getLength(); i++)
        {

                Element e = (Element) items.item(i);
                setHyperLinkParam(e.getAttribute("name").toString());
                setProductType(getTextValue(e,"productType"));
                System.out.print(e.getAttribute("name").toString());
                System.out.println(getTextValue(e,"productType"));

                pl.add(this);


         }

          for(int j=0; j < pl.size(); j++){
            System.out.print("getHyperLinkParam: " + pl.get(j).getHyperLinkParam());
            System.out.println("getProductType: " + pl.get(j).getProductType());
          }

Manufacture.java

    @WebMethod(operationName = "getProductList")
public ProductList getProductList() {
    try {
        ProductList productlist = new ProductList();
        if(productlist == null){
            return null;
        }else{
            return productlist;
        }
    } catch(Exception e){
        System.out.println("error: " + e.getMessage());
        return null;
    }
}

index.jsp

    <%
try {
org.soen487.supplychain.manufacturer.Manufacture_Service service = new org.soen487.supplychain.manufacturer.Manufacture_Service();
org.soen487.supplychain.manufacturer.Manufacture port = service.getManufacturePort();
// TODO process result here
org.soen487.supplychain.manufacturer.ProductList result = port.getProductList();
out.println("Result = "+result);

} catch (Exception ex) {
// TODO handle custom exceptions here
}
%>

Answer 1

这就是问题所在：

pl.add(this);

您一次又一次地添加相同的参考 (this)。 this 的值 只是对“当前”对象的引用 - 而您正在循环中修改该对象的内容。

在循环的每次迭代中，您应该创建一个新的、单独的对象，设置 那个 对象的属性，然后将对该对象的引用添加到列表中。

老实说，您将this 添加到列表中有点奇怪 - 通常这样的方法会在某个知道如何解析 XML 的类中，而不是数据项的实例本身。不清楚pl 是在哪里声明的，但您应该真正考虑一下程序的结构。

Answer 2

您不断将相同的对象 (this) 添加到列表中。尽管您更改了成员，但列表中的所有引用仍然引用同一个对象。

您应该创建一个新对象并添加它。

查看我的其他帖子

Answer 3

我试图从这里发布的内容中猜测 ProductList 和 Product 的整体结构。问题是列表元素的列表和字段似乎位于单个类的上下文中，即 ProductList。这不行 - 需要两个类。

// stores the data coming from a single Element ("item") of the document
class Product {
    private String productType;
    private String hyperLinkParam;
    public setHyperLinkParam( String hlp ){
        hyperLinkParam = hlp;
    }
    public setProductType( String pt ){
        productType = pt;
    }
}

// Container for a list of products from an inventory
class ProductList {
    private ArrayList <Product> pl = new ArrayList<Product> ();
    public ProductList() {
        DocumentBuilderFactory dbf = DocumentBuilderFactory.newInstance();
        DocumentBuilder db = dbf.newDocumentBuilder();
        InputStream url = getClass().getResourceAsStream("inventory.xml");
        Document doc = db.parse(url);
        doc.getDocumentElement().normalize();

        // loop through each item
        NodeList items = doc.getElementsByTagName("item"); 
        for (int i = 0; i < items.getLength(); i++){
            Element e = (Element) items.item(i);
            // create the single product from the current item
            Product prod = new Product();
            prod.setHyperLinkParam( e.getAttribute("name").toString() );
            prod.setProductType( getTextValue( e, "productType") );
            // add it to the list
            pl.add( prod );
        }
    }

    void showList(){
        for( Product prod: pl ){
        System.out.print("getHyperLinkParam: " + prod.getHyperLinkParam());
            System.out.println("getProductType: " + prod.getProductType());
        }
    }
}

注意：如果 ProductList 和 Product 的构造位于具有工厂方法 makeProductList 和 makeProduct 的工厂类中，那么一切都会变得更加清晰和清晰。 ProductList 应该有一个方法 addProduct 将 add 委托给它的 pl 成员。并且关于如何从 XML 中获取产品列表的知识应该在构造函数中 there 和 not，并且类似地，获取 Product 的字段值的方式来自“item”元素的不属于 Product 或 ProductList 的代码。

Answer 4

试试

ProductList obj=new ProductList();
// some work on this object and then store it in List
pl.add(obj);

它应该可以工作，因为它会给你新鲜的对象