konwerter liczb arabskich na rzymskie

Question

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);


    }

 

Comments

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

Nie widzę RomanFive

---

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

Answer 1

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

Comments

takie coś zrobiłem, ale nie działa nie wiedzieć czemu:

public static void main(String[] args) {

        int ArabicNumber = 990;

        int thousand = ArabicNumber / 1000;
        int hundred = ArabicNumber / 100 % 10;
        int tens = ArabicNumber / 10 % 10;
        int units = ArabicNumber % 10;

        StringBuilder RomanString = new StringBuilder();

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

        for (int i = 0; i < thousand; i++) {
            RomanString.append(RomanThousand);
        }

        for (int i = 0; i < hundred; i++) {
            if (hundred > 3) {
                switch (hundred) {
                    case 4: {
                        RomanString.append("CD");
                        break;
                    }
                    case 5: {
                        RomanString.append("D");
                        break;
                    }
                    case 6: {
                        RomanString.append("DC");
                        break;
                    }
                    case 7: {
                        RomanString.append("DCC");
                        break;
                    }
                    case 8: {
                        RomanString.append("DCCC");
                        break;
                    }
                    case 9: {
                        RomanString.append("CM");
                        break;
                    }
                }
            }
            RomanString.append(RomanHundred);
        }

        for (int i = 0; i < tens; i++) {
            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;
                    }
                }
            }
            RomanString.append(RomanTen);
        }

        for (int i = 0; i < units; i++) {
            if (units > 3) {
                switch (units) {
                    case 4: {
                        RomanString.append("IV");
                        break;
                    }
                    case 5: {
                        RomanString.append("V");
                        break;
                    }
                    case 6: {
                        RomanString.append("VI");
                        break;
                    }
                    case 7: {
                        RomanString.append("VII");
                        break;
                    }
                    case 8: {
                        RomanString.append("VIII");
                        break;
                    }
                    case 9: {
                        RomanString.append("IX");
                        break;
                    }
                }
            }
            RomanString.append(RomanOne);
        }
        System.out.println(RomanString);
    }

 

---

case nie musi mieć nawiasów

---

przejrzyście to wygląda wtedy, ale to chyba nie jest przyczyna?

---

nie wiem co Ci nie działa

---

Już ogarnąłem, bo przedtem przy ifach break mi wychodził z całej pętli for, a tereaz przy case break wychodzi z samego switcha, wiec Twoja opcja chyba nie dziala.

---

a po co Ci for jak z niego wychodzisz?  Nie lepiej dać do switch default w którym to uzyjesz fora do dadania odpowiedniej ilosci znaków?

dzięki temu pozbędziesz się zagnieżdżenia ifów

---

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);
                    }
                }
            }
        }

 

---

tak i zastanów się czy ten if na początku jest Ci potrzebny?

---

tez prawda.

Dzieki wielkie.

Tylko tyle da się zmienić w moim kodzie?

---

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

---

Faktycznie. Dzięki! :)