gaogaotiantian / biliscope

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

Remove unnecessary cache function #234

Closed F-park closed 1 week ago

F-park commented 2 weeks ago

函数定义离得太远,所以将代码内联并更新了数组内容。

gaogaotiantian commented 1 week ago

这里不是一个逻辑的问题,是一个可读性的问题。cache && ["stat", "info", "relation", "count"].every((key) => cache[key])是一个不太容易读懂的逻辑,过了一阵之后你再看这段代码你会完全不知道这里是要干什么。所以保留一层抽象是非常有必要的。如果未来对cache进行了修改,或者cache必要的内容又有了变化,就可以在这个抽象里继续完成,而不是不断增大这个if statement里的内容。