Wie kommt dieses Ergebnis zu stande

[Nico1999]

ich würde gerne wissen wie der Computer auf dieses wierde ergebnis kommt
hier ist mein Code
alle sachen die ich verwendet habe sind getestet also sollte funktieren

    protected HashMap<Integer, Field[]> createBoard() {
        HashMap <Integer, Field[]>startingBoard=new HashMap<>();
        Field[] row=new Field[brettGröße];
        for(int collumnumber=0;collumnumber<this.brettGröße;collumnumber++) {
            startingBoard.put(collumnumber, row);
            for(int rownumber=0;rownumber<this.brettGröße;rownumber++) {
                if(isBlackField(collumnumber, rownumber)) {
                    if(collumnumber<this.brettGröße/2-1) {
                        startingBoard.get(collumnumber)[ rownumber]=new Field(new Coordinates(collumnumber, rownumber), new Figur(Typ.PAWN, Color.BLACK), Color.BLACK);
                    }else if(collumnumber>this.brettGröße/2) {
                        startingBoard.get(collumnumber)[ rownumber]=new Field(new Coordinates(collumnumber, rownumber), new Figur(Typ.PAWN, Color.WHITE), Color.BLACK );
                    }else {
                        startingBoard.get(collumnumber)[ rownumber]=new Field(new Coordinates(collumnumber, rownumber), new Figur(), Color.BLACK );
                    }
                }else {
                    startingBoard.get(collumnumber)[ rownumber]=new Field(new Coordinates(collumnumber, rownumber), new Figur(), Color.WHITE );
                }
            }
        }return startingBoard;
        
    
    }

Der Output ist jetzt das er für die X koordinate nur 7 printet egal welches Feld
alles andere funktiert Danke schonmal für eure Antworten
(die Y koordinate macht er komplett Richtig )
und Ich wollte ma fragen wie viel Variablen ein Klasse maximal haben sollte

 

[mihe7]

Der Output ist jetzt das er für die X koordinate nur 7 printet egal welches Feld

Welcher Output? Und warum hast Du hier eine HashMap, wenn Du nur Indizes als Keys hast? Dann kannst Du auch einfach ein Array verwenden. Den Code kannst Du auch noch schöner aufteilen.

 

[Marinek]

Deine Modellierung eines Schachbretts ist extrem kompliziert.

Man kann ein Schachbrett simpel mit

Figur[][] brett = new Figur[][];

annehmen.

Dann spart man sich das Field und Coordinates.

Deine Frage kann aber hier nicht beantwortet werden, weil weder die Berechnung noch der Input bekannt sind.

