diff --git a/source/general/game_state/card.cpp b/source/general/game_state/card.cpp
index 0015d98f163d8511cd872a70d2f748794be4bf1a..edf1f37f6d5c46b6610cc64ce99de3bc2a72f740 100644
--- a/source/general/game_state/card.cpp
+++ b/source/general/game_state/card.cpp
@@ -36,7 +36,7 @@ card *card::from_json(const rapidjson::Value &json) {
}
}
-// is this even correct ?
+
void card::write_into_json(rapidjson::Value &json, rapidjson::Document::AllocatorType& allocator) const {
unique_serializable::write_into_json(json, allocator);
diff --git a/source/general/game_state/card.hpp b/source/general/game_state/card.hpp
index 05839b9ebf629aea9f1b224a578a4c90f80ecc62..5f2340d07984f7af0b6689be6076960a6781c1d2 100644
--- a/source/general/game_state/card.hpp
+++ b/source/general/game_state/card.hpp
@@ -12,7 +12,7 @@ private:
serializable_value<int>* _suit; // '0' for spades, '1' for clubs, '2' for diamonds, '3' for hearts
// from_diff constructor
- card(std::string id); // what is this exactly ?
+ card(std::string id);
// deserialization constructor
card(std::string id, serializable_value<int>* val, serializable_value<int>* suit);
public:
diff --git a/source/general/game_state/game_state.hpp b/source/general/game_state/game_state.hpp
index 573a21be681e42e9cfca48c35ec90753e34b2711..8721869400e05a7234076d42651dc8556b195fe9 100644
--- a/source/general/game_state/game_state.hpp
+++ b/source/general/game_state/game_state.hpp
@@ -10,9 +10,6 @@
#include "../serialization/serializable_value.h"
#include "../serialization/unique_serializable.h"
-//TODO: check if legal turn for standing and everything
-//TODO: make bets before ppl play
-
class game_state : public unique_serializable {
public:
@@ -30,7 +27,7 @@ private:
serializable_value<bool>* _is_finished;
serializable_value<int>* _round_number;
serializable_value<int>* _current_player_idx;
- serializable_value<int>* _starting_player_idx; // very possible this variable is NOT needed
+ serializable_value<int>* _starting_player_idx;
// from_diff constructor
game_state(std::string id);
@@ -85,9 +82,6 @@ public:
// end of round functions
void wrap_up_round(std::string& err);
- // functions from our SDS
- // void compute_dealers_hand(); // does hardcoded actions for dealer
-
//Flag that setup needs to be called
bool needs_setup = false;
#endif
diff --git a/source/general/game_state/player.cpp b/source/general/game_state/player.cpp
index 54e5a6d17947d25f44087c449b3258ca7e79b9de..30a6873db64baf0a4fcc65aedfad08f7e9498ba0 100644
--- a/source/general/game_state/player.cpp
+++ b/source/general/game_state/player.cpp
@@ -113,7 +113,7 @@ void player::wrap_up_round(int dealer_points, std::string& err) {
}
}
-
+// hit and stand are implemented directly in game_state
bool player::make_bet(int bet_size, std::string &err) {
if(bet_size > this->get_money()) {
err = "bet_size is bigger than amount of money the player " + this->_player_name->get_value() + " has.";
diff --git a/source/general/game_state/player.hpp b/source/general/game_state/player.hpp
index 57e0047bbb5bdaf19ffc678c1541a2776986be46..fc29e76018f0332a3947da0d0b786ff727cda6b1 100644
--- a/source/general/game_state/player.hpp
+++ b/source/general/game_state/player.hpp
@@ -10,7 +10,6 @@
#include "../serialization/unique_serializable.h"
#include "../serialization/serializable_value.h"
-//TODO: hit implemented in game_state, also set flag if over 21
class player : public unique_serializable {
private:
@@ -21,7 +20,7 @@ private:
hand* _player_hand;
-#ifdef BLACKJACK_SERVER // is this macro needed?
+#ifdef BLACKJACK_SERVER
std::string _game_id;
#endif
@@ -58,13 +57,12 @@ public:
#ifdef BLACKJACK_SERVER
// state update functions
- void wrap_up_round(int dealer_points, std::string& err); //TODO
+ void wrap_up_round(int dealer_points, std::string& err);
void setup_round(std::string& err);
- // player actions (probably not needed)
+ // player action
bool make_bet(int bet_size, std::string &err);
- // helper functions for game_state
// helper functions to calculate winnings
void won_round();
void draw_round();
diff --git a/source/general/game_state/shoe.hpp b/source/general/game_state/shoe.hpp
index bf154daf61f8dda49b332607659da26f09ef38ce..975589e882ca5cf491b8ee1c68875956a892fb4a 100644
--- a/source/general/game_state/shoe.hpp
+++ b/source/general/game_state/shoe.hpp
@@ -11,8 +11,7 @@
#include "../serialization/serializable_value.h"
#include "../../../rapidjson/include/rapidjson/document.h"
-//TODO: get rid of player in draw card, you just draw a card from the shoe, then you add it to a
-//player's hand through fct like hit, stand etc
+
class shoe : public unique_serializable {
private:
std::vector<card*> _cards;
diff --git a/source/general/network/requests/join_game_request.cpp b/source/general/network/requests/join_game_request.cpp
index bd0ecc1ad87ed7d23c5d75e88db88ec2a6ec029e..93914f37c962984c2774659cd99ba27ee9c2e6ce 100644
--- a/source/general/network/requests/join_game_request.cpp
+++ b/source/general/network/requests/join_game_request.cpp
@@ -15,7 +15,6 @@ join_game_request::join_game_request(std::string player_id, std::string name)
_player_name(name)
{ }
-// do we even want to request joining specific games?
join_game_request::join_game_request(std::string game_id, std::string player_id, std::string name)
: client_request( client_request::create_base_class_properties(RequestType::join_game, uuid_generator::generate_uuid_v4(), player_id, game_id) ),
_player_name(name)
diff --git a/source/general/network/requests/start_game_request.cpp b/source/general/network/requests/start_game_request.cpp
index c779eeebedb47d3bd27399369fb53df418af3278..ac5972aae08d9d25661fc1930acd17bfbc6c0d10 100644
--- a/source/general/network/requests/start_game_request.cpp
+++ b/source/general/network/requests/start_game_request.cpp
@@ -1,7 +1,7 @@
#include "start_game_request.hpp"
#ifdef BLACKJACK_SERVER
-#include "../../../server/game_instance_manager.hpp" //not planned for now ?
+#include "../../../server/game_instance_manager.hpp"
#include "../../../server/game_instance.hpp"
#endif
diff --git a/source/general/network/responses/answer_rqst_response.cpp b/source/general/network/responses/answer_rqst_response.cpp
index 369ad9677c97b9084b031654a48ddff98626c4dc..68bd9cd6d3a52dcc23d04a650cf52765b7c9b641 100644
--- a/source/general/network/responses/answer_rqst_response.cpp
+++ b/source/general/network/responses/answer_rqst_response.cpp
@@ -4,7 +4,7 @@
#include "../../game_state/game_state.hpp"
#ifdef BLACKJACK_CLIENT
-#include "../../../client/GameController.hpp" //adapt this if needed!
+#include "../../../client/GameController.hpp"
#endif
diff --git a/source/general/network/responses/answer_rqst_response.hpp b/source/general/network/responses/answer_rqst_response.hpp
index fc1dd1da962996a20ccf8cf38fa3ffd6f4c8cf25..4f45cb0759c4e15af6d7fe5935275ce0151bd2e0 100644
--- a/source/general/network/responses/answer_rqst_response.hpp
+++ b/source/general/network/responses/answer_rqst_response.hpp
@@ -9,7 +9,7 @@ class answer_rqst_response : public server_response {
private:
bool _success;
std::string _err;
- std::string _req_id; //not mentioned in SDS!
+ std::string _req_id;
rapidjson::Value* _state_json = nullptr;
answer_rqst_response(base_class_properties props, std::string req_id, bool success, rapidjson::Value* state_json, std::string& err);
diff --git a/source/general/network/responses/change_gamestate_msg.cpp b/source/general/network/responses/change_gamestate_msg.cpp
index 825443b05eb26d695f5be4c0a8b2468056d16b89..4fd99379b77638da16932b258bf632ba4cdc4fc3 100644
--- a/source/general/network/responses/change_gamestate_msg.cpp
+++ b/source/general/network/responses/change_gamestate_msg.cpp
@@ -4,7 +4,7 @@
#include "../../serialization/json_utils.h"
#ifdef BLACKJACK_CLIENT
-#include "../../../client/GameController.hpp" //adapt if necessary
+#include "../../../client/GameController.hpp"
#endif
change_gamestate_msg::change_gamestate_msg(server_response::base_class_properties props, rapidjson::Value* state_json) :
@@ -50,7 +50,7 @@ rapidjson::Value* change_gamestate_msg::get_state_json() const {
void change_gamestate_msg::Process() const {
try {
game_state* state = game_state::from_json(*_state_json);
- GameController::updateGameState(state); //do we have this function?
+ GameController::updateGameState(state);
} catch(std::exception& e) {
std::cerr << "Failed to extract game_state from change_gamestate_msg" << std::endl
diff --git a/source/general/network/responses/server_response.cpp b/source/general/network/responses/server_response.cpp
index be5517db5c7b289010fc0bb0535ef36d487ddaff..7e4bff10ff05221cd9764031d8efd90d93a1e11f 100644
--- a/source/general/network/responses/server_response.cpp
+++ b/source/general/network/responses/server_response.cpp
@@ -7,7 +7,7 @@
// for deserialization
const std::unordered_map<std::string, ResponseType> server_response::_string_to_response_type = {
{"req_response", ResponseType::req_response},
- {"state_diff_msg", ResponseType::state_diff_msg}, //is this even needed ? not in SDS !
+ {"state_diff_msg", ResponseType::state_diff_msg},
{"change_gamestate", ResponseType::change_gamestate}
};
// for serialization
@@ -61,12 +61,12 @@ server_response *server_response::from_json(const rapidjson::Value& json) {
ResponseType response_type = server_response::_string_to_response_type.at(type);
if (response_type == ResponseType::req_response) {
- return answer_rqst_response::from_json(json); //not sure if correct
+ return answer_rqst_response::from_json(json);
}
else if (response_type == ResponseType::change_gamestate) {
- return change_gamestate_msg::from_json(json); //not sure if correct
+ return change_gamestate_msg::from_json(json);
} else {
- throw BlackjackException("Encountered unknown ServerResponse type " + type); //should response_type not be string?
+ throw BlackjackException("Encountered unknown ServerResponse type " + type);
}
}
throw BlackjackException("Could not determine type of ClientRequest");