Refactored code based on Copilot suggestions

todo-app-encora/src/main/java/com/assigment/todoapp/repository/ToDoRepository.java

@@ -1,48 +1,16 @@
 package com.assigment.todoapp.repository;
 
 import java.util.List;
-import java.util.stream.Collectors;
-
-import org.springframework.stereotype.Repository;
+import java.util.UUID;
 
 import com.assigment.todoapp.domain.ToDoItem;
-
-
-/* 
- @Repository 
-public interface ToDoRepository extends JpaRepository<ToDoItem, UUID> to create the interface and use de Database
-*/
-
-@Repository
-public class ToDoRepository  { // interface 
-
-
-	private List<ToDoItem> todoItems;
-
-    public ToDoRepository(List<ToDoItem> todoItems) {
-        this.todoItems = todoItems;
-    }
-
-	public List<ToDoItem>  fetchAllToDoItems () {
-		return todoItems;
-	}
-
-	public List<ToDoItem> findByDone(boolean aux) {
-        return todoItems.stream()
-                .filter(item -> item.isDone() == aux)
-                .collect(Collectors.toList());
-    }
-
-    public List<ToDoItem> findByNameContainingIgnoreCase(String name) {
-        return todoItems.stream()
-                .filter(item -> item.getName().toLowerCase().contains(name.toLowerCase()))
-                .collect(Collectors.toList());
-    }
-
-    public List<ToDoItem> findByPriority(String priority) {
-    	return todoItems.stream()
-                .filter(item -> item.getPriority().equalsIgnoreCase(priority))
-                .collect(Collectors.toList());
-    }
 
-}
+public interface ToDoRepository {
+    List<ToDoItem> fetchAllToDoItems();
+    List<ToDoItem> findByDone(boolean done);
+    List<ToDoItem> findByNameContainingIgnoreCase(String name);
+    List<ToDoItem> findByPriority(String priority);
+    ToDoItem findById(UUID id);
+    void save(ToDoItem todoItem);
+    void deleteById(UUID id);
+}

todo-app-encora/src/main/java/com/assigment/todoapp/repository/ToDoRepositoryImpl.java

@@ -0,0 +1,63 @@
+package com.assigment.todoapp.repository;
+
+import java.util.List;
+import java.util.UUID;
+import java.util.stream.Collectors;
+
+import org.springframework.stereotype.Repository;
+
+import com.assigment.todoapp.domain.ToDoItem;
+
+@Repository
+public class ToDoRepositoryImpl implements ToDoRepository {
+
+    private final List<ToDoItem> todoItems;
+
+    public ToDoRepositoryImpl(List<ToDoItem> todoItems) {
+        this.todoItems = todoItems;
+    }
+
+    @Override
+    public List<ToDoItem> fetchAllToDoItems() {
+        return todoItems;
+    }
+
+    @Override
+    public List<ToDoItem> findByDone(boolean done) {
+        return todoItems.stream()
+                .filter(item -> item.isDone() == done)
+                .collect(Collectors.toList());
+    }
+
+    @Override
+    public List<ToDoItem> findByNameContainingIgnoreCase(String name) {
+        return todoItems.stream()
+                .filter(item -> item.getName().toLowerCase().contains(name.toLowerCase()))
+                .collect(Collectors.toList());
+    }
+
+    @Override
+    public List<ToDoItem> findByPriority(String priority) {
+        return todoItems.stream()
+                .filter(item -> item.getPriority().equalsIgnoreCase(priority))
+                .collect(Collectors.toList());
+    }
+
+    @Override
+    public ToDoItem findById(UUID id) {
+        return todoItems.stream()
+                .filter(item -> item.getId().equals(id))
+                .findFirst()
+                .orElseThrow(() -> new RuntimeException("ToDoItem not found"));
+    }
+
+    @Override
+    public void save(ToDoItem todoItem) {
+        todoItems.add(todoItem);
+    }
+
+    @Override
+    public void deleteById(UUID id) {
+        todoItems.removeIf(item -> item.getId().equals(id));
+    }
+}