Wie deklariert man eine Methode, die ein Objekt zurückgeben soll?

[scratchy1]

Hallo Leute,
Wie deklariert man eine Methode, die ein Objekt zurückgeben soll?
Viele Grüße

 

[Robat]

public Object foo() {
    return someObject;
}

 

[scratchy1]

Hi Robat,
Also könnte ich dann auch sowas machen:?

 public Object  multiplyBy(Fraction f) {
        int c;
        int d;
        int e;
        int g;
        Fraction w = new Fraction(c, d);
        Fraction b = new Fraction(e, g);
        f = new Fraction((c * e), (d * g));
        return this;
    }

 

[Robat]

Die Methode sollte wohl einer ein Fraction-Objekt zurückgeben:

public Fraction multiplyBy(Fraction f) {
    .... 
    return this;
}

BTW: Bist du dir sicher, dass die Methode das macht, was du dir erhoffst?

 

[scratchy1]

Also die Aufgabe ist:
Fraction-Objekte haben eine Methode multiplyBy, deren einziger formaler
Parameter eine Referenz auf ein weiteres Fraction-Objekt ist. Die Methode
gibt ein neues Fraction-Objekt zurück, welches das Produkt der beiden Brüche
(Nachrichtenempfänger und Parameter) repräsentiert.
D.h. ich bin mir nicht sicher.

 

[Robat]

Dann solltest du wohl eher das neue Fraction-Objekt zurückgeben. In deinem Fall wohl f.

 

[scratchy1]

ok, habs gemerkt und geändert, danke 🙂. Kannst Du mir noch einen Tipp geben, warum es erforderlich ist, die
Variablen c,d,e und g zu initialisieren, denn ich bekomme dort überall "the local variable may not have been initialized"?

 

[Robat]

Anders als Instanzvariablen, bekommen lokale Variablen in Java keinen default Wert. Da du auf eine nicht initialisierte Variable zugreifen willst, kommt es zu dem Fehler.

 

[scratchy1]

Ach ok danke, ich habe diese Variablen jetzt "globalisiert" 🙂.

 

[Robat]

Hast du dein Programm mal ausprobiert? So wie der Code oben ist sollte es eigentlich nicht funktionieren.

 

[mrBrown]

Ach ok danke, ich habe diese Variablen jetzt "globalisiert" 🙂.

Das ist ungefähr so wie "Durch mein geöffnetes Dachfenster schneit es rein, deshalb zieh ich jetzt unter die Brücke".

Wenn du Variablen nicht brauchst (wie in deinem Fall, sonst würdest du ja irgendwas mit ihnen machen), entfern sie einfach.

 

[scratchy1]

Hallo Leut,
Ich hab jetzt mal die Variablen entfernt und versuche es jetzt ohne 😉 .
Die Aufgabe lautet:
Erstellen Sie eine Klasse Fraction zur Beschreibung von Brüchen, welche die folgenden
Vorgaben umsetzt:
• Ein Fraction-Objekt hat zwei ganzzahlige Instanzvariablen numerator und
denominator (Zähler und Nenner). Auf diese Instanzvariablen soll aus fremden
Klassen nicht zugegriffen werden können.
• Die Klasse Fraction deklariert einen Konstruktor, dem zwei ganzzahlige Parameter
übergeben werden müssen. Der Konstruktor soll die Instanzvariablen des
Bruchs mit denWerten der übergebenen Parameter initialisieren. Der Konstruktor
soll deklarieren, eine Ausnahme des Typs IllegalArgumentException auslösen
zu können. Diese Ausnahme soll ausgelöst werden, wenn dem Konstuktor für
den Nenner der Wert 0 übergeben wird.
• Fraction-Objekte haben eine Methode multiplyBy, deren einziger formaler
Parameter eine Referenz auf ein weiteres Fraction-Objekt ist. Die Methode
gibt ein neues Fraction-Objekt zurück, welches das Produkt der beiden Brüche
(Nachrichtenempfänger und Parameter) repräsentiert.
• Fraction-Objekte haben eine parameterlose Methode toString. Die Methode
gibt ein String-Objekt zurück, welches das Fraction-Objekt in der Form
Zähler-Bruchstrich-Nenner repräsentiert, also z.B. "3/4" für den Bruch mit dem
Zähler 3 und dem Nenner 4.
Testen Sie Ihre Klasse ausgiebig. Erstellen Sie dazu eine Testklasse mit main-Methode!

Mein jetzt geänderter Code lautet:

class Fraction {
    int numerator;
    int denominator;

    Fraction(int numerator, int denominator) {
        this.numerator = numerator;
        this.denominator = denominator;
        try {
            if (denominator == 0) {
                throw new Exception();
            }
        } catch (IllegalArgumentException e) {
            System.out.println("Division durch 0 nicht erlaubt.");

        } catch (Exception e) {
            System.out.println("");
        }
    }

