vue-music-webapp icon indicating copy to clipboard operation
vue-music-webapp copied to clipboard

Published 20 hours ago • verified publisher icon caijinyc

几点改进的建议

Open hugetiny opened this issue 5 years ago • 1 comments

用户体验很好啊，居然是自己不靠UI框架写出来的，不错。 关于几点改进的建议 1.axios promise后面最好跟一下catch捕获错误，then里面只能打印请求成功但非200的错误信息。 2.既然用了Vuex，那么异步方法全部写到action里面去，用生命周期控制。尽量少用ref,props等，保持代码整洁。 3.一些接口有点变化，比如newsong，lyric。 我也试试大改一下你的代码

hugetiny avatar Nov 15 '19 12:11 hugetiny

还有我发现你不太用computed 一些异步的state应该用computed+mapstate自动刷新更新比较好

hugetiny avatar Nov 19 '19 00:11 hugetiny