Software zum Umrechnen römischer Zahlen

[alicedeh]

Grüße,

ich möchte ein Programm entwickeln welches römische Zahlen in nummerische umwandelt.

Mein Code:

import java.util.Scanner;
public class Rom {
  
  public static void main(String[] args) throws InterruptedException{
    Scanner scan = new Scanner(System.in);
    
    String eingabe = "";
    int ausgabe = 0;
    
    System.out.print("Bitte geben Sie die römische Zahl ein: ");
    eingabe = scan.next();
    
    switch (eingabe) {
      case eingabe.equals("I") : 
      ausgabe = 1;
      System.out.println(ausgabe); 
      break;
      case eingabe.equals("i") : 
      ausgabe = 1;
      System.out.println(ausgabe); 
      break;
      case eingabe.equals("V") : 
      ausgabe = 5;
      System.out.println(ausgabe); 
      break;
      case eingabe.equals("v") :
      ausgabe = 5;
      System.out.println(ausgabe); 
      break;
      case eingabe.equals("X"): 
      ausgabe = 10;
      System.out.println(ausgabe); 
      break;
      case eingabe.equals("x") :
      ausgabe = 10;
      System.out.println(ausgabe); 
      break;
      case eingabe.equals("L"): 
      ausgabe = 50;
      System.out.println(ausgabe); 
      break;
      break;
      case eingabe.equals("l") :
      ausgabe = 50;
      System.out.println(ausgabe); 
      break;
      case eingabe.equals("C") : 
      ausgabe = 100;
      System.out.println(ausgabe); 
      break;
      break;
      case eingabe.equals("c") : 
      ausgabe = 100;
      System.out.println(ausgabe); 
      break;
      case eingabe.equals("D") : 
      ausgabe = 500;
      System.out.println(ausgabe); 
      break;
      break;
      case eingabe.equals("d") :
      ausgabe = 500;
      System.out.println(ausgabe); 
      break;
      case eingabe.equals("M") : 
      ausgabe = 1000;
      System.out.println(ausgabe); 
      break;
      break;
      case eingabe.equals("m") :
      ausgabe = 1000;
      System.out.println(ausgabe); 
      break;
      case "0" : 
      System.out.println("Bitte geben Sie eine römische Zahl ein!");
      break;
      default:System.out.println("Die Umrechnung einer römischen Zahl,");
      System.out.println("die aus mehreren Zeichen besteht, ist eine komplizierte Angelegenheit."); 
      
    } // end of switch
  }
}

Dabei stoße ich auf folgende Fehler:

"Compiliere E:\Java-Projekte\Verzweigungen\Rom.java mit Java-Compiler
Rom.java:15:26: error: incompatible types: boolean cannot be converted to String
case eingabe.equals("I") :"

Wie ihr sieht habe ich nirgends ein Boolean festgemacht und verstehe auch nicht weshalb er da meckert.

Kann mir jemand weiterhelfen?

Gruß

 

[Ruzmanz]

Es musst case "I": lauten. Dort musst du kein equals verwenden.

 

[alicedeh]

Es musst case "I": lauten. Dort musst du kein equals verwenden.

Du bist der Boss!

Check, problem solved.

Danke

 

[VfL_Freak]

Moin,

BTW: warum denn soviele doppelte breaks ??

Gruß Klaus

 

[alicedeh]

Moin,

BTW: warum denn soviele doppelte breaks ??

Gruß Klaus

Hatte ich erst später bemerkt =D War natürlich beim kopieren ein Fehler...^^ Doppeltes break geht natürlich nicht =)

 

[Thallius]

Du Must auch nicht für große und kleine Zeichen den Code 2x schreiben. Ein

case 'i':
case 'I':
    ....
    Break;

Tut es auch

Gruß

Claus

 

[Thallius]

Richtig schön wird der Code wenn du eine Key Value Collection nimmst und einfach als Key den Buchstaben benutzt. Dann wird dein programm ein Zweizeiler.

Gruß

Claus

 

[consider]

Ach, na wenn wir dabei sind mische ich auch mal mit :

Möglich wäre auch toUpperCase() auf den Eingabestring, die Ausgabe als String und println nur ein mal am Schluss:

String ausgabe;

switch (eingabe.toUpperCase()) {
			case "I":
				ausgabe = "1";
				break;
			case "V":
				ausgabe = "5";
				break;
			case "X":
				ausgabe = "10";
				break;
			case "L":
				ausgabe = "50";
				break;
			case "C":
				ausgabe = "100";
				break;
			case "D":
				ausgabe = "500";
				break;
			case "M":
				ausgabe = "1000";
				break;
			case "0":
				ausgabe = "Bitte geben Sie eine römische Zahl ein!";
				break;
			default:
				ausgabe = "Die Umrechnung einer römischen Zahl, \ndie aus mehreren Zeichen besteht, ist eine komplizierte Angelegenheit.";
			} // end of switch
			System.out.println(ausgabe);

Wird zumindest um einiges kürzer und DRY.

 

[alicedeh]

@Consider und Thallius:

Ich wende nur Methoden an die ich aus dem Unterricht kenne.
Bin für weitere Tipps offen.

Danke an consider.

 

[alicedeh]

Ich schließe damit ab:

#1 (unsauber)

import java.util.Scanner;
public class Rom {
  
