c++: vectorのソートにつきまして

Question

質問内容

std::vector をソートしたのですが不可解な挙動に遭遇しております。
vectorをソートした直後は想定通りにソートされているのですが、別の箇所から同じvectorにアクセスするとソートされていません。

func1()でvector内のMyStructのvalueの値で昇順にソートします。
func2()で確認するとソートされていませんん。

struct MyStruct {
  int value;
}

std::map<string, std::vector<MyStruct>> mp;

void func1(){
    for (auto m: mp) {

        // vectorの内容をMyStructのvalueで昇順にソート
        std::sort(
            mp.second.begin(),
            mp.second.end(),
            [](const MyStruct &x, const MyStruct &y) { return x->value > y->value; });

        // vectorの並びを表示: 想定通りソートされている
        for (MyStruct &s : mp.second) {
            Logger::writeLog("MyStruct value=%d", s->value);
        }
    }
}

void func2(string &key){
    auto &m = mp[key];
    for (MyStruct &s : m) {
        // vectorの並びを表示: ソートする前の順序に戻っている
        Logger::writeLog("MyStruct value=%d", s->value);
    }
}

std::sortが破壊的ではないのかと思ったのですが、それだとどちらとも失敗するはずですし・・・。
iteratorをend()のまま再利用しているわけでもないですし。(それだと結果が何も表示されないですが。)

どこに誤りがあるのでしょうか？

どうぞ宜しくお願いいたします。

Comments

[fana]

そのコードではコンパイルが通らないのでどうしようもない．
問題が再現する「動作可能な」コードを提示してください．

[guriguri]

再現するコードがかなり複雑なので端折ってました。
mpのループの箇所が間違っておりました。
ご返信ありがとうございました。

Answer 1

for (auto m: mp)ではmには要素のコピーが入っています。
for (auto& m: mp)とすればうまく行くと思います。
また、ループ内のmpはmのタイプミスでしょう。

Comments

[yuki23]

forループの中で`m`を使っていないので、それだけでは直らないかと

[SHOMI]

コピペせずに書いていてfor内のmpはmの打ち間違いかなと

[SHOMI]

だとするとfor部分も記述通りかわからないですね…
fanaさんも書かれていますが、動作に関係のない箇所は省略しても良いのでコンパイルの通る実際のコードを貼り直してください。

[guriguri]

すみません、コードを端折ってました。
`for (auto& m: mp)`が正解でした。
参照には気をつけていたのですが外のループなので全く気になっておりませんでした。
ありがとうございました。