【问题标题】:Throwing array out of bounds exception in for loop when array size is unknown (multiplicative persistence)当数组大小未知时,在 for 循环中抛出数组越界异常(乘法持久性)
【发布时间】:2016-11-06 15:19:17
【问题描述】:

我正在在线进行编码挑战,我应该编写一个接收正参数(“num”)并返回其乘法持久性的类。这是您必须将“num”中的数字相乘直到达到一位数的次数。

例如,39 = 3的乘法持久化。这是因为:

  1. 3 * 9 = 27
  2. 2 * 7 = 14
  3. 1 * 4 = 4

这是到目前为止的整个程序:

import java.lang.reflect.Array;
import java.util.ArrayList;
import java.util.List;

public class Persist {

    public static void main(String[] args) {

        persistence(39);
    }

    public static int persistence(long num) {

        int persistenceValue = 0;
        List<Long> digitList = new ArrayList<Long>();
        long lastDigit;

        //Resolves if num is single digit

        if (num <= 9) {

            return 0;
        }

        //Takes each digit of number and stores it to digitList (backwards)

        while (num > 0) {

            lastDigit = (num % 10);
            digitList.add(lastDigit);
            num = num / 10;
        }

        //Takes each digit in digitList and stores it in array in correct order
        for (Long d : digitList) {

            Long[] currentDigitArray = new Long[digitList.size()];

            for (int i = 0; i < currentDigitArray.length; i++) {

                currentDigitArray[currentDigitArray.length - i] = d;
            }

            persistenceValue = currentDigitArray.length;

            while (persistenceValue > 1) {

                List<Long> productList = multiplyDigits(currentDigitArray);
                persistenceValue++;
            }
        }

        return persistenceValue;
    }

    public static List multiplyDigits(Long[] currentDigitArray) {
        //multiplies each digit    

        List<Long> productList = new ArrayList<Long>();

        for (int i = 0; i < currentDigitArray.length; i++) {

            Long product = currentDigitArray[i] * currentDigitArray[i + 1];
            productList.add(product);
        }
        return productList;
    }
}

我在第 52 行的 for 循环中不断遇到数组越界异常:

//Takes each digit in digitList and stores it in an array
for (Long d : digitList) {

    Long[] currentDigitArray = new Long[digitList.size()];

    for (int i = 0; i < currentDigitArray.length; i++) {

        currentDigitArray[currentDigitArray.length - i] = d; 
        // ^ exception is thrown here ^
    }

很明显,我在 Google 上查找了这个,就像一个好的堆栈溢出用户一样。数组索引越界异常是由于程序试图访问一个 Java 异常而引发的。元素位于数组限制之外的位置,因此是“超出范围”。

问题是我不知道该数组将在前面有多大,因为这完全取决于用户传递了多少数字。我硬编码了 39,但最终我希望用户能够输入任意数量的内容。

那么我还能如何获取digitList 中的每个数字并将其存储在数组中?

这部分已经解决了,但是现在我在第78行遇到了类似的问题:

public static List multiplyDigits(Long[] currentDigitArray) {
    //multiplies each digit    

    List<Long> productList = new ArrayList<Long>();

    for (int i = 0; i < currentDigitArray.length; i++) {
        Long product = currentDigitArray[i] * currentDigitArray[i + 1];
        //^This line here
        productList.add(product);

    }
    return productList;
}

我觉得这是一个非常相似的问题,但不太知道如何解决它。

【问题讨论】:

  • 你知道数组的长度不能超过 18 位,而且你也不需要使用数组或列表,我会尽量不使用。
  • currentDigitArray[currentDigitArray.length - i -1 ] = d; 将解决问题。因为在第一次迭代时它会超过数组的限制
  • 数组中的最后一个元素是currentDigitArray[currentDigitArray.length - i - 1],因为数组从0开始。
  • multipleDigits 创建一个新的productList,您将其丢弃。
  • 所以这很奇怪。我试过currentDigitArray[currentDigitArray.length - i -1 ] = d;,现在我得到了这个:Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 2 at Persist.multiplyDigits(Persist.java:78) at Persist.persistence(Persist.java:61) at Persist.main(Persist.java:19) /home/work/.cache/netbeans/8.2/executor-snippets/run.xml:53: Java returned: 1 BUILD FAILED (total time: 0 seconds)

标签: java arrays indexoutofboundsexception


【解决方案1】:

这个任务

currentDigitArray[currentDigitArray.length - i] = d;

应该是

currentDigitArray[currentDigitArray.length - 1 - i] = d;

为了避免这个问题。

话虽如此,您可以通过执行乘法来完全避免数组。回想一下,你做乘法的顺序不会改变结果。因此,您可以从数字后面开始乘法,并得出相同的解决方案。

【讨论】:

  • 我现在在第 78 行遇到了类似的问题:Long product = currentDigitArray[i] * currentDigitArray[i + 1];
  • @DaveCat 那是因为您需要在数组结束前一位数停止循环。
【解决方案2】:

数组是零索引的,所以currentDigitArray.length 总是会越界。事实上,由于附加的身份属性,当i 为 0 时,currentDigitArray.length - i 将变为 currentDigitArray.length。要解决此问题,只需在索引计算中减去一个额外的 1:

currentDigitArray[currentDigitArray.length - i - 1] = d;

对于第二个问题,你迭代一个元素太多,你需要提前停止一个元素:

for (int i = 0; i < currentDigitArray.length - 1; i++) {

【讨论】:

  • 这并没有明确解决第二个问题(当条件为
  • 答案是在第二个问题出现之前发布的。
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2015-01-28
相关资源
最近更新 更多