spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Tatsächlich, war grade schon dabei, ein eigenes Struct zu schreiben noch nie vorher gesehen / genutzt ..

Aktuell hab ichs so aufgeschrieben



aber

1. kann man die Bedingungen garantiert noch runterkürzen

2. gibt es eine Out Of Range Exception, sobald er durch die fünfte Bedingung einen maximalen Index setzt, den es gar nicht mehr gibt.

Ich arbeite jetzt erstmal an 2., aber das ist ja schonmal schön kurz.

bin ich grade zu blöd für solch ein "einfaches" Problem?

Der Spieler weiß, auf welcher Zelle er steht. Setzen tue ich


so

Spielerposition im 2D Array herausfinden

	C#-Quelltext
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18	int cellIndexHorizontal = 0; int cellIndexVertical = 0; for (int x = 0; x < mapCells.GetLength(0); x++) { for (int y = 0; y < mapCells.GetLength(1); y++) { Cell mapCell = mapCells[x, y]; if (mapCell.Equals(playerInfo.CurrentCell)) { cellIndexHorizontal = x; cellIndexVertical = y; } } } Vector2Int targetCellIndex = new Vector2Int(cellIndexHorizontal, cellIndexVertical);

Laut Debugger scheint die Spieler Position immer korrekt zu sein. Danach mache ich weiter mit dem Code Vorschlag von @TTGC

Zielposition berechnen

C#-Quelltext

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

Vector2Int targetCellIndex = new Vector2Int(cellIndexHorizontal, cellIndexVertical); // Startposition int distance = 1; // Startwert while (true) { Vector2Int currentCellIndex = targetCellIndex + movementDirection * distance; // neue Zielposition bool cellExists = currentCellIndex.x >= 0 && currentCellIndex.y >= 0 && currentCellIndex.x < mapCells.GetLength(0) && currentCellIndex.y < mapCells.GetLength(1); // Zelle innerhalb des Spielfeldes? if (cellExists && !mapCells[currentCellIndex.x, currentCellIndex.y].IsObstacle) // Zelle im Spielfeld und kein Hindernis { distance++; } else { break; } } targetCellIndex += movementDirection * distance; // eigentliche Zielposition setzen return mapCells[targetCellIndex.x, targetCellIndex.y];

cellExists scheint laut Debugger ebenfalls richtig gesetzt zu werden. Kommt er in eine out of Range Exception, ist der Index 1 zu viel. Für einen weiteren kleinen Denkanstoß wäre ich sehr dankbar ...

Sorry, ich habe nicht weit genug gedacht. Den Block oben kannst du streichen. Der Spieler hat jetzt nur noch einen Vector2Int Den muss er haben, weil falls es Abstände zu den Zellen geben sollte, kann ich ja nicht mit den wirklichen Positionen rechnen. Also kennt der Spieler jetzt nur noch die Koordinaten der aktuellen Zelle.

Ebenso gehe ich davon aus, dass man den bool, ob eine Zelle innerhalb der Karte liegt, noch kürzen kann. Ich wollte aber erstmal die Exception rausbekommen.

Zielberechnung

C#-Quelltext

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25

public Vector2Int GetTargetCellIndex(Vector2Int movementDirection) { Vector2Int targetCellIndex = new Vector2Int(playerInfo.CurrentCellIndex.x, playerInfo.CurrentCellIndex.y); int distance = 1; while (true) { Vector2Int currentCellIndex = targetCellIndex + movementDirection * distance; bool cellExists = currentCellIndex.x >= 0 && currentCellIndex.y >= 0 && currentCellIndex.x < mapCells.GetLength(0) && currentCellIndex.y < mapCells.GetLength(1); if (cellExists && !mapCells[currentCellIndex.x, currentCellIndex.y].IsObstacle) distance++; else break; } if(distance > 1) targetCellIndex += movementDirection * distance; return targetCellIndex; }

Vor return prüfe ich nochmal, ob sich die Position überhaupt verändern würde. Sonst darf ich ja keine Richtung draufrechnen. Das passt aber noch nicht so ganz, es gibt immer noch die Exception..

stimmt, die Distanz auf 0 zu setzen (bei keiner Bewegung), bzw. eins abzuziehen, war eine gute Idee.

Dieser Code hier berechnet erstmal fehlerfrei die korrekte Zielposition

Berechnung

C#-Quelltext

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30

public Vector2Int GetTargetCellIndex(Vector2Int movementDirection) { Vector2Int targetCellIndex = new Vector2Int(playerInfo.CurrentCellIndex.x, playerInfo.CurrentCellIndex.y); int distance = 1; while (true) { Vector2Int currentCellIndex = targetCellIndex + movementDirection * distance; bool cellExists = currentCellIndex.x >= 0 && currentCellIndex.y >= 0 && currentCellIndex.x < mapCells.GetLength(0) && currentCellIndex.y < mapCells.GetLength(1); if (cellExists && !mapCells[currentCellIndex.x, currentCellIndex.y].IsObstacle) { distance++; } else { break; } } distance--; targetCellIndex += movementDirection * distance; return targetCellIndex; }

Den bool, der prüft, ob eine Zelle noch auf der Karte liegt, könnte ich so abkürzen


aber lesbarer gestaltet es den Code nicht wirklich glaube ich und ob dieser Code optimierter ist, ja vielleicht ein wenig..

Ich werde ihn mal ausprobieren. Erstmal ist das Thema hier aber abgeschlossen Jetzt hab ich mich aber auch echt blöd angestellt gehabt vielen Dank euch!