java 将阿拉伯数字转换为罗马数字

Question

我正在为我拥有的 java 类编写这个程序。该程序的要点是将阿拉伯数字输入转换为罗马数字。

这是我的代码:

import java.util.Scanner;

public class RomanDigit{
public static void main(String arg []){
    System.out.println("Please enter a number to be converted to roman numerals: ");
    Scanner scan = new Scanner (System.in);
    int number = scan.nextInt();
    String romanDigit = "";
    if (number < 0 || number > 3999) {
        System.out.println( "This number cannot be converted");
    }
    else
    {
        System.out.println("The number you input, converted into roman numerals is: " + romanDigit);
        System.out.println(convert(number));
    }
}



    public static String romanDigit(int n, String one, String five, String ten){

    if(n >= 1)
    {
        if(n == 1)
        {
            return one;
        }
        else if (n == 2)
        {
            return one + one;
        }
        else if (n == 3)
        {
            return one + one + one;
        }
        else if (n==4)
        {
            return one + five;
        }
        else if (n == 5)
        {
            return five;
        }
        else if (n == 6)
        {
            return five + one;
        }
        else if (n == 7)
        {
            return five + one + one;
        }
        else if (n == 8)
        {
            return five + one + one + one;
        }
        else if (n == 9)
        {
            return one + ten;
        }

    }
    return "";
}

public static String convert(int number){

    String romanOnes = romanDigit( number%10, "I", "V", "X");
    number /=10;
    String romanTens = romanDigit( number%10, "X", "L", "C");
    number /=10;
    String romanHundreds = romanDigit(number%10, "C", "D", "M");
    number /=10;
    String romanThousands = romanDigit(number%10, "M", "", "");

    String result = romanThousands + romanHundreds + romanTens + romanOnes;
    return result;

}
}

我现在根据给定的建议编辑了代码。我注意到我漏掉了数字/= 10;最后声明。由于某种原因我之前删除了它并忘记将其放回去。现在似乎工作正常。除了 number/=10 语句之外它不起作用的原因是我将 If 语句放在了错误的位置。

Answer 1

您应该将 convert 方法更改为如下所示。

public static String convert(int number) {
    if (number < 0 || number > 3999) {
        return "This number cannot be converted";
    }

    String romanOnes = romanDigit(number % 10, "I", "V", "X");
    number /= 10;

    String romanTens = romanDigit(number % 10, "X", "L", "C");
    number /= 10;

    String romanHundreds = romanDigit(number % 10, "C", "D", "M");
    number /= 10;

    String romanThousands = romanDigit(number % 10, "M", "", "");
    number /= 10;

    String result = romanThousands + romanHundreds + romanTens + romanOnes;
    return result;
}

此外，您应该从 romanDigit 方法中删除对大于零且小于 4000 的数字的检查。

正如 @eis 所注意到的，检查是否数字对于转换有效，您应该在 convert 方法之外进行检查。在本例中，在 main 方法中。