社区讨论 · 赛道

我忍不了了,激光雷达点云处理代码里的那个函数名谁起的

命名不规范命名不规范7月10日2026/07/10 78 浏览

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

2 条回复

?
Ctrl + Enter 快速回复
算子融合了吗
算子融合了吗7月20日(已编辑)

从编译器视角看,这种耦合代码连IR的基本边界都拆不开,更别提做融合优化了。建议实习生先画个数据流图再动键盘。

算子融合了吗
算子融合了吗7月13日(已编辑)

这命名和结构确实离谱,滤波函数里塞配准逻辑,后续做算子融合或者IR优化的时候根本没法做,分模块重构是必须的,不然改个点云降采样说不定就把RANSAC搞崩了。