鲁树人 - Unlock Music Git Service

lsr commented on pull request um/web#5

2022-11-21 19:01:58 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

确实会坏掉，请忽略之前的建议。

// 文件顶部定义
const fromGBK = (text) => iconv.decode(new Buffer(text

lsr created pull request um/web#6

2022-11-21 18:16:32 +00:00

优化 QMCCache 解密过程

lsr commented on pull request um/web#5

2022-11-21 18:09:48 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

也行

lsr commented on pull request um/web#5

2022-11-21 01:02:13 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

但这样的话不应该是捕捉出错信息并提示用户吗。

我看上面还有个 try/catch 块，但是错误直接吞掉了。

lsr commented on pull request um/web#5

2022-11-21 01:00:02 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

也行

lsr commented on pull request um/web#5

2022-11-21 00:59:51 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

参见上面的那条回复

lsr commented on pull request um/web#5

2022-11-21 00:59:30 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

直接重构吧，不然很多代码都要字符串、数组转来转去。

明明是同一个项目里的代码…

lsr commented on pull request um/web#5

2022-11-21 00:58:47 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

建议改一下 wasm 的返回值或 js 接收这个值的时候做类型转换。

lsr commented on pull request um/web#5

2022-11-21 00:57:52 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

但是要注意这两个实现不一致。遇到报告错误的时候，排查问题会很麻烦。

lsr commented on pull request um/web#5

2022-11-20 15:46:27 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

不需要 == undefined 部分

lsr commented on pull request um/web#5

2022-11-20 15:46:27 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

为什么一开始不传入一个整数值呢

lsr commented on pull request um/web#5

2022-11-20 15:46:27 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

同上，为什么一开始就不使用字符串数组来储存艺术家信息？

lsr commented on pull request um/web#5

2022-11-20 15:46:27 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

picture 换成 albumCover 也许会好一点。

lsr commented on pull request um/web#5

2022-11-20 15:46:27 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

这个提示应该没有必要，因为用户可以看到预览更新？

lsr commented on pull request um/web#5

2022-11-20 15:46:27 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

没有 v2

lsr commented on pull request um/web#5

2022-11-20 15:46:27 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

no error 应为 unknown error

lsr commented on pull request um/web#5

2022-11-20 15:46:27 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

还是动态生成 mask 的方案… 直接删掉这部分也许会更好？

lsr commented on pull request um/web#5

2022-11-20 15:46:27 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

musicMeta.common.artists == undefined ? musicMeta.common.artist : musicMeta.common.artists.toString() 可以替换为 `String(musicMeta.common.artists

lsr commented on pull request um/web#5

2022-11-20 15:46:27 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm

未使用的变量，后面直接内嵌了

lsr commented on pull request um/web#5

2022-11-20 15:46:27 +00:00

Add Tag Edit Function & Wasm for Qmc & Kgm