최근에는 사소한 건축 문제가 발생했습니다. 내 코드에는 다음과 같은 간단한 저장소가 있습니다 (코드는 C #입니다).
var user = /* create user somehow */;
_userRepository.Add(user);
/* do some other stuff*/
_userRepository.SaveChanges();SaveChanges 데이터베이스 변경 사항을 커밋하는 간단한 래퍼였습니다.
void SaveChanges()
{
_dataContext.SaveChanges();
_logger.Log("User DB updated: " + someImportantInfo);
}그런 다음 얼마 후에 사용자가 시스템에서 생성 될 때마다 전자 메일 알림을 보내는 새로운 논리를 구현해야했습니다. 많은 전화가 있었다 때문에 _userRepository.Add()및 SaveChanges시스템의 주위에, 나는 업데이트하기로 결정 SaveChanges다음과 같이 :
void SaveChanges()
{
_dataContext.SaveChanges();
_logger.Log("User DB updated: " + someImportantInfo);
foreach (var newUser in dataContext.GetAddedUsers())
{
_eventService.RaiseEvent(new UserCreatedEvent(newUser ))
}
}이런 식으로 외부 코드는 UserCreatedEvent를 구독하고 알림을 보내는 데 필요한 비즈니스 로직을 처리 할 수 있습니다.
그러나 내가 수정 한 내용 SaveChanges이 단일 책임 원칙 을 위반 한 것으로 나타 났 으며, 이는 SaveChanges어떠한 사건도 막고 저장하지 않아야한다는 점을 지적했습니다 .
이것이 유효한 포인트입니까? 여기서 이벤트를 발생시키는 것은 로깅과 본질적으로 같은 것 같습니다. 기능에 일부 기능을 추가하는 것입니다. 그리고 SRP는 함수에서 로깅 또는 실행 이벤트를 사용하는 것을 금지하지 않으며 그러한 논리를 다른 클래스로 캡슐화해야한다고 저장소가 다른 클래스를 호출하는 것이 좋습니다.
22
당신의 레토르트는 " 좋습니다. 어떻게 SRP를 위반하지 않고 여전히 단일 수정 지점을 허용하도록 작성 하시겠습니까?"
—
Robert Harvey
내 관찰은 이벤트를 제기해도 추가 책임이 추가되지 않는다는 것입니다. 실제로는 정반대입니다. 책임을 다른 곳에서 위임합니다.
—
Robert Harvey
귀하의 동료가 옳다고 생각하지만 귀하의 질문은 유효하고 유용합니다.
—
Andres F.
단일 책임에 대한 결정적인 정의는 없습니다. SRP를 위반한다고 지적한 사람은 개인 정의를 사용하여 정확하며 자신의 정의를 사용하여 올바른 것입니다. 이 이벤트는 다른 유사한 기능이 다른 방식으로 수행되는 일회성 이벤트가 아니라는 점에서 디자인이 완벽하다고 생각합니다. 일관성을 유지하는 것이 SRP와 같은 모호한 지침보다주의를 기울이는 것보다 훨씬 중요합니다.
—
덩크