juk3n/fluffy · 1fbb4dc

⑂ 1fbb4dc

46 prevent from duplication when adding new game (#47)Unverified

* moved add game code to separate function in application class

* added game name duplication validation

---------

Co-authored-by: Adrian Duraj <adrian.duraj@onmail.com>

Adrian Duraj committed 2026-09-21T07:39:14+02:00 Browse files

1fbb4dc parent: 21b1737

modified app/include/fluffyApplication.hpp +1 -1 Viewed

@@ -36,7 +36,7 @@ fluffy commands:\n\
36	36	show Show all games added to library\n\
37	37	rm <game_name> Remove game from a library"};
38	38
39		- auto addGame(Database& database, std::string name, std::string path) -> void;
	39	+ auto addGame(std::string name, std::string path) -> void;
40	40
41	41	auto removeGame(Database& database, std::string name) -> void;
42	42

      show                          Show all games added to library\n\
      rm <game_name>                Remove game from a library"};
--    auto addGame(Database& database, std::string name, std::string path) -> void;
++    auto addGame(std::string name, std::string path) -> void;
      auto removeGame(Database& database, std::string name) -> void;

modified app/src/fluffyApplication.cpp +25 -14 Viewed

@@ -31,20 +31,9 @@ auto FluffyApplication::handleCommand(
31	31	gameRepository->deleteGame(gameName);
32	32	}
33	33	else if (command == "add") {
34		- std::string gameName{arguments[0]};
35		- auto localPath = std::filesystem::path(arguments[1]);
36		- std::string globalPath =
37		- std::filesystem::absolute(localPath).lexically_normal().string();
38		- std::string temp{};
39		- for (const auto a : globalPath) {
40		- if (a != ' ') {
41		- temp += a;
42		- } else {
43		- temp += "\' \'";
44		- }
45		- }
46		-
47		- gameRepository->addGame(gameName, temp);
	34	+ std::string gameName = arguments[0];
	35	+ std::string gamePath = arguments[1];
	36	+ this->addGame(gameName, gamePath);
48	37	}
49	38	else if (command == "play") {
50	39	std::string gameName{arguments[0]};
@@ -61,6 +50,28 @@ auto FluffyApplication::handleCommand(
61	50	}
62	51	}
63	52
	53	+auto FluffyApplication::addGame(std::string name, std::string path) -> void {
	54	+ std::string gameName{name};
	55	+ auto localPath = std::filesystem::path(path);
	56	+ std::string globalPath =
	57	+ std::filesystem::absolute(localPath).lexically_normal().string();
	58	+ std::string temp{};
	59	+ for (const auto a : globalPath) {
	60	+ if (a != ' ') {
	61	+ temp += a;
	62	+ } else {
	63	+ temp += "\' \'";
	64	+ }
	65	+ }
	66	+ Game currentGame = gameRepository->getGameByName(gameName);
	67	+ if (currentGame.getName().length() != 0) {
	68	+ output->printMessage("Game already exist");
	69	+ }
	70	+ else {
	71	+ gameRepository->addGame(gameName, temp);
	72	+ }
	73	+
	74	+}
64	75
65	76	auto FluffyApplication::handleFlags(std::vector<std::string> flags) -> void {
66	77	for (const auto& flag : flags) {

          gameRepository->deleteGame(gameName);
      }
      else if (command == "add") {
--        std::string gameName{arguments[0]};
++        std::string gameName = arguments[0];
--        auto localPath = std::filesystem::path(arguments[1]);
++        std::string gamePath = arguments[1];
--        std::string globalPath =
++        this->addGame(gameName, gamePath);
--            std::filesystem::absolute(localPath).lexically_normal().string();
--        std::string temp{};
--        for (const auto a : globalPath) {
--        if (a != ' ') {
--            temp += a;
--        } else {
--            temp += "\' \'";
--        }
--        }
--
--        gameRepository->addGame(gameName, temp);
      }
      else if (command == "play") {
          std::string gameName{arguments[0]};
      }
  }
++auto FluffyApplication::addGame(std::string name, std::string path) -> void {
++    std::string gameName{name};
++    auto localPath = std::filesystem::path(path);
++    std::string globalPath =
++        std::filesystem::absolute(localPath).lexically_normal().string();
++    std::string temp{};
++    for (const auto a : globalPath) {
++        if (a != ' ') {
++            temp += a;
++        } else {
++            temp += "\' \'";
++        }
++    }
++    Game currentGame = gameRepository->getGameByName(gameName);
++    if (currentGame.getName().length() != 0) {
++        output->printMessage("Game already exist");
++    }
++    else {
++        gameRepository->addGame(gameName, temp);
++    }
++
++}
  auto FluffyApplication::handleFlags(std::vector<std::string> flags) -> void {
      for (const auto& flag : flags) {