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

Comment model에 삭제 여부를 나타내는 field 추가 #29

Open
tictactoeid opened this issue Jan 4, 2021 · 4 comments
Open

Comment model에 삭제 여부를 나타내는 field 추가 #29

tictactoeid opened this issue Jan 4, 2021 · 4 comments

Comments

@tictactoeid
Copy link
Contributor

Comment쪽 개발을 시작했는데, model에 comment가 삭제되었는지를 나타내는 field 하나를 추가해야 할 것 같습니다.

Post의 경우 DELETE api를 통해 게시글 하나를 삭제하면 DB에서도 실제로 삭제되는 것으로 구현하였습니다.
그런데, 댓글도 그렇게 삭제할 경우 원 댓글에 달려 있던 대댓글을 표시할 때 문제가 생길 것 같습니다.

예를 들어 3번 Comment에 대댓글로 4번, 5번 Comment가 달려 있는 상황에 3번을 삭제할 경우,
4번, 5번 Commentcomment_id 값은 3이 되는데 DB에 id=3인 Comment가 존재하지 않아 프론트에서 삭제할 때 문제가 될 것 같습니다.

그래서 deleted라는 Boolean field를 하나 만들고, 기본값을 False로 하고, DELETE api에서는 이걸 True로 바꾸는 식으로 구현하고자 하는데 어떻게 생각하시나요?

field name이나 구현 방식에 대한 의견 부탁드립니다.

@tictactoeid
Copy link
Contributor Author

추가로,
comment_iddefault 값(대댓글이 아닌 경우)을 0으로 하는 것은 어떤가요?
어차피 id는 1부터 시작하니 False를 나타내는 0이 더 적합한 것 같습니다.

@ghost
Copy link

ghost commented Jan 4, 2021

일단 comment_iddefault 값을 0 으로 하면 더 적합할 것 같다는 의견에 찬성입니다

/

그리고 첫 글의 상황에 대해서 deleted 라는 boolean field 가 필요한 이유는
Screen Shot 2021-01-04 at 5 23 41 PM
위처럼 부모댓글이 사라졌을 때도 대댓글은 남아있어야 하기 때문인건가요?

@tictactoeid
Copy link
Contributor Author

네 대댓글을 남기기 위해서입니다.

@ghost
Copy link

ghost commented Jan 4, 2021

그런 의미에서 말씀해주신거라면 deleted 가 필요할 것 같습니다, 말씀해주신대로 구현하면 좋을 것 같습니다

추가적으로, 실제로 삭제되었기에 내용도 남아있으면 안되므로
글 내용도 삭제된 댓글입니다.
글 작성자도 (삭제) 로 바꾸면 좋을 것 같습니다

@ghost ghost added the Completion label Jan 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant