Improve the interface of the tile loader

Allow the loader to support multiple tilesets with tiles of different
size

include/domain/Vector.hpp

@@ -10,6 +10,7 @@ struct Vector2D {
 };
 
 using Vector2i = Vector2D<int>;
+using Vector2u = Vector2D<unsigned int>;
 
 } // namespace Domain
 

include/interface/Tile.hpp

@@ -1,8 +1,8 @@
 #ifndef INTERFACE_TILE_HPP
 #define INTERFACE_TILE_HPP
 
-#include "domain/Shape.hpp"
 #include "domain/Sprite.hpp"
+#include "domain/Vector.hpp"
 #include <SFML/Graphics/Sprite.hpp>
 #include <SFML/Graphics/Texture.hpp>
 #include <memory>
@@ -13,8 +13,8 @@ namespace Interface {
 
 class TileManager {
 public:
-  void load(const std::string &path, const Domain::IntRect &textureRect,
-            int tileSize);
+  bool load(const std::string &path, const Domain::Vector2u &tileSize,
+            unsigned int tileCount, unsigned int columnCount);
 
   const sf::Sprite &get(Domain::Sprite::ID spriteId) const;
 

src/interface/Tile.cpp

@@ -1,24 +1,32 @@
 #include "interface/Tile.hpp"
+#include "domain/Vector.hpp"
 #include <SFML/Graphics/Rect.hpp>
 
 namespace Interface {
 
-void TileManager::load(const std::string &    path,
-                       const Domain::IntRect &textureRect, int tileSize) {
+bool TileManager::load(const std::string &     path,
+                       const Domain::Vector2u &tileSize, unsigned int tileCount,
+                       unsigned int columnCount) {
   auto texture = std::make_unique<sf::Texture>();
 
-  if (!texture->loadFromFile(path, {textureRect.left, textureRect.top,
-                                    textureRect.width, textureRect.height})) {
-    // Throw exception
+  if (!texture->loadFromFile(path)) {
+    return false;
   }
 
-  for (int y = 0; y < textureRect.height; y += tileSize) {
-    for (int x = 0; x < textureRect.width; x += tileSize) {
-      _sprites.emplace_back(*texture, sf::IntRect{x, y, tileSize, tileSize});
+  const auto rowCount = tileCount / columnCount;
+  for (unsigned int y = 0; y < rowCount; ++y) {
+    for (unsigned int x = 0; x < columnCount; ++x) {
+      _sprites.emplace_back(*texture, sf::IntRect{
+                                          static_cast<int>(x * tileSize.x),
+                                          static_cast<int>(y * tileSize.y),
+                                          static_cast<int>(tileSize.x),
+                                          static_cast<int>(tileSize.y),
+                                      });
     }
   }
 
   _textures.push_back(std::move(texture));
+  return true;
 }
 
 const sf::Sprite &TileManager::get(Domain::Sprite::ID spriteId) const {