Work on texture class

+ Set up TextureManager class to handle static texture map

app/main.cpp

@@ -8,8 +8,8 @@
 
 #include <QApplication>
 #include <QLabel>
+#include <QSurfaceFormat>
 
-#include <mainwidget.h>
 #include <mainwindow.h>
 
 

src/camera3d.cpp

@@ -23,8 +23,8 @@ const QMatrix4x4 & Camera3D::toMatrix()
 {
   mWorld.setToIdentity();
   // Qt6 renamed QMatrix4x4::conjugate() to conjugated()
-  mWorld.rotate(mTransform.rotation().conjugated());
+  mWorld.rotate(mTransform.getRotation().conjugated());
-  mWorld.translate(-mTransform.translation());
+  mWorld.translate(-mTransform.getTranslation());
   return mWorld;
 }
 

src/camera3d.h

@@ -24,18 +24,18 @@ public:
   // Accessors
   inline Transform3D & transform() { return mTransform;}
   inline const QVector3D & translation() const
-  { return mTransform.translation();}
+  { return mTransform.getTranslation();}
   inline const QQuaternion & rotation() const
-  { return mTransform.rotation();}
+  { return mTransform.getRotation();}
   const QMatrix4x4 & toMatrix();
 
   // Queries
   inline QVector3D forward() const
-  { return mTransform.rotation().rotatedVector(LocalForward);}
+  { return mTransform.getRotation().rotatedVector(LocalForward);}
   inline QVector3D right() const
-  { return mTransform.rotation().rotatedVector(LocalRight);}
+  { return mTransform.getRotation().rotatedVector(LocalRight);}
   inline QVector3D up() const
-  { return mTransform.rotation().rotatedVector(LocalUp);}
+  { return mTransform.getRotation().rotatedVector(LocalUp);}
 
 private:
   Transform3D mTransform;

src/mainwidget.cpp

@@ -62,8 +62,8 @@ void MainWidget::initObjects()
   // The Object class only stores basic QOpenGL* members and shape data
   // + Within mainwidget, mObject serves as a basic QOpenGL example
   mObject = new Object("testObject");
-  mObject->setVertices(Cube(QTK_DRAW_ELEMENTS).vertices());
+  mObject->setVertices(Cube(QTK_DRAW_ELEMENTS).getVertices());
-  mObject->setIndices(Cube(QTK_DRAW_ELEMENTS).indices());
+  mObject->setIndices(Cube(QTK_DRAW_ELEMENTS).getIndexData());
   mObject->mProgram.create();
   mObject->mProgram.addShaderFromSourceFile(QOpenGLShader::Vertex,
                                             ":/solid-ambient.vert");
@@ -79,13 +79,15 @@ void MainWidget::initObjects()
   mObject->mVBO.setUsagePattern(QOpenGLBuffer::StaticDraw);
   mObject->mVBO.bind();
 
-  mObject->mVBO.allocate(mObject->vertices().data(),
+  mObject->mVBO.allocate(
-                         mObject->vertices().size()
+      mObject->getVertices().data(),
-                         * sizeof(mObject->vertices()[0]));
+      mObject->getVertices().size() * sizeof(mObject->getVertices()[0])
+  );
 
   mObject->mProgram.enableAttributeArray(0);
-  mObject->mProgram.setAttributeBuffer(0, GL_FLOAT, 0,
+  mObject->mProgram.setAttributeBuffer(
-                                       3, sizeof(mObject->vertices()[0]));
+      0, GL_FLOAT, 0, 3, sizeof(mObject->getVertices()[0])
+  );
   mObject->mProgram.setUniformValue("uColor", QVector3D(1.0f, 0.0f, 0.0f));
   mObject->mProgram.setUniformValue("uLightColor", WHITE);
   mObject->mProgram.setUniformValue("uAmbientStrength", 0.75f);
@@ -116,8 +118,8 @@ void MainWidget::paintGL()
   mObject->mProgram.setUniformValue("uModel", mObject->mTransform.toMatrix());
   mObject->mProgram.setUniformValue("uView", Scene::Camera().toMatrix());
   mObject->mProgram.setUniformValue("uProjection", Scene::Projection());
-  glDrawElements(GL_TRIANGLES, mObject->indices().size(),
+  glDrawElements(GL_TRIANGLES, mObject->getIndexData().size(),
-                 GL_UNSIGNED_INT, mObject->indices().data());
+                 GL_UNSIGNED_INT, mObject->getIndexData().data());
   mObject->mVAO.release();
   mObject->mProgram.release();
 }

src/mainwidget.h

@@ -22,7 +22,7 @@ class Model;
 class Object;
 class Scene;
 class Skybox;
