Litt skriving på oblig3.md

inf112-v21 · Mar 26, 2021 · 0a642bb · 0a642bb
1 parent 1e716ba
commit 0a642bb
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 3 deletions.
diff --git a/Deliverables/5H Project Board.png b/Deliverables/5H Project Board.png
diff --git a/Deliverables/5H Project Board2.png b/Deliverables/5H Project Board2.png
diff --git a/Deliverables/GUI_design.png b/Deliverables/GUI_design.png
diff --git a/Deliverables/ObligatoriskOppgave3.md b/Deliverables/ObligatoriskOppgave3.md
@@ -100,6 +100,7 @@ som kan tildeles, kan de som ikke har tildelte oppgaver jobbe med andre oppgaver
 - GUI  
     - Kunne klikke på kort for å velge dem istedenfor bruke tastatur
     - Knapp for å sende kort
+ - Legge til en phase "EnOfRegister" som håndterer alt som skjer mellom hvert register, f.eks. laser, og andre brettfunksjoner. 
 
 
 
@@ -134,7 +135,7 @@ For å bygge koden:
 - Kjør `mvn clean install`. (Testet til å funke på alle platformer, skal få build success)
 
 #### Bygging:
-- Vi har alle windows som OS og bruker Travis for å sjekke om det bygger på Linux/Mac.
+- Vi har alle windows som OS og bruker Travis for å sjekke om det bygger på Linux/Mac. Ifølge Travis bygger prosjektet vårt.
 
 #### Klassediagram:
 Vi har generert klassedigram med Object Aid Explorer. Dette er en Eclipse Plugin som finnes her: https://marketplace.eclipse.org/content/objectaid-uml-explorer.
@@ -202,8 +203,17 @@ I denne innleveringen har vi lagt til følgende funksjonalitet:
 - GUI
     - Vise kort i hånd
     - Vise valgte kort
+    - Vise liv, skade og spillebrikke i GUI
+    - Litt annet i GUI for senere implementasjoner (knapper o.l.)
 
 - Flagg oppførsel følger reglene
-- Laser og kollisjon (ikke implementert)
+- Laser
+    - Ikke implementert i selve spilllogikken da vi ikke har laget logikken for fasen hvor laserer avfyres
+- Kollisjon
+  - Ikke implementert i flyttelogikken, da vi avventer dette.
 
 #### Kommentarer til kode og utførelse
+
+Commits er som forrige gang ujevne. Dette har seg med at det er mange commits fra mindre prosjekter (laser, kollisjon og tester for disse) samt bugfixes, mens de andre gruppemedlemmene jobbet med å 
+få til GUI. Dette viste seg å være en svært vanskelig jobb, og mesteparten av arbeidet fra disse gruppemedlemmene ble da å få forståelse. Vi prøvde så å implementere GUI i felleskap og fikk det ikke helt til
+men til slutt klarte Skjalg å få til en kjørende versjon basert på forarbeidet til resten. Dette fører til ujevne commits da arbeidet til de som originalt jobbet med GUI ikke er reflektert i commitsene.
diff --git a/Deliverables/Referat/Referat_15 → Deliverables/Referat/Referat_15.txt b/Deliverables/Referat/Referat_15 → Deliverables/Referat/Referat_15.txt
diff --git a/src/main/java/inf112/skeleton/app/Game.java b/src/main/java/inf112/skeleton/app/Game.java
@@ -721,7 +721,6 @@ private Direction getNewDirection(Direction dir, boolean clockwise){
      * able to see the final position of the player. (There is probably a cleaner way to do this than sleep the thread.)
      */
     private void endTurn(){
-        fireLasers();
         ArrayList<Player> toBeRemoved = new ArrayList<>(); //List of players that died this round
         for(Player player : alivePlayerList){
             if(player.isDead() || player.getPlayerNum() > numPlayers){    //If player died