Konstruktor Problem

[Brexoi]

Hallo liebe Community,

ich hab folgendes Problem, ich muss ein UML-Diagramm in einen Code umsetzen und halt dann die ganzen Methoden und Konstruktoren mit leben befüllen usw.
Nun ist unten eine von fünf Klassen eingefügt wurden, wobei es hauptsächlich nur um die eine Klasse momentan geht.

Ich brauche folgendes:
Der parametrisierte Konstruktor soll die übergebenen Werte den entsprechenden Attributen der Klasse zuweisen. Der Konstruktor soll prüfen ob die übergebenen Werte im zulässigen Bereich liegen und falls nicht, die Attribute auf die Grenzwerte des zulässigen Bereiches setzten. Der zulässige Bereich liegt zwischen 0 und maxpos. Der Standardkonstruktor soll die Koordinaten mit dem Wert 511 initialisieren.
Die Methoden getXpos, bzw. getYpos sind die Getter-Methoden der Attribute xpos und ypos.
Die Methode setXYpos soll die übergebenen Parameter den Attributen zuweisen.
Die Methode verschiebeUm soll eine Verschiebung des Punktes um die als Parameter übergebenen Werte durchführen und als Rückgabe den verschobenen Punkt liefern.

Ich hab es mit einer IF/ELSE anweisung versucht aber bekommen nur fehler ohne die Anweisungen so wie der Code unten steht, funktioniert das Programm (zwischentest) einwandfrei, aber nicht wie es soll,..
Für Vorschläge und hilfestellungen währe ich sehr dankbar.

Ps: Ich habe bewusst meine ansätze zur IF/ELSE anweisung weg gelassen, damit man sich nicht darauf beschränkt, über verschiedene lösungswege oder ansätze bin ich immer dankbarer, da ich vergleichen kann und vielleicht das eine oder andere, anders in meinem vorhaben einsetzen könnte woran ich vielleicht in dem augenblick nicht gedacht habe.

public class Punkt {
   
   private final static int maxpos = 1023;
   private int xpos;
   private int ypos;

   public Punkt(int xpos, int ypos) {
     this.xpos = xpos;
     this.ypos = ypos;
   }

   public Punkt() {
     this.xpos = 0;
     this.ypos = 0;
   }

   public int getXpos() {
     return this.xpos;
   }

   public void setXYpos(int xpos, int ypos) {
     this.xpos = xpos;
     this.ypos = ypos;
   }

   public int getYpos() {
     return this.ypos;
   }

   public Punkt verschiebeUm(int deltax, int deltay) {
     return new Punkt(xpos + deltax, ypos + deltay);
   }

   public Punkt verschiebeUm(Punkt deltaxy) {
     return new Punkt(xpos + deltaxy.xpos, ypos + deltaxy.ypos);
   }

   public String toString() {
     return "(" + xpos + "," + ypos + ")";
   }
}

 

[InfectedBytes]

Ich verstehe dein Problem nicht ganz, du sollst doch nichts anderes machen als die Grenzen prüfen:

public Punkt(int xpos, int ypos) {
  if(xpos<0) {
    this.xpos = 0;
  } else if(xpos>maxpos) {
    this.xpos=maxpos;
  } else {
    this.xpos=xpos;
  }
  //das gleiche für ypos
}

Oder ist deine Frage eigentlich eine andere gewesen?

 

[Brexoi]

Nein, ist schon richtig, hatte kein "this.xpos = 0" , bin noch so recht neu in der Welt von Java, könntest du/jemand mir erklären warum ich an dem Punkt nochmal this.xpos = 0 setze?

 

[InfectedBytes]

Da primitive Klassenvariablen mit 0 initialisiert werden, könnte man sich das this.xpos=0 auch sparen, jedoch sollte man auch den Fall explizit aufschreiben, da es somit übersichtlicher ist.
Folgendes würde auch gehen, ist aber ggf. etwas "unsauber"

public Punkt(int xpos, int ypos) {
  if(xpos<0) {
    //nichts machen, wodurch der initialwert von 0 beibehalten wird
  } else if(xpos>maxpos) {
    this.xpos=maxpos;
  } else {
    this.xpos=xpos;
  }
  //das gleiche für ypos
}

