Diving-Fish / maimaidx-prober

舞萌 DX 查分器
https://www.diving-fish.com/maimaidx/prober
MIT License
681 stars 52 forks source link

没有针对网络异常做合适的错误处理 #86

Closed Janet-Baker closed 1 year ago

Janet-Baker commented 1 year ago

众所周知,新版更新后服务器一直是炸的,查看成绩页面巨卡。

然后查分也炸了。

连接异常可以尝试重试或者跳过当前难度,但是不应该挂掉啊。

登录成功,代理已开启到127.0.0.1:8033
自动修改代理设置失败。请尝试手动修改代理。
使用此软件则表示您同意共享您在微信公众号舞萌 DX、中二节奏中的数据。
您可以在微信客户端访问微信公众号舞萌 DX、中二节奏的个人信息主页进行分数导入,如需退出请直接关闭程序或按下 Ctrl + C
正在导入 Basic 难度……2023/06/23 17:52:21 [004] WARN: Error copying to client: readfrom tcp 127.0.0.1:8033->127.0.0.1:56590: read tcp 192.168.31.20:56591->202.89.233.100:443: wsarecv: An existing connection was forcibly closed by the remote host.
2023/06/23 17:52:50 [008] WARN: Error copying to client: readfrom tcp 192.168.31.20:56612->124.251.34.42:443: read tcp 127.0.0.1:8033->127.0.0.1:56611: wsarecv: An existing connection was forcibly closed by the remote host.
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x40 pc=0x12cc1e5]

goroutine 51 [running]:
main.commit({0x13e63e0?, 0xc00019e080?})
        C:/Users/DivingFish/Github/maimaidx-prober/proxy/cmd/maimaidx-prober-proxy/main.go:71 +0x65
main.fetchData(0xc00015c400, {0xc0001a6018, 0x1, 0x1})
        C:/Users/DivingFish/Github/maimaidx-prober/proxy/cmd/maimaidx-prober-proxy/main.go:102 +0x5e5
created by main.main.func2
        C:/Users/DivingFish/Github/maimaidx-prober/proxy/cmd/maimaidx-prober-proxy/main.go:223 +0x28f
Janet-Baker commented 1 year ago

https://pic.imgdb.cn/item/649595571ddac507cc0d899e.png

resp就是个nil,还往里塞呢……

根本没做错误处理……旧版本华立的服务端还好,新版华立服务端太差劲差劲,不做错误处理的后果就凸显出来了。

Janet-Baker commented 1 year ago

用 chrome android inspect 试了一下,华立网太差,单个难度加载了六分多钟……