Java中如何纠正Arraylist索引越界长度错误

Question

我正在为学校写一个充当收银机的程序。我要求输入项目的价格，并将它们播放到一个正在进行的ArrayList中，直到用户输入-1或0。0是在错误的情况下重新输入以前的价格，-1终止循环。

我得到了

我在Eclipse中运行Java11。

在不使用该方法的情况下，代码运行良好，因为我减少了计数器，并且在循环的下一次迭代中，无论是否删除了以前的数组位置，都将覆盖它。方法本身被设置为删除ArrayList.size()-1，以便删除最后一个条目。我已经用-2和0试过了，但它仍然出界。

我通读了以前的问题，许多人没有填充数组。因此，我运行了一个打印存根，以确保ArrayList已经正确填充，并且它已经填充：当两个项被放置到ArrayList中时，大小等于2。错误代码也上升更多的项目，我放入代码，但总是项目-1索引超出边界的项目-1长度，我肯定我犯了一个菜鸟错误，但我找不到它，它是让我疯了！

对于完整的错误上下文：

线程“main”java.lang.indexoutOfboundsException：在java.base/jdk.internal.util.preconditions.outofbounds(preconditions.java:64)，在java.base/jdk.internal.util.preconditions.outofboundscheckindex（preconditions.java:70)，在java.base/jdk.internal.util.preconditions.checkindex（preconditions.java:248)，在(c_m_ideaproject.java:76)

// declare our array list which will hold our prices!
ArrayList<Double> prices = new ArrayList<Double>();


// declaring variables to terminate loop, count prices, total prices, and current entry for conditions
        int counter  = 0;
        double entry = 0;
        double total = 0;

// our loop to continuously add prices to the array list from input
while (entry != -1) {

    System.out.println("Enter a price for item #" + (counter+1) + ": ");
    entry = myInput.nextDouble();

// if the entry is a price we will add it to prices and continue with the loop

    if (entry != 0 && entry != -1) {
            prices.add(entry);
            total += entry;
            counter++;
    }

//if the entry is 0 we will revert back to the previous iteration to re-add

    else if (entry == 0.0) {
        total -= prices.get(counter-1);
        removeLastEntry(prices);
        counter--;
    }

public static void removeLastEntry(ArrayList<Double> anArrayList) {
    anArrayList.remove(anArrayList.size()-1);
}

Answer 1

您可以添加一个检查以查看列表是否为空：

// declare our array list which will hold our prices!
ArrayList<Double> prices = new ArrayList<Double>();


// declaring variables to terminate loop, count prices, total prices, and current entry for conditions
        int counter  = 0;
        double entry = 0;
        double total = 0;

// our loop to continuously add prices to the array list from input
while (entry != -1) {

    System.out.println("Enter a price for item #" + (counter+1) + ": ");
    entry = myInput.nextDouble();

// if the entry is a price we will add it to prices and continue with the loop

    if (entry != 0 && entry != -1) {
            prices.add(entry);
            total += entry;
            counter++;
    }

//if the entry is 0 we will revert back to the previous iteration to re-add

    else if (entry == 0.0) {
        total -= prices.get(counter-1);
        removeLastEntry(prices);
        counter--;
    }

public static void removeLastEntry(ArrayList<Double> anArrayList) {
    if(!anArrayList.isEmpty()) {
        anArrayList.remove(anArrayList.size()-1);
    }
}