Changeset 95595de in opengl-game

Timestamp:

Jul 2, 2018, 2:48:01 AM (6 years ago)

Author:

Dmitry Portnoy <dmp1488@…>

Branches:

feature/imgui-sdl, master, points-test

Children:

ebaa95c

Parents:

58088c0

Message:

Update each object's bounding center as the object moves, and remove asteroids from the scene once they leave the visible area.

File:

• new-game.cpp (modified) (12 diffs)

new-game.cpp

@@ -36 +36 @@
 struct SceneObject {
    unsigned int id;
+
+   // Currently, model_transform should only have translate, and rotation and scale need to be done in model_base since
+   // they need to be done when the object is at the origin. I should change this to have separate scale, rotate, and translate
+   // matrices for each object that can be updated independently and then applied to the object in that order.
    mat4 model_mat, model_base, model_transform;
+   mat4 translate_mat; // beginning of doing what's mentioned above
    GLuint shader_program;
    unsigned int num_points;
@@ -95 +100 @@
                   GLuint ubo,
                   GLuint model_mat_idx_vbo);
-void removeObjectFromScene(int objectId, GLuint ubo);
+void removeObjectFromScene(SceneObject& obj, GLuint ubo);
 
 void calculateObjectBoundingBox(SceneObject& obj);
@@ -186 +191 @@
 float FAR_CLIP = 100.0f;
 
-// Should really have some array or struct of UI-related variables
+// TODO: Should really have some array or struct of UI-related variables
 bool isRunning = true;
 
@@ -795 +800 @@
    obj.model_base = T_model * R_model * scale(mat4(1.0f), vec3(0.1f, 0.1f, 0.1f));
 
+   obj.translate_mat = T_model;
+
    addObjectToSceneDuringInit(obj);
 
@@ -951 +958 @@
 
          frame_count++;
-      }
-
-      elapsed_seconds_spawn += elapsed_seconds;
-      if (elapsed_seconds_spawn > 0.5f) {
-         spawnAsteroid(vec3(getRandomNum(-1.3f, 1.3f), getRandomNum(-3.0f, -1.0f), getRandomNum(-5.5f, -4.5f)), color_sp,
-            shaderBufferInfo,
-            points_vbo,
-            colors_vbo,
-            selected_colors_vbo,
-            texcoords_vbo,
-            normals_vbo,
-            ubo,
-            model_mat_idx_vbo);
-
-         elapsed_seconds_spawn = 0.0f;
       }
 
@@ -996 +988 @@
 
       if (curState == STATE_GAME) {
+
+         elapsed_seconds_spawn += elapsed_seconds;
+         if (elapsed_seconds_spawn > 0.5f) {
+            spawnAsteroid(vec3(getRandomNum(-1.3f, 1.3f), getRandomNum(-3.0f, -1.0f), getRandomNum(-5.5f, -4.5f)), color_sp,
+               shaderBufferInfo,
+               points_vbo,
+               colors_vbo,
+               selected_colors_vbo,
+               texcoords_vbo,
+               normals_vbo,
+               ubo,
+               model_mat_idx_vbo);
+
+            elapsed_seconds_spawn -= 0.5f;
+         }
+
          /*
          if (clickedObject == &objects[0]) {
@@ -1027 +1035 @@
             if (!objects[i].deleted) {
                transformObject(objects[i], translate(mat4(1.0f), vec3(0.0f, 0.0f, 0.04f)), ubo);
+
+               // TODO: Should really compare against the near clipping plane, but also account for
+               // the camera's translation and rotation
+               if (objects[i].bounding_center.z - objects[i].bounding_radius > 3.0f) {
+                  removeObjectFromScene(objects[i], ubo);
+               }
             }
          }
 
          if (key_state[GLFW_KEY_SPACE] == GLFW_PRESS) {
-            removeObjectFromScene(0, ubo);
+            removeObjectFromScene(objects[0], ubo);
          }
       }
@@ -1357 +1371 @@
 
    calculateObjectBoundingBox(obj);
+
+   obj.bounding_center = vec3(obj.translate_mat * vec4(obj.bounding_center, 1.0f));
 
    objects.push_back(obj);
@@ -1398 +1414 @@
 }
 
-void removeObjectFromScene(int objectId, GLuint ubo) {
-   SceneObject& obj = objects[objectId];
-
+void removeObjectFromScene(SceneObject& obj, GLuint ubo) {
    if (!obj.deleted) {
       // Move the object outside the render bounds of the scene so it doesn't get rendered
@@ -1447 +1461 @@
    GLfloat radius_z = max_z - obj.bounding_center.z;
 
+   // This actually underestimates the radius. Might need to be fixed at some point.
    obj.bounding_radius = radius_x;
    if (obj.bounding_radius < radius_y)
@@ -1655 +1670 @@
    obj.model_mat = obj.model_transform * obj.model_base;
 
+   obj.bounding_center = vec3(transform * vec4(obj.bounding_center, 1.0f));
+
    glBindBuffer(GL_UNIFORM_BUFFER, ubo);
    glBufferSubData(GL_UNIFORM_BUFFER, obj.ubo_offset * sizeof(mat4), sizeof(mat4), value_ptr(obj.model_mat));
@@ -1896 +1913 @@
    obj.model_base = T * R * scale(mat4(1.0f), vec3(0.1f, 0.1f, 0.1f));
 
+   obj.translate_mat = T;
+
    addObjectToScene(obj, shaderBufferInfo,
                   points_vbo,