Skip to content
This repository has been archived by the owner on Apr 8, 2024. It is now read-only.

Update Konzept.md #22

Merged
merged 5 commits into from
Feb 11, 2018
Merged

Update Konzept.md #22

merged 5 commits into from
Feb 11, 2018

Conversation

FrederikSchlemmer
Copy link
Contributor

@FrederikSchlemmer FrederikSchlemmer commented Jan 30, 2018

Issue #8

Copy link
Member

@silasmahler silasmahler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich habe mal den Text durchgelesen und mir sind noch kleine Fehler aufgefallen, ich hab die mal nicht direkt geändert, weil ich den Reviewprozess auf Github ausprobieren wollte :octocat:


Jeder Agent implementiert das Interface **BrainySnakePlayer**, welches folgende Methoden vorgibt:
- **String getPlayerName()**
-- Agent wird nach dem gesetztem Spielernamen gefragt
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gesetzten


### Objekt: PlayerState

Der Agent erhält das Objekt PlayerState in seiner Methode *handlePlayerStatusUpdate()* übergeben. In dem Objekt kann der Zustand der eigenen Schlange abgefragt werden.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bekommt übergeben oder erhält und dann ohne übergeben

|:----------------|:-------------------------------|:-----------------------------:|
|movesPlayed / getMovesPlayed()| Anzahl der gespielten Runden.|int
|movesRemaining / getMovesRemaining()|Anzahl der übrigen zu spielenden Runden.|int
|playerPoints / getPlayerPoints()|Aktuelle Spielerpunkte (Gesamtzahl von Kopf und Körperteileanzahl).|int
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kopf- Körperteilanzahl

|movesRemaining / getMovesRemaining()|Anzahl der übrigen zu spielenden Runden.|int
|playerPoints / getPlayerPoints()|Aktuelle Spielerpunkte (Gesamtzahl von Kopf und Körperteileanzahl).|int
|playersHead / getPlayersHead()|Aktuelle Position des Kopfes von der Schlange.|Point2D
|playersTail / getPlayersTail()|Aktuelle Position des Körpers von der Schlange. Es wird nur der letzte Punkt des Körpers zurückgegeben. Falls kein Körper vorhanden wird null zurückgegeben.|Point2D
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

von der Schlanger -> der Schlange

|movesPlayed / getMovesPlayed()| Anzahl der gespielten Runden.|int
|movesRemaining / getMovesRemaining()|Anzahl der übrigen zu spielenden Runden.|int
|playerPoints / getPlayerPoints()|Aktuelle Spielerpunkte (Gesamtzahl von Kopf und Körperteileanzahl).|int
|playersHead / getPlayersHead()|Aktuelle Position des Kopfes von der Schlange.|Point2D
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

von der Schlanger -> der Schlange

| Name / Methode |Beschreibung |Datentyp |
|---------------- |------------------------------- |:-----------------------------:|
|currentOrientation/ getCurrentOrientation()| Aktuelle Orientierung der Schlange.|Orientation
|viewWidth / getViewWidth()|Feldbreite in Feldern aus Sicht der Schlange. Das Feld beginnt vor dem Kopf der Schlange. Die Aufteilung findet in gleichen Anteilen nach rechts & links der Schlange statt.|int
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in gleichen Anteilen rechts und links der Schlange


### Objekt: RoundEvent

Jede Runde werden für die jeweiligen Schlangen verschiedene RoundEvents berechnet. Diese RoundEvents können in dem PlayerState ausgelesen werden.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

im PlayerState


## Projektstruktur (Gradle)

Jeder Player (Agent) wird in ein **seperates Modul** ausgelagert, welches in den Core improtiert wird.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

importiert

## Projektstruktur (Gradle)

Jeder Player (Agent) wird in ein **seperates Modul** ausgelagert, welches in den Core improtiert wird.
Diese Auslagerung verhindert den Zugriff von den Agent auf fremden Code.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

des Agenten

**Laufzeit Lösung:** Der Aufruf jedes Agenten erfolgt ein eigenem Thread, welcher nach einer angemessenen Zeit terminiert, sofern keine Antwort eingegangen ist.
Der Agent wird eingefroren (PlayerStatus frozen), wechselt zur Kennzeichnung des Status die Farbe und es werden zu diesem Zug keine Aktionen für diesen Agenten ausgeführt.

### Der Agent greift auf Fremden Code zu
**Problem:** Der Agent könnte **Public Methoden** der Simulation der oder die **Public Methoden** anderer Agenten zugreifen.
__Problem:__ Der Agent könnte **Public Methoden** der Simulation der oder die **Public Methoden** anderer Agenten zugreifen.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

könnte auf Methoden der Simulation oder anderer Agenten zugreifen

@FrederikSchlemmer
Copy link
Contributor Author

@silasmahler Ich habe die Anmerkungen bezüglich Rechtschreibfehlern abgearbeitet. In den offenen Punkten konnte ich keinerlei verbesserte Verständlichkeit bzw. Notwendigkeit sehen.

Gerne können diese Punkte basierend auf persönlicher Präferenz eigenständig geändert werden.

@RudiKlassen-zz RudiKlassen-zz dismissed silasmahler’s stale review February 11, 2018 21:52

Requested changes are done

Copy link
Member

@RudiKlassen-zz RudiKlassen-zz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good state so far

@RudiKlassen-zz RudiKlassen-zz merged commit 37302ad into master Feb 11, 2018
@RudiKlassen-zz RudiKlassen-zz deleted the Player-API-Dokumentation branch February 11, 2018 21:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants