safe guard code + make internal private

package/cpp/core/RendererWrapper.cpp

@@ -16,10 +16,20 @@ void RendererWrapper::loadHybridMethods() {
 }
 
 bool RendererWrapper::beginFrame(std::shared_ptr<SwapChainWrapper> swapChain, double frameTimeNanos) {
-  return _renderer->beginFrame(swapChain->getSwapChain().get(), frameTimeNanos);
+  if (swapChain->getSwapChain() == nullptr) {
+    throw std::runtime_error("SwapChain is null");
+  }
+
+  SwapChain* swapChainPtr = swapChain->getSwapChain().get();
+  return _renderer->beginFrame(swapChainPtr, frameTimeNanos);
 }
 void RendererWrapper::render(std::shared_ptr<ViewWrapper> view) {
-  _renderer->render(view->getView().get());
+  if (view->getView() == nullptr) {
+    throw std::runtime_error("View is null");
+  }
+
+  View* viewPtr = view->getView().get();
+  _renderer->render(viewPtr);
 }
 void RendererWrapper::endFrame() {
   _renderer->endFrame();

package/cpp/core/SceneWrapper.cpp

@@ -5,5 +5,5 @@ void margelo::SceneWrapper::loadHybridMethods() {
 }
 
 void margelo::SceneWrapper::addEntity(std::shared_ptr<EntityWrapper> entity) {
-  _scene->addEntity(entity->entity);
+  _scene->addEntity(entity->getEntity());
 }

package/cpp/core/ViewWrapper.h

@@ -12,7 +12,7 @@ using namespace filament;
 
 class ViewWrapper : public HybridObject {
 public:
-  explicit ViewWrapper(const std::shared_ptr<View>& view) : _view(std::move(view)) {}
+  explicit ViewWrapper(std::shared_ptr<View> view) : _view(view) {}
 
   void loadHybridMethods() override;
 

package/cpp/core/utils/EntityWrapper.h

@@ -9,10 +9,15 @@ using namespace utils;
 
 class EntityWrapper : public HybridObject {
 public:
-  explicit EntityWrapper(Entity entity) : entity(entity) {}
+  explicit EntityWrapper(Entity entity) : _entity(entity) {}
 
   void loadHybridMethods() override;
 
-  Entity entity;
+  const Entity& getEntity() {
+    return _entity;
+  }
+
+private:
+  Entity _entity;
 };
 } // namespace margelo