Methoden Verständnisfrage zur Methoden

[g_gex]

Guten Morgen

ich verstehe bei dieser Klasse nicht wofür wir die Methode setFlaeche(..) bzw. setUmfang(...) brauchen. Kann man die Methoden eigentlich auch löschen, weil die ja garnicht gebraucht werden.

public class Kreis {
	double radius;

	Kreis() {
		radius = 0;
	}

	Kreis(double r) {
		radius = r;
	}

	double getRadius() {
		return radius;
	}

	void setRadius(double r) {
		radius = r;
	}

	double getUmfang() {
		return 2 * Math.PI * radius;
	}

	double getFlaeche() {
		return Math.PI * radius * radius;
	}

	void setUmfang(double u) {
		radius = u / (2 * Math.PI);
	}

	void setFlaeche(double f) {
		radius = Math.sqrt(f / Math.PI);
	}

}

und die main-Klasse

public class KreisTest {

	public static void main(String[] args) {
String kreisradius;
		double r;
		kreisradius = JOptionPane
				.showInputDialog("Geben Sie einen Kreisradius ein:");
		
		r = Double.parseDouble(kreisradius);
		Kreis k = new Kreis(r);
		System.out.println("Kreisradius: " + k.getRadius());
		System.out.println("Umfang: " + k.getUmfang());
		System.out.println("Fläche: " + k.getFlaeche());
		

	}

}

 

[Tom299]

Wenn du Fläche und Umfang nicht setzen willst, kannst du die Methoden löschen oder auskommentieren, falls du sie vielleicht später doch noch brauchst. Aber das kannst nur du wissen, was du damit anstellen willst ;-)

 

[g_gex]

Das war eine Übung aus einem Buch, ich habe die Aufgabe etwas anders gelöst, aber inwieweit mein Ergebnis richtig ist das weiß ich nicht. Wofür brauche ich den die Methode setFlaeche() bzw setUmfang().

public class Kreis {

	double radius;

	public Kreis() {

		radius = 0;

	}

	public Kreis(double radius) {
		this.radius = 0.0;
	}

	public double getRadius() {
		return radius;
	}

	public void setRadius(double radius) {
		this.radius = radius;
	}

	public double getUmfang() {
		double umfang;

		umfang = 2.0 * 3.1415936 * radius;

		return umfang;

	}

	public void setUmfang(double u) {

	}

	public double getFlaeche() {
		double flaeche;

		flaeche = 3.1415926 * radius * radius;

		return flaeche;
	}

//main-Klasse

import javax.swing.JOptionPane;

public class KreisTest {

	public static void main(String[] args) {
		

 String eingabe = JOptionPane.showInputDialog(null,
		 "Geben Sie den Kreisradius ein: ");
		 double r = Double.parseDouble(eingabe);
		
		 Kreis k = new Kreis();      //oder Kreis k = new Kreis(r); ?
		
		 k.setRadius(r);
		 System.out.println("Umfang des Kreises: ");
		 System.out.println(k.getUmfang());
		 System.out.println("Flaeche des Kreises: ");
		 System.out.println(k.getFlaeche());
 }
}

 

[Dompteur]

Für dein Beispiel brauchst du die beiden Methoden (setFlaeche, setUmfang) nicht. Dein Beispiel ist also nicht falsch, wenn du sie einfach löscht.

Ob man die beiden Methoden anbietet oder nicht ist eine Design Frage.

Wenn du eine Klasse planst, dann überlegst du dir normalerweise, welche Eigenschaften deine Klasse haben soll. Also wie das Interface aussehen soll. Dabei versuchst du, den Satz der angebotenen Methoden abzurunden.
Wenn du zum Beispiel Setter für einige Instanzvariablen brauchst, dann Implementierst du normalerweise Setter für alle interessanten Instanzvariablen.

Ob es nun bei einem Kreis sinnvoll ist die beiden Setter-Methoden anzubieten, hängt vom Anwendungsumfeld ab. Wenn du deinen Kreis immer über die Angabe des Radius definierst, dann brauchst du sie nicht. Wenn du aber deine Kreis-Klasse für ein Umfeld schreibst, in dem es um Umwandlungen verschiedener Kreiseigenschaften geht, dann solltest du die Methoden drin lassen.

In kleineren Übungsprojekten ist das meist egal. Wenn du drauf kommst, dass dir eine Methode fehlt, dann schreibst du sie dir einfach bei Bedarf. Aber stell dir einmal ein Team vor, wo einer für die Klasse Kreis zuständig ist und ein anderer sie verwenden will. Dann bedeutet jede fehlende Methode, dass du sie anfordern und dann darauf warten musst.

 

[strußi]

du solltest die methodenvariablen radius, flaeche und umfang aus den methoderümpfen in den klassenrumpf "ziehen", im überladenen konstruktor (puplic Kreis( double radius)) den radius an deine objektvariable radius übergeben, sonst ist das ganze ding nutzlos. außderdem kannst du die berechnung von fläche und umfang im konstruktor/ in der setRadius-Methode aufrufen, und gibst dann nur noch den Wert von umfang und flaeche in den methoden getUmfang/Flaeche zurück.

 

[g_gex]

Kann jetzt nicht so ganz folgen was sie meinen.
Was meinen sie mit nutzlos?Also warum wäre es nutlos?

public Kreis(double radius) {
		this.radius = radius;
	}

 

[Dompteur]

Kann jetzt nicht so ganz folgen was sie meinen.
Was meinen sie mit nutzlos?Also warum wäre es nutlos?

public Kreis(double radius) {
		this.radius = radius;
	}

Er bezieht sich auf die Zeile 12 in der ersten Klasse in deinem heutigen Beitrag von 09:26.
Da steht

public Kreis(double radius) {
   this.radius = 0.0;
}

 

[strußi]

@Dompteur: Danke für die richtigstellung
@g_gex: wenn du den übergabeparameter in deinem Konstruktor nicht verwendest kannst du genau so gut den parameterlosen konstruktor verwenden