std::map循环遍历删除元素出现诡异现象

	std::map<int, int,cmp> m1{ { 1, 1 }, { 3, 3 }, { 2, 2 } };
	m1.insert(std::make_pair(4, 4));

	for (auto it = m1.begin(); it != m1.end();)
	{
		if (it->first == 3 || it->first == 2)
			m1.erase(it);
		else
			++it;
	}

这段代码在运行的时候会出现if语句执行完继续执行else语句。为什么会出现这种现象?

先来看一下map中的erase函数:

可以看到erase并没有返回值,不像vector一样会erase会有返回值,表示被删除元素下一个元素的位置。

分析上述诡异现象:当元素是2或者3的时候erase(it)会导致it迭代器失效,但是没有更新迭代器,而是继续继续判断else语句成立所以会继续执行++it,导致最终程序奔溃。

解决上述问题:如下代码

    std::map<int, int,cmp> m1{ { 1, 1 }, { 3, 3 }, { 2, 2 } };
	m1.insert(std::make_pair(4, 4));

	for (auto it = m1.begin(); it != m1.end();)
	{
		if (it->first == 3 || it->first == 2)
			m1.erase(it++);
		else
			++it;
	}
	
	for (auto it : m1)
		std::cout << it.first << " " << it.second << "\n";

在上述正确代码中erase参数是it++,因为it再给erase传参之后,在迭代器没有失效之前进行了++操作指向下一个元素,所以这种能够正确删除元素。或者可以定义一个临时变量。如下:

	std::map<int, int,cmp> m1{ { 1, 1 }, { 3, 3 }, { 2, 2 } };
	m1.insert(std::make_pair(4, 4));

	for (auto it = m1.begin(); it != m1.end();)
	{
		auto tmp = it;
		++tmp;
		if (it->first == 3 || it->first == 2)
		{
			m1.erase(it);
			it = tmp;
		}
		else
			++it;
	}
	
	for (auto it : m1)
		std::cout << it.first << " " << it.second << "\n";

注:不可以是前置++

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 打赏
    打赏
  • 2
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论 2
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包

打赏作者

weixin_41318405

你的鼓励将是我创作的最大动力

¥1 ¥2 ¥4 ¥6 ¥10 ¥20
扫码支付:¥1
获取中
扫码支付

您的余额不足,请更换扫码支付或充值

打赏作者

实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值