  public static void main(String[] args) throws InterruptedException{
    Scanner scan = new Scanner(System.in);
    
    String eingabe = "";
    int ausgabe = 0;
    
    System.out.println("Römischer Kalender wird geladen.");
    System.out.println("");
    System.out.print("-");
    Thread.sleep(500);
    System.out.print(" - ");
    Thread.sleep(500);
    System.out.print("-");
    Thread.sleep(500);
    System.out.print(" - ");
    Thread.sleep(500);
    System.out.print("-");
    Thread.sleep(500);
    System.out.print(" - ");
    Thread.sleep(500);
    System.out.print("-");
    Thread.sleep(1500);
    System.out.print(" - ");
    Thread.sleep(200);
    System.out.print("-");
    Thread.sleep(200);
    System.out.print(" - ");
    
    System.out.println("");
    System.out.println("");
    
    System.out.print("Bitte geben Sie die römische Zahl ein: ");
    eingabe = scan.next();
    
    System.out.println("");
    
    System.out.println("Es wird gerechnet.");
    
    System.out.println("");
    System.out.print("-");
    Thread.sleep(500);
    System.out.print(" - ");
    Thread.sleep(500);
    System.out.print("-");
    Thread.sleep(500);
    System.out.print(" - ");
    Thread.sleep(500);
    System.out.print("-");
    Thread.sleep(500);
    System.out.print(" - ");
    Thread.sleep(500);
    System.out.print("-");
    Thread.sleep(1500);
    System.out.print(" - ");
    Thread.sleep(200);
    System.out.print("-");
    Thread.sleep(200);
    System.out.print(" - ");
    
    System.out.println("");
    
    switch (eingabe) {
      case "I" : 
      ausgabe = 1;
      System.out.println("I = " + ausgabe); 
      break;
      case "i" : 
      ausgabe = 1;
      System.out.println("i = " + ausgabe); 
      break;
      case "V" : 
      ausgabe = 5;
      System.out.println("V = " + ausgabe); 
      break;
      case "v" :
      ausgabe = 5;
      System.out.println("v = " + ausgabe); 
      break;
      case "X": 
      ausgabe = 10;
      System.out.println("X = " + ausgabe); 
      break;
      case "x" :
      ausgabe = 10;
      System.out.println("x = " + ausgabe); 
      break;
      case "L": 
      ausgabe = 50;
      System.out.println("L = " + ausgabe); 
      break;
      case "l" :
      ausgabe = 50;
      System.out.println("l = " + ausgabe); 
      break;
      case "C" : 
      ausgabe = 100;
      System.out.println("C = " + ausgabe); 
      break;
      case "c" : 
      ausgabe = 100;
      System.out.println("c = " + ausgabe); 
      break;
      case "D" : 
      ausgabe = 500;
      System.out.println("D = " + ausgabe); 
      break;
      case "d" :
      ausgabe = 500;
      System.out.println("d = " + ausgabe); 
      break;
      case "M" : 
      ausgabe = 1000;
      System.out.println("M = " + ausgabe); 
      break;
      case "m" :
      ausgabe = 1000;
      System.out.println("m = " + ausgabe); 
      break;
      case "" : 
      System.out.println("Bitte geben Sie eine römische Zahl ein!");
      break;
      default:System.out.println("Die Umrechnung einer römischen Zahl, die aus mehreren Zeichen besteht,");
      System.out.println("ist eine komplizierte Angelegenheit."); 
      
    } // end of switch
  }
}

 

[alicedeh]

#2 (elegant) THX@consider

import java.util.Scanner; 
public class Rom_elegant {
  
  public static void main(String[] args) throws InterruptedException{
    Scanner scan = new Scanner(System.in);
    
    String eingabe = "";
    String ausgabe = "";
    
    System.out.println("Römischer Kalender wird geladen.");
    System.out.println("");
    System.out.print("-");
    Thread.sleep(500);
    System.out.print(" - ");
    Thread.sleep(500);
    System.out.print("-");
    Thread.sleep(500);
    System.out.print(" - ");
    Thread.sleep(500);
    System.out.print("-");
    Thread.sleep(500);
    System.out.print(" - ");
    Thread.sleep(500);
    System.out.print("-");
    Thread.sleep(1500);
    System.out.print(" - ");
    Thread.sleep(200);
    System.out.print("-");
    Thread.sleep(200);
    System.out.print(" - ");
    
    System.out.println("");
    System.out.println("");
    
    System.out.print("Bitte geben Sie die römische Zahl ein: ");
    eingabe = scan.next();
    
    System.out.println("");
    
    System.out.println("Es wird gerechnet.");
    
    System.out.println("");
    System.out.print("-");
    Thread.sleep(500);
    System.out.print(" - ");
    Thread.sleep(500);
    System.out.print("-");
    Thread.sleep(500);
    System.out.print(" - ");
    Thread.sleep(500);
    System.out.print("-");
    Thread.sleep(500);
    System.out.print(" - ");
    Thread.sleep(500);
    System.out.print("-");
    Thread.sleep(1500);
    System.out.print(" - ");
    Thread.sleep(200);
    System.out.print("-");
    Thread.sleep(200);
    System.out.print(" - ");
    
    System.out.println("");
    
    switch (eingabe.toUpperCase()) {
      case "I" : 
      ausgabe = "1";
      break;
      case "V" : 
      ausgabe = "5";
      break;
      case "X": 
      ausgabe = "10";
      break;
      case "L": 
      ausgabe = "50";
      break;
      case "C" : 
      ausgabe = "100";
      break;
      case "D" : 
      ausgabe = "500";
      break;
      case "M" : 
      ausgabe = "1000";
      break;
      case "" : 
      System.out.println("Bitte geben Sie eine römische Zahl ein!");
      break;
      default:System.out.println("Die Umrechnung einer römischen Zahl, die aus mehreren Zeichen besteht,");
      System.out.println("ist eine komplizierte Angelegenheit."); 
      
    } // end of switch
    System.out.println(eingabe + " = " + ausgabe);
  }
}