konwerter liczb arabskich na rzymskie

pytanie zadane 4 lipca 2018 w Java przez periedynek Obywatel (1,320 p.)

Hej, to mój drugi już temat odnośnie tego tematu.

Piszę teraz zamianę liczb arabskich an rzymskie.

Napisałem coś takiego, ale domyślam się, że to boli w oczy i jest zahardkodowane, wiecie jak to może usprawnić?

 public static void main(String[] args) {

        int ArabicNumber = 1131;

        int tysiac = ArabicNumber / 1000;
        int setki = ArabicNumber / 100 % 10;
        int dziesiatki = ArabicNumber / 10 % 10;
        int jednosci = ArabicNumber % 10;

        StringBuilder RomanString = new StringBuilder();

        String RomanThousand = "M";
        String RomanHundred = "C";
        String RomanTen = "X";
        String RomanOne = "I";

        for (int i = 0; i < tysiac; i++) {
            RomanString.append(RomanThousand);
        }
        for (int i = 0; i < setki; i++) {
            if (setki > 3) {
                if (setki == 4) {
                    RomanString.append("CD");
                    break;
                }

                if (setki == 5) {
                    RomanString.append("D");
                    break;
                }
                if (setki == 6) {
                    RomanString.append("DC");
                    break;
                }
                if (setki == 7) {
                    RomanString.append("DCC");
                    break;
                }
                if (setki == 8) {
                    RomanString.append("DCCC");
                    break;
                }

                if (setki == 9) {
                    RomanString.append("CM");
                    break;
                }
            }
            RomanString.append(RomanHundred);
        }
        for (int i = 0; i < dziesiatki; i++) {
            if (dziesiatki > 3) {
                if (dziesiatki == 4) {
                    RomanString.append("XL");
                    break;
                }
                if (dziesiatki == 5) {
                    RomanString.append("L");
                    break;
                }
                if (dziesiatki == 6) {
                    RomanString.append("LX");
                    break;
                }
                if (dziesiatki == 7) {
                    RomanString.append("LXX");
                    break;
                }
                if (dziesiatki == 8) {
                    RomanString.append("LXXX");
                    break;
                }
                if (dziesiatki == 9) {
                    RomanString.append("XC");
                    break;
                }
            }
            RomanString.append(RomanTen);
        }
        for (int i = 0; i < jednosci; i++) {
            if (jednosci > 3) {
                if (jednosci == 4) {
                    RomanString.append("IV");
                    break;
                }
                if (jednosci == 5) {
                    RomanString.append("V");
                    break;
                }
                if (jednosci == 6) {
                    RomanString.append("VI");
                    break;
                }
                if (jednosci == 7) {
                    RomanString.append("VII");
                    break;
                }
                if (jednosci == 8) {
                    RomanString.append("VIII");
                    break;
                }
                if (jednosci == 9) {
                    RomanString.append("IX");
                    break;
                }
            }
            RomanString.append(RomanOne);
        }

        System.out.println(RomanString);


    }

komentarz 4 lipca 2018 przez Mariusz08 Maniak (62,300 p.)

String RomanThousand = "M";
       String RomanHundred = "C";
       String RomanTen = "X";
       String RomanOne = "I";

Nie widzę RomanFive

komentarz 4 lipca 2018 przez periedynek Obywatel (1,320 p.)
edycja 4 lipca 2018 przez periedynek

Bo u mnie to nie jest potrzebne. Popatrz jak działają u mnie pętle.

1 odpowiedź

odpowiedź 4 lipca 2018 przez Mateusz51 Nałogowiec (28,180 p.)
wybrane 4 lipca 2018 przez periedynek

Najlepsza

Jak już używasz takiej ilości ifów i nawet z break to zamień to na switch case

Pokaż 6 poprzednich komentarzy

komentarz 4 lipca 2018 przez periedynek Obywatel (1,320 p.)

A okej, o to Ci chodziło?

if (tens > 3) {
            switch (tens) {
                case 4: {
                    RomanString.append("XL");
                    break;
                }
                case 5: {
                    RomanString.append("L");
                    break;
                }
                case 6: {
                    RomanString.append("LX");
                    break;
                }
                case 7: {
                    RomanString.append("LXX");
                    break;
                }
                case 8: {
                    RomanString.append("LXXX");
                    break;
                }
                case 9: {
                    RomanString.append("XC");
                    break;
                }
                default: {
                    for (int i = 0; i < tens; i++) {
                        RomanString.append(RomanTen);
                    }
                }
            }
        }

1	komentarz 4 lipca 2018 przez Mateusz51 Nałogowiec (28,180 p.) tak i zastanów się czy ten if na początku jest Ci potrzebny?

komentarz 4 lipca 2018 przez periedynek Obywatel (1,320 p.)

tez prawda.

Dzieki wielkie.

Tylko tyle da się zmienić w moim kodzie?

komentarz 4 lipca 2018 przez Mateusz51 Nałogowiec (28,180 p.)

według konwencji javowych nie powinno nazywac sie zmiennych z wielkich liter.

Ale generalnie jak pozbyłeś się zagnieżdżonych ifów to jest okej. Nie ma co walczyc o perfekcyjny kawałek kodu gdy masz jeszcze wiele rzeczy do nauczenia

ewentualnie porozdzielaj to na funkcje