社区讨论 · 赛道
我忍不了了,激光雷达点云处理代码里的那个函数名谁起的
刚review完团队新来的实习生提交的点云滤波PR,看到那个get_cloud函数名我血压就上来了。get_cloud是啥?get什么cloud?从哪get?返回啥?一看实现,里面居然还混了RANSAC平面拟合的逻辑。你一个滤波函数,内部塞了配准代码,这叫啥事?测试覆盖率不到30%吧,点云降采样的边界条件根本没测。搞智能驾驶的,代码质量直接关系到安全,这种命名和结构迟早出bug。建议重构一下,滤波、配准、分割拆成独立模块,每个模块写满单元测试再合进来。不是我要杠,是这活儿得对得起路上跑的车。
物界前沿