Informatik Bruchrechner (Addition) hilfe?
In den Kommentaren in der 3. letzten Methode zur Berechnung stehen die angezeigten Fehler. Es sind 2 Klassen mit welchen gearbeitet wird, die eine für die Oberfläche und die andere zu Berechnung. Es geht um Addition mit Brüchen und wollte fragen, wie ich diese Fehler beheben kann, und ob ich was falsch gemacht habe. Danke
3 Antworten
- Du hast zwei aufeinanderfolgende return-Befehle in deiner Methode. Der zweite Befehl wird also nie ausgeführt, ist redundant.
- Du versuchst ein undefiniertes Symbol Zaehler zu nutzen. Wo hast du diese Methode denn definiert?
Methodennamen sollten übrigens mit einem Kleinbuchstaben beginnen und eine Tätigkeit ausdrücken, die ihren tatsächlichen Zweck beschreibt.
Ein Konstruktoraufruf außerhalb der Klasse kann nur nach dem new-Operator folgen und eine Methode kann ebenfalls ein Objekt zurückgeben. Bitte schau dir die erwähnten Themen (Konstruktoren, Methoden) nochmal in Büchern / Tutorials an.
Aber eine Methode wird ja wohl nicht groß geschrieben :D dachte ich. Daher ist meine Schlussfolgerung dass Nenner und Zähler Konstruktoren sind
Naja bin auch nicht die beste Programmierern. :D Also kann ich nicht 100% hinter dem stehen was ich sage. Aber das ist so meine Ansicht
Technisch betrachtet ist es egal, ob die Bezeichner mit Groß-Kleinbuchstaben beginnt oder nicht (so lange man sie nach erster Benennung stringent hält). Nach Java-Konvention schreibt man Klassennamen mit einem Großbuchstaben - und daher auch Konstruktoren (dann auch technisch bedingt, da der Konstruktor den gleichen Namen wie die Klasse haben muss).
Wären Nenner oder Zaehler jeweils Konstruktoren, müssten sie wie gesagt nach einem new-Operator aufgerufen werden, denn der schafft erst den Speicher für die neuen Objekte.
new Nenner(/* ... */);
new Zaehler(/* ... */);
Wie du erkannt hast, erwartet die add-Methode ein Objekt vom Typ Bruch als Rückgabewert. Das heißt, selbst wenn der Aufruf so wie in meinem Snippet erfolgen würde, wäre es unlogisch, denn ein Objekt Nenner und auch ein Objekt Zaehler ist jeweils kein Bruch. Nicht einmal eine Subklasse dessen.
Folgendermaßen könnte man nun nur noch eine Methode dahinter vermuten, die einfach ziemlich schlecht benannt wurde.
public Bruch Nenner(/* ... */) {
return new Bruch(/* ... */);
}
// same for Zaehler ...
Aber wie dem auch sei: Letztendlich bleiben die Symbole Nenner / Zaehler undefiniert und bedürfen Aufklärung. Ob die vom FS kommt, ist abzuwarten. Einen Lösungsvorschlag habe ich unten bereits gegeben.
Sollte es sich doch um Konstruktoren handeln, könnte diese Modifikation vorgenommen werden:
Zaehler ergebnisZaehler = // get result for Zaehler ...
Nenner ergebnisNenner = // get result for Nenner ...
return new Bruch(ergebnisZaehler, ergebnisNenner);
und eine passende Überladung des Bruch-Konstruktors müsste definiert werden.
Sollten die Objekte Zaehler / Nenner entsprechende Getter bereitstellen (die den konkreten numerischen Wert liefern), könnte man sich die Überladung auch sparen:
Zaehler ergebnisZaehler = // get result for Zaehler ...
Nenner ergebnisNenner = // get result for Nenner ...
return new Bruch(ergebnisZaehler.gibWert(), ergebnisNenner.gibWert());
Außerdem verlangt die Methode nach einem returnwert Bruch und nicht nach Zähler oder Nenner
Also ich weiß ja nicht genau was bei dir Zähler und Nenner sind. Aber Methoden sind das nicht.
Bitte zeige einmal die Definitionen von Nenner und Zaehler.
An sich würde es ja ausreichen, mit den vorhandenen Symbolen in der Methode zu arbeiten.
public Bruch add(int pz, int pn, int z2, int n2) {
// calculate ...
return new Bruch(ergebnisZaehler, ergebnisNenner);
}
Wobei die Variablen ergebnisZaehler und ergebnisNenner die jeweiligen berechneten Werte für Zähler und Nenner speichern.
Generell solltest du einmal schauen, bessere Bezeichner zu vergeben. pz, pn, ... sind nichtssagend. Außerdem vermischt du englische Namen (add) mit deutschen Namen (Bruch, Zaehler, ...). Das macht dein Programm nur mehr wirr und unleserlich.
Es tut mir leid, das zu sagen, aber leider kann man fast nichts auf den Bild erkennen... :(
Habe ich schon versucht... Und was rauskam, kannst du dir Bestimmt denken :(
Er returnt ein Objekt. Also das ist keine Methode sondern ein Konstruktor