Inflearn Community Q&A
asked
18일차 2교시 강의 중 그림에 이상한 부분이 있어 제보합니다.
Resolved
Written on
·
152
·
Edited
1
12:40분에 언급한 그림이 이상한 것 같습니다.
설명한 로직은 LastChangeListIndex ~ HistoryEnd -1까지 범위의 Changed 값을 HistoryEnd 위치의 Changed에 merge하여 저장하는 로직으로 보입니다.
코드 상으로는 Changed(== PossibleNewHistoryItem)에 해당하는 값은 HistoryEnd 인덱스에 위치하는 값이며, History의 순회 범위는 [LastChangeListIndex, HistoryEnd) 입니다.
따라서 그림에 "History_2 Change"에 해당하는 인덱스 값은 "HistoryEnd"가 아닌 HistoryEnd - 1이 되야 할 것 같아 글을 작성합니다. 감사합니다.
Answer 1
1
Haker
Instructor
감사합니다.
항상 HistoryEnd를 기반으로 새로운 History를 작성하고 있다보니, 이미 RepChangelistState의 새로운 History는 업데이트가 되었으니, 더 이상 HistoryEnd가 아니라 HistoryEnd-1이 겠군요.
언급해주신 내용이 맞습니다. 감사합니다.
더블 체크로서 수정된 아래 내용을 첨부합니다:
