Skip to content
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

音质选择有问题 #1588

Closed
JingBh opened this issue May 1, 2022 · 4 comments · Fixed by #1589
Closed

音质选择有问题 #1588

JingBh opened this issue May 1, 2022 · 4 comments · Fixed by #1589
Labels
bug Something isn't working fixed 已修复但是还没发布新版本

Comments

@JingBh
Copy link

JingBh commented May 1, 2022

#1585 新增对 Hi-Res 音质的支持后,当设置中选择 FLAC 音质,而所播放歌曲没有此音质时,将无法播放。

根据此段代码:

/**
* 获取音乐 url
* 说明 : 使用歌单详情接口后 , 能得到的音乐的 id, 但不能得到的音乐 url, 调用此接口, 传入的音乐 id( 可多个 , 用逗号隔开 ), 可以获取对应的音乐的 url,
* !!!未登录状态返回试听片段(返回字段包含被截取的正常歌曲的开始时间和结束时间)
* @param {string} id - 音乐的 id,例如 id=405998841,33894312
* @param {string} sqBr - flac(SQ)的比特率
* @param {string} hiResBr - hi-res的比特率
*/
export function getMP3(id, sqBr, hiResBr) {
const getBr = (quality, sqBr, hiResBr) => {
if (quality === undefined) return 320000;
if (quality === 'flac' && sqBr) return sqBr;
if (quality === '999000' && hiResBr) return hiResBr;
return quality;
};

qualityflacsqBrundefined,将返回 flac,不是合法参数,导致 NeteaseCloudMusicApi 将返回 400 错误,如图所示。

image
image

@memorydream
Copy link
Collaborator

memorydream commented May 1, 2022

我已经重写了音质选择器,正在进行一些测试
#1585 的修改是一个兼容性修改,选用hi-res即可回到之前的行为,暂时不要选择flac即可,当hi-res音源不可用时,会匹配到flac音源

@memorydream memorydream added the bug Something isn't working label May 1, 2022
@JingBh
Copy link
Author

JingBh commented May 1, 2022

Hi-Res 文件太大,且个人没有对这么高音质的需求。

@JingBh
Copy link
Author

JingBh commented May 1, 2022

另外:这样有问题的行为是不是不应该草率地 release 到新版本?

@memorydream
Copy link
Collaborator

memorydream commented May 1, 2022

另外:这样有问题的行为是不是不应该草率地 release 到新版本?

因为我没有测试出这样的问题,而且上一个版本存在着更多不可接受的bug,修复了这些真正最影响软件使用的bug后,我们认为现在已经是时候发布新的版本了

这个音质选择的问题确实是我没有写好,但是考虑到选择hi-res会回到之前的行为(即如果没有这个支持hi-res的pr,当在音质里选择无损时,会优先使用hi-res音频),在我们发新版本之前应该不会造成过大的问题

@memorydream memorydream added the fixed 已修复但是还没发布新版本 label May 1, 2022
@memorydream memorydream reopened this May 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed 已修复但是还没发布新版本
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants