#8-중복 데이터 방어로직 추가

[Sunmon]

[요약]
중복 데이터 방어로직 추가 및 테스트코드 작성

[목적]

• 중복된 데이터 삽입으로 인한 오류 방지

[목표]

• 중복 로그인 방지 기능 추가
• 중복 회원가입 방지 기능 추가
• 가입하지 않은 유저로 로그인 세션 생성 방지 기능 추가
• 팔로우 중복 추가 방지 기능 추가
• 각 목표에 대한 테스트코드 작성

[목표로 하지 않는 것]

• 로그인 기능 보완 (id, pw로 로그인하기)
• 오류 상세 정의 및 예외처리

[고려사항]

1. 오류를 반환하는 위치

• 오류를 반환하는 위치가 3곳이 존재함 (서비스, 레포지토리, DB)
  • DB단에서 오류를 반환함 -> OK
  • 서비스 OR 레포지토리에서 오류를 반환함 -> 둘 중 서비스 단에서 반환하도록 선택함
  • 레포지토리는 단순히 DB에 접근해서 데이터를 나르는 역할만 하는 것이기때문에 서비스가 오류 핸들링하는 위치로 적합하다고 판단했기 때문

2. 서비스 방어 로직 필요 유무

예: 회원가입시 이미 존재하는 회원은 에러를 반환함

function join() {
  // 기존에 이미 DB에 존재하는지 체크 -> 존재한다면 에러 반환
  if (UserDB.get(id)) {
    throw new Error('이미 존재하는 유저')
  }
  // 새로운 회원을 DB에 추가. -> pk, unique 조건이 존재하므로 중복되어 들어간다면 에러 반환
  UserDB.create(id);
}

DB 에서 새 회원을 추가할때 에러를 반환할 것인데, 굳이 if문을 추가하여 기존 회원이 존재하는지 체크할 필요가 있는지 의문입니다.
DB에 한 번 더 접근하기때문에 오히려 효율성이 떨어지는 것이 아닌가요?

---

this resolves #8

[f-lab-jayce]

제 개인적인 케이스로는 DB의 에러를 핸들링하는 방향으로 코드를 짜는 편입니다.