Question

for(i=0;i<in.length;i++)
    {
        for(j=i+1;j<in.length;j++)
        {
            //System.out.println("for"+in[i].val);
            //System.out.println("for "+in[j].val);
            if(comp(in[i].bin,in[j].bin,n)!=null)
            {
                String bi=comp(in[i].bin,in[j].bin,n);
                //System.out.println("is it null "+co[k]);
                coin.a(in[i],in[j],bi);//taken input
                co.add(coin);//now passing to array list HERE LIES THE PROBLEM!!!
                System.out.println(in[i].val+","+in[j].val+"="+coin.bin);
                in[i].combo=true;
                in[j].combo=true;
                k++;
            }   

        }
    }

55f96302硬币55f96302的地址 55f96302硬币地址@ 55f96302 55f96302硬币地址@ 55f96302 55f96302硬币地址@ 55f96302 55f96302硬币地址@ 55f96302 55f96302硬币地址@ 55f96302 55f96302硬币地址@ 55f96302 55f96302硬币地址@ 55f96302 55f96302硬币地址@ 55f96302

正如您所看到的，列表中所有元素的引用与＆＃34; coin＆＃34;相同。我该怎么做才能解决这个问题？

Answer 1

我猜你在一对几个类的集合上有一个引用构建没有任何问题。这是处理java中元素存储的正确方法。

Answer 2

如果我正确理解了这个问题，那么您的问题是您不希望列表中存储的项目与变量coin中存储的项目相同。在这种情况下，您应该创建存储在变量coin中的对象的精确副本，并将副本放入列表中。

事实上，在传递可变对象时应始终小心，例如将它们放入列表中。否则，您的程序可能变得难以调试。在Java中搜索防御性副本。

我可以想到两种创建防御性副本的方法：克隆和复制构造函数。但请注意。 implementing Cloneable is not trivial。

另外，我建议您考虑是否以及如何使您的班级couple不可变。在大多数情况下，具有不可变类（即，不必担心防御性复制）的优点超过了不利因素（即，创建新对象，导致存储器和CPU周期中的成本较高）。

TL; DR：未将相同引用传递给arraylist的唯一原因是该项目可能被有权访问它的每个人修改。创建一个防御性副本或使您的对象不可变。

编辑：更新完示例后，我会看到您的问题（至少我认为是这样）。我尝试将您的示例重建为最小的工作示例。你基本上拥有的是：

public class Coin {

    int value;

    public Coin(int value) {
        this.value = value;
    }

    public void setValue(int value) {
        this.value = value;
    }

    public static void main(String[] args) {
        Coin myCoin = new Coin(0);
        List<Coin> bankAccount = new ArrayList<Coin>();
        for (int coinValue = 0; coinValue < 5; coinValue ++) {
            myCoin.setValue(coinValue);
            bankAccount.add(myCoin);
        }
        for(Coin coin : bankAccount)
            System.out.println(coin);
    }

    @Override
    public String toString() {
        return "Coin: " + value;
    }

}

如果您运行此代码，则列表中的所有硬币都将具有值4.原因是，方法setValue（对应于您的方法a）会改变{{1}实例。解决此问题的最佳方法（如果可能）是使Coin不可变并在调用Coin时返回新的Coin实例。代码如下所示：

setValue

尝试运行此代码。您现在不会在列表中多次插入相同的对象。

但是，如果您的方法public class ImmutableCoin { final int value; public ImmutableCoin(int value) { this.value = value; } public ImmutableCoin setValue(int value) { return new ImmutableCoin(value); } public static void main(String[] args) { ImmutableCoin myCoin = new ImmutableCoin(0); List<ImmutableCoin> bankAccount = new ArrayList<ImmutableCoin>(); for (int coinValue = 0; coinValue < 5; coinValue ++) { myCoin = myCoin.setValue(coinValue); bankAccount.add(myCoin); } for(ImmutableCoin coin : bankAccount) System.out.println(coin); } @Override public String toString() { return "ImmutableCoin: " + value; } } 必须修改对象并且不允许返回新对象，那么您必须自己在循环中复制该对象，修改此复制，然后插入副本。代码看起来像这样：

这也有效。我希望这有帮助。

如何将数据输入到用户定义类的arraylist？

2 个答案: