Question

我必须重构看起来像这样的旧代码（我不是一个非常熟练的C ++编码器）

 std::set<SomeObject>::iterator it = setobject.begin();

 do {
     it->setProperty1ToNextValue();
     it->getProperty2();
     it->getProperty3();
     it++ 
 } while (it != setobject.end());

基本上我想迭代集合的元素并获取和设置/更新它们的一些属性。我不能使用原始集，因为我遇到了这个线程中描述的问题 the object has type qualifiers that are not compatible with the member function object type is const

我正在考虑用dequeue替换set（它将涉及一些重写），因为我将能够在dequeue的每个元素上设置和获取属性。这是一个好方法吗？

Answer 1

std::set的工作原理是根据对象的<运算符保持所有项目的顺序。您无法在对象上调用非const方法的原因是因为这些方法可能会更改对象<运算符返回的值，从而有效地重新排序＆＃34;没有std::set知道的引擎盖下的集合。

虽然您还没有指定足够的内容来为我们提供最佳答案，但这里有几种技术方法可以在您的设备上调用某些方法。您可以使用const_cast来调用您确定无法修改密钥的方法。或者您可以将项目放在矢量中，调用可能会修改＆＃34;键＆＃34;的方法，然后将它们放回原始集合中。

// Example program
#include <iostream>
#include <string>
#include <set>
#include <algorithm>

class SomeObject
{
    std::string key;
    int         data;

public:
    SomeObject( const std::string& key_, int data_ ) : key( key_ ), data( data_ )
    {}

    // For a item to be in a set, it must have a "<" operator that tells it how to order it
    bool operator <( const SomeObject& rhs ) const
    {
        return key < rhs.key;   
    }

    void setKey( const std::string& key_ )
    {
        key = key_;   
    }

    void setData( int data_ )
    {
        data = data_;   
    }
};

int main()
{   
     std::set< SomeObject > setobject;
     setobject.insert( SomeObject("c", 1 ) );
     setobject.insert( SomeObject("a", 1 ) );
     setobject.insert( SomeObject("b", 1 ) );

     // internally, the set will keep everything in order "a", "b", "c"

     // option 1 - use const_cast (risky!)
     {
         std::set< SomeObject >::iterator it = setobject.begin();

         do {
             // const_cast< SomeObject& >( *it ).setKey( "d" );  bad idea, now the set is jacked up because its not in the right order
             const_cast< SomeObject& >( *it ).setData( 2 );
             it++;
         } while (it != setobject.end());
     }

     // option 2 - put the items in the vector, call the methods, then put them back in the original set
     {
         std::vector< SomeObject > tempVec( std::begin( setobject ), std::end( setobject ) );
         std::vector< SomeObject >::iterator it = tempVec.begin();
         do {
             it->setKey( "d" ); 
             it->setData( 2 );
             it++;
         } while (it != tempVec.end());

         std::set< SomeObject > newSet( std::begin( tempVec ), std::end( tempVec ) );
         std::swap( newSet, setobject );  // put the new items back in the original setobject
     }



}

在C ++中替换集合的最佳方法

1 个答案: