agalassi / tablutcompetition

Software for the Tablut Students Competition

TablutCompetition

Software for the Tablut Students Competition

Installation on Ubuntu/Debian

From console, run these commands to install JDK 8 e ANT:

sudo apt update
sudo apt install openjdk-8-jdk -y
sudo apt install ant -y

Now, clone the project repository:

git clone https://github.com/AGalassi/TablutCompetition.git

Run the Server without Eclipse

The easiest way is to utilize the ANT configuration script from console. Go into the project folder (the folder with the build.xml file):

cd TablutCompetition/Tablut

Compile the project:

ant clean
ant compile

The compiled project is in the build folder. Run the server with:

ant server

Check the behaviour using the random players in two different console windows:

ant randomwhite

ant randomblack

At this point, a window with the game state should appear.

To be able to run other classes, change the build.xml file and re-compile everything

Replay function

Replay a game using the logfile

Example:

java -jar .\server.jar -g -R .\logs\PLAYER1_vs_PLAYER2_1652711382324_gameLog.txt

tablutcompetition's Issues

Unusual switch syntax

In ReplayServer.java, in the getStateFromLines(), the switch block uses an unusual syntax (-> instead of :, and lacks break commands), which is not compatible with Java 11

Mangiata re con trono

Ho notato che nella situazione in figura la mossa del nero C6 -> C5 non provoca la mangiata del re.

Nel seguente caso invece il re viene mangiato regolarmente.

Nel caso fosse utile allego anche i file di log delle due partite.

gameLog_1.txt
gameLog_2.txt

👻 NullPointerException 👻 when checking move

Commit ccc748c causes a NullPointerException to be thrown when checking the move at line 519. This is due to tin being null since the switch-case block which included the assignement to Turnwhite or Turnblack was moved after said line.

To replicate the behaviour run the server and two random players.

Giocatore Random rimane con una sola pedina: IllegalArgumentException

Giocando come giocatore Random alle linee 153 e 239 della classe TablutRandomClient Random.nextInt viene chiamato con argomento pawns.size() - 1 che risulta essere 0 essendoci solo una pedina rimasta.

Questo lancia una IllegalArgumentException: bound must be positive che fa perdere la partita al giocatore.

E' un problema secondario, e non relativo alle regole del gioco, ma volevo comunque segnalarlo.
Saluti

Bug relativi al "mangiare una pedina"

@AGalassi
Allego due screenshot per due possibili problemi.
Muovendo la pedina bianca da E3 a A3 ottengo che la pedina nera in posizione A4 viene mangiata (anche se l'accampamento di posizione A5 è coperto da una pedina nera). È un funzionamento corretto? Dalle regole delle slide non è chiaro se questo sia corretto o meno.

Secondo "problema": se muovo la pedina nera da posizione E2 a E3 NON mangio la pedina bianca in posizione E4, questo è corretto? Dalle regole si evince che il castello "fa da sponda" indipendentemente dal fatto che il re sia dentro o fuori dallo stesso. È corretto? Allego schermata anche di questo.

Compilazione server con Ant: abilitare GUI

Pare non sia possibile abilitare la GUI del server se compilato tramite Ant

Problema nel nome del giocatore nero

Al nome del giocatore nero non viene applicata la funzione di cancellazione dei caratteri invalidi. Nell'esempio allegato la stringa che viene passata è sempre la stessa ma il server li gestisce in maniera diversa.

Outdated README

• the README proposes Java 8, but the String.strip() method is used in ReplayServer.java, which is only compatible with Java 11+
• in the "Replay function" section of the README, it would be useful to also include the ant server-jar command, used to generate the server.jar file mentioned there

fromString() not implemented in Pawn enum

In ReplayServer, there is a call on a supposed State.Pawn.fromString() method, but it is not implemented in State.Pawn

State.java hashCode broken

Two different istances of it.unibo.ai.didattica.competition.tablut.domain.StateTablut have equal values but different hashCodes.
The problems is in turn related to .hashCode() invoked on java matrix representing the board. This is because hashCode default implementation for Array is the same of Object!

Here's a junit5 test case to reproduce the problem:
TestHashCodeState.txt

Cattura di un pezzo

Non ho capito una cosa delle regole: il Trono funge da sponda indipendentemente dal colore della pedina che deve essere mangiata giusto?
L'accampamento invece funge da sponda solo per i Neri che mangiano oppure anche per i Bianchi?

Errore cattura del re in F5

Con il re in F5, una nera in F4, l'altra in G5 e spostando l'ultima nera da G6 a F6 la cattura del re non viene comunicata e la partita continua, allego i log di seguito.

PartitaCompletaServer.txt

1558166254223_systemLog.txt

ErroreSpecificoServer.txt

MyBlackPlayer.txt

Specifico che l'errore si è presentato nonostante abbia fatto la pull delle modifiche apportate inerenti a un errore simile di poco tempo fa.

Aggiornamento stato nella gui

Problema che non c'entra con le regole del gioco:

Lanciando da terminale un random contro uno human, abbassando la finestra della gui e continuando a giocare da terminale lo stato viene comunicato correttamente a entrambi i giocatori, ma la gui non viene aggiornata e anche rialzandola la situazione non cambia.
Lo faccio presente casomai qualcuno lo utilizzasse, ho provato anche random vs random e human vs human e mi pare che in quei casi non si verifichi questo problema.