This repository was archived by the owner on Oct 18, 2025. It is now read-only.
refactor: use json instead of csv to store location history#63
Draft
PA733 wants to merge 1 commit intoella8192:masterfrom
Draft
refactor: use json instead of csv to store location history#63PA733 wants to merge 1 commit intoella8192:masterfrom
PA733 wants to merge 1 commit intoella8192:masterfrom
Conversation
This file contains hidden or 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
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
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 将 HistoricalLocation 部分重构，抛弃了 CSV 存储方式，改为 JSON 。
同时增加了迁移步骤，以兼容旧数据。
重构原因
JSON 可直接利用项目中已导入的 com.alibaba.fastjson2 进行处理，不需要增加额外的依赖。
CSV 要完整处理，需要大量的编码实现，可行但是代码可读性和效率会下降。
CSV 即使处理后，Kotlin 字面量也仍然需要对引号转义，否则会出现把
"123456"显示为123456之类的问题。演示
To-Do
测试
其他