diff --git a/src/main/java/com/dh/TaskManager/service/impl/TaskService.java b/src/main/java/com/dh/TaskManager/service/impl/TaskService.java
index 7e15a79..4e352a2 100644
--- a/src/main/java/com/dh/TaskManager/service/impl/TaskService.java
+++ b/src/main/java/com/dh/TaskManager/service/impl/TaskService.java
@@ -40,14 +40,14 @@ public TaskService(ITaskRepository taskRepository, ObjectMapper objectMapper) {
     /**
      * Crea una nueva tarea utilizando los datos proporcionados en el objeto TaskRequestDTO.
      *
-     * @param taskDTO El objeto TaskRequestDTO que contiene los datos de la tarea a crear.
+     * @param taskRequestDTO El objeto TaskRequestDTO que contiene los datos de la tarea a crear.
      * @return TaskResponseDTO El objeto TaskResponseDTO correspondiente a la tarea creada.
      */
     @Override
-    public TaskResponseDTO createTask(TaskRequestDTO taskDTO) {
-        Task task = objectMapper.convertValue(taskDTO, Task.class);
-        Task taskSaved = taskRepository.save(task);
-        return objectMapper.convertValue(taskSaved, TaskResponseDTO.class);
+    public TaskResponseDTO createTask(TaskRequestDTO taskRequestDTO) {
+        Task task = mapToEntity(taskRequestDTO);
+        taskRepository.save(task);
+        return mapToDTO(task);
     }
 
     /**
@@ -59,13 +59,10 @@ public TaskResponseDTO createTask(TaskRequestDTO taskDTO) {
      */
     @Override
     public TaskResponseDTO getTaskById(Long id) {
-        Task task = taskRepository.findById(id)
-                .orElse(null);
-
-        if (task != null) {
-            return mapToDTO(task);
-        }
-        throw new NotFoundException(NOT_FOUND_MESSAGE);
+        Task task = taskRepository.findById(id).orElseThrow(
+                () -> new NotFoundException(NOT_FOUND_MESSAGE)
+        );
+        return mapToDTO(task);
 
     }
 
@@ -81,7 +78,7 @@ public List<TaskResponseDTO> getAllTasks() {
         return taskRepository.findAll()
                 .stream() // Inicia un flujo de datos sobre la lista de tareas
                 .map(this::mapToDTO) // Mapea cada tarea a un objeto TaskResponseDTO utilizando un método de referencia
-                .collect(Collectors.toList()); // Recopila los resultados en una lista
+                .toList(); // Recopila los resultados en una lista
     }
 
     /**
@@ -96,39 +93,31 @@ public List<TaskResponseDTO> getAllTasks() {
      * TaskRequestToUpdateDTO. Si la tarea existe, se actualizan sus datos con los valores proporcionados
      * en el objeto TaskRequestToUpdateDTO, y luego se guarda la tarea actualizada en la base de datos.
      *      *
-     * @param taskRequestDTO El objeto TaskRequestToUpdateDTO que contiene los datos actualizados de la tarea.
+     * @param taskRequestToUpdateDTO El objeto TaskRequestToUpdateDTO que contiene los datos actualizados de la tarea.
      * @return TaskResponseDTO El objeto TaskResponseDTO correspondiente a la tarea actualizada.
      * @throws NotFoundException Si no se encuentra ninguna tarea con el identificador proporcionado.
      */
     @Override
     @Transactional
-    public TaskResponseDTO updateTask(TaskRequestToUpdateDTO taskRequestDTO) {
-        Task task = taskRepository.findById(taskRequestDTO.getId())
-                .orElse(null);
-
-        if (task != null) {
-            task.setTitle(taskRequestDTO.getTitle());
-            task.setDescription(taskRequestDTO.getDescription());
-
-            taskRepository.save(task);
-            return mapToDTO(task);
-        }
-        throw new NotFoundException(NOT_FOUND_MESSAGE);
+    public TaskResponseDTO updateTask(TaskRequestToUpdateDTO taskRequestToUpdateDTO) {
+        getTaskById(taskRequestToUpdateDTO.getId());
+        return mapToDTO(taskRepository.save(mapToEntity(taskRequestToUpdateDTO)));
     }
 
+    /**
+     * Elimina una tarea por su identificador único.
+     *
+     * @param id Identificador único de la tarea a eliminar.
+     *
+     * @throws NotFoundException Si no se encuentra ninguna tarea con el identificador proporcionado.     *
+     */
     @Override
     public void deleteTaskById(Long id) {
         // Busca la tarea en la base de datos utilizando el identificador proporcionado
-        Task task = taskRepository.findById(id)
-                .orElse(null);
+        getTaskById(id);
 
         // Si la tarea existe, actualiza sus datos y guarda la tarea actualizada en la base de datos
-        if (task != null) {
-            taskRepository.delete(task);
-        }
-
-        // Si la tarea no existe, lanza una excepción NotFoundException
-        throw new NotFoundException(NOT_FOUND_MESSAGE);
+        taskRepository.deleteById(id);
     }
 
     /**
@@ -142,12 +131,22 @@ private TaskResponseDTO mapToDTO(Task task) {
     }
 
     /**
-     * Convierte un objeto de tipo TaskResponseDTO a un objeto de tipo Task.
+     * Convierte un objeto de tipo TaskRequestDTO a un objeto de tipo Task.
+     *
+     * @param taskRequestDTO El objeto de tipo TaskRequestDTO a convertir.
+     * @return Task El objeto Task resultante de la conversión.
+     */
+    private Task mapToEntity(TaskRequestDTO taskRequestDTO) {
+        return objectMapper.convertValue(taskRequestDTO, Task.class);
+    }
+
+    /**
+     * Convierte un objeto de tipo TaskRequestDTO a un objeto de tipo Task.
      *
-     * @param taskResponseDTO El objeto de tipo TaskResponseDTO a convertir.
+     * @param taskRequestUpdateDTO El objeto de tipo TaskRequestDTO a convertir.
      * @return Task El objeto Task resultante de la conversión.
      */
-    private Task mapToEntity(TaskResponseDTO taskResponseDTO) {
-        return objectMapper.convertValue(taskResponseDTO, Task.class);
+    private Task mapToEntity(TaskRequestToUpdateDTO taskRequestUpdateDTO) {
+        return objectMapper.convertValue(taskRequestUpdateDTO, Task.class);
     }
 }