Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[6주차 미션] 구현 완료했습니다. 리뷰 부탁드립니다. #78

Open
wants to merge 9 commits into
base: kikat
Choose a base branch
from

Conversation

sunghyuki
Copy link

현재의 요구사항 정리

  • 질문 데이터를 완전히 삭제하는 것이 아니라 데이터의 상태를 삭제 상태로 변경한다.
  • 질문을 삭제할 때 답변 또한 삭제해야 하며, 답변의 삭제 또한 삭제 상태를 변경한다.
  • 로그인한 사용자와 질문한 사람이 같은 경우 삭제 가능하다.
  • 답변이 없는 경우 삭제가 가능하다.
  • 질문 데이터는 질문자와 답변 글의 모든 답변자가 같은 경우 삭제 가능하다.
  • 질문자와 답변자가 다른 경우 답변을 삭제할 수 없다.

현재의 요구사항을 고려했을 때 질문데이터와 답변데이터는 완전한 삭제가 아닌 삭제 상태를 갖는다. 그렇기 때문에 데이터는 삭제라는 '상태'를 가지고, view에서 보여질 때는 삭제 상태인지 아닌지를 판단 후 보여져야 한다. 데이터의 완전한 삭제를 고려하지 않으니 영속성 전이 속성 또한 아직은 필요하지 않을 것 같다.

질문 데이터가 삭제될 수 있는 선택지에는

  1. 댓글이 처음부터 달리지 않아 질문데이터가 보유한 댓글 갯수가 0인 경우
  2. 질문데이터가 보유한 댓글 데이터의 모든 삭제 상태가 true인 경우
  3. 질문데이터에 작성한 모든 댓글이 해당 질문을 작성한 사용자의 댓글인 경우이다.

기존 구현에서 수정되어야 할 부분

댓글을 생성 후 해당 사용자인지 아닌지를 검사 후 해당 사용자일 경우 댓글 삭제 버튼이 보이도록 구현을 생각했음. 생각을 다시해보니 댓글을 작성할 수 있는 권한은 해당 사용자에게만 있는데 작성 후 보여지는 페이지에서 작성자인지 아닌지를 검사할 필요가 없다고 생각함. -> (추후 수정 예정)

e.preventDefault() : 서버로 데이터 전송 기능을 막음

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant