投稿した質問数 2

2

먼저 아래 코드는 이메일 인증 여부를 확인한 뒤에 이메일 인증 처리를 수행합니다.

if (!mem.isEmailVerified()) {
    mem.verifyEmail();
}

여기서 isEmailVerified() 메서드와 verifyEmail() 메서드는 모두 mem의 내부 필드 데이터를 사용해서 상태를 확인하고 상태를 변경하죠. 게다가 '이메일 인증' 기능 자체는 '이메일이 이미 검증된 여부를 확인'하는 제약을 포함하고 있습니다. 즉 이메일 인증 기능은 검증 여부 확인을 포함해야 온전한 기능이 되는 거죠. 그래서 verifyEmail() 메서드를 다음과 같이 구현함으로써 '이메일 인증' 기능에 대한 캡슐화를 높일 수 있습니다.

void verifyEmail() {
    if (isEmailVerified()) {
        // 이미지 검증된 경우에 대한 처리
        // 예를 들어, 익셉션 발생 등
        throw new AlreadyEmailVerifiedException();
    }
    // 이메일 검증 상태로 변경하는 코드
    ...
}

또한 제약 확인과 상태 변경을 한 곳으로 모으면 다음과 같이 잘못된 코드를 만들 가능성도 줄어들죠.

if (mem.isEmailVerified()) { // 이미 검증했는데
    mem.verifyEmail(); // 또 검증, 그런데 verifyEmail에서 검증 여부를 처리하지 않으면 인증 로직을 다시 실행
}

만약에 verifyEmail()에서 이메일 인증 시간을 현재 시간으로 설정한다거나 하면, 위 코드는 잘못된 인증 시간을 갖게 됩니다.