댓글 수정 기능 구현 #303
Merged
댓글 수정 기능 구현 #303
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
댓글 수정 버튼과 ViewModel 간 바인딩을 구현했습니다.
답장 뿐 아니라 댓글 수정시에도 ReplyView가 사용되어 이름을 변경했습니다. 이 때 ReplyVIew에 들어가는 UILabel과 UIButton의 text도 바뀌므로 두 텍스트 값을 설정하기 위한 프로퍼티(labelText, buttonText) 설정도 함께 구현했습니다
수정 버튼, 신고 버튼 그리고 삭제 버튼이 눌렸을 때, 대상이 되는 id도 함게 반환하도록 재구현했습니다. 기존에는 BottomSheet를 누르는 즉시 id를 반환하였는데, 이 경우 문제가 될 수 있는 시나리오가 존재합니다. 예를 들면, 수정 버튼을 누르고 댓글을 수정하는 도중 다른 댓글의 옵션 버튼을 누르면 그 즉시 id가 바뀌게 됩니다. 이 경우 내가 수정하고자 했던 id와 다른 값으로 변경되기 때문에 문제가 야기될 수 있습니다.
답글, 댓글 수정, 댓글 삭제 시 필요한 타겟 ID를 두 Subject로 처리하고 있었는데, 이를 하나의 Subject로 통합하였습니다.
댓글 수정 옵션과 삭제 옵션을 두 가지로 나누어 Subject를 처리했으나, 코드 가독성과 확장성을 고려하여 하나의 Subject로 통합했습니다. 통합하는 과정에서 `Void`로 emit했던 것이 `CommentOption`이라는 새로운 열거형 타입으로 변경되었고, CommentOption은 댓글 생성, 수정, 삭제와 연관되어 동작합니다.
댓글을 수정할 수 있도록 기존 댓글 텍스트를 텍스트필드란에 삽입하는 코드를 구현했습니다.
WhiteHyun
added
🐛 bug
dlrjswns
approved these changes
Apr 20, 2023
Comment on lines
+28
to
+34
| var commentText: String { | ||
| get { | ||
| textView.text | ||
| } set { | ||
| textView.text = newValue | ||
| } | ||
| } |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📌 PR 요약
🌱 작업한 내용
🌱 PR 포인트
ReplyView를 DecoratorView로 변경했습니다.
답장 뿐 아니라 댓글 수정할 때도 ReplyView가 사용되기 때문입니다.
댓글 수정, 댓글 작성, 답글 작성, 댓글 삭제를 각각 분리하여 구현하되, 비슷한 성격을 가진 Item들을 묶어 관리하였습니다.
CommentOptionSubject: 현재 댓글 정보를 처리하기 위한 옵션을 처리합니다. (댓(답)글인지, 삭제인지, 수정인지)TargetIDSubject:replyID와recentSelectedCommentID를 묶었습니다. 답글이든, 삭제이든, 수정이든 결국에는 타겟팅 되어있는 ID를 가지고 처리하는 녀석들이기 때문에, 하나의 Subject로 관리했습니다.BottomSheet를 띄울 때 commentID도 initializer에 넣어 처리하도록 구현했습니다.
기존에는
BottomSheet를 요청한Cell의commentID를 미리 갖고있었으나, 나중에다른 Cell의BottomSheet요청 버튼을 누른 뒤 나가버리게 되면 commentID가 재갱신되어버리는 상황이 발생되기 때문에 시나리오상 버그가 발생할 우려가 존재했습니다. 따라서, 확실하게 BottomSheet로부터댓글 수정 버튼,댓글 삭제 버튼이 눌려졌다면 delegate에서 commentID와 함께 호출하는 방식을 사용하도록 하여 사용자로부터 의도치 않는 현상을 방지할 수 있도록 수정하였습니다.📸 스크린샷
📮 관련 이슈