Java Backend

[Jw456]

Jetzt muss ich im Prinzip nur noch die set Methode erstellen oder?

    public void setzFeld(int zeile, int spalte, Spielfeld wert) {
       
       
           
        }

So ist ja der Aufbau. Die Zuweisung kann ich doch bestimmt mit der Datenklasse machen oder? Also wieder eine Datenklasse erstellen?

nein eine neue DatenKlasse brauchst du nicht du kannst aber das Objekt was du aus der User Eingabe Methode zurück bekommst gleich an die Methode Übergeben. Zeile Spalte sind ja in dem Objekt.

 

[kneitzel]

Jetzt muss ich im Prinzip nur noch die set Methode erstellen oder?

    public void setzFeld(int zeile, int spalte, Spielfeld wert) {
       
       
           
        }

So ist ja der Aufbau. Die Zuweisung kann ich doch bestimmt mit der Datenklasse machen oder? Also wieder eine Datenklasse erstellen?

Du kannst eine Methode schreiben, die ein Feld setzt. Die sollte dann in der Klasse Spielbrett (Du machst es ja gerne auf Deutsch, ich meine die Klasse Board, die ich schon einmal skizziert habe) sein. Und statt zeile/spalte wäre dann eine Instant der Klasse Position (oder wie immer du die gerne nennen möchtest, aber bitte nicht Datenklasse!) zu übergeben.

 

[Jw456]

nein eine neue DatenKlasse brauchst du nicht du kannst aber das Objekt was du aus der User Eingabe Methode zurück bekommst gleich an die Methode Übergeben. Zeile Spalte sind ja in dem Objekt.

ergenzung
PS du willst doch auch einen Spieler auf das Feld setzen. also soltest du auch den Spieler übergeben.
den du auf das Feld setzen willst.

 

[kneitzel]

ergenzung
PS du willst doch auch einen Spieler auf das Feld setzen. also soltest du auch den Spieler übergeben.
den du auf das Feld setzen willst.

Den hat er ja im Parameter wert. Da stehst ja ein Spieler oder eben das leere Feld drin.

 

[Jw456]

Den hat er ja im Parameter wert. Da stehst ja ein Spieler oder eben das leere Feld drin.

ich denke das ist sein Board nicht der Payer
er wird wohl Spielfeld und Spieler daraus gemacht haben.

edit: schaue post #22

 

[kneitzel]

Die Methode wird hoffentlich Bestandteil des Spielbretts / Boards sein und dazu dienen, ein einzelnes Feld zu setzen. Und das wären - nach meinem Vorschlag, ein Wert aus der Menge FIRST_PLAYER, SECOND_PLAYER und EMPTY.

 

[Jw456]

Die Methode wird hoffentlich Bestandteil des Spielbretts / Boards sein und dazu dienen, ein einzelnes Feld zu setzen. Und das wären - nach meinem Vorschlag, ein Wert aus der Menge FIRST_PLAYER, SECOND_PLAYER und EMPTY.

"wert" ist aber vom Datentyp Spielfeld also das Board . wo ist jetzt Spieler 1 oder 2 ?

das Board ist eine Instanzvariable die braucht er nicht zu übergeben . aber ob es sich um Spieler 1 oder 2 handelt schon. Er will ja den Spieler auf das mit Zeile und Spalte angeben Feld setzen.

 

[Jw456]

@Maxel33​

Um hier den Überblick zubehalten wäre es wohl gut deine bisherige gesamte Struktur zu zeigen.

​

 

[Maxel33]

Hallo, sorry das ich gestern abend nicht mehr konnte.

Hier mein Code:

import java.util.Scanner;

import com.sun.java.util.jar.pack.Instruction.Switch;

public class Spielfeld {
    
    Eingabe[][] array1 = new Eingabe[3][3];
    
    
    public Spielfeld() {
    for(int zeile = 0; zeile < array1.length; zeile++) {
        for(int spalte = 0; spalte < array1.length;spalte++) {
        array1[zeile][spalte] = Eingabe.EMPTY;
        if(spalte < 2) {
            System.out.print("|");
        }
    }
        
        System.out.println();
        if(zeile < 2) {
        System.out.println("------");
        }
    }
}
    
