diff --git a/src/main/java/it/polimi/ingsw/gc14/Controller/ClientController.java b/src/main/java/it/polimi/ingsw/gc14/Controller/ClientController.java index 0322b97..6121fcd 100644 --- a/src/main/java/it/polimi/ingsw/gc14/Controller/ClientController.java +++ b/src/main/java/it/polimi/ingsw/gc14/Controller/ClientController.java @@ -86,11 +86,7 @@ public class ClientController { } else { - try { - client.drawUpperTribeCard(playerUsername, pos); - } catch (RemoteException e) { - throw new RuntimeException(e); - } + client.drawUpperTribeCard(playerUsername, pos); } } @@ -105,11 +101,7 @@ public class ClientController { if(!Objects.equals(playerUsername, miniModel.currentState.getCurrentPlayer().getUserName())) view.showError("It's not your turn!"); else - try { - client.drawLowerTribeCard(playerUsername, pos); - } catch (RemoteException e) { - throw new RuntimeException(e); - } + client.drawLowerTribeCard(playerUsername, pos); } @@ -123,11 +115,7 @@ public class ClientController { if(!Objects.equals(playerUsername, miniModel.currentState.getCurrentPlayer().getUserName())) view.showError("It's not your turn!"); else { - try { - client.drawUpperBuildingCard(playerUsername,pos); - } catch (RemoteException e) { - throw new RuntimeException(e); - } + client.drawUpperBuildingCard(playerUsername,pos); } } @@ -143,11 +131,7 @@ public class ClientController { if(!Objects.equals(playerUsername,miniModel.currentState.getCurrentPlayer().getUserName())) view.showError("It's not your turn!"); else { - try { - client.drawLowerBuildingCard(playerUsername,pos); - } catch (RemoteException e) { - throw new RuntimeException(e); - } + client.drawLowerBuildingCard(playerUsername,pos); } } @@ -161,11 +145,7 @@ public class ClientController { if(!Objects.equals(playerUsername,miniModel.currentState.getCurrentPlayer().getUserName())) view.showError("It's not your turn!"); else { - try { - client.skipTurn(playerUsername); - } catch (RemoteException e) { - throw new RuntimeException(e); - } + client.skipTurn(playerUsername); } } @@ -182,11 +162,7 @@ public class ClientController { if(!Objects.equals(playerUsername,miniModel.currentState.getCurrentPlayer().getUserName())) view.showError("It's not your turn!"); else { - try { - client.pickOptionalTribeCard(playerUsername,pos); - } catch (RemoteException e) { - throw new RuntimeException(e); - } + client.pickOptionalTribeCard(playerUsername,pos); } } @@ -201,11 +177,7 @@ public class ClientController { if(!Objects.equals(playerUsername, miniModel.currentState.getCurrentPlayer().getUserName())) view.showError("It's not your turn!"); else { - try { - client.pickOptionalBuildingCard(playerUsername,pos); - } catch (RemoteException e) { - throw new RuntimeException(e); - } + client.pickOptionalBuildingCard(playerUsername,pos); } } @@ -219,11 +191,7 @@ public class ClientController { if(!Objects.equals(playerUsername, miniModel.currentState.getCurrentPlayer().getUserName())) view.showError("It's not your turn!"); else { - try { - client.noOptionalCard(playerUsername); - } catch (RemoteException e) { - throw new RuntimeException(e); - } + client.noOptionalCard(playerUsername); } } @@ -237,11 +205,7 @@ public class ClientController { if(!Objects.equals(playerUsername, miniModel.currentState.getCurrentPlayer().getUserName())) view.showError("It's not your turn!"); else { - try { - client.slotChoice(playerUsername,pos); - } catch (RemoteException e) { - throw new RuntimeException(e); - } + client.slotChoice(playerUsername,pos); } } @@ -250,11 +214,7 @@ public class ClientController { if(!Objects.equals(playerUsername, miniModel.currentState.getCurrentPlayer().getUserName())) view.showError("It's not your turn!"); else { - try { - client.totemChoice(playerUsername, String.valueOf(miniModel.availableTotems.get(pos))); - } catch (RemoteException e) { - throw new RuntimeException(e); - } + client.totemChoice(playerUsername, String.valueOf(miniModel.availableTotems.get(pos))); } } diff --git a/src/main/java/it/polimi/ingsw/gc14/Model/OrderLogicCard.java b/src/main/java/it/polimi/ingsw/gc14/Model/OrderLogicCard.java index 1fac78a..d49b955 100644 --- a/src/main/java/it/polimi/ingsw/gc14/Model/OrderLogicCard.java +++ b/src/main/java/it/polimi/ingsw/gc14/Model/OrderLogicCard.java @@ -70,9 +70,12 @@ public abstract class OrderLogicCard implements Serializable { playerList=new ArrayList<>(); } + if(players.stream().anyMatch(x->x.getUserName().equals(player.getUserName()))) + { + return; + } playerList.removeIf(x->player.getUserName().equals(x.player.getUserName())); playerList.add(new OrderPlayer(player,false)); - players.removeIf(x->player.getUserName().equals(x.getUserName())); players.add(player); } diff --git a/src/main/java/it/polimi/ingsw/gc14/Network/IClient.java b/src/main/java/it/polimi/ingsw/gc14/Network/IClient.java index 23a8f3b..6534082 100644 --- a/src/main/java/it/polimi/ingsw/gc14/Network/IClient.java +++ b/src/main/java/it/polimi/ingsw/gc14/Network/IClient.java @@ -17,7 +17,7 @@ public interface IClient { public void drawLowerBuildingCard(String playerUsername,int pos) ; - public void skipTurn(String playerUsername) throws RemoteException; + public void skipTurn(String playerUsername); diff --git a/src/main/java/it/polimi/ingsw/gc14/View/TUI/TUI.java b/src/main/java/it/polimi/ingsw/gc14/View/TUI/TUI.java index a023341..2ccaf7e 100644 --- a/src/main/java/it/polimi/ingsw/gc14/View/TUI/TUI.java +++ b/src/main/java/it/polimi/ingsw/gc14/View/TUI/TUI.java @@ -158,14 +158,13 @@ public class TUI implements IView { public void renderTotems() { clearTerminal(); var table = new AsciiTable(BorderStyle.ROUNDED, model.availableTotems.size()); - List lines = model.availableTotems.stream().map(Totems::toString).toList(); - int i=0; - for(String line : lines) + List lines =new ArrayList<>(); + for(int i=0;i