-class Texture;
+class OpenGLTextureFactory;
 
 class MainWidget : public QOpenGLWidget,
                    protected QOpenGLFunctions {

src/mesh.h

@@ -86,11 +86,11 @@ struct ShapeBase {
       : mVertices(v), mColors(c), mIndices(i), mTexCoords(t), mNormals(n)
   {}
 
-  inline const Vertices & vertices() const { return mVertices;}
+  inline const Vertices & getVertices() const { return mVertices;}
-  inline const Indices & indices() const { return mIndices;}
+  inline const Indices & getIndexData() const { return mIndices;}
-  inline const Colors & colors() const { return mColors;}
+  inline const Colors & getColors() const { return mColors;}
-  inline const TexCoords & texCoords() const { return mTexCoords;}
+  inline const TexCoords & getTexCoords() const { return mTexCoords;}
-  inline const Normals & normals() const { return mNormals;}
+  inline const Normals & getNormals() const { return mNormals;}
 
 protected:
   DrawMode mDrawMode;

src/meshrenderer.cpp

@@ -29,9 +29,6 @@ MeshRenderer::MeshRenderer(const char * name, const ShapeBase & shape)
 
 MeshRenderer::~MeshRenderer()
 {
-  if (mHasTexture) {
-    mTexture->destroy();
-  }
   sInstances.remove(mName);
 }
 
@@ -68,9 +65,9 @@ void MeshRenderer::init()
 
   // Combine position and color data into one vector, allowing us to use one VBO
   Vertices combined;
-  combined.reserve(vertices().size() + colors().size());
+  combined.reserve(getVertices().size() + getColors().size());
-  combined.insert(combined.end(), vertices().begin(), vertices().end());
+  combined.insert(combined.end(), getVertices().begin(), getVertices().end());
-  combined.insert(combined.end(), colors().begin(), colors().end());
+  combined.insert(combined.end(), getColors().begin(), getColors().end());
 
   mVBO.allocate(combined.data(),
                 combined.size() * sizeof(combined[0]));
@@ -82,7 +79,7 @@ void MeshRenderer::init()
   // Enable color attribute, setting offset to total size of vertices()
   mProgram.enableAttributeArray(1);
   mProgram.setAttributeBuffer(1, GL_FLOAT,
-                              vertices().size() * sizeof(vertices()[0]),
+                              getVertices().size() * sizeof(getVertices()[0]),
                               3, sizeof(QVector3D));
 
   mVBO.release();
@@ -96,15 +93,15 @@ void MeshRenderer::draw()
   mProgram.bind();
   mVAO.bind();
 
-  if(mHasTexture) {
+  if(mTexture.hasTexture()) {
-    mTexture->bind();
+    mTexture.getOpenGLTexture().bind();
   }
 
   // TODO: Automate uniforms some other way
   setUniformMVP();
 
   if (mShape.mDrawMode == QTK_DRAW_ARRAYS) {
-    glDrawArrays(mDrawType, 0, vertices().size());
+    glDrawArrays(mDrawType, 0, getVertices().size());
   }
   else if (mShape.mDrawMode == QTK_DRAW_ELEMENTS
            || mShape.mDrawMode == QTK_DRAW_ELEMENTS_NORMALS) {
@@ -112,8 +109,8 @@ void MeshRenderer::draw()
                    GL_UNSIGNED_INT, mShape.mIndices.data());
   }
 
-  if(mHasTexture) {
+  if(mTexture.hasTexture()) {
-    mTexture->release();
+    mTexture.getOpenGLTexture().release();
   }
 
   mVAO.release();
@@ -137,29 +134,17 @@ void MeshRenderer::setUniformMVP(const char * model, const char * view,
 void MeshRenderer::setColor(const QVector3D & color)
 {
   if (mShape.mColors.empty()) {
-    for (const auto & vertex : mShape.vertices()) {
+    for (const auto & vertex : mShape.getVertices()) {
       mShape.mColors.push_back(color);
     }
   }
   else {
-    for (int i = 0; i < mShape.colors().size(); i++) {
+    for (int i = 0; i < mShape.getColors().size(); i++) {
       mShape.mColors[i] = color;
     }
   }
 }
 
-void MeshRenderer::setTexture(const char * path)
-{
-  mTexture = new QOpenGLTexture(*Texture::initImage(path));
-  mHasTexture = true;
-}
-
-void MeshRenderer::setTexture(QOpenGLTexture * texture)
-{
-  mTexture = texture;
-  mHasTexture = true;
-}
-
 
 /*******************************************************************************
  * Inherited Virtual Member Functions

src/meshrenderer.h

@@ -11,15 +11,17 @@
 #include <mesh.h>
 #include <object.h>
 
+#include <utility>
+
 
 class MeshRenderer : public Object {
 public:
   // Delegate constructors
   MeshRenderer(const char * name, Vertices vertices, Indices indices,
                DrawMode mode=QTK_DRAW_ARRAYS)
-      : MeshRenderer(name, ShapeBase(mode, vertices, indices))
+      : MeshRenderer(name, ShapeBase(mode, std::move(vertices), std::move(indices)))
   {}
-  MeshRenderer(const char * name)
+  explicit MeshRenderer(const char * name)
       : MeshRenderer(name, Cube(QTK_DRAW_ELEMENTS))
   {}
   // Constructor
@@ -54,11 +56,6 @@ public:
   void setUniformMVP(const char * model="uModel", const char * view="uView",
                      const char * projection="uProjection");
 
-  // Sets the texture to the image at the given path
-  // + Sets mHasTexture to enable texture binding in draw()
-  void setTexture(const char * path);
-  void setTexture(QOpenGLTexture * texture);
-
   // These functions modify data stored in a VBO
   // + After calling them, the VBO will need to be reallocated
   void setShape(const Shape & value) override;

src/model.cpp

@@ -175,7 +175,7 @@ void Model::flipTexture(const std::string & fileName, bool flipX, bool flipY)
       texture.mTexture->destroy();
       texture.mTexture->create();
       texture.mTexture->setData(
-          *Texture::initImage(fullPath.c_str(), flipX, flipY));
+          *OpenGLTextureFactory::initImage(fullPath.c_str(), flipX, flipY));
       modified = true;
     }
   }
@@ -388,7 +388,7 @@ ModelMesh::Textures Model::loadMaterialTextures(
     // If the texture has not yet been loaded
     if (!skip) {
       ModelTexture texture;
-      texture.mTexture = Texture::initTexture2D(
+      texture.mTexture = OpenGLTextureFactory::initTexture2D(
           std::string(mDirectory + '/' + fileName.C_Str()).c_str(),
           false, false);
       texture.mID = texture.mTexture->textureId();
@@ -409,11 +409,12 @@ ModelMesh::Textures Model::loadMaterialTextures(
 void Model::sortModels()
 {
   auto cameraPos = Scene::Camera().transform();
-  auto cameraDistance = [&cameraPos](const ModelMesh &a, const ModelMesh &b)
+  auto cameraDistance =
+      [&cameraPos](const ModelMesh &a, const ModelMesh &b)
       {
-    // Sort by the first vertex position, since all transforms will be the same
+        // Sort by the first vertex position in the model
-    return (cameraPos.translation().distanceToPoint(a.mVertices[0].mPosition))
+        return (cameraPos.getTranslation().distanceToPoint(a.mVertices[0].mPosition))
-           < (cameraPos.translation().distanceToPoint(b.mVertices[0].mPosition));
+               < (cameraPos.getTranslation().distanceToPoint(b.mVertices[0].mPosition));
       };
   std::sort(mMeshes.begin(), mMeshes.end(), cameraDistance);
 }

src/model.h

@@ -25,6 +25,7 @@
 
 // QTK
 #include <transform3D.h>
+#include <object.h>
 
 
 struct ModelVertex {

src/object.h

@@ -14,6 +14,7 @@
 #include <QOpenGLVertexArrayObject>
 
 #include <mesh.h>
+#include <texture.h>
 
 
 class Object : public QObject {
@@ -32,19 +33,23 @@ public:
 
   ~Object() {}
 
-  inline const Vertices & vertices() { return mShape.mVertices;}
+  inline const Colors & getColors() { return mShape.mColors;}
-  inline const Indices & indices() { return mShape.mIndices;}
+  inline const Indices & getIndexData() { return mShape.mIndices;}
-  inline const Colors & colors() { return mShape.mColors;}
+  inline const Normals & getNormals() { return mShape.mNormals;}
-  inline const TexCoords & texCoords() { return mShape.mTexCoords;}
+  inline const Shape & getShape() { return mShape;}
-  inline const Normals & normals() { return mShape.mNormals;}
+  inline const TexCoords & getTexCoords() { return mShape.mTexCoords;}
-  inline QOpenGLTexture & texture() const { return *mTexture;}
+  inline Texture & getTexture() { return mTexture;}
+  inline const Vertices & getVertices() { return mShape.mVertices;}
 
-  virtual inline void setVertices(const Vertices & value) { mShape.mVertices = value;}
-  virtual inline void setIndices(const Indices & value) { mShape.mIndices = value;}
   virtual inline void setColors(const Colors & value) { mShape.mColors = value;}
-  virtual inline void setTexCoords(const TexCoords & value) { mShape.mTexCoords = value;}
+  virtual inline void setIndices(const Indices & value) { mShape.mIndices = value;}
   virtual inline void setNormals(const Normals & value) { mShape.mNormals = value;}
   virtual inline void setShape(const Shape & value) { mShape = value;}
+  virtual inline void setTexCoords(const TexCoords & value) { mShape.mTexCoords = value;}
+  virtual inline void setTexture(const char * path, bool flipX=false, bool flipY=false)
+  { mTexture.setTexture(OpenGLTextureFactory::initTexture2D(path, flipX, flipY));}
+  virtual inline void setTexture(QOpenGLTexture * value) { mTexture.setTexture(value);}
+  virtual inline void setVertices(const Vertices & value) { mShape.mVertices = value;}
 
   QOpenGLBuffer mVBO, mNBO;
   QOpenGLVertexArrayObject mVAO;
@@ -52,10 +57,9 @@ public:
 
   Transform3D mTransform;
   Shape mShape;
+  Texture mTexture;
 
   const char * mName;
-private:
-  QOpenGLTexture * mTexture;
 };
 
 #endif // QTK_OBJECT_H

src/resourcemanager.h

@@ -24,10 +24,9 @@ public:
    *    Should be given in qrc format: ':/path/to/asset.obj'
    * @return Absoulte system path to a qtk asset
    */
-  static std::string getPath(const std::string & path) {
+  static std::string getPath(const std::string & path)
   // Only construct qtk resource path if in qrc format; else return it as-is
-    return path[0] == ':' ? resourcesDir + path.substr(2) : path;
+  { return path[0] == ':' ? resourcesDir + path.substr(2) : path; }
-  }
 
   static std::string resourcesDir;
 } RM;

src/scene.cpp

@@ -64,12 +64,11 @@ void Scene::init()
   mTestPhong->mNBO.create();
   mTestPhong->mNBO.setUsagePattern(QOpenGLBuffer::StaticDraw);
   mTestPhong->mNBO.bind();
-  mTestPhong->mNBO.allocate(mTestPhong->normals().data(),
+  mTestPhong->mNBO.allocate(mTestPhong->getNormals().data(),
-                            mTestPhong->normals().size()
+                            mTestPhong->getNormals().size()
-                            * sizeof(mTestPhong->normals()[0]));
+                            * sizeof(mTestPhong->getNormals()[0]));
   mTestPhong->mProgram.enableAttributeArray(1);
-  mTestPhong->mProgram.setAttributeBuffer(1, GL_FLOAT, 0,
+  mTestPhong->mProgram.setAttributeBuffer(1, GL_FLOAT, 0, 3, sizeof(QVector3D));
-                                          3, sizeof(QVector3D));
   mTestPhong->mNBO.release();
   mTestPhong->mVAO.release();
   mTestPhong->mProgram.release();
@@ -89,9 +88,9 @@ void Scene::init()
   mTestAmbient->mNBO.create();
   mTestAmbient->mNBO.setUsagePattern(QOpenGLBuffer::StaticDraw);
   mTestAmbient->mNBO.bind();
-  mTestAmbient->mNBO.allocate(mTestAmbient->normals().data(),
+  mTestAmbient->mNBO.allocate(mTestAmbient->getNormals().data(),
-                              mTestAmbient->normals().size()
+                              mTestAmbient->getNormals().size()
-                              * sizeof(mTestAmbient->normals()[0]));
+                              * sizeof(mTestAmbient->getNormals()[0]));
   mTestAmbient->mProgram.enableAttributeArray(1);
   mTestAmbient->mProgram.setAttributeBuffer(1, GL_FLOAT, 0,
                                             3, sizeof(QVector3D));
@@ -113,9 +112,9 @@ void Scene::init()
   mTestDiffuse->mNBO.create();
   mTestDiffuse->mNBO.setUsagePattern(QOpenGLBuffer::StaticDraw);
   mTestDiffuse->mNBO.bind();
-  mTestDiffuse->mNBO.allocate(mTestDiffuse->normals().data(),
+  mTestDiffuse->mNBO.allocate(mTestDiffuse->getNormals().data(),
-                              mTestDiffuse->normals().size()
+                              mTestDiffuse->getNormals().size()
-                              * sizeof(mTestDiffuse->normals()[0]));
+                              * sizeof(mTestDiffuse->getNormals()[0]));
   mTestDiffuse->mProgram.enableAttributeArray(1);
   mTestDiffuse->mProgram.setAttributeBuffer(1, GL_FLOAT, 0,
                                             3, sizeof(QVector3D));
@@ -139,9 +138,9 @@ void Scene::init()
   mTestSpecular->mNBO.create();
   mTestSpecular->mNBO.setUsagePattern(QOpenGLBuffer::StaticDraw);
   mTestSpecular->mNBO.bind();
-  mTestSpecular->mNBO.allocate(mTestSpecular->normals().data(),
+  mTestSpecular->mNBO.allocate(mTestSpecular->getNormals().data(),
-                               mTestSpecular->normals().size()
+                               mTestSpecular->getNormals().size()
-                               * sizeof(mTestSpecular->normals()[0]));
+                               * sizeof(mTestSpecular->getNormals()[0]));
   mTestSpecular->mProgram.enableAttributeArray(1);
   mTestSpecular->mProgram.setAttributeBuffer(1, GL_FLOAT, 0,
                                              3, sizeof(QVector3D));
@@ -241,8 +240,7 @@ void Scene::init()
 
 
   // Test basic cube with phong.vert and phong.frag shaders
-  mMeshes.push_back(
+  mMeshes.push_back(new MeshRenderer("testLight", Triangle(QTK_DRAW_ELEMENTS)));
-      new MeshRenderer("testLight", Triangle(QTK_DRAW_ELEMENTS)));
   mMeshes.back()->mTransform.setTranslation(5.0f, 1.25f, 10.0f);
   mMeshes.back()->mTransform.scale(0.25f);
   mMeshes.back()->setDrawType(GL_LINE_LOOP);
@@ -250,8 +248,7 @@ void Scene::init()
   mMeshes.back()->setColor(GREEN);
   mMeshes.back()->init();
 
-  mMeshes.push_back(
+  mMeshes.push_back(new MeshRenderer("testPhong", Cube(QTK_DRAW_ARRAYS)));
-      new MeshRenderer("testPhong", Cube(QTK_DRAW_ARRAYS)));
   mMeshes.back()->mTransform.setTranslation(5.0f, 0.0f, 10.0f);
   mMeshes.back()->setShaders(":/phong.vert", ":/phong.frag");
   mMeshes.back()->setColor(QVector3D(0.0f, 0.25f, 0.0f));
@@ -262,9 +259,9 @@ void Scene::init()
   mMeshes.back()->mNBO.create();
   mMeshes.back()->mNBO.bind();
 
-  mMeshes.back()->mNBO.allocate(mMeshes.back()->normals().data(),
+  mMeshes.back()->mNBO.allocate(mMeshes.back()->getNormals().data(),
-                                mMeshes.back()->normals().size()
+                                mMeshes.back()->getNormals().size()
-                                * sizeof(mMeshes.back()->normals()[0]));
+                                * sizeof(mMeshes.back()->getNormals()[0]));
   mMeshes.back()->mProgram.enableAttributeArray(1);
   mMeshes.back()->mProgram.setAttributeBuffer(1, GL_FLOAT, 0,
                                               3, sizeof(QVector3D));
@@ -294,8 +291,7 @@ void Scene::init()
       new MeshRenderer("rightTriangle", Triangle(QTK_DRAW_ELEMENTS)));
   mMeshes.back()->mTransform.setTranslation(-5.0f, 0.0f, -2.0f);
 
-  mMeshes.push_back(
+  mMeshes.push_back(new MeshRenderer("centerCube", Cube(QTK_DRAW_ELEMENTS)));
-      new MeshRenderer("centerCube", Cube(QTK_DRAW_ELEMENTS)));
   mMeshes.back()->mTransform.setTranslation(-7.0f, 0.0f, -2.0f);
 
   mMeshes.push_back(
@@ -331,9 +327,9 @@ void Scene::init()
   mMeshes.back()->mNBO.bind();
   mMeshes.back()->mProgram.bind();
 
-  mMeshes.back()->mNBO.allocate(mMeshes.back()->normals().data(),
+  mMeshes.back()->mNBO.allocate(mMeshes.back()->getNormals().data(),
-                                mMeshes.back()->normals().size()
+                                mMeshes.back()->getNormals().size()
-                                * sizeof(mMeshes.back()->normals()[0]));
+                                * sizeof(mMeshes.back()->getNormals()[0]));
   mMeshes.back()->mProgram.enableAttributeArray(1);
   mMeshes.back()->mProgram.setAttributeBuffer(1, GL_FLOAT, 0,
                                               3, sizeof(QVector3D));
@@ -344,7 +340,8 @@ void Scene::init()
 
   // RGB Normals cube to show normals are correct with QTK_DRAW_ELEMENTS_NORMALS
   mMeshes.push_back(
-      new MeshRenderer("rgbNormalsCubeElementsTest", Cube(QTK_DRAW_ELEMENTS_NORMALS)));
+      new MeshRenderer("rgbNormalsCubeElementsTest",
+                       Cube(QTK_DRAW_ELEMENTS_NORMALS)));
   mMeshes.back()->mTransform.setTranslation(5.0f, 0.0f, 2.0f);
   mMeshes.back()->setShaders(":/rgb-normals.vert", ":/rgb-normals.frag");
   mMeshes.back()->init();
@@ -353,9 +350,9 @@ void Scene::init()
   mMeshes.back()->mNBO.bind();
   mMeshes.back()->mProgram.bind();
 
-  mMeshes.back()->mNBO.allocate(mMeshes.back()->normals().data(),
+  mMeshes.back()->mNBO.allocate(mMeshes.back()->getNormals().data(),
-                                mMeshes.back()->normals().size()
+                                mMeshes.back()->getNormals().size()
-                                * sizeof(mMeshes.back()->normals()[0]));
+                                * sizeof(mMeshes.back()->getNormals()[0]));
   mMeshes.back()->mProgram.enableAttributeArray(1);
   mMeshes.back()->mProgram.setAttributeBuffer(1, GL_FLOAT, 0,
                                               3, sizeof(QVector3D));
@@ -375,19 +372,16 @@ void Scene::init()
   mMeshes.back()->init();
   mMeshes.back()->mProgram.bind();
 
-  mMeshes.back()->setTexture(Texture::initTexture2D(":/crate.png"));
+  mMeshes.back()->setTexture(OpenGLTextureFactory::initTexture2D(":/crate.png"));
   mMeshes.back()->setUniform("uTexture", 0);
-  mMeshes.back()->texture().bind();
-
-  mMeshes.back()->texture().release();
 
   mMeshes.back()->mVAO.bind();
   mMeshes.back()->mNBO.destroy();
   mMeshes.back()->mNBO.create();
   mMeshes.back()->mNBO.bind();
-  mMeshes.back()->mNBO.allocate(mMeshes.back()->mShape.texCoords().data(),
+  mMeshes.back()->mNBO.allocate(mMeshes.back()->mShape.getTexCoords().data(),
-                                mMeshes.back()->mShape.texCoords().size()
+                                mMeshes.back()->mShape.getTexCoords().size()
-                                * sizeof(mMeshes.back()->mShape.texCoords()[0]));
+                                * sizeof(mMeshes.back()->mShape.getTexCoords()[0]));
   mMeshes.back()->mProgram.enableAttributeArray(1);
   mMeshes.back()->mProgram.setAttributeBuffer(1, GL_FLOAT, 0,
                                               2, sizeof(QVector2D));
@@ -406,13 +400,13 @@ void Scene::init()
   mMeshes.back()->mNBO.bind();
   mMeshes.back()->mProgram.bind();
 
-  mMeshes.back()->mNBO.allocate(mMeshes.back()->texCoords().data(),
+  mMeshes.back()->mNBO.allocate(mMeshes.back()->getTexCoords().data(),
-                                mMeshes.back()->texCoords().size()
+                                mMeshes.back()->getTexCoords().size()
-                                * sizeof(mMeshes.back()->texCoords()[0]));
+                                * sizeof(mMeshes.back()->getTexCoords()[0]));
   mMeshes.back()->mProgram.enableAttributeArray(1);
   mMeshes.back()->mProgram.setAttributeBuffer(1, GL_FLOAT, 0,
                                               3, sizeof(QVector3D));
-  mMeshes.back()->setTexture(Texture::initTexture2D(":/crate.png"));
+  mMeshes.back()->setTexture(OpenGLTextureFactory::initTexture2D(":/crate.png"));
 
   mMeshes.back()->mProgram.setUniformValue("uTexture", 0);
   mMeshes.back()->mProgram.release();
@@ -430,16 +424,16 @@ void Scene::init()
   mMeshes.back()->init();
   mMeshes.back()->mProgram.bind();
 
-  mMeshes.back()->setTexture(Texture::initCubeMap(":/crate.png"));
+  mMeshes.back()->setTexture(OpenGLTextureFactory::initCubeMap(":/crate.png"));
   mMeshes.back()->setUniform("uTexture", 0);
 
   mMeshes.back()->mVAO.bind();
   mMeshes.back()->mNBO.destroy();
   mMeshes.back()->mNBO.create();
   mMeshes.back()->mNBO.bind();
-  mMeshes.back()->mNBO.allocate(mMeshes.back()->mShape.texCoords().data(),
+  mMeshes.back()->mNBO.allocate(mMeshes.back()->mShape.getTexCoords().data(),
-                                mMeshes.back()->mShape.texCoords().size()
+                                mMeshes.back()->mShape.getTexCoords().size()
-                                * sizeof(mMeshes.back()->mShape.texCoords()[0]));
+                                * sizeof(mMeshes.back()->mShape.getTexCoords()[0]));
   mMeshes.back()->mProgram.enableAttributeArray(1);
   mMeshes.back()->mProgram.setAttributeBuffer(1, GL_FLOAT, 0,
                                               2, sizeof(QVector2D));
@@ -449,8 +443,7 @@ void Scene::init()
 
   // Create a cube with custom shaders
   // + Apply RGB normals shader and spin the cube for a neat effect
-  mMeshes.push_back(
+  mMeshes.push_back(new MeshRenderer("rgbNormalsCube", Cube(QTK_DRAW_ARRAYS)));
-      new MeshRenderer("rgbNormalsCube", Cube(QTK_DRAW_ARRAYS)));
   mMeshes.back()->mTransform.setTranslation(5.0f, 2.0f, -2.0f);
   mMeshes.back()->setShaders(":/rgb-normals.vert", ":/rgb-normals.frag");
   mMeshes.back()->init();
@@ -459,9 +452,9 @@ void Scene::init()
   mMeshes.back()->mNBO.bind();
   mMeshes.back()->mProgram.bind();
 
-  mMeshes.back()->mNBO.allocate(mMeshes.back()->normals().data(),
+  mMeshes.back()->mNBO.allocate(mMeshes.back()->getNormals().data(),
-                                mMeshes.back()->normals().size()
+                                mMeshes.back()->getNormals().size()
-                                * sizeof(mMeshes.back()->normals()[0]));
+                                * sizeof(mMeshes.back()->getNormals()[0]));
   mMeshes.back()->mProgram.enableAttributeArray(1);
   mMeshes.back()->mProgram.setAttributeBuffer(1, GL_FLOAT, 0,
                                               3, sizeof(QVector3D));
@@ -481,9 +474,9 @@ void Scene::init()
   mMeshes.back()->mVAO.bind();
   mMeshes.back()->mNBO.create();
   mMeshes.back()->mNBO.bind();
-  mMeshes.back()->mNBO.allocate(mMeshes.back()->normals().data(),
+  mMeshes.back()->mNBO.allocate(mMeshes.back()->getNormals().data(),
-                                mMeshes.back()->normals().size()
+                                mMeshes.back()->getNormals().size()
-                                * sizeof(mMeshes.back()->normals()[0]));
+                                * sizeof(mMeshes.back()->getNormals()[0]));
   mMeshes.back()->mProgram.enableAttributeArray(1);
   mMeshes.back()->mProgram.setAttributeBuffer(1, GL_FLOAT, 0,
                                               3, sizeof(QVector3D));
@@ -503,9 +496,9 @@ void Scene::init()
   mMeshes.back()->mVAO.bind();
   mMeshes.back()->mNBO.create();
   mMeshes.back()->mNBO.bind();
-  mMeshes.back()->mNBO.allocate(mMeshes.back()->normals().data(),
+  mMeshes.back()->mNBO.allocate(mMeshes.back()->getNormals().data(),
-                                mMeshes.back()->normals().size()
+                                mMeshes.back()->getNormals().size()
-                                * sizeof(mMeshes.back()->normals()[0]));
+                                * sizeof(mMeshes.back()->getNormals()[0]));
   mMeshes.back()->mProgram.enableAttributeArray(1);
   mMeshes.back()->mProgram.setAttributeBuffer(1, GL_FLOAT, 0,
                                               3, sizeof(QVector3D));
@@ -521,19 +514,19 @@ void Scene::init()
   mMeshes.back()->init();
   mMeshes.back()->mProgram.bind();
 
-  mMeshes.back()->setTexture(Texture::initTexture2D(":/crate.png"));
+  mMeshes.back()->setTexture(":/crate.png");
-  mMeshes.back()->setUniform("uTexture", 0);
-  mMeshes.back()->texture().bind();
 
-  mMeshes.back()->texture().release();
+  // Test assignment of Texture
+  mMeshes.back()->mTexture = Texture(":/crate.png");
+  mMeshes.back()->setUniform("uTexture", 0);
 
   mMeshes.back()->mVAO.bind();
   mMeshes.back()->mNBO.destroy();
   mMeshes.back()->mNBO.create();
   mMeshes.back()->mNBO.bind();
-  mMeshes.back()->mNBO.allocate(mMeshes.back()->mShape.texCoords().data(),
+  mMeshes.back()->mNBO.allocate(mMeshes.back()->mShape.getTexCoords().data(),
-                                mMeshes.back()->mShape.texCoords().size()
+                                mMeshes.back()->mShape.getTexCoords().size()
-                                * sizeof(mMeshes.back()->mShape.texCoords()[0]));
+                                * sizeof(mMeshes.back()->mShape.getTexCoords()[0]));
   mMeshes.back()->mProgram.enableAttributeArray(1);
   mMeshes.back()->mProgram.setAttributeBuffer(1, GL_FLOAT, 0,
                                               2, sizeof(QVector2D));
@@ -550,19 +543,17 @@ void Scene::init()
   mMeshes.back()->init();
   mMeshes.back()->mProgram.bind();
 
-  mMeshes.back()->setTexture(Texture::initTexture2D(":/crate.png"));
+//  mMeshes.back()->setTexture(OpenGLTextureFactory::initTexture2D(":/crate.png"));
+  mMeshes.back()->mTexture.setTexture(":/crate.png");
   mMeshes.back()->setUniform("uTexture", 0);
-  mMeshes.back()->texture().bind();
-
-  mMeshes.back()->texture().release();
 
   mMeshes.back()->mVAO.bind();
   mMeshes.back()->mNBO.destroy();
   mMeshes.back()->mNBO.create();
   mMeshes.back()->mNBO.bind();
-  mMeshes.back()->mNBO.allocate(mMeshes.back()->mShape.texCoords().data(),
+  mMeshes.back()->mNBO.allocate(mMeshes.back()->mShape.getTexCoords().data(),
-                                mMeshes.back()->mShape.texCoords().size()
+                                mMeshes.back()->mShape.getTexCoords().size()
-                                * sizeof(mMeshes.back()->mShape.texCoords()[0]));
+                                * sizeof(mMeshes.back()->mShape.getTexCoords()[0]));
   mMeshes.back()->mProgram.enableAttributeArray(1);
   mMeshes.back()->mProgram.setAttributeBuffer(1, GL_FLOAT, 0,
                                               2, sizeof(QVector2D));
@@ -605,7 +596,7 @@ void Scene::draw()
 {
   mSkybox.draw();
 
-  for (auto & model : mModels) model->draw();
+  for (const auto & model : mModels) model->draw();
 
   for (const auto & mesh : mMeshes) mesh->draw();
 
@@ -614,15 +605,15 @@ void Scene::draw()
                          mTestPhong->mTransform.toMatrix().normalMatrix());
   mTestPhong->setUniform(
       "uLightPosition",
-      MeshRenderer::getInstance("phongLight")->mTransform.translation());
+      MeshRenderer::getInstance("phongLight")->mTransform.getTranslation());
   mTestPhong->setUniform("uCameraPosition",
-                         Scene::Camera().transform().translation());
+                         Scene::Camera().transform().getTranslation());
   mTestPhong->mProgram.release();
   mTestPhong->draw();
 
   mTestAmbient->mProgram.bind();
   mTestAmbient->setUniform("uCameraPosition",
-                           Scene::Camera().transform().translation());
+                           Scene::Camera().transform().getTranslation());
   mTestAmbient->mProgram.release();
   mTestAmbient->draw();
 
@@ -631,8 +622,9 @@ void Scene::draw()
                            mTestDiffuse->mTransform.toMatrix().normalMatrix());
   mTestDiffuse->setUniform(
       "uLightPosition",
-      MeshRenderer::getInstance("diffuseLight")->mTransform.translation());
+      MeshRenderer::getInstance("diffuseLight")->mTransform.getTranslation());
-  mTestDiffuse->setUniform("uCameraPosition", Scene::Camera().transform().translation());
+  mTestDiffuse->setUniform("uCameraPosition",
+                           Scene::Camera().transform().getTranslation());
   mTestDiffuse->mProgram.release();
   mTestDiffuse->draw();
 
