단일 책임 원칙을 어겨서는 안됩니다.

[hannut91]

문제

객체가 변경해야 할 이유가 두 가지 이상이다. 이는 코드를 유지보수하기 어렵게 만든다.

예시

@Repository
public class TaskRepository {
	public ResponseEntity<Task> getTask(Long id) {
		Optional<Task> task = findTask(id);
		if (task.isEmpty()) {
			return new ResponseEntity(HttpStatus.NOT_FOUND);
		}
		return new ResponseEntity(task, HttpStatus.OK);
	}

	// ... 생략
}

Repository는 데이터를 관리하는 것을 추상화한 것입니다. 그런데 여기서 응답하는 타입이 ResponseEntity입니다.
ResponseEntity는 HTTP 응답을 어떻게 할지 결정하는데, 이 것을 여기에서 책임지는 순간, 내가 사용자에게 응답을 주는 것을
변경하려고 할 때 여기서 코드가 수정되어야 합니다. 즉 응답을 변경하고 싶을 때, 데이터를 어떻게 관리할지 두 가지 이유로 인해
이 객체가 변경이 일어납니다.

따라서 단일 책임 원칙을 지키도록 코드를 수정해야 합니다