Klassen Vergleich zweier Klassen

[GermanOakheart]

Guten Morgen / Abend, je nachdem wann ihr das hier lest,

Ich bin leider Gottes noch etwas unerfahren und werde mich wohl des öfteren in diesem Forum melden.
Für die folgende Aufgabenstellung bin ich auf ein kleines Problem gestoßen, ich hänge bei Aufgabe d:

Exercise 4b – Terms & Polynomials
Let’s do some more mathematical stuff to get even more familiar to OOP:
1) Develop a class Term that objects represent terms of type a * xb with
a. two attributes for the coefficient and the exponent
b. three useful constructors
c. a method to calculate the value for a given x
d. a method that can sum two terms if the exponent is equal​

Aufgabe a bis c waren kein Problem, bei d scheitert es allerdings.

Ich hab mir gedacht ich könnte einfach eine Methode in meine Klasse "Term" schreiben die die Exponenten beider erstellten Objekte vergleicht und dann zusammen fasst.
Das würde dann ungefähr so aussehen:

  //Zusammenfassen
   
    public boolean Zip(Term t1, Term t2) {
   
       this.t1 = t1;
       this.t2 = t2;
           if(t1.b == t2.b) {
               Term t3 = new Term( (t1.a * t2.a), (t2.b + t2.b));
               return true;
           }  else {
                   return false;
                   }
           }
}

Die Methode sollte funktioniere, allerdings kan ich mit t3 in meiner Main nichts machen, weil er meint, er kennt t3 nicht.

Über einen kleinen Denkanstoß würde ich mich freuen, gleich eine ganze Lösung muss nicht sein

 

[mrBrown]

t3 ist nur in dem Block bekannt, in dem er deklariert wurde - also bei dir nur innerhalb des if.

warum gibt die Methode denn einen boolean zurück, und nicht den neuen Term?

 

[GermanOakheart]

Ich könnte natürlich einen Term zurückgeben:

 public Term Zip(Term t1, Term t2) {
   
        this.t1 = t1;
        this.t2 = t2;
            if(t1.b == t2.b) {
             Term t3 = new Term( (t1.a * t2.a), (t2.b + t2.b));
                return t3;
            }  else {
           
                    }
            }

Dan habe ich aber das Problem das ich den Error habe, dass die function einen Term zurückgeben soll, auch wenn es nicht möglich ist beide Terme zusammenzufassen, deswegen das noch leere else statement.
Und dann noch das Problem das t3 in der Main nicht bekannt ist.

 

[mrBrown]

Du könntest null zurückgeben


Und als t3 musst du den rückgabewert deiner Funktion nutzen

 

[JStein52]

Was ist in deinem Code denn Exponent und was Koeffizient ?

Edit: und warum heisst deine Methode Zip(...) und nicht sum(...) ?

 

[VfL_Freak]

Moin,
etwa so :

public Term Zip( Term t1, Term t2 )
{
    this.t1 = t1;
    this.t2 = t2;
    if( t1.b == t2.b )
    {
         Term t3 = new Term( (t1.a*t2.a), (t2.b+t2.b) );
         return t3;
    }
    return null;  // oder entsprechendes !!
}

Falls das else unbedingt drin bleiben soll, dann muss auch hier ein return erfolgen !!
Also in allen denkbaren Zweigen, in denen die Methode verlassen werden kann !!

VG Klaus

EDIT:

warum heisst deine Methode Zip(...) und nicht sum(...) ?

gutes Argument

 

[JStein52]

gutes Argument

Was noch erschwerend dazu kommt: warum erzeugt sie nicht den SummenTerm sondern den Produktterm ?

 

[mrBrown]

Edit: und warum heisst deine Methode Zip(...) und nicht sum(...) ?

Das ist durchaus eine typische Benennung zum zusammenfassen zweier Dinge, zumindest in funktionalen Sprachen.

 

[JStein52]

zumindest in funktionalen Sprachen.

Was hat die Benennung mit der Programmiersprache zu tun ? Und ist Java hier nun eine funktionale Sprache ? Und davon abgesehen tut die Methode das falsche. Aber den Namen Zip kann man dann ja lassen da es egal ist was sie tut.

Edit: und würdest du die Methode die den Produktterm bilden soll dann ZipZip(...) nennen ?

 

[mrBrown]

Was hat die Benennung mit der Programmiersprache zu tun ? Und ist Java hier nun eine funktionale Sprache ? Und davon abgesehen tut die Methode das falsche. Aber den Namen Zip kann man dann ja lassen da es egal ist was sie tut.

Einiges - weiß man aber eigentlich auch, wenn man sich mal damit beschäftigt...