@@ -642,19 +634,21 @@ void Scene::draw()
       mTestSpecular->mTransform.toMatrix().normalMatrix());
   mTestSpecular->setUniform(
       "uLightPosition",
-      MeshRenderer::getInstance("specularLight")->mTransform.translation());
+      MeshRenderer::getInstance("specularLight")->mTransform.getTranslation());
-  mTestSpecular->setUniform("uCameraPosition", Scene::Camera().transform().translation());
+  mTestSpecular->setUniform("uCameraPosition",
+                            Scene::Camera().transform().getTranslation());
   mTestSpecular->mProgram.release();
   mTestSpecular->draw();
 }
 
 void Scene::update()
 {
-  auto position = MeshRenderer::getInstance("alienTestLight")->mTransform.translation();
+  auto position = MeshRenderer::getInstance(
+      "alienTestLight")->mTransform.getTranslation();
   Model::getInstance("alienTest")->setUniform(
       "uLight.position", position);
   Model::getInstance("alienTest")->setUniform(
-      "uCameraPosition", Scene::Camera().transform().translation());
+      "uCameraPosition", Scene::Camera().transform().getTranslation());
   auto posMatrix = Model::getInstance("alienTest")->mTransform.toMatrix();
   Model::getInstance("alienTest")->setUniform(
       "uMVP.normalMatrix", posMatrix.normalMatrix());
