mirror of
https://github.com/Cockatrice/Cockatrice.git
synced 2026-09-21 09:05:10 -07:00
8d30ac54f0
* [Game] Prevent spectator duplication when replaying joined events The spectator branch of eventJoin emitted spectatorJoined unconditionally even when the spectator was already present (e.g. replayed during a rewind). Guard it like the player branch and eventGameStateChanged, and make PlayerListWidget::addPlayer idempotent as defense in depth. * In resetChatAndPhase() (the rewound() handler), also clear all spectators from both PlayerManager and PlayerListWidget before the replay rebuilds from event 0. The forward replay then re-adds exactly the spectators whose join events fall within the new time range via eventGameStateChanged/eventJoin. --------- Co-authored-by: Lukas Brübach <Bruebach.Lukas@bdosecurity.de>
123 lines
3.1 KiB
C++
123 lines
3.1 KiB
C++
/**
|
|
* @file player_manager.h
|
|
* @ingroup GameLogicPlayers
|
|
*/
|
|
//! \todo Document this file.
|
|
|
|
#ifndef COCKATRICE_PLAYER_MANAGER_H
|
|
#define COCKATRICE_PLAYER_MANAGER_H
|
|
|
|
#include <QMap>
|
|
#include <QObject>
|
|
#include <libcockatrice/protocol/pb/serverinfo_playerproperties.pb.h>
|
|
|
|
class AbstractGame;
|
|
class PlayerLogic;
|
|
class PlayerManager : public QObject
|
|
{
|
|
Q_OBJECT
|
|
|
|
public:
|
|
PlayerManager(AbstractGame *_game, int _localPlayerId, bool _localPlayerIsJudge, bool localPlayerIsSpectator);
|
|
|
|
AbstractGame *game;
|
|
QMap<int, PlayerLogic *> players;
|
|
int localPlayerId;
|
|
bool localPlayerIsJudge;
|
|
bool localPlayerIsSpectator;
|
|
QMap<int, ServerInfo_User> spectators;
|
|
|
|
[[nodiscard]] bool isSpectator() const
|
|
{
|
|
return localPlayerIsSpectator;
|
|
}
|
|
|
|
[[nodiscard]] bool isJudge() const
|
|
{
|
|
return localPlayerIsJudge;
|
|
}
|
|
|
|
[[nodiscard]] int getLocalPlayerId() const
|
|
{
|
|
return localPlayerId;
|
|
}
|
|
|
|
[[nodiscard]] const QMap<int, PlayerLogic *> &getPlayers() const
|
|
{
|
|
return players;
|
|
}
|
|
|
|
[[nodiscard]] int getPlayerCount() const
|
|
{
|
|
return players.size();
|
|
}
|
|
|
|
[[nodiscard]] PlayerLogic *getActiveLocalPlayer(int activePlayer) const;
|
|
bool isLocalPlayer(int playerId);
|
|
|
|
PlayerLogic *addPlayer(int playerId, const ServerInfo_User &info);
|
|
|
|
void removePlayer(int playerId);
|
|
|
|
[[nodiscard]] PlayerLogic *getPlayer(int playerId) const;
|
|
|
|
void onPlayerConceded(int playerId, bool conceded);
|
|
|
|
[[nodiscard]] bool isMainPlayerConceded() const;
|
|
|
|
[[nodiscard]] bool isLocalPlayer(int playerId) const
|
|
{
|
|
return playerId == getLocalPlayerId();
|
|
}
|
|
|
|
[[nodiscard]] const QMap<int, ServerInfo_User> &getSpectators() const
|
|
{
|
|
return spectators;
|
|
}
|
|
|
|
[[nodiscard]] ServerInfo_User getSpectator(int playerId) const
|
|
{
|
|
return spectators.value(playerId);
|
|
}
|
|
|
|
[[nodiscard]] QString getSpectatorName(int spectatorId) const
|
|
{
|
|
return QString::fromStdString(spectators.value(spectatorId).name());
|
|
}
|
|
|
|
void addSpectator(int spectatorId, const ServerInfo_PlayerProperties &prop)
|
|
{
|
|
if (!spectators.contains(spectatorId)) {
|
|
spectators.insert(spectatorId, prop.user_info());
|
|
emit spectatorAdded(prop);
|
|
}
|
|
}
|
|
|
|
void removeSpectator(int spectatorId)
|
|
{
|
|
ServerInfo_User spectatorInfo = spectators.value(spectatorId);
|
|
spectators.remove(spectatorId);
|
|
emit spectatorRemoved(spectatorId, spectatorInfo);
|
|
}
|
|
|
|
/** @brief Remove all spectators, emitting the removal signal for each. */
|
|
void clearSpectators();
|
|
|
|
[[nodiscard]] AbstractGame *getGame() const
|
|
{
|
|
return game;
|
|
}
|
|
|
|
signals:
|
|
void playerAdded(PlayerLogic *player);
|
|
void playerRemoved(PlayerLogic *player);
|
|
void activeLocalPlayerConceded();
|
|
void activeLocalPlayerUnconceded();
|
|
void playerConceded(int playerId);
|
|
void playerUnconceded(int playerId);
|
|
void playerCountChanged();
|
|
void spectatorAdded(ServerInfo_PlayerProperties spectator);
|
|
void spectatorRemoved(int spectatorId, ServerInfo_User spectator);
|
|
};
|
|
|
|
#endif // COCKATRICE_PLAYER_MANAGER_H
|