search

AKitavtsev / telegram-vk-bot

BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

InitSession as parameter for LoopBot #13 #25

Closed AKitavtsev closed 3 years ago

olegromashin commented 3 years ago

Я может где-то недопонял архитектуру, но я немного по-другому обращаться к botInit предлагал. Перед запуском loopBot из main запускать initSession как было до этого. В самой функции loopBot не запускать botInit, а передавать это действие дальше в другие функции хэндла. В функциях хэндла, если в testException будет ошибка, вызывать botInit.

AKitavtsev commented 3 years ago

Я предлагаю утвердить мою редакцию InitSession. Я много думал о testException и пришел к выводу, что она должна обходиться без InitSession. А InitSession необходима для другого случая - когда мы получаем респонс с ошибкой. но эта уже другая ситуация. и она почти решена. Предлагаю влить ветку Issue13, заодно 17 (там просто не о чем). Я открою новую ветку. И ее рамках закончу вопросы с InitSession и testException OK?

olegromashin commented 3 years ago

Предлагаю влить ветку Issue13, заодно 17

Но в том виде как оно сделано в этом ПРе, initSession будет вызываться каждый раз в случае ВК, а это не то, что нам надо, разве нет?

Про 17 надо, чтобы Кирилл посмотрел. Кстати имена веткам тоже надо по смыслу изменений давать :)

AKitavtsev commented 3 years ago

Или не понял, или "разве нет" . Я же передаю в loopBot в качестве параметра InitSession (вместе с ее параметрами) только тогда, когда действительно нужна перезагрузка сессии (например, при старте, или когда закончился срок действия ключа или когда невозможно распарсить апдейт). В остальных случаях (нормальный цикл, да , думаю, и после Exeption передаю pure ()

olegromashin commented 3 years ago

А, вижу, да. Тогда твой вариант тоже хороший.