Ansonsten wird ein Parameter namens xpos deklariert, welcher nur eine lokale Variable ist, welche ungleich der Klassenvariable xpos ist.

 

[Brexoi]

Okay danke dir für die hilfe, eine frage hätte ich noch, ich bekomme ja nun durch den ganzen code "Koordinaten" als ergebnis ausgespuckt, wie kann ich diese mit einem Wert von bspw 123 initialisieren?

 

[InfectedBytes]

So wie du es bereits machst:

   public Punkt() {
     this.xpos = 0;
     this.ypos = 0;
   }

Bloß dann eben mit 123

 

[Brexoi]

Dann kann ich ihn auch direkt mit:

new Punkt(123, 123);//Aufruf mit 123 initialisiert mit oben verwendeten Code

Aufrufen, oder ist es nicht das selbe? Falls doch, ist es nicht ganz dass, was ich brauche oder ich verstehe die Aufgabe nicht Richtig,.. :noe:

In meiner Übungsaufgabe steht:
Der parametrisierte Konstruktor soll die übergebenen Werte den entsprechenden Attributen der Klasse zuweisen. Der Konstruktor soll prüfen ob die übergebenen Werte im zulässigen Bereich liegen und falls nicht, die Attribute auf die Grenzwerte des zulässigen Bereiches setzten. Der zulässige Bereich liegt zwischen 0 und maxpos. Der Standardkonstruktor soll die Koordinaten mit dem Wert 511 initialisieren.

Ansonsten ist folgendes mein Problem, ich hab in der main class folgenden Code:

public class Hauptprogramm {
    public static void main(String[] args) {
        
        
       System.out.println("Aufgabe 2a:");
        Punkt A = new Punkt();
        Punkt B = new Punkt(123456, 654321);
        Punkt C = new Punkt(123,-456);
        Punkt D = new Punkt(654,321);
        
        System.out.println("A=" + A);
        System.out.println("B=" + B);
        System.out.println("C=" + C);
        System.out.println("D=" + D);
        System.out.println("----------");
    }
}

Als Ausgabe soll rauskommen:
A=(511,511)
B=(1023,1023)
C=(123,0)
D=(654,321)

Aber es kommt nur folgendes raus:
A=(0,0)
B=(1023,1023)
C=(123,0)
D=(654,654)

Aber das hilft alles nicht bei meinem Problem ausser das ich bei Punkt A=511 stehen habe aber die anderen zahlen nicht verändert werden (bei ausgabe) und das ich nicht auf das vorgegebene Ergebnis somit komme, versteht jemand was ich meine oder mein Problem

Oder kann es einfach daran liegen das meine verschiebeUm methode nicht gefüllt ist? Da sehe ich gerade in der Aufgabenstellung das die Methode folgendes machen soll:
Die Methode verschiebeUm soll eine Verschiebung des Punktes um die als Parameter übergebenen Werte durchführen und als Rückgabe den verschobenen Punkt liefern. Beachten Sie: Die Methode kann als Parameter entweder zwei ganze Zahlen oder ein Objekt vom Typ Punkt erhalten.

 

[InfectedBytes]

Die Lösung zu deinem Konstruktor Problem habe ich doch hier schon beantwortet:

So wie du es bereits machst:

   public Punkt() {
     this.xpos = 0;
     this.ypos = 0;
   }

Bloß dann eben mit 123

Du musst nur anstatt 0 eben deine 511 reinschreiben.

Und bei den anderen machst du vermutlich folgendes falsch:

//....
this.ypos = xpos;
//...

Da muss natürlich schon die y Koordinate genutzt werden

 

[Brexoi]

Die Lösung zu deinem Konstruktor Problem habe ich doch hier schon beantwortet:


Du musst nur anstatt 0 eben deine 511 reinschreiben.

Und bei den anderen machst du vermutlich folgendes falsch:

//....
this.ypos = xpos;
//...

Da muss natürlich schon die y Koordinate genutzt werden

DANKE, ja ich hab es ja mit dem WERT 511 probiert aber es hat sich in der ausgabe nichts geändert ausser in Punkt A deswegen dachte ich, es liegt nicht daran, es war dennoch richtig und lag an ypos = xpos -.-
Ich hab es mir den ganzen tag mehrmals angeschaut und nicht gesehen,..