@@ -666,11 +660,12 @@ void Scene::update()
       "uMVP.projection", Scene::Projection());
   Model::getInstance("alienTest")->mTransform.rotate(0.75f, 0.0f, 1.0f, 0.0f);
 
-  position = MeshRenderer::getInstance("spartanTestLight")->mTransform.translation();
+  position = MeshRenderer::getInstance(
+      "spartanTestLight")->mTransform.getTranslation();
   Model::getInstance("spartanTest")->setUniform(
       "uLight.position", position);
   Model::getInstance("spartanTest")->setUniform(
-      "uCameraPosition", Scene::Camera().transform().translation());
+      "uCameraPosition", Scene::Camera().transform().getTranslation());
   posMatrix = Model::getInstance("spartanTest")->mTransform.toMatrix();
   Model::getInstance("spartanTest")->setUniform(
       "uMVP.normalMatrix", posMatrix.normalMatrix());
@@ -687,11 +682,11 @@ void Scene::update()
   MeshRenderer::getInstance("testPhong")->mTransform.rotate(
       0.75f, 1.0f, 0.5f, 0.0f);
   MeshRenderer::getInstance("testPhong")->mProgram.bind();
-  position = MeshRenderer::getInstance("testLight")->mTransform.translation();
+  position = MeshRenderer::getInstance("testLight")->mTransform.getTranslation();
   MeshRenderer::getInstance("testPhong")->setUniform(
       "uLight.position", position);
   MeshRenderer::getInstance("testPhong")->setUniform(
-      "uCameraPosition", Scene::Camera().transform().translation());
+      "uCameraPosition", Scene::Camera().transform().getTranslation());
   posMatrix = MeshRenderer::getInstance("testPhong")->mTransform.toMatrix();
   MeshRenderer::getInstance("testPhong")->setUniform(
       "uMVP.normalMatrix", posMatrix.normalMatrix());