    public void print() {
        for(int zeile = 0; zeile < array1.length; zeile++) {
            for(int spalte = 0; spalte < array1.length;spalte++) {
            array1[zeile][spalte] = Eingabe.EMPTY;
            if(spalte < 2) {
                System.out.print("|");
            }
            switch(array1[zeile][spalte]) {
            case PLAYER_ONE:
                System.out.print("O");
                break;
            case PLAYER_TWO:
                System.out.print("X");
                break;
            case EMPTY:
                System.out.print(" ");
                break;
            }
        }
            
            System.out.println();
            if(zeile < 2) {
            System.out.println("------");
            }
        }
    }
    
    public Datenklasse eingabeDerStelle() {
        System.out.println("Gebe die Stelle an: ");
        Scanner scan = new Scanner(System.in);
        int zeile = scan.nextInt();
        int spalte = scan.nextInt();
        Datenklasse dt = new Datenklasse(zeile, spalte);
        return dt;
    }
    
    public boolean PrüfenObWertGültigIst(int zeile, int spalte) {
        if(zeile >=  0) {
            return true;
        }else if(zeile <=2) {
            return true;
        }else if(spalte >=0) {
            return true;
        }else if(spalte <= 2) {
            return true;
        }else {
            return false;
        }

    }

    public boolean PrüfungEinesLeerenFeldes(int zeile, int spalte) {
        
        //entweder so
        return array1[zeile][spalte] == Eingabe.EMPTY;
        
        //oder so
        
//        if(array1[zeile][spalte] == Eingabe.EMPTY) {
//            return true;
//        }else {
//            return false;
//        }
    }
    
    
    public void setzFeld(int zeile, int spalte, Spielfeld wert) {
        
        
            
        }
    }

 

[Maxel33]

Du kannst eine Methode schreiben, die ein Feld setzt. Die sollte dann in der Klasse Spielbrett (Du machst es ja gerne auf Deutsch, ich meine die Klasse Board, die ich schon einmal skizziert habe) sein. Und statt zeile/spalte wäre dann eine Instant der Klasse Position (oder wie immer du die gerne nennen möchtest, aber bitte nicht Datenklasse!) zu übergeben.

Das verstehe ich leider nicht. Also soll ich jetzt meine Datenklasse an die Methode übergeben?
Und wie kann ich genau an diesem Punkt/Koordinaten dann das X oder O setzen?

 

[Maxel33]

Eventuell so?

    public void setzFeld(int zeile, int spalte, Eingabe value) {
        value = array1[zeile][spalte];
        System.out.print(array1[zeile][spalte]);
        }

 

[Jw456]

Ich sehe einiges was nicht so passt.

Der user macht eine Eingabe in der Methode eingabeDerStelle() zurück bekommst du ein Objekt vom Typ „Datenklasse“
In dem Objekt ist die Zeile und Spalte.

Jetzt willst du die Eingabe prüfen in „PrüfenObWertGültigIst“ aber was übergingst du Zeile und Spalte. Woher hast du die zwei werte die sind doch in dem Objekt.

Was musst du übergeben das Objekt von der User Eingabe.

Bei den anderen Methoden das gleiche.

 

[kneitzel]

Eventuell so?

    public void setzFeld(int zeile, int spalte, Eingabe value) {
        value = array1[zeile][spalte];
        System.out.print(array1[zeile][spalte]);
        }

Die Zuweisung ist falsch herum. Ansonsten ist das prinzipiell richtig (Aber die Bezeichner sind gewöhnungsbedürftig).

 

[Jw456]

Die Zuweisung ist falsch herum. Ansonsten ist das prinzipiell richtig (Aber die Bezeichner sind gewöhnungsbedürftig).

ja richtig falschrum aber woher hat er die zeile und Spalte?

import java.util.Scanner;

import com.sun.java.util.jar.pack.Instruction.Switch;

public class Spielfeld {
    
