1

文字列をキーとして、ベクトルを値として持つ 3 つのマップがあります。

std::map<std::string, std::vector<int> >          m_attri;   ///< Attributes of type int
std::map<std::string, std::vector<double> >       m_attrd;   ///< Attributes of type double
std::map<std::string, std::vector<std::string> >  m_attrs;   ///< Attributes of type std::string

それらすべてのサイズを変更する関数があり、 SetNum() を呼び出します

void SetNum(size_t n)
{
    std::cout << "setting num: " << n << std::endl;
    m_num = n;
    for (auto attr : m_attri) {
        attr.second.resize(n, 0);
        std::cout<<attr.second.size();
    }
    for (auto attr : m_attrd) {
        attr.second.resize(n, 0.0);
        std::cout<<attr.second.size();
    }
    for (auto attr : m_attrs) {
        attr.second.resize(n, "");
        std::cout<<attr.second.size();
    }
    std::cout<<std::endl;
}

これはすべて機能しているように見えます。setnum(1) と言うと、出力として 1 の束が得られます。

問題はここにあります。 Set という別の関数があります

template<typename T>
inline void AttributeContainer::Set(size_t node_idx, const std::string& name, T value)
{
    std::cout << GetNum() << node_idx << ',' << std::endl;
    if( node_idx >= GetNum()){
        SetNum(node_idx+1);
    }
    auto attr_kv = Attributes<T>().find(name);
    if (attr_kv != Attributes<T>().end()) {
        std::cout<<"Containersize: " << attr_kv->second.size() << ',' << "id:"<< node_idx<< std::endl;
        attr_kv->second.at(node_idx) = value;
        std::cout<<"ContainersizeDone: " << attr_kv->second.size() << std::endl;
    } else {
        throw std::runtime_error("AttributeContainer::Set(): attribute not found: " + name);
    }
}

node_idx は 0 で、ベクトルが見つかりました。しかし、サイズが変更されたことを確認する出力が得られても、サイズが変更されたことを確認する出力が得られるため、サイズ変更が機能しなかったようです。サイズ変更に成功しましたか？

4

1 に答える 1

4

SetNum では、マップを値で反復処理しています (この投稿を参照してください) 。

その結果、保存されたベクトルのコピーにサイズ変更を適用しています。

あなたが望むのは、 auto& を使用することです:

void SetNum(size_t n)
{
    std::cout << "setting num: " << n << std::endl;
    m_num = n;
    for (auto& attr : m_attri) {
        attr.second.resize(n, 0);
        std::cout<<attr.second.size();
    }
    for (auto& attr : m_attrd) {
        attr.second.resize(n, 0.0);
        std::cout<<attr.second.size();
    }
    for (auto& attr : m_attrs) {
        attr.second.resize(n, "");
        std::cout<<attr.second.size();
    }
    std::cout<<std::endl;
}
于 2014-05-17T20:12:57.597 に答える