GraiaCommunity / Docs

Graia 社区文档
https://graiax.cn/
Other
21 stars 8 forks source link

Add AldotaiBot #67

Closed kaixinol closed 1 year ago

kaixinol commented 1 year ago

Check List

假设修改的是 Markdown

你添加/修改/删除了啥

add my bot

Redlnn commented 1 year ago

你的 Bot 存在以下问题,不够 awesome:

  1. 不应更改 sys.path,且你在多个地方重复应用了 sys.path.append("../")
  2. 数据库操作没有使用 ORM,也没有检查是否被注入,存在安全风险
  3. 无意义的变量名,如:x = SqlLink(...)
  4. loguru.logger.xxx(...) -> logger.xxx(...),应使用 from loguru import logger
  5. 读写设置不够优雅,可以试试 Kayaku
  6. 代码格式化及导入优化未完成
  7. print(...) 未删除干净,运行时不应使用 print 输出 debug 信息
  8. 请不要硬编码 Bot QQ号、VerifyKey 等隐私信息,包含隐私的配置文件请不要上传至仓库
  9. 使用 Alconna 解析消息时,请优先考虑 Alconna-Graia 替代原先写法
  10. 请尽量避免使用 evalexec
RF-Tar-Railt commented 1 year ago

追加一点: 考虑代码内使用 Alconna 作为消息解析,请优先考虑 Alconna-Graia 替代原先写法

kaixinol commented 1 year ago

我会尽快更改不符合awesome标准的代码,请暂时不要关闭此PR

Redlnn commented 1 year ago

SQLAlchemy 未使用异步写法,建议改进,可以参考社区其他 bot 的实现