-
Notifications
You must be signed in to change notification settings - Fork 33
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
feat(music_new): new interfaces for music, platform and API #15
base: master
Are you sure you want to change the base?
Conversation
…l params; make PropertyRequestor.invoke private
今天我会先完成一下 在玩状态 和 qq音乐的封面解析,发布一下新的版本后合并这个pr |
我把这个简化了一下,去掉了 我之后有空的时候把单元测试也写一下再把代码推上来 |
解释一下新提交的更改: 音乐属性分三大类:
我把 另外我不清楚 |
这个设计很赞,这个也是我一直想做的,这样搜索的体验会好很多 |
….music_new.netease.netease_music; add a ClientSession parameter to getter functions and functions making aiohttp request; add unit tests for APIs
给音乐,音乐平台和 API 设计了几个新的类。还没做完想先听听意见
Platform
负责通过搜索和歌单、专辑来找到音乐MusicPiece
是音乐的实例。绝大部分属性都允许按需加载,减少不必要的延迟PropertyRequestor
把 API 和用到的参数封装到了一起。如果同时从多个地方调用或者短时间内多次调用可以减少重复请求的数量(我不确定这个东西有没有必要)其实
NeteaseMusic
除了媒体 URL 以外都可以快速批量获取。我这么写只是为了展示一下功能