Skip to content

Commit

Permalink
refactor : ResponseStatus 어노테이션 사용으로 변경
Browse files Browse the repository at this point in the history
  • Loading branch information
tmxhsk99 committed Jul 13, 2023
1 parent d488924 commit c7b8cc5
Showing 1 changed file with 6 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
import com.codesoom.assignment.models.request.TaskCreate;
import com.codesoom.assignment.models.request.TaskEdit;
import org.springframework.http.HttpStatus;
import org.springframework.http.ResponseEntity;
import org.springframework.web.bind.annotation.*;

import java.util.List;
Expand All @@ -29,22 +28,22 @@ public Task detail(@PathVariable Long id) {
return taskService.getTask(id);
}

@ResponseStatus(value = HttpStatus.CREATED)
@PostMapping("/tasks")
public ResponseEntity<Task> create(@RequestBody TaskCreate taskCreate) {
public Task create(@RequestBody TaskCreate taskCreate) {
taskCreate.validate();
Task task = taskService.createTask(taskCreate);
return ResponseEntity.status(HttpStatus.CREATED).body(task);
return taskService.createTask(taskCreate);
}

@RequestMapping(value = "/tasks/{id}", method = {RequestMethod.PUT, RequestMethod.PATCH})
public Task update(@PathVariable Long id,@RequestBody TaskEdit task) {
public Task update(@PathVariable Long id, @RequestBody TaskEdit task) {
task.setId(id);
return taskService.updateTask(task);
}

@ResponseStatus(value = HttpStatus.NO_CONTENT)
@DeleteMapping("/tasks/{id}")
public ResponseEntity<Void> delete(@PathVariable Long id) {
public void delete(@PathVariable Long id) {
taskService.deleteTask(id);
return ResponseEntity.noContent().build();
}
}

0 comments on commit c7b8cc5

Please sign in to comment.