FDUCSLG / pafd-automated

120 stars 1.51k forks source link

定位信息会多加一个 “上海市 ” #24

Open FY-Han opened 2 years ago

FY-Han commented 2 years ago

1c73a3b665c404e96c85e8fbedcb141

SunflowerAries commented 2 years ago

似乎没什么影响?我用现在 master 分支上的代码就正常填写?

FY-Han commented 2 years ago

似乎没什么影响?我用现在 master 分支上的代码就正常填写?

不过这不就一眼被老师发现有问题吗hh,如果细看的话,因为正常的话只有一个上海市

YangMindiDeMajia commented 2 years ago

在验证码没出时,针对原来的那个代码,我的处理是:

if province == "上海市" : self.last_info.update( { "tw" : "13", "province": "上海市", "city" : "上海市", "area" : " ".join(("上海市", "徐汇区")) } ) else: self.last_info.update( { "tw" : "13", "province": province, "city" : city, "area" : " ".join((province, city, district)) } )

lees2019 commented 2 years ago

这个能保证到定位是上海吗

ghost commented 2 years ago

今天刚用了一下 还是有两个上海市诶

YangMindiDeMajia commented 2 years ago

这个能保证到定位是上海吗

只要你上次打卡是在上海,即使是原来的程序也能保证。

YangMindiDeMajia commented 2 years ago

今天刚用了一下 还是有两个上海市诶

因为现在的代码并没有接受我这个建议;而且你也可以看见这会使得定位落在徐汇区而不是杨浦区,仅适合在枫林校区的小伙伴;如果你是别的校区的,可以稍微魔改一下代码,比如:

if province == "上海市" : self.last_info.update( { "tw" : "13", "province": "上海市", "city" : "上海市", "area" : " ".join((province, district)) } ) else: self.last_info.update( { "tw" : "13", "province": province, "city" : city, "area" : " ".join((province, city, district)) } )

简单地说,就是把字典中area的内容不再针对性地赋值,而是采用上一次的值赋值, 但我记得在早年验证码尚未出现时我针对早版本的自动打卡的代码的如此处理似乎并没有成功?反倒是采用

"area" : " ".join(("上海市", "徐汇区"))

成功了一段时间。

但主要事实证明辅导员或者别的什么人并不常看这玩意儿,所以也就无所谓了,还不如就用现在的大众版,这样如果出现什么问题也不需要我个人去修理。

ghost commented 2 years ago

今天刚用了一下 还是有两个上海市诶

因为现在的代码并没有接受我这个建议;而且你也可以看见这会使得定位落在徐汇区而不是杨浦区,仅适合在枫林校区的小伙伴;如果你是别的校区的,可以稍微魔改一下代码,比如:

if province == "上海市" : self.last_info.update( { "tw" : "13", "province": "上海市", "city" : "上海市", "area" : " ".join((province, district)) } ) else: self.last_info.update( { "tw" : "13", "province": province, "city" : city, "area" : " ".join((province, city, district)) } )

简单地说,就是把字典中area的内容不再针对性地赋值,而是采用上一次的值赋值, 但我记得在早年验证码尚未出现时我针对早版本的自动打卡的代码的如此处理似乎并没有成功?反倒是采用

"area" : " ".join(("上海市", "徐汇区"))

成功了一段时间。

但主要事实证明辅导员或者别的什么人并不常看这玩意儿,所以也就无所谓了,还不如就用现在的大众版,这样如果出现什么问题也不需要我个人去修理。

谢谢谢谢!!已经改成功了