V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX  ›  pf94  ›  全部回复第 1 页 / 共 1 页
回复总数  4
这个代码在我这里是不合格的,从上往下说

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 检查
德国留学只用英语还行,找工作不会德语,那你的选择范围就只限大城市,情况和国内北漂差不多,月光存不了钱,通勤时间长,只是不加班而已。
我在德国和德国人合伙开的 IT 公司,我们经历了各人种的正式员工、实习生,结论就是不会德语的坚决不招。真想在小国家找英语工作躺平北欧或者荷兰最好。
117 天前
回复了 dododada 创建的主题 程序员 算法工程师的工程能力问题
我在我们公司做的范围比这个还广,有 20%股份。
在生产层面有这个技术广度的不可能甘心当“算法工程师”的,也不希望去大厂,因为会局限到某几项技能。我想做的是创造自己的产品。
河西给河西打的概率:3/10 * 3/10 = 9/100
河东给河东打的概率:7/10 * 7/10 = 49/100
跨河概率:100/100 - 9/100 - 49/100 = 41/100

大概 41 万
关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   5157 人在线   最高记录 6679   ·     Select Language
创意工作者们的社区
World is powered by solitude
VERSION: 3.9.8.5 · 15ms · UTC 07:35 · PVG 15:35 · LAX 00:35 · JFK 03:35
Developed with CodeLauncher
♥ Do have faith in what you're doing.