-
Notifications
You must be signed in to change notification settings - Fork 4
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
Feature/#011 순서있는 리스트 숫자 구현 #214
The head ref may contain hidden characters: "Feature/#011_\uC21C\uC11C\uC788\uB294_\uB9AC\uC2A4\uD2B8_\uC22B\uC790_\uAD6C\uD604"
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고많으셨습니다!
updateEditorState(); | ||
if (prevBlock.type === "ol" && nextBlock?.type === "ol") { | ||
editorCRDT.LinkedList.updateAllOrderedListIndices(); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이전과 이후가 "ol"이여야 실행되게 수정됐군요! 확인했습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다 !!
📝 변경 사항
🔍 변경 사항 설명
CRDT의 BlockLinkedList에
updateAllOrderedListIndices
메서드를 구현해서 현재 연산이 순서 리스트일 경우, 모든 블록을 순회하며 리스트의 index를 갱신합니다. 기존에는 각 케이스별로 코드를 작성했는데 이렇게 하다보니까 에러도 많이 발생하고 코드의 길이가 늘어나서 LinkedList 메서드로 분리해서 사용하는 식으로 구현했습니다.블록을 모두 순회하는 로직이다보니 자주 실행하면 성능에 영향을 줄 수 있을것 같아 순서리스트 연산일때만 실행시키도록 분기 처리했습니다.
🙏 질문 사항
📷 스크린샷 (선택)
orderedlist.mov
✅ 작성자 체크리스트