Erste Schritte Instanzmethoden Rechnen (Bruch)

[Zipe]

Hey Leute,

Ich möchte Instanzmethoden für eine Klasse "Bruch" erstellen. Hierbei soll der Bruch mit einer anderen Zahl multipliziert, dividiert, addiert, subtrahiert werden.

Nun soll den jeweiligen Methoden nur einen Parameter vom Typ Bruch übergeben werden, und es soll eine neue Instanz vom Typ Bruch zurückgegeben werden, die das Ergebnis enthält.

Hier die Klasse "Bruch"

public class Bruch {
	int zaehler;
	int nenner;
	Bruch nBruch = new Bruch();

hier der Methodenansatz

	public Bruch mul(nBruch){
		int faktor;
		faktor = IOTools.readInteger("Faktor?: ");
		Bruch neuerBruch = new Bruch();
		neuerBruch = nBruch * faktor;
		return neuerBruch;
	}

Hier zeigt mir eclipse aber einige Fehlermeldungen. Ich verstehe nicht ganz, wie ich es hinkriege, dass ich einen Parameter vom Typ Bruch übergeben kann. Dachte das würde klappen, wenn ich oben eine variable nBruch erzeuge.

Irgendwelche Tipps?

Mfg

 

[Ruzmanz]

Du hast bei deinem Parameter keinen Typ angegeben:

