✨ feat: 新增列表缓存#687
Conversation
Summary of ChangesHello @imsyy, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 此拉取请求的核心目标是引入列表数据缓存机制，以优化应用程序中歌单、专辑和电台等列表的加载性能和用户体验。通过在 CacheService 中实现 Gzip 压缩，不仅加快了数据检索速度，还降低了存储开销。此外，还包含了一些用户界面和数据处理的细节改进，以及对 Discord Rich Presence 功能的初步集成，旨在提升整体应用的响应性和稳定性。 Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
本次 PR 引入了对歌单、专辑和电台列表的缓存机制，这是一个很棒的功能，能够有效提升应用的性能和用户体验。整体实现非常出色，考虑到了缓存版本控制、Gzip 压缩、后台静默更新以及并发请求处理等多个方面。
代码层面，我提出了一些建议：
- 在新的
useListDataCache组合式函数中，建议将用于调试的console.log调用替换为在生产环境中会被移除的日志方案。 - 在多个列表视图组件（
album、playlist、radio）中，backgroundCheck函数的cached参数目前是any类型，建议为其指定具体的ListCacheData类型，以增强代码的类型安全性和可维护性。
除了这些小建议，整个 PR 的代码质量很高。合并后将显著改善列表页面的加载速度。
|
|
||
| try { | ||
| await cacheManager.set("list-data", key, jsonStr); | ||
| console.log(`✅ List cache saved: ${key}`); |
There was a problem hiding this comment.
该文件中使用了大量的 console.log 和 console.error 进行日志记录。虽然这在开发过程中很有用，但这些日志也会输出到生产环境的控制台，可能会造成不必要的干扰，并可能暴露应用的内部工作细节。建议使用专门的日志库，或者将这些日志包装在开发模式的条件判断中（例如 if (import.meta.env.DEV) { ... }），以避免在生产环境中输出。此建议适用于此文件中的所有 console.* 调用。
| console.log(`✅ List cache saved: ${key}`); | |
| if (import.meta.env.DEV) console.log(`✅ List cache saved: ${key}`); |
| if (currentRequestId.value !== id) return; | ||
| setDetailData(formatCoverList(detail.album)[0]); | ||
| // 获取专辑歌曲 | ||
| const ids: number[] = detail.songs.map((song: any) => song.id as number); |
There was a problem hiding this comment.
在 backgroundCheck 函数中，cached 参数的类型是 any。为了更好的类型安全和代码可读性，建议使用具体的 ListCacheData 类型。
你可以在文件顶部修改导入语句：
import { useListDataCache, type ListCacheData } from "@/composables/List/useListDataCache";然后更新函数签名。
const backgroundCheck = async (id: number, cached: ListCacheData) => {
| }; | ||
|
|
||
| // 后台检查更新 | ||
| const backgroundCheck = async (id: number, cached: any) => { |
There was a problem hiding this comment.
与 album.vue 中的情况类似，backgroundCheck 函数中的 cached 参数类型为 any。建议使用 ListCacheData 类型以增强类型安全。
你可以在文件顶部修改导入语句：
import { useListDataCache, type ListCacheData } from "@/composables/List/useListDataCache";然后更新函数签名。
const backgroundCheck = async (id: number, cached: ListCacheData) => {
| }; | ||
|
|
||
| // 后台检查更新 | ||
| const backgroundCheck = async (id: number, cached: any) => { |
There was a problem hiding this comment.
与 album.vue 和 playlist.vue 中的情况类似，backgroundCheck 函数中的 cached 参数类型为 any。建议使用 ListCacheData 类型以增强类型安全。
你可以在文件顶部修改导入语句：
import { useListDataCache, type ListCacheData } from "@/composables/List/useListDataCache";然后更新函数签名。
const backgroundCheck = async (id: number, cached: ListCacheData) => {
|
/gemini review |
There was a problem hiding this comment.
Code Review
此次更新引入了列表（专辑、歌单、电台）的缓存机制，这是一个非常出色的性能和用户体验优化。整体实现考虑周全，采用了“缓存优先，网络兜底”并结合后台更新的策略，同时处理了版本控制、Gzip 压缩和并发请求等问题。代码结构清晰，改动在各个视图中保持了一致性。
我发现了一个 Bug，即在切换电台页面时，旧的歌曲列表没有被正确清除，会导致数据显示错误。此外，我还针对一处硬编码的 URL 提出了可维护性建议。
总体而言，这是一次高质量的提交。在修复提到的 Bug 后，代码便可合并。
| if (detailData.value?.id !== id) { | ||
| setDetailData(null); | ||
| } |
There was a problem hiding this comment.
这里存在一个 Bug：在缓存未命中的情况下，从一个电台页面导航到另一个电台页面时，旧电台的歌曲列表（listData）没有被清空。后续调用的 getRadioAllProgram 方法会使用 appendListData 将新电台的歌曲追加到旧列表的末尾，导致显示错误的歌曲列表。
为了修复这个问题，在清除 detailData 的同时，也应该清空 listData。
if (detailData.value?.id !== id) {
setDetailData(null);
setListData([]);
}
| const goDownload = () => { | ||
| emit("close"); | ||
| window.open(packageJson.github + "/releases", "_blank"); | ||
| window.open("https://splayer.imsyy.top/download.html", "_blank"); |
There was a problem hiding this comment.
将下载链接硬编码到代码中会降低可维护性。如果未来下载页面的 URL 发生变化，就需要修改代码并重新发布。
建议将这个 URL 移到 package.json 文件中，就像之前的 github 链接一样。这样可以更方便地管理和更新。
例如，在 package.json 中添加：
"downloadUrl": "https://splayer.imsyy.top/download.html"然后在代码中使用：
window.open(packageJson.downloadUrl, "_blank");
2 participants
No description provided.