search

gaogaotiantian / biliscope

Bilibili chrome extension to show uploader's stats
MIT License
575 stars 45 forks source link

Clean code #105

Closed F-park closed 9 months ago

F-park commented 9 months ago

整理代码所做的事

gaogaotiantian commented 9 months ago

在ui卡里的url不要动了，意义不大。直接的HTML还清楚一些。

JavaScript里的可以改掉。既然都改了，把constant.js里面的URL后面的/拿掉，变成https://space.bilibili.com，这样你就可以写成${BILIBILI_SPACE_URL}/${data['mid']}之类的，会好看一些。尽量不要在里面用加法，假设data['mid']不存在，这里会直接报错的，可能会影响程序的正常运行。如果不做运算，这里就是一个undefined，只是读不出来那个图而已。

F-park commented 9 months ago

在ui卡里的url不要动了，意义不大。直接的HTML还清楚一些。

JavaScript里的可以改掉。既然都改了，把constant.js里面的URL后面的/拿掉，变成https://space.bilibili.com，这样你就可以写成${BILIBILI_SPACE_URL}/${data['mid']}之类的，会好看一些。尽量不要在里面用加法，假设data['mid']不存在，这里会直接报错的，可能会影响程序的正常运行。如果不做运算，这里就是一个undefined，只是读不出来那个图而已。

谢谢天哥，学到了，已经改掉了。