warning: ‘operator=‘ should check for assignment to self to avoid problems with dynamic memory

看似合理 operator= 的实现,要是出现 assignment to self(自赋值)则是不安全的。(它也不是 exception-safe(异常安全)。

防止这个错误的传统方法是在 operator= 的开始处通过 identity test(一致性检测)来阻止 assignment to self(自赋值)。

    SmallString &SmallString::operator=(const SmallString &s)    {
        Reset();
        unicode_ = s.unicode_;
        if (s.name_)        {
            size_t len = strlen(s.name_);
            name_ = new char[len + 1];
            strcpy(name_, s.name_);  // NOLINT
        }
        if (s.wname_)        {
            size_t len = wcslen(s.wname_);
            wname_ = new wchar_t[len + 1];
            wcscpy(wname_, s.wname_);
        }
        return *this;
    }

修改为:

    SmallString &SmallString::operator=(const SmallString &s)    {
        if (this == &s)
            return *this;

        Reset();
        unicode_ = s.unicode_;
        if (s.name_)        {
            size_t len = strlen(s.name_);
            name_ = new char[len + 1];
            strcpy(name_, s.name_);  // NOLINT
        }
        if (s.wname_)        {
            size_t len = wcslen(s.wname_);
            wname_ = new wchar_t[len + 1];
            wcscpy(wname_, s.wname_);
        }
        return *this;
    }

  • 3
    点赞
  • 1
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

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

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

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

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

余额充值