From 4acb80eea0246dae2eb782b07fcb5613a3f3224f Mon Sep 17 00:00:00 2001
From: aniruddhzaveri <aniruddh.zaveri@tum.de>
Date: Mon, 2 Dec 2024 13:44:07 +0100
Subject: [PATCH] simon feedback

---
 .../cit/aet/artemis/assessment/service/ResultService.java | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/main/java/de/tum/cit/aet/artemis/assessment/service/ResultService.java b/src/main/java/de/tum/cit/aet/artemis/assessment/service/ResultService.java
index 8d7db4ae46ca..030b2d290ca8 100644
--- a/src/main/java/de/tum/cit/aet/artemis/assessment/service/ResultService.java
+++ b/src/main/java/de/tum/cit/aet/artemis/assessment/service/ResultService.java
@@ -686,9 +686,7 @@ private List<FeedbackDetailDTO> aggregateUsingLevenshtein(List<FeedbackDetailDTO
         for (FeedbackDetailDTO base : feedbackDetails) {
             boolean isMerged = false;
 
-            for (int i = 0; i < processedDetails.size(); i++) {
-                FeedbackDetailDTO processed = processedDetails.get(i);
-
+            for (FeedbackDetailDTO processed : processedDetails) {
                 // Ensure feedbacks have the same testCaseName and taskName
                 if (base.testCaseName().equals(processed.testCaseName()) && base.taskName().equals(processed.taskName())) {
                     double similarity = NameSimilarity.levenshteinSimilarity(base.detailText().getFirst(), processed.detailText().getFirst());
@@ -701,10 +699,10 @@ private List<FeedbackDetailDTO> aggregateUsingLevenshtein(List<FeedbackDetailDTO
                         long mergedCount = processed.count() + base.count();
 
                         // Replace the processed entry with the updated one
+                        processedDetails.remove(processed);
                         FeedbackDetailDTO updatedProcessed = new FeedbackDetailDTO(mergedCount, 0, mergedTexts, processed.testCaseName(), processed.taskName(),
                                 processed.errorCategory());
-
-                        processedDetails.set(i, updatedProcessed);
+                        processedDetails.add(updatedProcessed); // Add the updated entry
                         isMerged = true;
                         break; // No need to check further
                     }