文章目录
- 1. 问题现象
- 2. 问题描述
- 3. 问题分析
- 4. 问题根因
- 5. 问题修复
- 6. 补充:类成员变量定义为引用类型
1. 问题现象
- 其中curr_lanes是一个目标上一帧的当前车道current_lanes_
- curr_lane是lane_id对应的LaneInfo信息
- 现象:在lane_info->lane().successor_id_size()或lane_info->lane().successor_id().size()为1的情况下,通过for循环去访问lane_info->lane().successor_id()的时候出现了coredump
2. 问题描述
-
(1) 问题必现,问题现象自相矛盾。size为1但是却无法访问,按理来说如果可迭代对象使用for循环访问,为空的情况是不会出问题的,for循环就跳过了
-
(2) 类Lane_info的结构,其中Lane是protobuf结构,如下
class LaneInfo { const Lane &lane_; // 所包含的lane的信息, 居然定义的是引用 { id_; length_; type_; turn_; successor_ids_; predecessor_ids_; … }; points; unit_directions_; segments_; total_length_; road_id; … };
3. 问题分析
- (1)怀疑Lane的proto实例创建时add_successor_id没有添加成功
- 检查发现添加方式没有问题,从地图收到的successor_id也不为空,正常填写的
- (2)怀疑LaneInfo实例没有创建成功
- 添加打印发现,LaneInfo创建过程未出现失败
- (3)怀疑是智能指针引用问题
- std::shared_ptr指针指向内容被异常释放,检查下来也没发现
- (4)怀疑是引用的原变量已经不在了,仍然尝试去访问
- 检查发现就是这个原因。LaneInfo类中的&lane_引用很关键
4. 问题根因
- 满足地图更新条件时,上一次的map_结果会被清理(智能指针管理);但是目标上一帧的current_lanes(类型是LaneInfo)仍去索引lane_信息
- 问题发生过程:
- (1) 第一次地图更新时,创建了static_map_->hdmap_->{map_, lanes_}的关联关系。其中lanes_结构:<lane_id, LaneInfo>
- (2)目标第一帧的current_lanes根据static_map_记录了当时的<lane_id, LaneInfo>信息,且由于是proto结构,LaneInfo中copy了&lane_的信息,对应的proto字段bit均已赋值,比如has_successor_id字段、succcessor_id_size字段等;本次访问没问题
- (3)第二次更新地图时,static_map_关联到了新的hdmap_,即static_map_->new_hdmap_->{map_, lanes_}的关联关系,此时lanes信息已完全变更
- (4)目标第二帧再去访问第一帧的current_lanes_发现lane proto的successor_id_size有效但是去访问successor_id的时候却找不到原变量,因此失败。即被引用对象已被删除
5. 问题修复
- (1)static_map_全程只关联一个hdmap_,每次地图更新也只更新该hdmap_。这里的static_map_是一个伪静态类
- (2)static_map_->hdmap->{map_, lanes_}中的lanes_不做清理,避免重复构建相同lane的LaneInfo,也避免在查询目标前后继车道的过程中出现异常,如目标已记录的successor_id在已更新的地图lanes_中却找不到
6. 补充:类成员变量定义为引用类型
- (1)C++初始化包括声明时即定义,或者在初始化列表中进行初始化。构造函数分为初始化列表和函数体
- (2)C++类成员变量定义为引用类型,不能用默认函数初始化,必须定义构造函数来初始化引用成员变量,否则会报引用未初始化错误
- (3)构造函数的形参也必须是引用类型,如LaneInfo(const Lane& lane) : lane_(lane) {}
防止引用绑定内存错误,在构造函数结束后所指向内存生命周期已结束,会有问题 - (4)不能在构造函数里初始化,必须在初始化列表中进行初始化
- (5)初始化后的引用变量所占用的内存空间和普通变量相同,可通过类的大小进行测试
【参考文章】
[1]. c++之类内定义引用成员
[2]. 引用变量所占用的内存空间
[3]. C++类成员变量定义为引用
created by shuaixio, 2024.02.18