OOP Methoden mit einander Verknüpfen

[Zeppi]

Moin, ich möchte zwei Methoden mit einander Verknüpfen, bzw. eine Referenz einer Klasse in einer anderen verwenden.
Jetzt möchte ich quasi eine get Methode machen in klasse2, in der die Werte von klasse1 als Referenz genommen werden als Index.
[CODE lang="java" title="methode in klasse2"]public klasse1 get(int i) {
klasse1[] x = new klasse1; //so wäre meine erste Überlegung,
// aber klappt nicht
return x;
}[/CODE]
[CODE lang="java" title="Main"]
public static void main(String[] args) {
final int ANZAHL = 10;
final int X = 9;
final int Y = 7;

klasse2 a = new klasse2(ANZAHL, X, Y);
for (int i=0; i<ANZAHL; i++) {
System.out.println(a.get(i).x+", "+a.get(i).y);[/CODE]
[CODE lang="java" title="klasse1"]public class klasse1 {
int x;
int y;
public klasse1(int X, int Y) {
java.util.Random zufall = new java.util.Random();
this.x= zufall.nextInt(X);
this.y= zufall.nextInt(Y);
}[/CODE]
[CODE lang="java" title="klasse2"]public class klasse2 {
klasse1[] x;

public klasse2(int ANZAHL, int X, int Y) {
klasse1 x[] = new klasse1[ANZAHL];
for (int i = 0; i < ANZAHL; i++) {
x = new klasse1(X,Y);
}

}[/CODE]


Falls jemand ein Tipp hat oder etwas anderes auffällt, wäre ich dankbar über eine Rückmeldung

 

[kneitzel]

Also ich verstehe erst einmal nicht, was Du genau machen willst. Wo soll der erste Code mit der get Methode denn hin?

Aber betrachten wir einmal Deinen Code etwas im Detail:

public klasse1 get(int i) {
    klasse1[] x = new klasse1[i];
        return x[i];
    }

Du erzeugst ein neues Array vom Typ klasse1 mit Größe i. Das bedeutet das Array hat die Elemente 0, 1, ... i-1.
Dann versuchst du das Element mit Nr i zurück zu geben -> IndexOutOufBoundsException, da das letzte Element i-1 ist und nicht i!
Desweiteren macht es kein Sinn, denn alle Elemente des neuen Arrays sind null.... Evtl. ist das ja in der Klasse, die eine Instanzvariable x hat vom Typ Array von klasse1 - dann wäre das get evtl. nur das einfach return, ds Du eh schon hast?

public class klasse2 {
    klasse1[] x;

    public klasse2(int ANZAHL, int X, int Y) {
    klasse1   x[] = new klasse1[ANZAHL];
        for (int i = 0; i < ANZAHL; i++) {
            x[i] = new klasse1(X,Y);
        } 
      
    }

Klasse2 hat eine Instanzvariable x, aber im Konstruktor initialisierst Du diese Variable nicht sondern erstellst eine lokale Variable mit gleichem Namen und gleichem Typ. Da willst Du das "klasse1" vor dem x und die eckigen Klammern nach dem x vielleicht löschen. Damit würde dann x initialisiert und gefüllt. Zusammen mit der ersten get Methode hättest Du dann etwas, das was machen würde ...

Weiterhin ein paar Hinweise:
Klassen fangen mit einem Großbuchstaben an, also Klasse1, Klasse2.
Parameter und Variablem fangen mit einem kleinen Buchstaben an, also anzahl, x, y
ANZAHL wäre die Schreibweise für eine Konstante (Also komplett Uppercase und dann zwischen Wortteilen ein _).

 

[Zeppi]

Also ich verstehe erst einmal nicht, was Du genau machen willst. Wo soll der erste Code mit der get Methode denn hin?

Aber betrachten wir einmal Deinen Code etwas im Detail:

public klasse1 get(int i) {
    klasse1[] x = new klasse1[i];
        return x[i];
    }

Du erzeugst ein neues Array vom Typ klasse1 mit Größe i. Das bedeutet das Array hat die Elemente 0, 1, ... i-1.
Dann versuchst du das Element mit Nr i zurück zu geben -> IndexOutOufBoundsException, da das letzte Element i-1 ist und nicht i!
Desweiteren macht es kein Sinn, denn alle Elemente des neuen Arrays sind null.... Evtl. ist das ja in der Klasse, die eine Instanzvariable x hat vom Typ Array von klasse1 - dann wäre das get evtl. nur das einfach return, ds Du eh schon hast?

public class klasse2 {
    klasse1[] x;

    public klasse2(int ANZAHL, int X, int Y) {
    klasse1   x[] = new klasse1[ANZAHL];
        for (int i = 0; i < ANZAHL; i++) {
            x[i] = new klasse1(X,Y);
        }
     
    }

Klasse2 hat eine Instanzvariable x, aber im Konstruktor initialisierst Du diese Variable nicht sondern erstellst eine lokale Variable mit gleichem Namen und gleichem Typ. Da willst Du das "klasse1" vor dem x und die eckigen Klammern nach dem x vielleicht löschen. Damit würde dann x initialisiert und gefüllt. Zusammen mit der ersten get Methode hättest Du dann etwas, das was machen würde ...

Weiterhin ein paar Hinweise:
Klassen fangen mit einem Großbuchstaben an, also Klasse1, Klasse2.
Parameter und Variablem fangen mit einem kleinen Buchstaben an, also anzahl, x, y
ANZAHL wäre die Schreibweise für eine Konstante (Also komplett Uppercase und dann zwischen Wortteilen ein _).

Klasse, mit deinen Tipps läuft das Programm jetzt.
Mache leider als Anfänger noch zu viele "einfache Fehler".

 

[kneitzel]

Mache leider als Anfänger noch zu viele "einfache Fehler

Das ist am Anfang normal. Einfach dran bleiben und versuchen, viel praktisch zu machen!

Und wir helfen gerne.

 

[M.L.]

Anfänger ... "einfache Fehler"

Man sollte für sich selbst inkl. Nachfolger leserliche Programme schreiben, Stichwort auch "Clean Code Developer" und "Coding Conventions". Bemühungen, auf "Teufel-komm-raus" möglichst viel komplizierten (Java)Code mit x Implikationen und Abhängigkeiten zu schreiben, sollte man also reduzieren. Vor allem denke man an den externen Eindruck, wenn man irgendwann selbst nicht mehr erklären, warum ausgerechnet diese Vorgehensweise (im Ggs. zu Alternativen) gewählt wurde...

 

[kneitzel]

Man sollte für sich selbst inkl. Nachfolger leserliche Programme schreiben, Stichwort auch "Clean Code Developer" und "Coding Conventions". Bemühungen, auf "Teufel-komm-raus" möglichst viel komplizierten (Java)Code mit x Implikationen und Abhängigkeiten zu schreiben, sollte man also reduzieren. Vor allem denke man an den externen Eindruck, wenn man irgendwann selbst nicht mehr erklären, warum ausgerechnet diese Vorgehensweise (im Ggs. zu Alternativen) gewählt wurde...

Oder kurz zusammen gefasst: Schreibe Code immer so, als müsste ein Axt schwingender Wahnsinniger Deinen Code lesen, der weiß, wo Du wohnst 🙂

 

[M.L.]

Axt

Der "Wahnsinnige" könnte ja im Gegenteil über die geringe Komplexität des Codes verärgert sein... Ein aktuelles Beispiel, wie man die Axt an zuviel Code in einer Methode ansetzen kann: https://medium.com/javarevisited/a-key-to-clean-code-small-methods-cff120538eba