Everyone-s-delivery / Web

모두의 배달 Web Service Repository
https://everyone-s-delivery.com
2 stars 1 forks source link

[chore] Entity Setter 제거 #58

Closed rnjstjdgh closed 2 years ago

rnjstjdgh commented 2 years ago

Setter 제거 이유

 참고: 이론적으로 Getter, Setter 모두 제공하지 않고, 꼭 필요한 별도의 메서드를 제공하는게 가장 이상적
이다. 하지만 실무에서 엔티티의 데이터는 조회할 일이 너무 많으므로, Getter의 경우 모두 열어두는 것이
편리하다. Getter는 아무리 호출해도 호출 하는 것 만으로 어떤 일이 발생하지는 않는다. 하지만 Setter는
문제가 다르다. Setter를 호출하면 데이터가 변한다. Setter를 막 열어두면 가까운 미래에 엔티티에가 도대
체 왜 변경되는지 추적하기 점점 힘들어진다. 그래서 엔티티를 변경할 때는 Setter 대신에 변경 지점이 명확
하도록 변경을 위한 비즈니스 메서드를 별도로 제공해야 한다.
rnjstjdgh commented 2 years ago

추가 링크