1
0
mirror of https://github.com/inf112-v20/Fiasko.git synced 2025-03-04 09:09:46 +01:00

Fikser problemer med kodestil

This commit is contained in:
Kristian Knarvik 2020-03-25 14:43:54 +01:00
parent b2f8abd28c
commit 88d5686137
4 changed files with 7 additions and 8 deletions
Deliverables
src/main/java/inf112/fiasko/roborally

@ -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, 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 var vi bedre rustet til utfordringene som kom med SARS-CoV-2 pandemien og de inførte restriksjoner
det har medbragt. det har medbragt.
### Retrospektiv ### Retrospektiv
#### Plan #### Plan
Det vi planlagte var å ha to fysiske møter og ett digitalt møte per korte sprint (en uke) 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 #### Forbedringspunkter
* Forbedre bruk av tester. Bruke mer test driven development. Vi har til nå skrevet teser etter vi har skrevet kode * 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. noe som har ført til noen bugs som kanskje kunne vært unngått med tdd.
### Prioritering av oppgaver ### Prioritering av oppgaver
Vi må bli ferdig med runder og kortvelging først. Etter dette prioriterer vi å få nettverksfunksjonaliteten på plass. 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. 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 * Som fase trenger jeg å kunne kjøre programmeringskort etter høyest verdi for å bestemme
rekkefølgen på trekkene til robotene. rekkefølgen på trekkene til robotene.
* Programmeringskort er sorterbare. * Programmeringskort er sorterbare.
* Programmeringskort kan hentes og gis videre. * Programmeringskort kan hentes og gis videre.
* Som tannhjul trenger jeg å kunen snu roboter for gjennomføre min funksjon. * 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. * Tannhjul vrir roboter som står på de 90 grader i retningen tannhjulet har.

@ -42,7 +42,6 @@ public class CardChoiceScreen extends InputAdapter implements Screen {
private final List<CardRectangle> chosenCards; private final List<CardRectangle> chosenCards;
private final int maxCards; private final int maxCards;
private final Stage stage; private final Stage stage;
final TextButton confirmCards;
/** /**
* Initializes a new card choice screen * Initializes a new card choice screen
@ -72,7 +71,7 @@ public class CardChoiceScreen extends InputAdapter implements Screen {
stage = new Stage(); stage = new Stage();
inputMultiplexer.addProcessor(stage); inputMultiplexer.addProcessor(stage);
confirmCards = new SimpleButton("Confirm cards", roboRallyWrapper.font).getButton(); TextButton confirmCards = new SimpleButton("Confirm cards", roboRallyWrapper.font).getButton();
stage.addActor(confirmCards); stage.addActor(confirmCards);
confirmCards.setY(viewport.getWorldHeight() + 60); confirmCards.setY(viewport.getWorldHeight() + 60);
confirmCards.setX(15); confirmCards.setX(15);

@ -10,7 +10,7 @@ import com.badlogic.gdx.scenes.scene2d.ui.TextButton;
* This class generates a simple text button using a default skin * This class generates a simple text button using a default skin
*/ */
public class SimpleButton { public class SimpleButton {
private TextButton button; private final TextButton button;
/** /**
* Instantiates a new simple button * Instantiates a new simple button

@ -447,7 +447,7 @@ public class Board {
* @param direction The direction something is going * @param direction The direction something is going
* @return True if a wall would stop its path * @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) && return hasWallFacing(robotPosition, direction) || (isValidPosition(newPosition) &&
hasWallFacing(newPosition, Direction.getReverseDirection(direction))); hasWallFacing(newPosition, Direction.getReverseDirection(direction)));
} }