search

rhoentier / ss22_fp_mapc-group_5

4 stars 0 forks source link

Neue Aktion anlegen: Rotation der Attached Elemente #16

Closed A-V-Lorenz closed 2 years ago

A-V-Lorenz commented 2 years ago

Action: rotate

Rotates the agent (and all attached things) 90 degrees in the given direction. For each attached thing, its final position after the rotation has to be free.

Parameter: cw, ccw

Beispiel: new Action("rotate", "cw");

A-V-Lorenz commented 2 years ago

Das issue ist nicht erfüllt.

Für das rotieren der Elemente muss zuerst die Abfrage des Rotationsziels erfolgen. Erst wenn frei, darf rotiert werden.

rhoentier commented 2 years ago

Ah okay, das hab ich übersehen. Wenn solche Abfragen mit in das Ticket sollen, dann würde ich Vorschlagen, dass wir die direkt mit reinschreiben :-)

Soll die Abfrage über unsere interne Map laufen, damit wir keinen Schritt brauchen, oder soll eine Abfrage an den Server geschickt werden?

seblod90 commented 2 years ago

Ich würde das über die Map machen. Bei mehreren attached-Blocks kann es sein, dass man dann einige Felder prüfen muss.

Ich schaffe das leider nicht mehr bis heute Abend, würde das aber dann zeitnah übernehmen. Wenn die Funktion bereits irgendwo benötigt wird, bitte nochmal melden. (nicht dass hierdurch was anderes still steht)

rhoentier commented 2 years ago

Ich hab das grade gemacht. Aber @seblod90, schau am besten nochmal drüber, ob ich deine Methoden alle richtig interpretiert habe

seblod90 commented 2 years ago

Sieht gut aus. Danke nochmal fürs übernehmen ;)

Sollen wir solche Ideen bzw. ToDos für später auch als Issues aufnehmen und mit einem Tag versehen?

seblod90 commented 2 years ago

Vielleicht macht es Sinn, dem MapTile eine Eigenschaft "walkable" (o.ä.) zu geben. Da könnte man sich das mit den Abfragen etwas einfacher machen. obstacle, block, entity und dispenser wären dann nicht walkable, alle anderen schon.

erledigt. Macht den Code etwas einfacher.