Create TaskService.java

* move repository logic from controller class to service

Author: codeherk

src/main/java/com/codeherk/taskapi/controller/TaskController.java

@@ -2,9 +2,9 @@
 
 import java.util.List;
 
-import com.codeherk.taskapi.exception.TaskNotFoundException;
 import com.codeherk.taskapi.model.Task;
-import com.codeherk.taskapi.repo.TaskRepository;
+import com.codeherk.taskapi.service.TaskService;
+import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.web.bind.annotation.DeleteMapping;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.PathVariable;
@@ -16,51 +16,40 @@
 @RestController
 public class TaskController {
 
-    private final TaskRepository repository;
+    @Autowired
+    private final TaskService taskService;
 
-    TaskController(TaskRepository repository) {
-        this.repository = repository;
+    TaskController(TaskService taskService) {
+        this.taskService = taskService;
     }
 
 
     // Aggregate root
     // tag::get-aggregate-root[]
     @GetMapping("/tasks")
-    List<Task> all() {
-        return repository.findAll();
+    List<Task> getTasks() {
+        return taskService.getTasks();
     }
     // end::get-aggregate-root[]
 
     @PostMapping("/tasks")
-    Task newTask(@RequestBody Task newTask) {
-        return repository.save(newTask);
+    Task createTask(@RequestBody Task newTask) {
+        return taskService.createTask(newTask);
     }
 
-    // Single item
-
+    // Single task
     @GetMapping("/tasks/{id}")
-    Task one(@PathVariable Long id) {
-
-        return repository.findById(id)
-                .orElseThrow(() -> new TaskNotFoundException(id));
+    Task getTaskById(@PathVariable Long id) {
+        return taskService.getTaskById(id);
     }
 
     @PutMapping("/tasks/{id}")
-    Task replaceTask(@RequestBody Task newTask, @PathVariable Long id) {
-
-        return repository.findById(id)
-                .map(Task -> {
-                    Task.setDescription(newTask.getDescription());
-                    return repository.save(Task);
-                })
-                .orElseGet(() -> {
-                    newTask.setId(id);
-                    return repository.save(newTask);
-                });
+    Task updateTask(@RequestBody Task newTask, @PathVariable Long id) {
+        return taskService.updateTask(newTask, id);
     }
 
     @DeleteMapping("/tasks/{id}")
     void deleteTask(@PathVariable Long id) {
-        repository.deleteById(id);
+        taskService.deleteTask(id);
     }
 }

src/main/java/com/codeherk/taskapi/service/TaskService.java

@@ -0,0 +1,47 @@
+package com.codeherk.taskapi.service;
+
+import com.codeherk.taskapi.exception.TaskNotFoundException;
+import com.codeherk.taskapi.model.Task;
+import com.codeherk.taskapi.repo.TaskRepository;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.stereotype.Service;
+
+import java.util.List;
+
+@Service
+public class TaskService {
+    @Autowired
+    private final TaskRepository repository;
+
+    public TaskService(TaskRepository repository) {
+        this.repository = repository;
+    }
+
+    public List<Task> getTasks() {
+        return repository.findAll();
+    }
+
+    public Task getTaskById(Long id) {
+        return repository.findById(id).orElseThrow(() -> new TaskNotFoundException(id));
+    }
+
+    public Task createTask(Task newTask) {
+        return repository.save(newTask);
+    }
+
+    public Task updateTask(Task newTask, Long id) {
+        return repository.findById(id)
+                .map(Task -> {
+                    Task.setDescription(newTask.getDescription());
+                    return repository.save(Task);
+                })
+                .orElseGet(() -> {
+                    newTask.setId(id);
+                    return repository.save(newTask);
+                });
+    }
+
+    public void deleteTask(Long id) {
+        repository.deleteById(id);
+    }
+}