std :: mapと同等のremove_if

特定の条件に基づいて、マップからさまざまな要素を削除しようとしました。STLアルゴリズムを使用してどうすればよいですか？

最初は使用を考えましたがremove_if、連想コンテナに対してremove_ifが機能しないため、それは不可能です。

マップで機能する「remove_if」と同等のアルゴリズムはありますか？

簡単なオプションとして、マップをループして消去することを考えました。しかし、マップをループして安全なオプションを消去していますか？（イテレータは消去後に無効になるため）

私は次の例を使用しました：

bool predicate(const std::pair<int,std::string>& x)
{
    return x.first > 2;
}

int main(void) 
{

    std::map<int, std::string> aMap;

    aMap[2] = "two";
    aMap[3] = "three";
    aMap[4] = "four";
    aMap[5] = "five";
    aMap[6] = "six";

//      does not work, an error
//  std::remove_if(aMap.begin(), aMap.end(), predicate);

    std::map<int, std::string>::iterator iter = aMap.begin();
    std::map<int, std::string>::iterator endIter = aMap.end();

    for(; iter != endIter; ++iter)
    {
            if(Some Condition)
            {
                            // is it safe ?
                aMap.erase(iter++);
            }
    }

    return 0;
}

ほとんど。

for(; iter != endIter; ) {
     if (Some Condition) {
          iter = aMap.erase(iter);
     } else {
          ++iter;
     }
}

もともとイテレータから要素を削除した場合、イテレータを2回インクリメントしていました。消去する必要がある要素をスキップする可能性があります。

これは、私が多くの場所で使用および文書化したことを見た一般的なアルゴリズムです。

[編集]イテレータは消去後に無効化されるのは正しいですが、消去される要素を参照するイテレータのみであり、他のイテレータは引き続き有効です。したがってiter++、erase()呼び出しで使用します。

std :: map（およびその他のコンテナー）のerase_if

私はこのために次のテンプレートを使用します。

namespace stuff {
  template< typename ContainerT, typename PredicateT >
  void erase_if( ContainerT& items, const PredicateT& predicate ) {
    for( auto it = items.begin(); it != items.end(); ) {
      if( predicate(*it) ) it = items.erase(it);
      else ++it;
    }
  }
}

これは何も返しませんが、項目をstd :: mapから削除します。

使用例：

// 'container' could be a std::map
// 'item_type' is what you might store in your container
using stuff::erase_if;
erase_if(container, []( item_type& item ) {
  return /* insert appropriate test */;
});

2番目の例（テスト値を渡すことができます）：

// 'test_value' is value that you might inject into your predicate.
// 'property' is just used to provide a stand-in test
using stuff::erase_if;
int test_value = 4;  // or use whatever appropriate type and value
erase_if(container, [&test_value]( item_type& item ) {
  return item.property < test_value;  // or whatever appropriate test
});

さて、std::experimental::erase_ifヘッダで利用可能です<experimental/map>。

参照：http : //en.cppreference.com/w/cpp/experimental/map/erase_if

このドキュメントは、優れたSGI STLリファレンスから入手しました。

マップには、新しい要素をマップに挿入しても、既存の要素を指すイテレータが無効にならないという重要な特性があります。マップから要素を消去しても、イテレータが無効になることはありません。ただし、消去される要素を実際に指しているイテレータは除きます。

したがって、消去する要素を指しているイテレータはもちろん無効になります。このようなことをしてください：

if (some condition)
{
  iterator here=iter++;
  aMap.erase(here)
}

元のコードには1つだけ問題があります。

for(; iter != endIter; ++iter)
{
    if(Some Condition)
    {
        // is it safe ?
        aMap.erase(iter++);
    }
}

ここでiterは、forループで1回インクリメントされ、eraseでもう1回インクリメントされます。これにより、おそらく無限ループが発生します。

ここにいくつかのエレガントなソリューションがあります。

for (auto it = map.begin(); it != map.end();)
{   
    (SomeCondition) ? map.erase(it++) : (++it);
}

以下のメモから：

http://www.sgi.com/tech/stl/PairAssociativeContainer.html

可変連想配列の値の型は割り当て可能である必要があり、ペアは割り当て可能ではないため、ペア連想コンテナは可変反復子を提供できません（トリビアル反復子の要件で定義されています）。ただし、ペア連想コンテナは、完全に一定ではないイテレータを提供できます。式（* i）.second = dが有効であるようなイテレータです。

最初

マップには、新しい要素をマップに挿入しても、既存の要素を指すイテレータが無効にならないという重要な特性があります。マップから要素を消去しても、イテレータが無効になることはありません。ただし、イテレータが実際に消去されている要素を指している場合は除きます。

第二に、次のコードは良いです

for(; iter != endIter; )
{
    if(Some Condition)
    {
        aMap.erase(iter++);
    }
    else
    {
        ++iter;
    }
}

関数を呼び出すとき、パラメーターはその関数の呼び出し前に評価されます。

したがって、eraseを呼び出す前にiter ++が評価されると、イテレータの++演算子は現在のアイテムを返し、呼び出し後に次のアイテムをポイントします。

私見ではremove_if()同等のものはありません。
マップを並べ替えることはできません。
ですからremove_if()、あなたが興味のあるペアをあなたが呼ぶことができる最後に置くことはできませんerase()。

Iron Saviorの回答に基づく。

template< typename ContainerT, class FwdIt, class Pr >
void erase_if(ContainerT& items, FwdIt it, FwdIt Last, Pr Pred) {
    for (; it != Last; ) {
        if (Pred(*it)) it = items.erase(it);
        else ++it;
    }
}

ContainerTアイテムを失い、それをイテレータから取得する方法があるかどうか知りたいです。

スティーブフォリーの答え私はより効率的に感じます。

簡単だが効率の悪い別のソリューションを次に示します。

ソリューションはremove_copy_if、使用する値を新しいコンテナーにコピーするために使用し、次に、元のコンテナーの内容を新しいコンテナーの内容と交換します。

std::map<int, std::string> aMap;

...
//Temporary map to hold the unremoved elements
std::map<int, std::string> aTempMap;

//copy unremoved values from aMap to aTempMap
std::remove_copy_if(aMap.begin(), aMap.end(), 
                    inserter(aTempMap, aTempMap.end()),
                    predicate);

//Swap the contents of aMap and aTempMap
aMap.swap(aTempMap);

2より大きいキーを持つすべての要素を消去したい場合、最良の方法は

map.erase(map.upper_bound(2), map.end());

ただし、範囲に対してのみ機能し、述部には機能しません。

私はこのように使います

 std::map<int, std::string> users;    
 for(auto it = users.begin(); it <= users.end()) {
    if(<condition>){
      it = users.erase(it);
    } else {
    ++it;
    }
 }