fix(animation): use findBone instead of manual reading from animation channels

Author: AhmedAmrNabil

src/common/animation/animation.hpp

@@ -33,12 +33,12 @@ namespace our {
                 channels[nodeName] = BoneAnimation(nodeName, channel);
             }
         }
-        BoneAnimation& findBone(const std::string& name) {
+        const BoneAnimation* findBone(const std::string& name) const {
             auto it = channels.find(name);
             if (it != channels.end()) {
-                return it->second;
+                return &it->second;
             }
-            throw std::runtime_error("BoneAnimation not found: " + name);
+            return nullptr;
         }
     };
 }  // namespace our

src/common/animation/animator.hpp

@@ -36,9 +36,8 @@ namespace our {
 
                 // animated local transform, fallback to bind pose
                 glm::mat4 localTransform = node.localTransform;
-                auto it = currentAnimation->channels.find(node.name);
-                if (it != currentAnimation->channels.end()) {
-                    localTransform = it->second.interpolate(currentTime);
+                if (const BoneAnimation* boneAnim = currentAnimation->findBone(node.name); boneAnim) {
+                    localTransform = boneAnim->interpolate(currentTime);
                 }
 
                 // accumulate from parent