Und die Funktion tut genau das, was der Name sagt - dass es nicht das in der Aufgabe gefordert ist, ist was anderes

 

[GermanOakheart]

Der Koeffizient ist a und der Exponent b.
Ich hatte gerade das Problem das er bei 0 gemeckert hat, hab nicht dran gedacht null zu schreiben

Der Code on Freak funktioniert ja, danke dafür, aber ich kriege es noch nicht hin t3 in der Main zu "greifen".
Wenn ich in der Main weiter mache, möchte ich mit einem Setter den x-Wert von t3 setzen (der Setter läuft) und den ganzen Term dan ausrechnen (die method läuft auch).
Aber wen ich jetzt sage:
t3.Set(2);
erkennt er t3 immer noch nicht obwohl ich ihn doch zurück gebe?

Zip war eine spontan Idee weil ich nicht an das zusammenrechnern sondern eher an das zusammepacker beider Terme gedacht habe

 

[JStein52]

sondern eher an das zusammepacker beider Terme gedacht habe

Trotzdem, wie nennst du dann die Methode die den Produktterm bilden soll ? Ist auch eine Zusammenfassung. Und deine Zip tut nicht was sie soll !

 

[JStein52]

Auch wenn @mrBrown jetzt abhebt, aber du so solltest deine Bennenungen noch mal überdenken, und wozu brauchst du Set(..) du brauchst eine Methode calculate(17);

 

[JStein52]

Und deine Zip bruacht keine zwei Parameter sondern einen. Und sie bildet dann die Summe von this und dem übergebenen Term.

Du könntest dann schreiben:

Term t1 = new Term(3,4);
Term t2 = new Term(4,4);
Term t3 = t1.produktTerm(t2); // oder summenTerm, je nachdem was du nun willst
t3.calculate(17);

 

[mrBrown]

Auch wenn @mrBrown jetzt abhebt

Der Boden ist halt irgendwann langweilig, so'n Jetpack ist da schon cool...

 

[GermanOakheart]

das mit dem Set ist wohl nur ein Zwischenschritt, da hast du Recht, den könnte ich rauslassen. Den Produktterm bilde ich mit calc, für calculation. Die richtige Bezeichnung sollte ich mir wohl angewöhnen allerdings ist das Programm so klein und mit Kommentaren gespickt das das kein Problem sein sollte.

Ich kann die Methode ja nicht in produktTerm oder summenTerm umbennenen, da ich den Exponenten addieren und die Koeffizienten multiplizieren muss. Deswegen zwei Werte rein und zwei Werte raus.

 

[mrBrown]

Ich kann die Methode ja nicht in produktTerm oder summenTerm umbennenen, da ich den Exponenten addieren und die Koeffizienten multiplizieren muss. Deswegen zwei Werte rein und zwei Werte raus.

Nein, zum einfachen summieren zweier Termine musst du das nicht machen - da lieg der Fehler in der Methode.

Übergeben musst du wirklich nur eins - das zweite ist this

 

[JStein52]

allerdings ist das Programm so klein und mit Kommentaren gespickt das das kein Problem sein sollte.

Ja, aber es ist noch ziemlich falsch. Aus deinem Codeschnipsel schliesse ich dass deine Klasse Term auch Attribute t1 und t2 besitzt ? wozu denn das ? Und zwei Werte rein und zwei raus ist eben falsch.

 

[GermanOakheart]

Ich brauche aber die Werte von a von t1 und t2 und die Werte von b von t1 und t2. Dann gebe ich entweder die Werte für a und b für t3 aus oder erstelle in der MEthode bereits t3 mit den vorhandenen Werten.

 

[mrBrown]

Du brauchst du Werte von this und die von einem übergebenem Term, aus denen kannst du den neuen berechnen.
Übergebe musst du aber eben nur einen, den andere hast du ja schon als this


Oder andere Frage: weist du was this ist?

 

[JStein52]

public class Term {
private int koeffizient;
private int exponent;
   public(Term(...) { /// geeignete Konstruktoren
   }
   public Term summenTerm(Term t) {
             // drumrum mal weggelassen
             Term summe = new Term(this.koeffizient+t.koeffizient, this.exponent);
             return summe;
    }

 

[GermanOakheart]

Naja, this verwende ich damit keine Konflikte bei gleichen Namen enstehen.

 

[JStein52]

this verwende ich damit keine Konflikte bei gleichen Namen enstehen

Ist zwar nicht falsch aber lange nicht alles was du dazu wissen solltest.

 

[JStein52]

Lies vielleicht nochmal hier: http://openbook.rheinwerk-verlag.de...tml#dodtpfad8dec2-3342-4f23-a2b0-aacb03d77cd3