错误:在只读对象中分配成员

error: assignment of member in read-only object

本文关键字:分配 成员 对象 只读 错误      更新时间:2023-10-16

IDEONE: http://ideone.com/uSqSq7

#include <iostream>
#include <algorithm>
#include <set>
#include <vector>
using namespace std;
struct node
{
    int value, position;
    bool left, right;
    bool operator < (const node& a) const
    {
        return value < a.value;
    }
};
int main()
{
    int n;
    cin >> n;
    vector < node > a(n);
    set < node > s;
    for (auto &i: a)
    {
        cin >> i.value;
        i.left=i.right=0;
    }
    a[0].position=1;
    s.insert(a[0]);
    for (int i=1; i<n; i++)
    {
        auto it=s.upper_bound(a[i]);
        auto it2=it; --it2;
        if (it==s.begin())
        {
            a[i].position=2*it->position;
            s.insert(a[i]);
            it->left=1;
        }
        else if (it==s.end())
        {
            a[i].position=2*(--it)->position+1;
            s.insert(a[i]);
            it->right=1;
        }
        else
        {
            if (it2->right==0)
            {
                a[i].position=2*it2->position+1;
                s.insert(a[i]);
                it2->right=1;
            }
            else
            {
                a[i].position=2*it->position;
                s.insert(a[i]);
                it->left=1;
            }
        }
    }
    for (auto i: a) cout << i.position << ' ';
}

当我编译此代码时,我得到

error: assignment of member ‘node::right’ in read-only object

我认为这与bool operator <中的const有关,但我无法摆脱它,因为有必要创建集合。

Angelika Langer曾经写过一篇关于这个问题的文章:Set Iterators是可变的还是不可变的?。

您可以通过将set排序无关紧要的Node成员定义为mutable来解决此问题:

mutable bool left, right;

(请参阅IDEone中的建筑版本。

就个人而言,我会考虑使用map将不可变部分映射到可变部分的设计。

问题:

请记住,std::set 中的键是常量。将密钥插入密钥集后,无法更改密钥。因此,当您取消引用迭代器时,它必然返回一个常量引用。

const node& n = (*it);
n->left = 1; //It will not allow you to change as n is const &.

溶液:

正如Ami Tavory回答的那样,你可以将左和右声明为可变的。