同事的code这是在故意埋坑吗？

本人MLE，最近发现合作的base不在美国的同事写的code,里面有很多关于parameters 挺复杂的list套list的情况。 就比如，他喜欢把各种params 一层一层地套起来，这些params 都是数字，然后再加上各种运算。
类似于：

Param_level1 [ param_level2][param_level3]

后面再加一个运算：
[number * param_level2[idx] for param_level2 in param_level1]

关键是上下十多行完全没有comment 这些操作是干什么的。大概50行有一个comment 说这一整个chunk是干什么的。

我感觉这种代码读起来挺费劲的， 而且一层套一层完全也没有必要。再加上没有注释。我不知道这是因为我太菜了，还是他故意埋坑让人看不懂的。
注：我们组写code没有任何guideline， manager 也不检查， 也没有PR。 code 写好能跑就拿给engineering 去上线了。MLE的模型都是内部自己用的，不面向客户。
这种写code的style属于正常吗？以后我写code也要这样吗？

埋坑不至于。这种坑自己都可能爬不起来。

可能就是菜。不是每个人写代码都能想得清清楚楚。

你把代码复制给ai让他帮你讲讲在干嘛，然后让ai帮你refactor，然后你写个代码把它改了。不久你可能就被当成owner了。

Shadyupup 发表于 2025-1-25 15:32
你把代码复制给ai让他帮你讲讲在干嘛，然后让ai帮你refactor，然后你写个代码把它改了。不久你可能就被当成 ...

AI 总结和修改这种屎山代码确实有奇效

不正常，纯属就是习惯不好

不好维护啊，尽然没有PR，什么公司，那么牛皮，不互相检查approve？
我知道有的高手喜欢写的复杂，就是abstract层太多，你这个显然不是

CBP 发表于 2025-01-24 17:38:38
不好维护啊，尽然没有PR，什么公司，那么牛皮，不互相检查approve？
我知道有的高手喜欢写的复杂，就是abstract层太多，你这个显然不是

其他同组的project 都有正规的git code repo PR。 就这个项目没有。 project lead 不咋给力， 我觉得就是为了赶进度出成果，再加上上面的LT不过问。 其实整个DS/MLE的code我觉得写得都不规范，都不是专业的程序员，好多各种各样的PHD转行来的。

很有可能是学统计的没有CS背景的人写的。

这是高手👍
防御性编程

for in 这种一键生成list的写法只适合一层循环简单操作，这么写纯属恶心人