Alpaca4610
commented
1 year ago 首先非常感谢你做出的插件，我最近一直有在使用它
我查看了你的代码，想提几点不成熟的小建议 我注意到，为了实现私聊功能，你加入了
if plugin_config.enable_private_chat:来启用针对私聊的处理，但是绝大部分的代码都是重复的，并且在nonebot初始化插件导入时会因为相同的触发命令而报几个warning。以下是我的几个修改意见：
第一种方法，你可以将
handler中的event:替换为MessageEvent，随后使用if isinstance(event, GroupMessageEvent):和if isinstance(event, PrivateMessageEvent):来判断当前消息是私聊还是群聊；第二种方法，你可以为相同的功能只保留一个
matcher，但是你可以写多个@matcher.handle匹配不同的类型。 希望这些对你有帮助，非常感谢你对nonebot社区的贡献。
好的，感谢大佬的建议！
QNLanYang
这样写真的可以吗，好像不是挺常规🤔（比较少见）前面不带上下文的群聊命令和这个不会出现冲突吗？
首先非常感谢你做出的插件，我最近一直有在使用它
我查看了你的代码，想提几点不成熟的小建议 我注意到，为了实现私聊功能，你加入了
if plugin_config.enable_private_chat:来启用针对私聊的处理，但是绝大部分的代码都是重复的，并且在nonebot初始化插件导入时会因为相同的触发命令而报几个warning。以下是我的几个修改意见：
第一种方法，你可以将
handler中的event:替换为MessageEvent，随后使用if isinstance(event, GroupMessageEvent):和if isinstance(event, PrivateMessageEvent):来判断当前消息是私聊还是群聊；第二种方法，你可以为相同的功能只保留一个
matcher，但是你可以写多个@matcher.handle匹配不同的类型。 希望这些对你有帮助，非常感谢你对nonebot社区的贡献。