Field[] row=new Field[brettGröße];
        for(int collumnumber=0;collumnumber<this.brettGröße;collumnumber++) {
            startingBoard.put(collumnumber, row); // Ist es Absicht, dass in jeder Spalte die gleiche Zeile ist!?

 

[yfons123]

Welcher Output? Und warum hast Du hier eine HashMap, wenn Du nur Indizes als Keys hast? Dann kannst Du auch einfach ein Array verwenden. Den Code kannst Du auch noch schöner aufteilen.

da ich ihm das ursprünglich vorgeschlagne habe um buchstaben + zahl zu haben... ala C6 un D4 also char als keys aber das wurde untershclagen

 

[Nico1999]

Welcher Output? Und warum hast Du hier eine HashMap, wenn Du nur Indizes als Keys hast? Dann kannst Du auch einfach ein Array verwenden. Den Code kannst Du auch noch schöner aufteilen.

da hast du recht später werde ich den code noch in weitere funktionen packen
wollte mal die HashMap ausprobieren

Field[] row=new Field[brettGröße];
        for(int collumnumber=0;collumnumber<this.brettGröße;collumnumber++) {
            startingBoard.put(collumnumber, row); // Ist es Absicht, dass in jeder Spalte die gleiche Zeile ist!?

das habe ich gemacht das ich keine NullpointerException bekomme

da ich ihm das ursprünglich vorgeschlagne habe um buchstaben + zahl zu haben... ala C6 un D4 also char als keys aber das wurde untershclagen

sry ich hatte dass mit den Buchstaben mal probiert aber ich wusste nicht wie ich mit denen rechnen soll

Das Ding ist halt ich hab mir das im debug mode angeguckt und wenn collumnumber sich erhöht auf 1 wird auch in dem
startingBoard.get(0)[0].getCoordinates().getX(), dass eigentlich schon richtig abgespeichert wurde jetzt auf 1 erhöht
}also wenn sich collumnumber auf 1 wird ein schon abgespeichertes Field im Nachhinnein verändern aber ich habe keine ahnung wo die herkommen soll?

 

[KonradN]

das habe ich gemacht das ich keine NullpointerException bekomme

Damit hast Du aber nur eine einzige row - und die hast Du mehrfach in der Liste eingefügt.
Somit überschreibst Du in der äußeren Schleife immer wieder die Werte der Row und Du hast am Ende nur die Werte vom letzten Durchgang.

Die Erzeugung der Row muss also in die Schleife:

        for(int collumnumber=0;collumnumber<this.brettGröße;collumnumber++) {
            Field[] row=new Field[brettGröße];
            startingBoard.put(collumnumber, row);

 

[mihe7]

wollte mal die HashMap ausprobieren

OK, man kann das Spielbrett natürlich als Menge von Feldern auffassen, die ihre Lage auf dem Spielbrett kennen. Für den Zugriff auf ein Feld über seine Koordinaten lässt sich eine Map verwenden, wobei es sich dann aber anbieten würde, auch Koordinaten auf Felder abzubilden.

Map<Coordinates, Field> board = new HashMap<>();

private void add(Field field) {
    board.put(field.getCoordinates(), field);
}

Ob das sinnvoll ist, ist eine ganz andere Frage. Für ein simples Schachbrett ist das wohl etwas übertrieben. Interessant kann so etwas zum Beispiel sein, wenn man große, "dünnbesetzte" Welten hat. Wenn man z. B. einen 1000 x 1000 x 1000 Würfel hat, in dem es 10.000 Felder gibt, dann wäre es ziemlich doof, für die 10.000 Felder Platz für 1.000.000.000 Array-Elemente zu reservieren.

 

[Nico1999]

Damit hast Du aber nur eine einzige row - und die hast Du mehrfach in der Liste eingefügt.
Somit überschreibst Du in der äußeren Schleife immer wieder die Werte der Row und Du hast am Ende nur die Werte vom letzten Durchgang.

Die Erzeugung der Row muss also in die Schleife:

        for(int collumnumber=0;collumnumber<this.brettGröße;collumnumber++) {
            Field[] row=new Field[brettGröße];
            startingBoard.put(collumnumber, row);

okay dass war mir nicht bewusst, dass das so einen großen unterschied macht.
danke

OK, man kann das Spielbrett natürlich als Menge von Feldern auffassen, die ihre Lage auf dem Spielbrett kennen. Für den Zugriff auf ein Feld über seine Koordinaten lässt sich eine Map verwenden, wobei es sich dann aber anbieten würde, auch Koordinaten auf Felder abzubilden.

Map<Coordinates, Field> board = new HashMap<>();

private void add(Field field) {
    board.put(field.getCoordinates(), field);
}

Ob das sinnvoll ist, ist eine ganz andere Frage. Für ein simples Schachbrett ist das wohl etwas übertrieben. Interessant kann so etwas zum Beispiel sein, wenn man große, "dünnbesetzte" Welten hat. Wenn man z. B. einen 1000 x 1000 x 1000 Würfel hat, in dem es 10.000 Felder gibt, dann wäre es ziemlich doof, für die 10.000 Felder Platz für 1.000.000.000 Array-Elemente zu reservieren.

wie wäre es den am platzsparenstens, weil später möchte ich auch noch einen algorithmus anwenden der mir für Dame(Spiel) alle möglichen Moves ausrechnet. Das angenehme an dem Feld Array ist ich kann erst mal alle Weißen Felder ausklammern weil auf denen steht ja eh nix drauf
ah und nice ich dachte man kann nur bei der einer Seite der HashMap eigene Klassen rein tun

 

[yfons123]

du kannst mit der hashmap genau das gleiche machen wie mit einem array da es nur ein Container außen rum ist mehr oder weniger

Speicher soll dich nicht interessieren


was für alle moves wichtig ist ist die Zeit, wenns 100 Jahre dauert hilft dir die Lösung relativ wenig

 

[mihe7]

wie wäre es den am platzsparenstens

Wie @yfons123 schon geschrieben hat: der Platz braucht Dich nicht zu interessieren. Wir reden hier von 64 Feldern.

Das angenehme an dem Feld Array ist ich kann erst mal alle Weißen Felder ausklammern weil auf denen steht ja eh nix drauf

So etwas wäre schon eher ein Argument, allerdings kann man das ganz leicht berechnen. Es gibt je Zeile 4 schwarze Felder, so dass sich unter der Annahme, dass (0,0) ein schwarzes Feld ist, der Spaltenindex für das (i+1)-te schwarze Feld mit 2*i + row % 2 berechnen lässt.

public void iterateBlackFields() {
    for (int row = 0; row < 8; row++) {
        for (int col = row % 2; col < 8; col += 2) {
            // (col, row) ist schwarzes Feld
        }
    }
}

ah und nice ich dachte man kann nur bei der einer Seite der HashMap eigene Klassen rein tun

Nö, das geht im Prinzip mit beliebigen Objekten. Einzige Voraussetzung ist, dass der equals/hashCode-Vertrag (zwei Objekte, die per equals gleich sind, müssen den gleichen hashCode liefern) eingehalten wird. Außerdem sollte man es tunlichst unterlassen, den Wert eines Schlüssels im Nachhinein zu ändern, weswegen sich unveränderliche Objekte besonders gut für Schlüssel eignen.

 

[Nico1999]

Nö, das geht im Prinzip mit beliebigen Objekten. Einzige Voraussetzung ist, dass der equals/hashCode-Vertrag (zwei Objekte, die per equals gleich sind, müssen den gleichen hashCode liefern) eingehalten wird. Außerdem sollte man es tunlichst unterlassen, den Wert eines Schlüssels im Nachhinein zu ändern, weswegen sich unveränderliche Objekte besonders gut für Schlüssel eignen.

also meine Coordinates sind final, aber jetzt wo ich das mit der einer HashMap <Coordinates, Figur> ausprobiert habe laufe ich jetzt in einen
java.lang.outOfMemoryError: Java heap space
im prinzip hab ich nur nochmal das selbe wie vorhergemacht ich habe keine Ahnung was da jetzt so schief geht

protected HashMap<Coordinates, Figur> createBoard(){
        HashMap<Coordinates, Figur> createBoard=new HashMap<Coordinates, Figur>();
        for(int i=0;i<brettGröße;i++) {
            for(int j=0;i<brettGröße;j++) {
                if(onCoordinatesXYIsBlackField(i, j)) {
                    if(i<brettGröße/2-1) {
                        createBoard.put(new Coordinates(i,j), new Figur(Typ.PAWN, Color.BLACK));
                    }else if(i>brettGröße/2) {
                        createBoard.put(new Coordinates(i,j), new Figur(Typ.PAWN, Color.WHITE));
                    }else {
                        createBoard.put(new Coordinates(i,j), new Figur());
                    }
                }else {
                    createBoard.put(new Coordinates(i,j), new Figur());
                }
            }
        }
        return createBoard;
    }

was muss ich da noch ändern ?
und wie rufe ich dann was aus der HashMap auf etwa so ?

HashMap<Coordinates, Figur> map=new HashMap<>();
map.get(new Coordinates(0,0))

 

[mihe7]

for(int j=0;i<brettGröße;j++) {

Das muss

for (int j = 0; j < brettGröße; j++) {

heißen.

 

[Jw456]

HashMap<Coordinates, Figur> map=new HashMap<>();
map.get(new Coordinates(0,0))

Wenn in der Map unter dem Key „new Coordinates(0,0)“ ein Eintag ist bekommst du das Value „ Fiur“ zurück wenn nicht dann „Null“



Wenn du eine Figur bewegst wirst du den alten Key ( das Feld auf dem Brett) löschen müssen „remove.“ Die Figur an der neuen stellen in der Map mit der neuen Pos (Key) mit „put“ speichern.

 

[Jw456]

Wenn in der Map unter dem Key „new Coordinates(0,0)“ ein Eintag ist bekommst du das Value „ Figur“ zurück wenn nicht dann „Null“

Kleine Berichtigung
Die Map wird sich als „ Key“ die Instanz von dem Objekt merken. Da du die nicht hast wirst du den Key (Instanz) wahrscheinlich nicht wider finden.
Wenn du also prüfen möchtest ob auf Feld 0,0 eine Figur ist wirst du mit „new Coordinates(0,0)“ kein glück haben da das ja jetzt ein anders Objekt ist als das was in der Map ist. Du wirst Null zurück bekommen.

 

[KonradN]

Kleine Berichtigung
Die Map wird sich als „ Key“ die Instanz von dem Objekt merken. Da du die nicht hast wirst du den Key (Instanz) wahrscheinlich nicht wider finden.
Wenn du also prüfen möchtest ob auf Feld 0,0 eine Figur ist wirst du mit „new Coordinates(0,0)“ kein glück haben da das ja jetzt ein anders Objekt ist als das was in der Map ist. Du wirst Null zurück bekommen.

Wenn er equals und Hashcode sinnvoll überschrieben hat, dann wird die Nutzung von Coordinates auch als Key in der HashMap funktionieren.
Und das wurde von @mihe7 in #10 auch entsprechend als Notwendigkeit formuliert, so dass ich davon ausgehen, dass es auch umgesetzt wurde.

 

[Jw456]

Wenn er equals und Hashcode sinnvoll überschrieben hat, dann wird die Nutzung von Coordinates auch als Key in der HashMap funktionieren.
Und das wurde von @mihe7 in #10 auch entsprechend als Notwendigkeit formuliert, so dass ich davon ausgehen, dass es auch umgesetzt wurde.

hoffen wir mal.