Umrechnen Aufgabe

[Hubson]

Hallo,

habe meine ersten Vorlesungen prog. hinter mir und hab auch schon einiges verstanden ^^.

Unser Prof. meint es gut mit uns und will das wir ein Programm schreiben welches verschiedene Umrechenarten beherrscht.

Ich habe bis jetzt eine main geschrieben die nichts macht als eine Klasse aufzurufen die das eigentlich Prog. starten soll.

Jetzt hab ich mit der eigentlichen Klasse begonnen die die tatsächliche Arbeit machen soll...
wollte nur mal wissen ob ich auf einem guten Weg bin oder ob ich schon jetzt in die falsche Richtung denke....hier mein bisher bescheidenes Werk....will keine Lösung des Programms sonder nur Hinweise wenn ich auf dem völlig falschen Weg bin bzw. irgendwo dringend was ändern muss...

/**
 * 
 */
package wi2012pr02;

import java.util.Scanner;


/**
 * @author Mats
 *
 */
public class Eingabe {

	/**
	 * @param args
	 */
	public void starte  (){
		// Attribute für die Abfragewerte
		// Anfangseinheit, Zieleinheit, Anfangswert, Auswahl Länge oder Gewicht
		int auswahl; // Auswahl für Gewicht oder Länge
		String anfangseinheit;// Meter, Kilo, etc
		String zieleinheit; // Pound, Yard, ...
		int anfangswert; 
		
		Scanner scan = new Scanner(System.in);
		System.out.println("Bitte wählen Sie zwischen Gewicht(1) oder Länge(2)aus.");
		auswahl = scan.nextInt();
		System.out.println("Bitte wählen Sie eine Starteinheit aus.");
		anfangseinheit = scan.next();
		System.out.println("Bitte wählen Sie eine Zieleinheit aus.");
		zieleinheit = scan.next();
		System.out.println("Bitte geben Sie den Wert ein, den Sie umrechnen möchten.");
		anfangswert = scan.nextInt();
		
		if (auswahl == 1){
			// rechne das Gewicht aus
		}else{
			// rechne die Länge aus
		}
				
		
	}

}

 

[Fab1]

Erster Eindruck ist auf alle Fälle gut. Du könntest dir allerdings noch die Java-Konventionen anschauen.

Bei

 if (auswahl == 1){
            // rechne das Gewicht aus
        }else{
            // rechne die Länge aus
        }

Würde ich statt dem else explizit auf die 2 prüfen. Denn, wenn man statt

1

die

11

eintippt, dann nimmt er auch die Länge.

Man könnte es zum Beispiel so machen.

if (auswahl == 1) {
			// rechne das Gewicht aus
		} else if (auswahl == 2) {

			// rechne die Länge aus
		} else {
			System.out
					.println("Sie haben sich wohl vertippt. Bitte wählen Sie zwischen Gewicht(1) oder Länge(2)aus.");
		}

Somit wären alle drei Fälle, die eintreten können abgedeckt.

Das ein oder andere könnte man evtl. auch in Methoden auslagern, müssen tut man das bei dem "großen" Programm allerdings noch nicht, wie ich finde.

 

[Hubson]

Erster Eindruck ist auf alle Fälle gut. Du könntest dir allerdings noch die Java-Konventionen anschauen.

Bei

 if (auswahl == 1){
            // rechne das Gewicht aus
        }else{
            // rechne die Länge aus
        }

Würde ich statt dem else explizit auf die 2 prüfen. Denn, wenn man statt

1

die

11

eintippt, dann nimmt er auch die Länge.

Man könnte es zum Beispiel so machen.

if (auswahl == 1) {
			// rechne das Gewicht aus
		} else if (auswahl == 2) {

			// rechne die Länge aus
		} else {
			System.out
					.println("Sie haben sich wohl vertippt. Bitte wählen Sie zwischen Gewicht(1) oder Länge(2)aus.");
		}

Somit wären alle drei Fälle, die eintreten können abgedeckt.

Das ein oder andere könnte man evtl. auch in Methoden auslagern, müssen tut man das bei dem "großen" Programm allerdings noch nicht, wie ich finde.

ok danke für den tip, werde das ändern, die konvention und ich sind noch nich so dicke, bitte das zu verzeihen ^^
das programm wird noch größer, ich soll sowohl von metrisch in angelsächsisch als auch polnisch und zurück umrechnen können

 

[pzypher]

das programm wird noch größer, ich soll sowohl von metrisch in angelsächsisch als auch polnisch und zurück umrechnen können

Anstatt mehrerer else if Abfragen könntest du dann den switch Befehl verwenden.

Gruß