pf94
2024-08-13 05:22:56 +08:00
这个代码在我这里是不合格的,从上往下说
1. import 顺序问题,前面已经有人说了;其他还有空格等格式问题下面就不提了
2. 30-52 行,无意义的字典定义,并且缩进有问题
3. 初始化全局变量,这个设计问题很大,导致大部分函数有副作用
4. get_information:第一个带副作用的函数,58 、60-63 行全角半角字符混用(同样的问题后面不提了)
5. 85 行:注释风格不统一,58 行的在函数定义前一行,参数空格问题
6. 115 行:注释错误,并不能 catch 不是图片的情况,因为 try 里面非图片情况没有报错而是直接返回 False
7. 139 行:经典整个 if 作为一个函数,没有处理 else 情况,if message[history_id]["history_content"][-1]["role"] != "user": ...可以让整个函数少一层缩进
8. 177 行:意义不明的变量命名
9. 181 行:pic_url 命名风格不统一,information, history 等都是整个单词,省略 picture 毫无意义
10. 189 行:循环外使用循环内定义的变量
中等问题是代码风格不统一,大问题是大量使用副作用函数
建议加上 type hint ,多用 black 和 mypy 检查