removed usless code added constexpr

include/omath/pathfinding/Astar.h

@@ -10,7 +10,7 @@
 
 namespace omath::pathfinding
 {
-    class Astar
+    class Astar final
     {
     public:
         [[nodiscard]]

include/omath/pathfinding/NavigationMesh.h

@@ -12,19 +12,12 @@
 
 namespace omath::pathfinding
 {
-    struct NavigationVertex
-    {
-        Vector3 origin;
-        std::vector<NavigationVertex*> connections;
-    };
-
-
     class NavigationMesh final
     {
     public:
 
         [[nodiscard]]
-        std::expected<Vector3, std::string> GetClossestVertex(const Vector3& point) const;
+        std::expected<Vector3, std::string> GetClosestVertex(const Vector3& point) const;
 
 
         [[nodiscard]]

include/omath/prediction/Target.h

@@ -13,7 +13,15 @@ namespace omath::prediction
     public:
 
         [[nodiscard]]
-        Vector3 PredictPosition(float time, float gravity) const;
+        constexpr Vector3 PredictPosition(float time, float gravity) const
+        {
+            auto predicted = m_origin + m_velocity * time;
+
+            if (m_isAirborne)
+                predicted.z -= gravity * std::pow(time, 2.f) * 0.5f;
+
+            return predicted;
+        }
 
         Vector3 m_origin;
         Vector3 m_velocity;

source/pathfinding/Astar.cpp

@@ -23,8 +23,8 @@ namespace omath::pathfinding
         std::unordered_map<Vector3, PathNode> closedList;
         std::unordered_map<Vector3, PathNode> openList;
 
-        const auto startVertex = navMesh.GetClossestVertex(start).value();
-        const auto endVertex = navMesh.GetClossestVertex(end).value();
+        const auto startVertex = navMesh.GetClosestVertex(start).value();
+        const auto endVertex = navMesh.GetClosestVertex(end).value();
 
         openList.emplace(startVertex, PathNode{std::nullopt, 0.f});
 

source/pathfinding/NavigationMesh.cpp

@@ -7,7 +7,7 @@
 #include <algorithm>
 namespace omath::pathfinding
 {
-    std::expected<Vector3, std::string> NavigationMesh::GetClossestVertex(const Vector3 &point) const
+    std::expected<Vector3, std::string> NavigationMesh::GetClosestVertex(const Vector3 &point) const
     {
         const auto res = std::ranges::min_element(m_verTextMap,
             [&point](const auto& a, const auto& b)

source/prediction/Target.cpp

@@ -3,18 +3,9 @@
 //
 
 #include "omath/prediction/Target.h"
-#include <cmath>
 
 
 namespace omath::prediction
 {
-    Vector3 Target::PredictPosition(const float time, const float gravity) const
-    {
-        auto predicted = m_origin + m_velocity * time;
 
-        if (m_isAirborne)
-            predicted.z -= gravity * std::pow(time, 2.f) * 0.5f;
-
-        return predicted;
-    }
 }