    public Bruch mul(Bruch nBruch){
            int faktor;
            faktor = IOTools.readInteger("Faktor?: ");
            Bruch neuerBruch = new Bruch();
            neuerBruch = nBruch * faktor;
            return neuerBruch;
        }

 

[Zipe]

Ah stimmt.

Hatte auch allgemein einen Fehler drin. Brauche garkeine Variable "faktor", sondern ich sollte den Parameter Bruch mit dem neuen Bruch multiplizieren.

Jetzt habe ich allerdings das Problem, dass ich Bruch * Bruch multiplizieren will und er das mit dem Typ "Bruch" nicht kann.

public Bruch mul(Bruch nBruch){
		Bruch neuerBruch = new Bruch();
		neuerBruch = neuerBruch * nBruch; // "operator * is undefined for [...]"
		return neuerBruch;

Jemand ne Idee, wie ich die beiden Brüche multipliziert bekomme?

 

[eRaaaa]

Wie multipliziert man denn Brüche? Du hast doch jeweils einen Zähler und Nenner der beiden Bruch-Objekten, damit sollte sich das umsetzen lassen (int * int geht schließlich).

ich denke auch nicht dass du ein neues Objekt erstellen sollst in der Methode, sondern es bezieht sich auf den bruch auf den du die Methode aufrufst (this!)

 

[Zipe]

Also muss ich den zaehler von "nBruch" mit dem zaehler von "neuerBruch" multiplizieren (beim nenner das gleiche)?

Wenn ich keinen neuen Bruch instanziere, welche Brüche multipliziere ich dann miteinander? Ich weiß schon was du meinst, da ich ja in der Testklasse nachher irgendwas wie "testBruch.mul(nBruch)" haben werde ...
Aber wie schreibe ich das? ???:L (das würde mir wahrscheinlich auch bei meinen anderen Methoden helfen, da ich jeweils den "aktuellen Bruch" in den Methoden weiterbearbeiten soll.

 

[ThIwOl]

Zähler * Zähler
______________
Nenner * Nenner

 

[nomuri]

Na den übergebenen Bruch mit dem aktuellen.. Sprich nBruch.zaehler * this.zaehler und nBruch.nenner * this.nenner

 

[Zipe]

und in welche Variable speichere ich das dann?
Kriege dann immer die Fehlermeldung "Can not convert from int to 'Bruch'" ... Da ich ja versuche das ganze wieder in einem Bruch zu speichern, der dann ausgegeben werden soll.

 

[ThIwOl]

Jeder Bruch hat doch eine Zähler und Nenner-Variable. Darin speicherst du das Ergebnis.
Sprich: this.zaehler = this.zaehler * nBruch.nenner;

 

[nomuri]

Also mal langsam. Du benötigst eine Zaehler und eine Nenner variable. Da speicherst du dann jeweils einen Wert rein aus dem Objekt was du anlegst.. z.B.

Bruch b1 = new Bruch(1,2);

Somit hast du jetzt einen Bruch 1/2

Jetzt möchtest du den b1 mit einem anderen Bruch multiplizieren.
Sprich du legst einen 2. Bruch an

Bruch b2 = new Bruch(1,4);

Dann multiplizierst du die beiden miteinander.

b1.multipliziere(b2);

Jetzt gehst du in deine Bruch Klasse und dort hast du den aktuellen Bruch b1 zerteilt in Zaehler und Nenner. Dein Konstruktor muss natürlich dementsprechend aufgebaut sein.

Dann schreibst du deine multipliziere Methode.
Z.B. so:

public void multipliziere(Bruch bruch){

this.zaehler = bruch.zaehler * this.zaehler;
this.nenner = bruch.nenner * this.nenner;

}

Bruch bruch ist der übergebene Bruch b2 und this.zaehler & this.nenner sind die Bestandteile des Bruchs b1

 

[Zipe]

Ah okay

und gebe dann wieder nBruch aus? oder welchen Bruch.

Eine Frage noch. Das mit dem "this." ... habe in meinen Methoden jetzt fast alle mit nem neu instanzierten Bruch geschrieben, obwohl ich ja den "aktuellen" Bruch benutzen soll (wusste aber nicht genau was das heißen soll)

wie würde man denn zB das hier jetzt schreiben?

public double wert() {
		double dBruch;
		Bruch neuerBruch = new Bruch();
		dBruch = zaehler / nenner;
		return dBruch;
	}

(versuche hier den Bruch als double-Wert zurückzugeben)

Edit: Die Methoden soll ich erst woanders aufrufen, dafür muss halt erstmal die Klasse stimmen. Muss ich oben in der Klasse zB. sowas deklarieren?

public class Bruch {
	int zaehler;
	int nenner;
	Bruch bruch = new Bruch();

 

[nomuri]

Hast du überhaupt mal richtig gelesen was ich dir geschrieben habe?
Und wieso einen Bruch als double Wert? Du möchtest den Bruch in eine Dezimalzahl umwandeln oder was? Dann kannst du auch direkt mit Dezimalzahlen rechnen.. wofür dann der Aufwand über die Brüche?

Zum ausgeben kannst du dir einfach ne toString methode schreiben...

 

[Zipe]

Ja ich habs mir durchlesen.

Also b1 und b2 würde ich dann in einer Testmethode initialisieren. Und darauf dann meine Methoden anwenden.

z.B. b1.mul(nBruch);

Allerdings müssen dafür meine Methoden ja erst richtig sein?!
Das mit dem double-Wert ist reine Spielerei, hat keinen tieferen Sinn, außer halt zu lernen Instanzmethoden zu schreiben.

Edit: Ja die toString Methode habe ich schon geschrieben

 

[nomuri]

Zeig doch mal deinen kompletten Quellcode, dann kann man dir ggf besser helfen.

 

[Zipe]

public class Bruch {
	int zaehler;
	int nenner;
	Bruch nBruch = new Bruch();

	public Bruch(int zaehler, int nenner) {
		this.zaehler = zaehler;
		this.nenner = nenner;

	}

	public Bruch() {
	}

	public String toString() {
		String bruch = "zaehler/nenner";
		return bruch;
	}

	public double wert() {
		double dBruch;
		Bruch neuerBruch = new Bruch();
		dBruch = zaehler / nenner;
		return dBruch;
	}
	public Bruch mul(Bruch nBruch){
		this.zaehler = this.zaehler*nBruch.zaehler;
		this.nenner = this.nenner*nBruch.zaehler;
		return nBruch;
	}
}

 

[nomuri]

Wofür hast du zaehler und nenner variable und dazu nochn Bruch?
Ist einfach nicht nötig.. ich würde es so machen.

public class Bruch {


	private int zaehler;
	private int nenner;
	
	
	public Bruch(int zaehler, int nenner)
	{
		this.zaehler = zaehler;
		this.nenner = nenner;
	}
	
	
	public void multipliziere(Bruch b)
	{
		
		this.zaehler *= b.zaehler;
		this.nenner *= b.nenner;
		
	}
	
	
	public String toString()
	{
	
		return this.zaehler + " / " + this.nenner;
			
	}

}

und in der Main methode/Main klasse dann

public class BruchTest {

	
	public static void main(String[] args) {
		
		
		
		Bruch b1 = new Bruch(1,2);
		Bruch b2 = new Bruch(1,4);
		
		
		b1.dividiere(b2);
		System.out.println(b1.toString());
		
		

	}

}

Das ist quasi genau das was ich dir in meinem Post vorhin gesagt habe.

 

[Zipe]

Danke dir! Würde meine "mul" Methode denn so wie ich sie geschrieben hab auch funktionieren?

Wie würdest du die Methode "wert" schreiben?

 

[nomuri]

Deine Methode gibt nBruch zurück. Warum? Du schreibst deine Ergebnisse ja in this.zaehler und this.nenner und nicht in nBruch..

Was soll deine Methode wert denn genau machen? Einfach die Umrechnung in Dezimal?
Nimmst den aktuellen Bruch bestehend aus this.zaehler und this.nenner und rechnest den Bruch in Dezimal um. Kannst du mit oder ohne returnwert machen.. bleibt dir frei gestellt.

 

[Zipe]

Okay glaub ich habs.

Danke für die Geduld mit mir!