@@ -727,7 +722,7 @@ void Scene::update()
   static float translateX = 0.025f;
   float limit = -9.0f; // Origin position.x - 2.0f
   float posX =
-      MeshRenderer::getInstance("topTriangle")->mTransform.translation().x();
+      MeshRenderer::getInstance("topTriangle")->mTransform.getTranslation().x();
   if (posX < limit || posX > limit + 4.0f) {
     translateX = -translateX;
   }

src/skybox.cpp

@@ -15,13 +15,13 @@
 Skybox::Skybox(std::string right, std::string top, std::string front,
                std::string left, std::string bottom, std::string back,
                const std::string & name)
-    : mCubeMap(Texture::initCubeMap(
+    : mCubeMap(OpenGLTextureFactory::initCubeMap(
     QImage(right.c_str()).mirrored(), QImage(top.c_str()),
     QImage(front.c_str()), QImage(left.c_str()),
     QImage(bottom.c_str()), QImage(back.c_str()))),
       mVBO(QOpenGLBuffer::VertexBuffer),
-      mVertices(Cube(QTK_DRAW_ELEMENTS).vertices()),
+      mVertices(Cube(QTK_DRAW_ELEMENTS).getVertices()),
-      mIndices(Cube(QTK_DRAW_ELEMENTS).indices())
+      mIndices(Cube(QTK_DRAW_ELEMENTS).getIndexData())
 { init();}
 
 Skybox::Skybox(std::string name)

src/texture.cpp

@@ -11,8 +11,9 @@
 
 #include <texture.h>
 
+std::unordered_map<std::string, Texture *> TM::textureMap;
 
-QImage * Texture::initImage(const char * image, bool flipX, bool flipY)
+QImage * OpenGLTextureFactory::initImage(const char * image, bool flipX, bool flipY)
 {
   // Qt6 limits loaded images to 256MB by default
   QImageReader::setAllocationLimit(512);
@@ -26,7 +27,7 @@ QImage * Texture::initImage(const char * image, bool flipX, bool flipY)
   return loadedImage;
 }
 
-QOpenGLTexture * Texture::initTexture2D(const char * texture,
+QOpenGLTexture * OpenGLTextureFactory::initTexture2D(const char * texture,
                                                      bool flipX, bool flipY)
 {
   QImage * image = initImage(texture, flipX, flipY);
@@ -39,14 +40,14 @@ QOpenGLTexture * Texture::initTexture2D(const char * texture,
   return newTexture;
 }
 
-QOpenGLTexture * Texture::initCubeMap(const char * tile)
+QOpenGLTexture * OpenGLTextureFactory::initCubeMap(const char * tile)
 {
   return initCubeMap(QImage(tile), QImage(tile),
                      QImage(tile), QImage(tile),
                      QImage(tile), QImage(tile));
 }
 
-QOpenGLTexture * Texture::initCubeMap(
+QOpenGLTexture * OpenGLTextureFactory::initCubeMap(
     const char * right, const char * top,
     const char * front, const char * left,
     const char * bottom, const char * back)
@@ -56,7 +57,7 @@ QOpenGLTexture * Texture::initCubeMap(
                      QImage(bottom), QImage(back));
 }
 
-QOpenGLTexture * Texture::initCubeMap(
+QOpenGLTexture * OpenGLTextureFactory::initCubeMap(
     QImage right, QImage top,
     QImage front, QImage left,
     QImage bottom, QImage back)

src/texture.h

@@ -11,10 +11,9 @@
 
 #include <QOpenGLTexture>
 
-
+class OpenGLTextureFactory {
-class Texture {
 public:
-  ~Texture() {}
+  ~OpenGLTextureFactory() = default;
 
   // QImage
   static QImage * initImage(const char * image,
@@ -36,7 +35,57 @@ public:
 
 private:
   // Private ctor to prevent creating instances of this class
-  Texture() {}
+  OpenGLTextureFactory() = default;
+};
+
+class Texture;
+typedef class TextureManager {
+public:
+  static inline Texture * getInstance(std::string path)
+  { return textureMap.count(path) > 0 ? textureMap[path] : Q_NULLPTR; }
+  static inline bool addInstance(const std::string & path, Texture * texture)
+  { return textureMap.emplace(path, texture).second ? true : false; }
+
+private:
+  static std::unordered_map<std::string, Texture *> textureMap;
+} TM;
+
+class Texture {
+//  explicit Texture(QOpenGLTexture * texture) : mOpenGLTexture(texture) { }
+public:
+  Texture() = default;
+  Texture(const Texture & value) {
+    if (TM::getInstance(value.mPath)) {
+      mOpenGLTexture = TM::getInstance(value.mPath)->mOpenGLTexture;
+    }
+    else {
+      mOpenGLTexture = OpenGLTextureFactory::initTexture2D(value.mPath);
+      TM::addInstance(value.mPath, this);
+    }
+    mPath = value.mPath;
+  }
+  explicit Texture(const char * path, bool flipX=false, bool flipY=false) :
+    mPath(path) {
+    if (TM::getInstance(path) == Q_NULLPTR) {
+      mOpenGLTexture = OpenGLTextureFactory::initTexture2D(path, flipX, flipY);
+      TM::addInstance(path, this);
+    }
+    else {
+      mOpenGLTexture = TM::getInstance(path)->mOpenGLTexture;
+    }
+  }
+//  ~Texture() { if (mOpenGLTexture) mOpenGLTexture->destroy();}
+
+  inline QOpenGLTexture & getOpenGLTexture() { return *mOpenGLTexture;}
+
+  inline void setTexture(const char * path, bool flipX=false, bool flipY=false)
+  { mOpenGLTexture = OpenGLTextureFactory::initTexture2D(path, flipX, flipY);}
+  inline void setTexture(QOpenGLTexture * texture) { mOpenGLTexture = texture;}
+
+  inline bool hasTexture() const { return mOpenGLTexture != Q_NULLPTR;}
+
+  QOpenGLTexture * mOpenGLTexture = Q_NULLPTR;
+  const char * mPath;
 };
 
 #endif // QTOPENGL_TEXTURE_H

src/transform3D.cpp

@@ -90,17 +90,17 @@ const QMatrix4x4 & Transform3D::toMatrix()
  * Queries
  ******************************************************************************/
 
-QVector3D Transform3D::forward() const
+QVector3D Transform3D::getForward() const
 {
   return mRotation.rotatedVector(LocalForward);
 }
 
-QVector3D Transform3D::up() const
+QVector3D Transform3D::getUp() const
 {
   return mRotation.rotatedVector(LocalUp);
 }
 
-QVector3D Transform3D::right() const
+QVector3D Transform3D::getRight() const
 {
   return mRotation.rotatedVector(LocalRight);
 }
@@ -113,16 +113,16 @@ QVector3D Transform3D::right() const
 QDebug operator<<(QDebug dbg, const Transform3D & transform)
 {
   dbg << "Transform3D\n{\n";
-  dbg << "Position: <" << transform.translation().x() << ", "
+  dbg << "Position: <" << transform.getTranslation().x() << ", "
-      << transform.translation().y() << ", "
+      << transform.getTranslation().y() << ", "
-      << transform.translation().z() << ">\n";
+      << transform.getTranslation().z() << ">\n";
-  dbg << "Scale: <" << transform.scale().x() << ", "
+  dbg << "Scale: <" << transform.getScale().x() << ", "
-      << transform.scale().y() << ", "
+      << transform.getScale().y() << ", "
-      << transform.scale().z() << ">\n";
+      << transform.getScale().z() << ">\n";
-  dbg << "Rotation: <" << transform.rotation().x() << ", "
+  dbg << "Rotation: <" << transform.getRotation().x() << ", "
-      << transform.rotation().y() << ", "
+      << transform.getRotation().y() << ", "
-      << transform.rotation().z() << " | " <<
+      << transform.getRotation().z() << " | " <<
-      transform.rotation().scalar() << ">\n}";
+      transform.getRotation().scalar() << ">\n}";
   return dbg;
 }
 

src/transform3D.h

@@ -77,14 +77,14 @@ public:
   //
   // Accessors
 
-  inline const QVector3D & translation() const { return mTranslation;}
+  inline const QVector3D & getTranslation() const { return mTranslation;}
-  inline const QVector3D & scale() const { return mScale; }
+  inline const QVector3D & getScale() const { return mScale; }
-  inline const QQuaternion & rotation() const { return mRotation; }
+  inline const QQuaternion & getRotation() const { return mRotation; }
   const QMatrix4x4 & toMatrix();
 
-  QVector3D forward() const;
+  QVector3D getForward() const;
-  QVector3D up() const;
+  QVector3D getUp() const;
-  QVector3D right() const;
+  QVector3D getRight() const;
 
   static const QVector3D LocalForward, LocalUp, LocalRight;