java - 克隆与仅仅赋值有何不同(为什么要费心实现 Cloneable 接口(interface))？

Question

我正在尝试创建一个类的深拷贝。这是我尝试复制的类，最后一个方法是复制方法:

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

public class catalog implements Cloneable{

    List<product> cat = new ArrayList<>();
    int counter;
    final int capacity = 2;

    public void addProduct(product x){
        cat.add(x);
    }
    public void addProduct(String name, int id, int price, int stock) throws Exception{
        for(int i = 0; i < cat.size(); i++){
                product temp = cat.get(i);
                if(temp.name == name){
                    temp.stock = temp.stock + stock;
                    return;
                }
                if(temp.id == id){
                    System.out.println("Product with this ID already exists");
                    return;
                }
        }
        if(cat.size() >= capacity){
            //throw new customException("Not enough capacity");
            throw new customException();
        }
        product temp = new product();
        temp.name = name;
        temp.id = id;
        temp.price = price;
        temp.stock = stock;
        cat.add(temp);
        }
    public void showSelection(){
        for(product temp : cat){
            System.out.println("Product " + temp.name + " ID: " + temp.id
                    + " Price: " + temp.price + " In stock: " + temp.stock);
        }
    }
    public void removeFromStock(String name){
        for(int i = 0; i < cat.size(); i++){
            product temp = cat.get(i);
            if(temp.name == name){
                temp.stock = temp.stock - 1;
                cat.set(i, temp);
            }
        }
    }

    public Object clone(){
        catalog b = new catalog();
        for(int i = 0; i <= cat.size(); i++){
            product temp, temp1;
            temp1 = cat.get(i);
            temp = (product)temp1.clone();
            b.addProduct(temp);
        }
        return b;
    }
}

这是主要内容:

public class testMoney {

    public static void main(String[] args) {

        automatasklase test = new automatasklase();
        test.addMoney(50);
        System.out.println(test.getMoney());
        product cola = new product();
        cola.id = 1;
        cola.price = 2;
        cola.name = "Cola";
        productsExpire Milk = new productsExpire();
        Milk.name = "Milk";
        Milk.id = 2;
        Milk.price = 10;
        Milk.stock = 10;
        Milk.isExpired = true;
        catalog menu = new catalog();
        menu.addProduct(cola);
        menu.addProduct(Milk);
        menu.showSelection();
        try{
        menu.addProduct("Pepsi" ,3, 5, 108);
        }catch(Exception e){
            System.out.println("Exception occured = " +e);
        }
        try {
            menu.addProduct("Cola", 1, 3, 7);
        } catch (Exception e) {
            System.out.println("Exception occured = " + e.getMessage());
        }
        menu.showSelection();
        test.buyProduct(menu, 2);
        System.out.println(test.getMoney());
        menu.showSelection();
        Milk.isAvailable();
        product copyy = (product)cola.clone();
        System.out.println("-------------");
        System.out.println(cola.id + cola.name + cola.price + cola.stock);
        System.out.println(copyy.id + copyy.name + copyy.price + copyy.stock);

        // BELOW THE PROBLEM OCCURS!!!

        catalog newtest = (catalog) menu.clone();

        // ABOVE THE IS THE PROBLEM!!!!

    }

}

问题如下:

Exception in thread "main" java.lang.IndexOutOfBoundsException: Index: 2, Size: 2
    at java.util.ArrayList.rangeCheck(Unknown Source)
    at java.util.ArrayList.get(Unknown Source)
    at AutomatasA.catalog.clone(catalog.java:58)
    at AutomatasA.testMoney.main(testMoney.java:46)

您会注意到，我已经在目录本身的克隆方法中克隆了产品类对象，但是您可能会注意到主类已经对其进行了测试并且它可以工作，因此类产品的克隆成功了。

你能看出问题所在吗？我知道还会有很多编码礼仪的小错误，也许还有更简单的做事方法，但我只是一个初学者。我必须实现深度复制，它必须有意义，所以克隆产品类是不够的，我想创建一种克隆整个目录的方法，所以如果机器要更换为更新的机器，你就可以使用该方法。

顺便说一下，整个项目正在为虚拟小吃机创建一个代码，就像你输入钱，按你想要的项目的数字，然后得到它的代码一样。

感谢您的帮助!

Answer 1

您的异常是由这一行引起的:

for(int i = 0; i <= cat.size(); i++){

应该是

for(int i = 0; i < cat.size(); i++){

但是你真的不应该像那样实现你自己的深度克隆。如果您想要一个副本，请为其创建一个单独的方法，或一个复制构造函数。