From 2bff8c999edde11270ecaf6fbd2d24f54d0e360b Mon Sep 17 00:00:00 2001 From: HampusM Date: Wed, 29 Jun 2022 22:15:36 +0200 Subject: refactor: cell helper take matrix as shared pointer --- src/game/cell_helper.hpp | 4 ++-- src/game/cell_helper_impl.hpp | 11 ++++++----- 2 files changed, 8 insertions(+), 7 deletions(-) (limited to 'src/game') diff --git a/src/game/cell_helper.hpp b/src/game/cell_helper.hpp index ee5ab27..cf84a75 100644 --- a/src/game/cell_helper.hpp +++ b/src/game/cell_helper.hpp @@ -13,7 +13,7 @@ template class CellHelper : public ICellHelper { public: - explicit CellHelper(const IMatrix &matrix) noexcept; + explicit CellHelper(std::shared_ptr> matrix) noexcept; [[nodiscard]] auto is_cell_dying(const Vector2 &cell_pos) const noexcept -> bool override; @@ -26,7 +26,7 @@ public: -> std::vector override; private: - const IMatrix &_matrix; + std::shared_ptr> _matrix; static auto _get_position_neighbours(const Vector2 &position) noexcept -> std::list; diff --git a/src/game/cell_helper_impl.hpp b/src/game/cell_helper_impl.hpp index b7da413..bd40794 100644 --- a/src/game/cell_helper_impl.hpp +++ b/src/game/cell_helper_impl.hpp @@ -6,18 +6,19 @@ template constexpr auto has_matrix_value( - const IMatrix &matrix, + const std::shared_ptr> &matrix, const MatrixElement &value) noexcept { return [&matrix, &value](const Vector2 &pos) { - return matrix.get(pos) == value; + return matrix->get(pos) == value; }; } template -CellHelper::CellHelper(const IMatrix &matrix) noexcept - : _matrix(matrix) +CellHelper::CellHelper( + std::shared_ptr> matrix) noexcept + : _matrix(std::move(matrix)) { } @@ -75,7 +76,7 @@ auto CellHelper::find_neighbours(const Vector2 &cell_pos) const n std::vector cell_positions = {}; const auto matrix_size = - Bounds({.width = _matrix.get_column_cnt(), .height = _matrix.get_row_cnt()}); + Bounds({.width = _matrix->get_column_cnt(), .height = _matrix->get_row_cnt()}); const auto neighbours = _get_position_neighbours(cell_pos); -- cgit v1.2.3-18-g5258