Question

下面，我试图编写一个程序来将罗马数字转换为十进制值，但我似乎无法弄清楚为什么输入测试值 MCMLXXVIII 时会打印出 1088 ，而不是 1978 。如果有人能告诉我我在哪里出错了，将不胜感激。

import java.util.Scanner;
public class RomanNumerals {

public static void main(String[] args) {
    Scanner console =  new Scanner(System.in);
    int total = 0, strLength;

    String romanNum;
    System.out.print("Enter roman numeral to convert: ");

    romanNum = console.next();
    total = convertNum(romanNum);

    System.out.println("The roman numeral converted is: " + total);
    console.close();
}

public static int convertNum(String romanNum) {
    int total = 0;
    while (!romanNum.isEmpty())  {
        if ((romanNum.length() == 1) || valueOf(romanNum.charAt(0)) >= 
        valueOf(romanNum.charAt(1))) {
            total += valueOf(romanNum.charAt(0));
            romanNum = romanNum.substring(1);
        } else {
            total += (romanNum.charAt(1)) - (romanNum.charAt(0));
            romanNum = romanNum.substring(2);
        }
    }
    return total;
}

/**
 * Gives the value of the Roman numeral
 *   
 * @param numeral a single Roman numeral
 * @return the decimal value of numeral
 */
public static int valueOf(char numeral) {
    if (numeral == 'I') {
        return 1;
    }
    if (numeral == 'V') {
        return 5;
    }
    if (numeral == 'X') {
        return 10;
    }
    if (numeral == 'L') {
        return 50;
    }
    if (numeral == 'C') {
        return 100;
    }
    if (numeral == 'D') {
        return 500;
    }
    // must be an M
    return 1000;
    }
}

我肯定有使用数组或其他方法执行此操作的简便方法，但是对于当前正在使用的类，我们不允许使用数组，而必须使用诸如方法和循环之类的基本Java工具，但不是数组，因为我们还没有得到它们。

Answer 1

您忘记在valueOf()分支中呼叫else：

...

} else {
    total += valueOf(romanNum.charAt(1)) - valueOf(romanNum.charAt(0));
    romanNum = romanNum.substring(2);
}

将罗马数字转换为十进制值错误

1 个答案: