Implemented most of main logic of the VoteHandler

This commit is contained in:
Alexander Sazonov 2022-04-03 18:25:37 +02:00
parent 6b55ed8756
commit 110a7a342d
3 changed files with 101 additions and 5 deletions

View File

@ -1,5 +1,6 @@
package ch.unibas.dmi.dbis.cs108.gamelogic; package ch.unibas.dmi.dbis.cs108.gamelogic;
import ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur.GhostPlayer;
import ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur.Passenger; import ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur.Passenger;
import ch.unibas.dmi.dbis.cs108.multiplayer.server.ClientHandler; import ch.unibas.dmi.dbis.cs108.multiplayer.server.ClientHandler;
@ -15,9 +16,14 @@ import ch.unibas.dmi.dbis.cs108.multiplayer.server.ClientHandler;
* <p>TODO: Think about if the timer needs to be implemented here or in the Game class * <p>TODO: Think about if the timer needs to be implemented here or in the Game class
*/ */
public class VoteHandler { public class VoteHandler {
public static void ghostVote(Passenger[] passengers) { public void ghostVote(Passenger[] passengers) {
// array to collect votes for all players during voting, i.e. votes for player 1 are saved in
// votesForPlayers[0]
int[] votesForPlayers = new int[6];
// Walk through entire train, ask ghosts to ghostify and humans to wait // Walk through entire train, ask ghosts to ghostify and humans to wait
// TODO: Messages in for-loop should be handled by ServerGameInfoHandler // TODO: Messages in for-loop should probably be handled by ServerGameInfoHandler
for (Passenger passenger : passengers) { for (Passenger passenger : passengers) {
if (passenger.getIsGhost()) { if (passenger.getIsGhost()) {
passenger.send("Vote on who to ghostify!"); passenger.send("Vote on who to ghostify!");
@ -25,9 +31,94 @@ public class VoteHandler {
passenger.send("Please wait, ghosts are active"); passenger.send("Please wait, ghosts are active");
} }
} }
for (Passenger passenger : passengers) { for (Passenger passenger : passengers) {
// TODO (Alex): Count received votes and deal with results // collecting the votes - distribute them among the vote counters for all players
// Note: Each voting collects votes for all players even though some might not be concerned
// (i.e. ghosts during ghost vote). Those players will then get 0 votes so it doesn't matter.
// TODO: Perhaps the vote results should be handled by ClientGameInfoHandler
if (passenger.getHasVoted()) { if (passenger.getHasVoted()) {
for (int i = 0; i < votesForPlayers.length; i++) {
if (passenger.getVote() == i) {
votesForPlayers[i]++;
}
}
}
}
// count the votes - determine which player has the most votes by going through the
// votesForPlayers array
int currentMax = 0;
for (int i = 0; i < votesForPlayers.length; i++) {
if (votesForPlayers[i] > currentMax) {
currentMax = votesForPlayers[i];
}
}
// ghostify the player with most votes
for (int i = 0; i < votesForPlayers.length; i++) {
if (votesForPlayers[i] == currentMax) { // if player has most votes
GhostifyHandler gh = new GhostifyHandler();
gh.ghostify(passengers[i]);
passengers[i].send(
"You are now a ghost!"); // TODO: ServerGameInfoHandler might deal with this one
}
}
}
public void humanVote(Passenger[] passengers) {
// very similar to ghostVote, differs mostly in the way votes are handled
// array to collect votes for all players during voting, i.e. votes for player 1 are saved in
// votesForPlayers[0]
int[] votesForPlayers = new int[6];
// Walk through entire train, ask humans to vote and ghosts to wait
// TODO: Messages in for-loop should probably be handled by ServerGameInfoHandler
for (Passenger passenger : passengers) {
if (passenger.getIsGhost()) {
passenger.send("Please wait, humans are active");
} else {
passenger.send("Vote for a ghost to kick off!");
}
}
for (Passenger passenger : passengers) {
// collecting the votes - distribute them among the vote counters for all players
// TODO: Perhaps the vote results should be handled by ClientGameInfoHandler
if (passenger.getHasVoted()) {
for (int i = 0; i < votesForPlayers.length; i++) {
if (passenger.getVote() == i) {
votesForPlayers[i]++;
}
}
}
}
// count the votes - determine which player has the most votes by going through the
// votesForPlayers array
int currentMax = 0;
for (int votesForPlayer : votesForPlayers) {
if (votesForPlayer > currentMax) {
currentMax = votesForPlayer;
}
}
// deal with voting results
for (int i = 0; i < votesForPlayers.length; i++) {
if (votesForPlayers[i] == currentMax) { // if player has most votes
if (!passengers[i].getIsGhost()) { // if player with most votes is human, notify everyone about it
for (Passenger passenger : passengers) {
passenger.send(
"You voted for a human!"); // TODO: ServerGameInfoHandler might be better to use here
}
}
if (passengers[i].getIsGhost()) { // if player is a ghost
// Now the case "ghost is og" and the case "ghost is not og" need to be handled
/* TODO: I don't know how to get the information about a ghost being the OG because I'm accessing the players
via the Passenger class which can't use the getIsOG method from the Ghost class. I (Alex) will try to
solve this issue but if anyone can help please do!
*/
}
} }
} }
} }

View File

@ -13,4 +13,6 @@ public class Ghost extends Passenger {
public boolean getIsOG() { public boolean getIsOG() {
return isOG; return isOG;
} }
public void setIsOG(boolean og) { isOG = og; };
} }

View File

@ -16,6 +16,7 @@ public class Passenger {
protected Boolean kickedOff; //true if the player has been voted off. protected Boolean kickedOff; //true if the player has been voted off.
protected ClientHandler clientHandler;//the socket for the client associated with this Passenger, for NPCs, this can be null. protected ClientHandler clientHandler;//the socket for the client associated with this Passenger, for NPCs, this can be null.
protected boolean hasVoted; //true if the player gave his vote during voting time protected boolean hasVoted; //true if the player gave his vote during voting time
protected int vote; //saves the number of the player this passenger voted for during voting (0-5)
/** /**
* Sends a protocol message to the respective player. * Sends a protocol message to the respective player.
@ -83,7 +84,9 @@ public class Passenger {
return isPlayer; return isPlayer;
} }
public boolean getHasVoted() { return hasVoted; }; public boolean getHasVoted() { return hasVoted; }; // returns true if player already voted during a voting
public int getVote() { return vote; }
public ClientHandler getClientHandler() { public ClientHandler getClientHandler() {
return clientHandler; return clientHandler;