gaogaotiantian / biliscope

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

Fix the issue of not accessing the last page of data #120

Closed noonisy closed 8 months ago

noonisy commented 8 months ago

两个 request 问题

具体更改

图一是修改前的情况

Snipaste_2024-01-03_19-58-00

图二是修改后的情况

Snipaste_2024-01-03_20-01-41

F-park commented 8 months ago

我觉得统一改成下面这样会好点,前面不使用变量 pn,页面参数固定为 1 就好了。pn 在 while 前初始化就好,减小变量作用域。

requestSearchPage(userId, 1, map).then((data) => { ... });
requestGuardPage(roomId, uid, 1, map).then((data) => { ... });
let pn = 1;
while (...) {
    pn += 1;
    ...
}

看了下天哥说的,这前面的判断确实也能去掉。

if (count > ... ) {
gaogaotiantian commented 8 months ago

是的,没必要让pn跨域,在local里去declare一个let pn = 1然后再做这个循环操作就行。外面那个if的check都可以省了。