Predidit / oneAnime

一款简洁清爽无广告的看番软件。 一款带弹幕的 anime1 第三方客户端,界面符合 Material You 规范。
GNU General Public License v3.0
592 stars 11 forks source link

Fix: Stop sorting returned anime list from remote #12

Closed songbirdzz closed 2 months ago

Predidit commented 2 months ago

此修改会导致未添加追番的番剧,观看进度丢失。

songbirdzz commented 2 months ago

fixed

Predidit commented 2 months ago

你测试过当网络环境有问题时, newlist 为空,所有数据可能被清除的情况吗

songbirdzz commented 2 months ago

https://github.com/Predidit/oneAnime/pull/12/files#diff-6d4dfc03634238e658178f3b77a3aff95173d813e554a3c3a1929b36f66f4663R37

这里有判断。

另外现在用来显示的数据和用来存储的数据应该可以更解耦一点,以后想重构的话可以考虑改进一下。

Predidit commented 2 months ago

看上去这个 PR 没有问题了。

不过我想再确认一下,这样修改之后,刷新列表会有延迟和卡顿吗,毕竟这样的处理是完全没有优化的。

songbirdzz commented 2 months ago

不会... 就是一个一千多项的数组而已, $O(n^2)$ 的复杂度也完全无所谓;再加之刷新这个操作非常稀少,更可以忽略不计。