lambda匿名函数使用中的坑

void RoleManager::selectRole(const std::string &roleUid, const CallbackFunc &callback)
{
    ValueMap opData;
    opData["op"] = "selectrole";
    opData["roleuid"] = roleUid;
    Remote::getInstance()->execForeGroundOp(opData, [&](int result, ValueMap& respVm)
    {
                if (result == 0)
                {
                    m_role.m_uid = respVm["roleuid"].asString();
                    if (m_role.readLocalRoleData(roleUid))
                    {
                       ......
                    }
                    else
                    {

                       ......               

                    }

    }

其中CallbackFunc的定义为:

 typedef std::function<void(int, ValueMap&)> CallbackFunc;



这里埋着一个大地雷,看到了吗,我足足调了一下午,log打了几十行,然后编译再编译。


roleUid是从外部传来的 引用,注意是引用,当execForeGroundOp被调用时,roleUid已经被外部修改了,或者它仅仅是存在栈里,走到execForeGroundOp函数时,那roleUid变量早就无效了。


正确的写法,应该使用roleUid的复制,而不是引用,改法如下:

void RoleManager::selectRole(const std::string &roleUid, const CallbackFunc &callback)
{
    ValueMap opData;
    opData["op"] = "selectrole";
    opData["roleuid"] = roleUid;
    Remote::getInstance()->execForeGroundOp(opData, [this,roleUid](int result, ValueMap& respVm)
    {
                if (result == 0)
                {
                    m_role.m_uid = respVm["roleuid"].asString();
                    if (m_role.readLocalRoleData(roleUid))
                    {
                       ......
                    }
                    else
                    {

                       ......               

                    }

    }


  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 打赏
    打赏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

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

打赏作者

峻峰飞阳

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

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

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

打赏作者

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

抵扣说明:

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

余额充值