    public Object multiplyBy(Fraction f) {
       f=( ((Object.numerator)*(f.numerator)),((Object.denominator)*(f.denominator));
        return f;
    }

    public String toString() {

        String strich = (strich.numerator + "/" + strich.denominator);
        return strich;
    }
}

public class FractionTest {
    public static void main(String[] args) {
        // TODO Auto-generated method stub
        Fraction bruch = new Fraction(1, 2);
        Fraction bruch2 = new Fraction(2, 3);
        Object f = bruch.multiplyBy(bruch2);
        System.out.println("Ergebnis" + f.toString());
    }
}

 

[temi]

Die Methode
gibt ein neues Fraction-Objekt zurück

1. Deine Methode gibt ein Object-Objekt zurück.

f=( ((Object.numerator)*(f.numerator)),((Object.denominator)*(f.denominator));

2. Das stimmt so nicht. Die Klasse "Object" kennt z.B. keinen "numerator". Außerdem sollst du ja ein neues Fraction-Objekt zurückgeben, welches dem Produkt aus "this" und "f" entspricht.

3. Der "try..catch"-Abschnitt im Konstruktor ist völlig überflüssig. Was du möchtest ist:

• Prüfen, ob der Nenner Null ist
• und falls das zutrifft eine Exception auslösen.

Du löst eine Exception aus und fängt sie sofort wieder ab.

 

[Robat]

Ein Fraction-Objekt hat zwei ganzzahlige Instanzvariablen numerator und
denominator (Zähler und Nenner). Auf diese Instanzvariablen soll aus fremden
Klassen nicht zugegriffen werden können.

Der letzte Teil fehlt hier noch. Stichwort: Access-Modifier

eine Ausnahme des Typs IllegalArgumentException auslösen
zu können. Diese Ausnahme soll ausgelöst werden, wenn dem Konstuktor für
den Nenner der Wert 0 übergeben wird

Lässt sich auch einfach als

if(nenner == 0) {
    throw new IllegalArgumentException("...");
}

 

[scratchy1]

1. Deine Methode gibt ein Object-Objekt zurück.

2. Das stimmt so nicht. Die Klasse "Object" kennt z.B. keinen "numerator". Außerdem sollst du ja ein neues Fraction-Objekt zurückgeben, welches dem Produkt aus "this" und "f" entspricht.

Ok, danke, ich fang dann mal hier an, also bei der Methode multiplyby():
Wie deklariere ich die korrekt? Ich probierte

public Fraction multiplyBy(Fraction f) {
         
       Fraction c=( ((this.numerator)*(f.numerator)),((this.denominator)*(f.denominator));
       return c;
    }

Ist immer noch falsch, aber sieht es nicht besser aus als zuvor?

 

[temi]

return new Fraction(......)

Grob übersetzt: "gebe zurück" "neu" "Fraktion", was ungefähr genau der Forderung entspricht.

 

[scratchy1]

Ok, also müsste

public Fraction multiplyBy(Fraction f) {
         
        return new Fraction( ((this.numerator)*(f.numerator)),((this.denominator)*(f.denominator));
       
    }

korrekt sein?

 

[temi]

Ok, also müsste korrekt sein?

Was hältst du von ausprobieren?

Edit: Von den Klammern solltest du einige entfernen. In diesem Fall sorgen sie eher für Unübersichtlichkeit. Es reicht ein Klammerpaar.

 

[scratchy1]

Ok danke temi, zum Ausprobieren muss ich gleich noch die toString-Methode

public String toString() {

        return new String strich = (this.numerator + "/" + this.denominator);
       
    }
}

und die main-Methode zum Testen richtig hinbekommen. 🙂

 

[flopalko]

Du solltest dich wirklich mal mit den Grundlagen vertraut machen und dir genau anschauen, wo du welche Statements wie verwenden darfst...

 

[scratchy1]

So jetzt hab ich die toString-Methode auch hinbekommen:

public String toString() {

        return  String.format("%s / %s ",numerator, denominator);
       
    }

Mein letztes Problem ist jetzt, dass bei public class FractionTest die Fehlermeldung steht:
"the public type FractionTest must be defined in its own file.", obwohl ich schonmal ein Bsp. gesehen habe (Buch),
wo alles in einer Datei war.

 

[White_Fox]

Hat doch neulich irgendwo erst wer geschrieben (ich glaub mihe7): Du kannst pro Datei nur eine (öffentliche) Klasse deklarieren. Kann es sein daß du diese Regel verletzt?

 

[scratchy1]

Ach danke, nee, hat sich erledigt 😉
PS: nee ich hab zwar nur eine public, aber ich wills mir angewöhnen dann 2 Dateien zu nutzen.