OOP Getter, Setter und andere Probleme

[Noel]

Hallo,

es fällt mir schwer durchzublicken was genau "gutes OOP" heißt. Ich hab jetzt ein Programm geschrieben, dass Celsius in Fahrenheit umrechnet.

Es wäre mir fürs Selbststudium wichtig, wenn jemand vielleicht mal schauen könnte was an meinem geschrieben Code zu verbessern wäre.

zB

-> Konstruktor so ok? kommt mir irgendwie komisch vor, da das was eigentlich da drin steht doch in eine Methode gehoert oder?

-> Sind die Getter überflüssig? Brauch ich Setter?

-> Sind die Variablen so gut genutzt bzw deklariert? (private/public?)

-> cels = richtigeEingabe("^[0-9]?[0-9].?[0-9]*?$"); Soll EIGENTLICH nur 1 oder 2 Zahlen vor dem Komma und 1 oder 2 Zahlen hinter dem Komma zulassen. Das Kommas [.] soll ebenfalls optional sein

Klasse Temperatur

package nichtStatisch;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.regex.Matcher;
import java.util.regex.Pattern;


public class Tempertaur {

	private double fahrenheit;
	private double celsius;
	private String cels;

	public void tempUmrechnen() {
		fahrenheit = (celsius * 9.0 / 5.0) + 32.0;
		System.out.println("Gegeben sind: " + celsius
				+ "° Grad Celsius und das entspricht: " + fahrenheit
				+ "° Fahrenheit");
	}

	/**
	 * 
	 * Liest die Daten durch <code>input</code> ein!
	 * 
	 * @param input
	 * @return Eingabe
	 */
	public String richtigeEingabe(String input) {
		Pattern p = Pattern.compile(input);
		String richtig = null;
		boolean repeat = true;
		while (repeat) {
			try {
				BufferedReader eingabe = new BufferedReader(
						new InputStreamReader(System.in));
				richtig = eingabe.readLine();
				Matcher m = p.matcher(richtig);
				repeat = !m.matches();
			} catch (IOException ioe) {
				System.out.println("Das war zu erwarten! ");
				ioe.printStackTrace();
			}
		}
		return richtig.trim();
	}

	// GETTER - public

	public double getCelsius() {
		return celsius;
	}

	public double getFahrenheit() {
		return fahrenheit;
	}

	public String getCels() {
		return cels;
	}

	// SETTER - private 

	// Konstruktor
	
	public Tempertaur() {
		System.out
				.println("Geben Sie die Tempperatur an: ");
		cels = richtigeEingabe("^[0-9]?[0-9].?[0-9]*?$");
		try {
			celsius = Double.parseDouble(cels);
		} catch (NumberFormatException e) {
			System.out.println("Schlimme Exception! ");
			e.printStackTrace();
		}
	}
}

Klasse BerechneTemperatur

package nichtStatisch;

import java.io.IOException;
import nichtStatisch.Tempertaur;

;

public class BerechneTemperatur {

	/**
	 * Die <code>Main-Methode</code> nimmt 2 Werte an:
	 * <ul>
	 * <li>Celsius</li>
	 * <li>Fahrenheit</li>
	 * </ul>
	 * rechnet dann Celsius in Fahrenheit um! <br>
	 * <br>
	 */
	public static void main(String[] args) throws IOException {

		Tempertaur test = new Tempertaur();
		test.tempUmrechnen();

	}

}

Vielen Dank

 

[Raistlin]

Aaaalso, um schon mal ein paar deiner Fragen zu beantworten:
Die Variablen sind als private schon mal so ok. Man sollte, jedenfalls laut meinem Prof, Variablen nach möglichkeit immer private halten. Die getter und setter brauchst du dann natürlich, um auf die Variablen zugreifen zu können.
Was den Konstruktor angeht, der ist, ehrlich gesagt, Stuhl. Der Konstruktor dient ja nur dazu, dir ein Objekt zu erstellen, das gewisse Datenelemente hat. In deinem Fall sollte der Konstruktor vielleicht so aussehen:

public Temperatur(double celsius){
      this.celsius = celsius;
}

und wenn du keine Clesius-Zahl mitgeben willst, nimmst du den Standard-Konstruktor, der dann so aussieht:

public Temperatur(){
       this.celsius = 0;
}

Das Komma bei der Celsius-Zahl kann leider (meiner Meinung nach) nicht optional sein, da du die Variable Celsius ja schon als double gewählt hast.

Hoffe, dir ist fürs erste ein bisschen geholfen.

 

[Noel]

Ja danke ein wenig

Es geht mir auch nicht um den Variablenwert von double, sondern um den regulären Ausdruck:

("^[0-9]?[0-9].?[0-9]*?$")

Nur wie verhält sich das dann mit den Gettern & Settern wenn ich die Daten eh über die Tastatur eingeben lasse?

 

[Final_Striker]

Die Eingabemethode für die Werte gehört nicht in die Klasse Temperatur. Stell dir mal vor du willst die Klassen in eine GUI verwenden, dann müsstest du die Klasse ja extra anpassen, sehr unpraktisch.

Die Klasse bekommt Werte. Woher die Werte kommen (Tastatur, Datenbank oder GUI) kann ihr herzlichst egal sein.

 

[bLaSt]

Ich hätte nur grundsätzlich anzumerken,dass es gerade bei komplexeren und längeren Programmen sinnvoll ist die main-Methode in eine eigene Klasse auszulagern.
In deinem aktuellen Beispiel ist das kein Problem, da deine main-Methode sehr klein ist.
Aber bei größeren Programmen wird sie sehr schnell viel größer und eine zusätzliche Klasse für die main-Methode steigert die Übersicht.

 

[Gast2]

hm, eigentlich sollten in der main Methode nur ein paar Zeilen drin stehen um das Programm zu starten (GUI erstellen oder ähnliches).
Zumindest meiner ansicht nach

 

[Noel]

Die Eingabemethode für die Werte gehört nicht in die Klasse Temperatur. Stell dir mal vor du willst die Klassen in eine GUI verwenden, dann müsstest du die Klasse ja extra anpassen, sehr unpraktisch.

Die Klasse bekommt Werte. Woher die Werte kommen (Tastatur, Datenbank oder GUI) kann ihr herzlichst egal sein.

Genau darum geht es. Also schreibe ich dafuer eine eingene Methode?

@EikeB danke aber es ist eine eigene Klasse.


Kann mir snst noch jemand was zu dem regulären Ausdruck sagen?

cels = richtigeEingabe("^[0-9]?[0-9].?[0-9]*?$"); Soll EIGENTLICH nur 1 oder 2 Zahlen vor dem Komma und 1 oder 2 Zahlen hinter dem Komma zulassen. Das Kommas [.] soll ebenfalls optional sein

 

[Gast2]

der Post war auch auf meinen Vorposter bezogen.
Wenn man die main-Methode in ne andere Klasse auslagern muss damits übersichtlich bleibt, dann läuft da mMn was falsch

 

[bLaSt]

der Post war auch auf meinen Vorposter bezogen.
Wenn man die main-Methode in ne andere Klasse auslagern muss damits übersichtlich bleibt, dann läuft da mMn was falsch

Es war damit nur gemeint, dass es übersichtlicher ist es auszulagern, so hab ichs halt gelernt. Damit ist nicht gemeint,dass man ez die main-Methode besser zuspamen kann
Aber sobald man mal ein paar Eingaben vom User übernehmen will und diese dann zuordnen will plus Exception Behandlung kanns schon mal mehr werden. Ist ja auch geschmackssache, ich finds so übersichtlicher, sollte keiner angegriffen werden der das anderst macht