Fix results

frontend/js/fizika.js

@@ -1,7 +1,6 @@
 var isSearch = false;
 var totalPoints = 0;
-var currentPoints = 0;
+var loadedQuestions = [];
-var correctAnswersGiven = 0;
 var timer = 0;
 var startTimer = 0;
 var reviewMode = 0;
@@ -17,7 +16,7 @@ function aspect() {
 async function ajaxLoad(type) {
   reviewMode = 0;
   totalPoints = 0;
-  currentPoints = 0;
+  loadedQuestions = [];
   $("#state").html("");
   $("#state2").html("");
   $("#percentage").html("");
@@ -92,6 +91,8 @@ async function ajaxLoad(type) {
     ) {
       $("#megoldas").show();
       $("#state").html("Feladatok sikeresen letöltve!");
+      startTimer = 1;
+      timer = 0;
     }
   } catch (error) {
     $("#loadingGif").hide();
@@ -116,66 +117,73 @@ async function ajaxLoad(type) {
   }
 }
 
-function showCorrect(id, correctAns) {
+// Colour a single question green/red depending on whether the user selected
-  teszt(id, correctAns);
+// its correct answer, and report whether they got it right.
-  eval(
+function gradeQuestion(question) {
-    "$('" +
+  var div = "#feladat" + question.id;
-      "#label" +
+  var selected = "#form" + question.id + " #rad" + question.correct;
-      id +
+  var isCorrect = $(selected).is(":checked");
-      ".rad" +
+  $(div).animate({ backgroundColor: isCorrect ? "#C6FF8C" : "#FF808C" }, 1100);
-      correctAns +
+  return isCorrect;
-      "').css('background-color', '#C6FF8C');",
+}
+
+// Grade every question currently on screen and return the score as a string
+// percentage (e.g. "66.67"). Grading the whole known set in one pass replaces
+// the old per-question click handlers, which accumulated across loads and
+// could save bogus results.
+function scoreLoadedQuestions() {
+  var correctAnswersGiven = 0;
+  loadedQuestions.forEach(function (question) {
+    if (gradeQuestion(question)) correctAnswersGiven++;
+  });
+  var percentage = (correctAnswersGiven / totalPoints) * 100;
+  percentage = Math.round(percentage * 100) / 100;
+  return percentage.toFixed(2);
+}
+
+// "Kiértékelés": grade the answers and, unless we are already reviewing, save
+// the result once.
+function evaluate() {
+  if (!loadedQuestions.length) return;
+  var percentage = scoreLoadedQuestions();
+  $("#percentage").html("Eredmény: " + percentage + "%");
+  $("#state").html("Válaszok leellenőrizve!");
+
+  if (isLocal && !reviewMode) {
+    saveResult(percentage);
+    reviewMode = 1;
+  }
+}
+
+// "Helyes megoldások": reveal the correct answers. This is a review action, so
+// we enter review mode *before* grading to guarantee it can never overwrite a
+// previously saved result.
+function showCorrect() {
+  if (!loadedQuestions.length) return;
+  reviewMode = 1;
+  var percentage = scoreLoadedQuestions();
+  loadedQuestions.forEach(function (question) {
+    $("#label" + question.id + ".rad" + question.correct).css(
+      "background-color",
+      "#C6FF8C",
     );
+  });
+  $("#percentage").html("Eredmény: " + percentage + "%");
   $("#state").html("Helyes válaszok bejelölve!");
+  $("#state2").show();
   $("#state2").html(
     "(Ellenőrző mód, az itteni eredményeid nem kerülnek elmentésre, a módból való kilépéshez tölts be egy új tesztsort!)",
   );
 }
 
-function teszt(id, correctAns) {
+function saveResult(percentage) {
-  var div = "#feladat" + id;
-  var correct = "#form" + id + " #rad" + correctAns;
-  var isCorrect = $(correct).is(":checked");
-  currentPoints++;
-  if (isCorrect) {
-    $(div).animate({ backgroundColor: "#C6FF8C" }, 1100);
-    correctAnswersGiven++;
-  } else {
-    $(div).animate({ backgroundColor: "#FF808C" }, 1100);
-  }
-  if (currentPoints >= totalPoints) {
-    var percentage = (correctAnswersGiven / totalPoints) * 100;
-    percentage = Math.round(percentage * 100) / 100;
-    percentage = percentage.toFixed(2);
-    $("#percentage").html("Eredmény: " + percentage + "%");
-    $("#state").html("Válaszok leellenőrizve!");
-    if (isLocal && !reviewMode) {
   var datum = new Date();
   var ido = Math.round(timer / 60);
-      eval(
+  localStorage["teszt" + numberOfPreviousTests] = percentage;
-        "localStorage.teszt" +
+  localStorage["teszt" + numberOfPreviousTests + "date"] =
-          numberOfPreviousTests +
+    datum.toLocaleDateString();
-          " = '" +
+  localStorage["teszt" + numberOfPreviousTests + "time"] = ido;
-          percentage +
+  localStorage["teszt" + numberOfPreviousTests + "total"] = totalPoints;
-          "'",
-      );
-      eval(
-        "localStorage.teszt" +
-          numberOfPreviousTests +
-          "date = '" +
-          datum.toLocaleDateString() +
-          "'",
-      );
-      eval(
-        "localStorage.teszt" + numberOfPreviousTests + "time = '" + ido + "'",
-      );
-      eval(
-        "localStorage.teszt" +
-          numberOfPreviousTests +
-          "total = '" +
-          totalPoints +
-          "'",
-      );
   startTimer = 0;
   timer = 0;
   $("#state2").show();
@@ -183,11 +191,6 @@ function teszt(id, correctAns) {
     "Eredményed mentésre került! Ellenőrző módba belépve az eredményeid nem kerülnek tárolásra. A módból való kilépéshez tölts be egy új tesztsort!",
   );
   eredmeny();
-      reviewMode = 1;
-    }
-    currentPoints = 0;
-    correctAnswersGiven = 0;
-  }
 }
 
 function howMany() {
@@ -338,6 +341,14 @@ $(document).ready(function () {
   $("#load").click(function (event) {
     ajaxLoad(isSearch ? 1 : 3);
   });
+  $("#ans").click(function (event) {
+    event.preventDefault();
+    evaluate();
+  });
+  $("#cAns").click(function (event) {
+    event.preventDefault();
+    showCorrect();
+  });
   $(".scroll").click(function () {
     $("body").animate(
       {

frontend/js/load.js

@@ -72,13 +72,24 @@ const loadQuestions = async (
     );
   }
 
-  resultHtml = "";
-
   currentQuestions = currentQuestions.slice(0, questionCount);
+
+  // Remember the exact set being shown. The two action buttons ("Kiértékelés"
+  // and "Helyes megoldások") are bound once in fizika.js and grade this whole
+  // set in a single pass. Previously each question injected its own #ans/#cAns
+  // click handler; those were never removed, so they piled up across loads and
+  // ended up grading stale (already-removed) questions, saving bogus 0%
+  // results over the user's history.
+  loadedQuestions = currentQuestions.map(({ id, correct }) => ({
+    id: Number(id),
+    correct: Number(correct),
+  }));
+  totalPoints = loadedQuestions.length;
+
+  let resultHtml = "";
   currentQuestions.forEach(
-    ({ id, source, description, a, b, c, d, correct, image }, i) => {
+    ({ id, source, description, a, b, c, d, image }, i) => {
       const qid = Number(id);
-      const correctAnswer = Number(correct);
       const safeImage = image ? encodeURIComponent(image) : "";
       resultHtml += `
       <div class="feladat card" id="feladat${qid}">
@@ -104,31 +115,15 @@ const loadQuestions = async (
               : ""
           }
         </form>
-        <script type="text/javascript">
-          $(document).ready(function(){
-              totalPoints++;
-              $("#ans").click(function(event){
-                  event.preventDefault();
-                  teszt(${qid}, ${correctAnswer});
-              });
-              $("#cAns").click(function(event){
-                  event.preventDefault();
-                  showCorrect(${qid}, ${correctAnswer});
-              });
-          });
-        </script>
       </div>
       `;
     },
   );
 
-  resultHtml +=
+  if (currentQuestions.length === 0) {
-    currentQuestions.length === 0
+    resultHtml =
-      ? '<div class="buttonwrapper"><b style="font-size: 2rem;">Nem található a keresésnek megfelelő feladat!</b></div>'
+      '<div class="buttonwrapper"><b style="font-size: 2rem;">Nem található a keresésnek megfelelő feladat!</b></div>';
-      : `<script type="text/javascript">
+  }
-            startTimer = 1;
-            timer = 0;
-          </script>`;
 
   return resultHtml;
 };