3
import java.util.Scanner;


public class SumDigits {

public static void main(String[] args)
{
    // TODO Auto-generated method stub

    Scanner input = new Scanner(System.in);

    // prompt the user to enter a value and store it as the integer called number
    System.out.print("Enter an integer: ");
    double number = Math.abs(input.nextDouble());

    System.out.println("The sum of the digits is " + sumNumbers(number));

    input.close();
}
public static int sumNumbers (double number)
{

    int sum = 0;

    for (int i = 10, digit = 0; (number * 10) /i !=0; i *= 10, digit = (int)((number % i) - digit)/(i / 10))
    {
        sum += digit;
    }

    return sum;
}
}

在运行时,我收到错误消息

线程“主”java.lang.ArithmeticException 中的异常:/ 为零

参考第 25 行(我的 for 循环条件)。

在我尝试将数字的值类型转换为 int 之前,循环运行良好,我不确定为什么这会导致循环的任何部分将某个值除以零。我已经研究了有关使用有理表达式的条件的所有可能性,并且无法推断出任何分母都将设置为零的偶然性。无论输入什么数字,我都会收到此错误。如果不是因为我的教授提供的数字的值超过了他的一个测试用例中可以存储在 int 中的数字,我根本不会选择将数字保存为双精度数。该程序在类型转换之前运行良好,并为所有其他测试用例提供了正确答案。

4

3 回答 3

1

你在这里做

(number * 10) /i !=0

其中数字是双倍的。

不建议将浮点数用于比较,因为它们的存储方式(尾数和指数形式)。因此,如果此条件返回 true,请认为自己很幸运。

因此,您的循环有点永无止境。你得到算术异常的原因是你在这种无限循环中将 i 乘以 10。“i”达到 32 位的“整数”限制并溢出并最终使所有这些 32 位为 0。

实际上,i=0 并且以下两个 throw 都被零除异常

(number * 10) /i 

(number % i) - digit)/(i / 10)
于 2015-09-23T04:53:23.500 回答
0

ArithmeticException 的根本原因是不正确的循环条件。在循环中添加System.out.println("i=" + i + " i*10=" + (i * 10));会产生输出:

i=10 i*10=100
i=100 i*10=1000
i=1000 i*10=10000
i=10000 i*10=100000
i=100000 i*10=1000000
i=1000000 i*10=10000000
i=10000000 i*10=100000000
i=100000000 i*10=1000000000
i=1000000000 i*10=1410065408
i=1410065408 i*10=1215752192
i=1215752192 i*10=-727379968
i=-727379968 i*10=1316134912
i=1316134912 i*10=276447232
i=276447232 i*10=-1530494976
i=-1530494976 i*10=1874919424
i=1874919424 i*10=1569325056
i=1569325056 i*10=-1486618624
i=-1486618624 i*10=-1981284352
i=-1981284352 i*10=1661992960
i=1661992960 i*10=-559939584
i=-559939584 i*10=-1304428544
i=-1304428544 i*10=-159383552
i=-159383552 i*10=-1593835520
i=-1593835520 i*10=1241513984
i=1241513984 i*10=-469762048
i=-469762048 i*10=-402653184
i=-402653184 i*10=268435456
i=268435456 i*10=-1610612736
i=-1610612736 i*10=1073741824
i=1073741824 i*10=-2147483648
i=-2147483648 i*10=0
于 2015-09-23T04:48:25.017 回答
0

第一次控制进入循环时,值digit将为零,而不是您所期望的输入数字的最后一位。此外,您需要修复循环终止条件,因为这是错误的。例如,如果数字是 123,那么最后一次预期的迭代1230/1000 = 1而不是0循环不会结束,而是导致溢出。您可以使用以下内容:

public static int sumNumbers (double number)
    {
        int sum = 0;
        for (int i = 10, digit = (int)(number % i); digit != 0; i *= 10, digit = (int)((number % i) - digit)/(i / 10))
        {
            sum += digit;
        }

        return sum;
    }

这应该可行,但我再次建议您改进此代码,因为这不是理想的方式。

测试:

Enter an integer: 3456 The sum of the digits is 18

于 2015-09-23T04:37:13.487 回答