Ex02: Minor improvements

example02_threaded_raytracing/src/rayTracer.cpp

@@ -192,31 +192,28 @@ std::optional<glm::vec3> Ex02::RT::Internal::rayToSphere(glm::vec3 rayPos,
                                                          glm::vec3 spherePos,
                                                          float sphereRadius) {
   // check if there is collision
-  glm::vec3 tempVec = rayPos - spherePos;
+  const glm::vec3 tempVec = rayPos - spherePos;
   float temp = rayDirUnit.x * tempVec.x + rayDirUnit.y * tempVec.y +
                rayDirUnit.z * tempVec.z;
-  float delta = temp * temp;
+  float delta =
-  temp = tempVec.x * tempVec.x + tempVec.y * tempVec.y + tempVec.z * tempVec.z -
+      temp * temp - (tempVec.x * tempVec.x + tempVec.y * tempVec.y +
-         sphereRadius * sphereRadius;
+                     tempVec.z * tempVec.z - sphereRadius * sphereRadius);
-  delta -= temp;
-
   if (delta < 0.0F) {
     return {};
   } else {
     temp = rayDirUnit.x * tempVec.x + rayDirUnit.y * tempVec.y +
            rayDirUnit.z * tempVec.z;
-    float dist = -temp - std::sqrt(delta);
+    const float temp2 = std::sqrt(delta);
-    float dist2 = -temp + std::sqrt(delta);
+    const float dist = -temp - temp2;
-    float min = dist > dist2 ? dist2 : dist;
+    const float dist2 = -temp + temp2;
-    float max = dist > dist2 ? dist : dist2;
+    if (dist < 0.0F) {
-    if (min < 0.0F) {
+      if (dist2 < 0.0F) {
-      if (max < 0.0F) {
         return {};
       } else {
-        return {rayPos + rayDirUnit * max};
+        return {rayPos + rayDirUnit * dist2};
       }
     } else {
-      return {rayPos + rayDirUnit * min};
+      return {rayPos + rayDirUnit * dist};
     }
   }
 }