一时粗心罢了. 证明了静态类型的必要性. 这玩意不仅能过编译, 测试的时候也难发现bug
有些动态语言有类型标注的话,也能在编译的时候检测出来这种 bug
这个错误应该很常见吧,不论大厂小厂,想起前不久一个新闻,腾讯云sdk20万个发行版把GitHub搞崩溃了,https://www.toutiao.com/w/1832969904905292/ ,大厂的操作有时让人看不懂
都用python3了,为什么不加类型标注呢
类型标注解决不了问题的, 非常鸡肋. 只有像 typescript 那样用泛型才行. 类型标注 type_a | type_b
, 一层就是两种类型, 两层就是四种类型的组合, 多套几层看你怎么处理, 约束不了一点
一副高高在上的口气 各种质问 真看不懂这位兄弟的言行
其实这个错误本质上是,从不信任的、非结构化数据(环境变量),到信任的、结构化数据(程序的 config),必须 parsing. 任何 os.getenv() 拿到值就直接使用的代码都是危险的,只是有时候他们可以运行。应该用现成的库解决这类 config parsing 的问题
任何用户给的数据都不可信 嘿嘿
只有一行的commit,非常明显的错误,典型的不懂装懂凑commit数的。真给intel这个商标抹黑。
不在乎自己的错误,只在乎别人的说话语气,呵呵。
对于这种工具类的东西,使用“约定”比“兼容异常”更有效率。就那个项目而言(纯粹实验性项目),没必要检查安全性,使用者自负就行了。只是,这个代码质量,让我不想用了。太LOW,LOW爆了。
楼里大家说的没错,确实算不上什么大问题,很常见的疏忽。既体现不了作者的水平,也体现不了揪了这个毛病的楼主的水平。楼主要鄙视大厂员工,需要拿一点更硬的成就出来。
PR review 就事论事就好. 和是不是大厂员工没有任何关系.
有没有可能是对大厂有PTSD
n年前,金山的人说过这么一句话:这里是金山只用svn,不用git。 现在呢,只要动力,不要安全!
大厂应该有比较严格的代码审查机制,以这个例子来说,什么都没有。
1)改了代码,没有启动一下, 2)没人code review,说明项目没人在乎。
我研究了一下,这个项目其实比较鸡肋。比如:ollama llama.cpp 都尝试增加了对intel 卡的支持。这个项目这种状态,应该标记为 archive,不应该再当成正常项目了。
intel可不是普通的厂家,是这些年来IT发展的基石。他们的项目也能这样,说明什么?暮气很重了。还是说中国人主导的项目才这样?
當大廠用愛發電?大廠願意弄 open source 項目就偷笑了。愛用就用,覺得不喜歡就發 PR。
python3类型标注 支持泛型 也支持procotol 很强大呀,有好多种实现方式解决你说的这种问题。
我知道支持泛型, 但是楼上又没提. 而且大部分场合大部分用户最多写个类型标注