jixunmoe
5 Followers · 0 Following
  • Joined on 2022-04-27
Repositories
2
 Projects Packages Public Activity Starred Repositories
jixunmoe commented on pull request um/web#5 2022-11-21 00:59:30 +00:00
Add Tag Edit Function & Wasm for Qmc & Kgm

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

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

jixunmoe 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 接收这个值的时候做类型转换。

jixunmoe commented on pull request um/web#5 2022-11-21 00:57:52 +00:00
Add Tag Edit Function & Wasm for Qmc & Kgm

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

jixunmoe commented on pull request um/web#5 2022-11-20 15:46:27 +00:00
Add Tag Edit Function & Wasm for Qmc & Kgm
  • 不需要 == undefined 部分
jixunmoe commented on pull request um/web#5 2022-11-20 15:46:27 +00:00
Add Tag Edit Function & Wasm for Qmc & Kgm

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

jixunmoe commented on pull request um/web#5 2022-11-20 15:46:27 +00:00
Add Tag Edit Function & Wasm for Qmc & Kgm

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

jixunmoe 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 也许会好一点。

jixunmoe commented on pull request um/web#5 2022-11-20 15:46:27 +00:00
Add Tag Edit Function & Wasm for Qmc & Kgm

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

jixunmoe commented on pull request um/web#5 2022-11-20 15:46:27 +00:00
Add Tag Edit Function & Wasm for Qmc & Kgm

没有 v2

jixunmoe 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

jixunmoe commented on pull request um/web#5 2022-11-20 15:46:27 +00:00
Add Tag Edit Function & Wasm for Qmc & Kgm

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

jixunmoe 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
jixunmoe commented on pull request um/web#5 2022-11-20 15:46:27 +00:00
Add Tag Edit Function & Wasm for Qmc & Kgm

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

jixunmoe commented on pull request um/web#5 2022-11-20 15:46:27 +00:00
Add Tag Edit Function & Wasm for Qmc & Kgm
jixunmoe suggested changes for um/web#5 2022-11-20 15:46:27 +00:00
Add Tag Edit Function & Wasm for Qmc & Kgm
jixunmoe commented on pull request um/web#5 2022-11-20 15:46:26 +00:00
Add Tag Edit Function & Wasm for Qmc & Kgm

同上，

jixunmoe commented on pull request um/web#5 2022-11-20 15:46:26 +00:00
Add Tag Edit Function & Wasm for Qmc & Kgm
jixunmoe commented on pull request um/web#5 2022-11-20 15:46:26 +00:00
Add Tag Edit Function & Wasm for Qmc & Kgm

no error 应为 unknown error。

jixunmoe commented on pull request um/web#5 2022-11-20 15:46:26 +00:00
Add Tag Edit Function & Wasm for Qmc & Kgm

reserve => preserve？

jixunmoe commented on pull request um/web#5 2022-11-20 15:46:26 +00:00
Add Tag Edit Function & Wasm for Qmc & Kgm

可以将 frame.id 和 frame.value 都打印到 console，然后报错