search

wdvxdr1123 / ZeroBot

一个基于onebot协议的机器人Go开发框架
GNU General Public License v3.0
326 stars 71 forks source link

postHandler类型改为[]Rule，使功能与描述一致 #63

Closed yuanyan3060 closed 2 years ago

fumiama commented 2 years ago

我觉得在post加打断意义不大，因为post本身没有任何sort

yuanyan3060 commented 2 years ago

prehandler也没有sort，我觉得没有任何理由使两者不一致

fumiama commented 2 years ago

prehandler的匹配结果是否为真将会影响后续matcher是否被执行。posthandler不具备此功能，因为matcher匹配已经结束。

wdvxdr1123 commented 2 years ago

Post应该对应go中的defer， 我觉得不该打断

fumiama commented 2 years ago

Post应该对应go中的defer， 我觉得不该打断

是的，打断位于未知顺序的其他posthandler将会产生难以预料的后果。比如修改中的 https://github.com/wdvxdr1123/ZeroBot/blob/159f950857b03a468e01643ee03aab91d8176d77/extension/single/single.go#L59-L62 如果在他执行前被打断，反并发将永远死锁。

yuanyan3060 commented 2 years ago

那看来需要修改的应该是UsePostHandler的注释了

fumiama commented 2 years ago

那看来需要修改的应该是UsePostHandler的注释了

估计是个复制问题2333