From 88d5686137b025a1a663e09592c08c7c48c5b883 Mon Sep 17 00:00:00 2001 From: EpicKnarvik97 Date: Wed, 25 Mar 2020 14:43:54 +0100 Subject: [PATCH] Fikser problemer med kodestil --- Deliverables/Oblig3.md | 8 ++++---- .../fiasko/roborally/game_wrapper/CardChoiceScreen.java | 3 +-- .../fiasko/roborally/game_wrapper/SimpleButton.java | 2 +- src/main/java/inf112/fiasko/roborally/objects/Board.java | 2 +- 4 files changed, 7 insertions(+), 8 deletions(-) diff --git a/Deliverables/Oblig3.md b/Deliverables/Oblig3.md index 0e39f21..04ac0fe 100644 --- a/Deliverables/Oblig3.md +++ b/Deliverables/Oblig3.md @@ -8,7 +8,7 @@ Rollefordelingen i teamet fungerer fint sånn som vi har det. Siden vi har fra begynnelsen av prosjektet valgt å ha noen av møtene og parprogrammeringsøktene våre på Discord, var vi bedre rustet til utfordringene som kom med SARS-CoV-2 pandemien og de inførte restriksjoner det har medbragt. - + ### Retrospektiv #### Plan Det vi planlagte var å ha to fysiske møter og ett digitalt møte per korte sprint (en uke) @@ -28,7 +28,7 @@ Ellers har vi gjennomført planen som planlagt. #### Forbedringspunkter * Forbedre bruk av tester. Bruke mer test driven development. Vi har til nå skrevet teser etter vi har skrevet kode noe som har ført til noen bugs som kanskje kunne vært unngått med tdd. - + ### Prioritering av oppgaver Vi må bli ferdig med runder og kortvelging først. Etter dette prioriterer vi å få nettverksfunksjonaliteten på plass. Til slutt vil vi prioritere det grafiske og eventulle nice to haves. @@ -63,10 +63,10 @@ Implementere fase og alt av funksjoner en fase kommer til trenge. * Som fase trenger jeg å kunne kjøre programmeringskort etter høyest verdi for å bestemme rekkefølgen på trekkene til robotene. - + * Programmeringskort er sorterbare. * Programmeringskort kan hentes og gis videre. - + * Som tannhjul trenger jeg å kunen snu roboter for gjennomføre min funksjon. * Tannhjul vrir roboter som står på de 90 grader i retningen tannhjulet har. diff --git a/src/main/java/inf112/fiasko/roborally/game_wrapper/CardChoiceScreen.java b/src/main/java/inf112/fiasko/roborally/game_wrapper/CardChoiceScreen.java index 1010e50..5522097 100644 --- a/src/main/java/inf112/fiasko/roborally/game_wrapper/CardChoiceScreen.java +++ b/src/main/java/inf112/fiasko/roborally/game_wrapper/CardChoiceScreen.java @@ -42,7 +42,6 @@ public class CardChoiceScreen extends InputAdapter implements Screen { private final List chosenCards; private final int maxCards; private final Stage stage; - final TextButton confirmCards; /** * Initializes a new card choice screen @@ -72,7 +71,7 @@ public class CardChoiceScreen extends InputAdapter implements Screen { stage = new Stage(); inputMultiplexer.addProcessor(stage); - confirmCards = new SimpleButton("Confirm cards", roboRallyWrapper.font).getButton(); + TextButton confirmCards = new SimpleButton("Confirm cards", roboRallyWrapper.font).getButton(); stage.addActor(confirmCards); confirmCards.setY(viewport.getWorldHeight() + 60); confirmCards.setX(15); diff --git a/src/main/java/inf112/fiasko/roborally/game_wrapper/SimpleButton.java b/src/main/java/inf112/fiasko/roborally/game_wrapper/SimpleButton.java index e697353..7668d88 100644 --- a/src/main/java/inf112/fiasko/roborally/game_wrapper/SimpleButton.java +++ b/src/main/java/inf112/fiasko/roborally/game_wrapper/SimpleButton.java @@ -10,7 +10,7 @@ import com.badlogic.gdx.scenes.scene2d.ui.TextButton; * This class generates a simple text button using a default skin */ public class SimpleButton { - private TextButton button; + private final TextButton button; /** * Instantiates a new simple button diff --git a/src/main/java/inf112/fiasko/roborally/objects/Board.java b/src/main/java/inf112/fiasko/roborally/objects/Board.java index e55eabb..a7872f0 100644 --- a/src/main/java/inf112/fiasko/roborally/objects/Board.java +++ b/src/main/java/inf112/fiasko/roborally/objects/Board.java @@ -447,7 +447,7 @@ public class Board { * @param direction The direction something is going * @return True if a wall would stop its path */ - boolean moveIsStoppedByWall(Position robotPosition, Position newPosition, Direction direction) { + private boolean moveIsStoppedByWall(Position robotPosition, Position newPosition, Direction direction) { return hasWallFacing(robotPosition, direction) || (isValidPosition(newPosition) && hasWallFacing(newPosition, Direction.getReverseDirection(direction))); }