    Eingabe[][] array1 = new Eingabe[3][3];
    
    
    public Spielfeld() {
    for(int zeile = 0; zeile < array1.length; zeile++) {
        for(int spalte = 0; spalte < array1.length;spalte++) {
        array1[zeile][spalte] = Eingabe.EMPTY;
        if(spalte < 2) {
            System.out.print("|");
        }
    }
        
        System.out.println();
        if(zeile < 2) {
        System.out.println("------");
        }
    }
}
    
    public void print() {
        for(int zeile = 0; zeile < array1.length; zeile++) {
            for(int spalte = 0; spalte < array1.length;spalte++) {
            array1[zeile][spalte] = Eingabe.EMPTY;
            if(spalte < 2) {
                System.out.print("|");
            }
            switch(array1[zeile][spalte]) {
            case PLAYER_ONE:
                System.out.print("O");
                break;
            case PLAYER_TWO:
                System.out.print("X");
                break;
            case EMPTY:
                System.out.print(" ");
                break;
            }
        }
            
            System.out.println();
            if(zeile < 2) {
            System.out.println("------");
            }
        }
    }
    
    public Datenklasse eingabeDerStelle() {
        System.out.println("Gebe die Stelle an: ");
        Scanner scan = new Scanner(System.in);
        int zeile = scan.nextInt();
        int spalte = scan.nextInt();
        Datenklasse dt = new Datenklasse(zeile, spalte);
        return dt;
    }
    
    public boolean PrüfenObWertGültigIst( Datenklasse dt) {
                    
        if(dt.getZeile() >=  0) {
            return true;
        }else if(dt.getZeile() <=2) {
            return true;
        }else if(dt.getSpalte() >=0) {
            return true;
        }else if(dt.getSpalte() <= 2) {
            return true;
        }else {
            return false;
        }

    }

    
      public void setzFeld(Datenklasse dt , Eingabe value) {
          int zeile = dt.getZeile();
          int spalte =dt.getSpalte(); 
          
        array1[zeile][spalte] = value;
        System.out.print(array1[zeile][spalte]);
        }
    
    
    }

 

[Maxel33]

Ich sehe einiges was nicht so passt.

Der user macht eine Eingabe in der Methode eingabeDerStelle() zurück bekommst du ein Objekt vom Typ „Datenklasse“
In dem Objekt ist die Zeile und Spalte.

Jetzt willst du die Eingabe prüfen in „PrüfenObWertGültigIst“ aber was übergingst du Zeile und Spalte. Woher hast du die zwei werte die sind doch in dem Objekt.

Was musst du übergeben das Objekt von der User Eingabe.

Bei den anderen Methoden das gleiche.

Das klingt logisch. Im Prinzip so?

    public boolean PrüfenObWertGültigIst(Position dt) {
        if(dt.getZeile() >=  0) {
            return true;
        }else if(dt.getZeile()<=2) {
            return true;
        }else if(dt.getSpalte() >=0) {
            return true;
        }else if(dt.getSpalte() <= 2) {
            return true;
        }else {
            return false;
        }

    }

 

[kneitzel]

Die Logik der Methode ist einfach weiterhin falsch. Und wir hatten das schon mehrfach benannt. Ob Du nun Zeile / Spalte einzeln übergibst oder in einer Position ist erst einmal egal. Die Logik solltest Du erst einmal korrigieren.

Bei dir ist jeder Wert korrekt, denn entweder ist eine Zahl >= 0 oder eben <=2. Und die Spalte prüfst Du niemals bei der Logik....

 

[Jw456]

Ja die Logik habe ich nicht angepasst das ist dein Part.

 

[Maxel33]

Die Logik der Methode ist einfach weiterhin falsch. Und wir hatten das schon mehrfach benannt. Ob Du nun Zeile / Spalte einzeln übergibst oder in einer Position ist erst einmal egal. Die Logik solltest Du erst einmal korrigieren.

Bei dir ist jeder Wert korrekt, denn entweder ist eine Zahl >= 0 oder eben <=2. Und die Spalte prüfst Du niemals bei der Logik....

Also interessieren praktisch nur die false Werte?

 

[kneitzel]

Also interessieren praktisch nur die false Werte?

Wie Du es genau baust, kannst Du Dir überlegen.

Es gibt ein 08/15 Verfahren, das man nutzen kann:
- In der Methode erst alles, was eindeutig ist, abhandeln mit einem return true / false;
- Dann am Ende das übriggebliebene zurückgeben.

Das ginge dann über etwas wie

boolean isPositionValid(Position pos) {
    if (pos.getSpalte() < 0) return false;
    if ......
    if ......
    
    return true;
}

Aber das ist so nicht zwingend notwendig. Das ist viel Code für wenig Funktionalität.

Es reicht ja schon, die Kriterien zu verknüpfen. Was muss denn erfüllt sein?

Spalte größer gleich 0 UND Spalt kleiner gleich 2 UND Zeile größer gleich 0 und Zeile <= 2:

boolean isPositionValid(Position pos) {
    return pos.getSpalte() >= 0 && pos.getSpalte() <= 2 && pos.getZeile() >= 0 && pos.getZeile() <= 2;
}

Diese logische Verknüpfung solltest Du Dir noch einmal im Detail ansehen.

 

[Maxel33]

Wie Du es genau baust, kannst Du Dir überlegen.

Es gibt ein 08/15 Verfahren, das man nutzen kann:
- In der Methode erst alles, was eindeutig ist, abhandeln mit einem return true / false;
- Dann am Ende das übriggebliebene zurückgeben.

Das ginge dann über etwas wie

boolean isPositionValid(Position pos) {
    if (pos.getSpalte() < 0) return false;
    if ......
    if ......
   
    return true;
}

Aber das ist so nicht zwingend notwendig. Das ist viel Code für wenig Funktionalität.

Es reicht ja schon, die Kriterien zu verknüpfen. Was muss denn erfüllt sein?

Spalte größer gleich 0 UND Spalt kleiner gleich 2 UND Zeile größer gleich 0 und Zeile <= 2:

boolean isPositionValid(Position pos) {
    return pos.getSpalte() >= 0 && pos.getSpalte() <= 2 && pos.getZeile() >= 0 && pos.getZeile() <= 2;
}

Diese logische Verknüpfung solltest Du Dir noch einmal im Detail ansehen.

Okay, vielen Dank für die nette und aufschlussreiche Antwort. Ich wusste nicht einmal, dass man von der Syntax her sowas machen kann.

 

[Maxel33]

Der Rest der anderen Methoden müsste aber stimmen, oder?

 

[Maxel33]

Mein weiteres Vorgehen wird jetzt wie folgt aussehen:
1) ich erstelle eine neue Klasse "Spielregeln". In dieser werde ich dann die Regeln überbringen durch Methoden. -> 3 in einer Reihe, 3 in Quer usw.
Dann soll eine Win Nachricht ausgegeben werden. Dazu nutze ich auch wieder eine Methode. Man kann dafür ja eventuell wieder die if Anweisungen nutzen.

 

[Jw456]

Das klingt logisch. Im Prinzip so?

    public boolean PrüfenObWertGültigIst(Position dt) {
        if(dt.getZeile() >=  0) {
            return true;
        }else if(dt.getZeile()<=2) {
            return true;
        }else if(dt.getSpalte() >=0) {
            return true;
        }else if(dt.getSpalte() <= 2) {
            return true;
        }else {
            return false;
        }

    }

Bei diesen Code überlege dir was return macht.
Es beendet die Methode.
Wenn das erste if true ist wird die Methode beendet.
Sollte sie aber nicht denn du bist mit deinen prüfen noch nicht fertig.

Was passiert wenn 5 eingehen wird.?

 

[Maxel33]

Bei diesen Code überlege dir was return macht.
Es beendet die Methode.
Wenn das erste if true ist wir die Methode beendet.
Sollte sie aber nicht denn du bist mit deinen prüfen noch nicht fertig.

Was passiert wenn 5 eingehen wird.?

Ach jetzt verstehe ich. Das durchläuft praktisch diese If Anweisung und wenn schon die erste Anweisung true ist. Wird abgebrochen?

Das klingt natürlich super logisch..

Also muss man immer erst die Bedingungen aufzählen, die ausgeschlossen werden sollen.

 

[Jw456]

Oder das Ergebnis der if in einer Variablen festhalten.