この中で右辺値

Question

とペアを構築：

vector<pair<string,int>> vp; 
    string s; 
    int i; 
    while(cin>>s>>i) vp.push_back({s,i}); 

は私が最後の行について興味があります。ペアはpush_backの呼び出しで作成されるため、コピーされずに移動されます。

ではなく、このように書かれていなかった場合は、次の組の構成は同じリソースを必要とする一方で、

vector<pair<string,int>> vp; 
    string s= ...; 
    int i= ...; 
    pair<string,int> p{s,i}; 
    vp.push_back(p); 

今ペアが命名され、p、したがって、それはもはや右辺値であるため、セマンティクスを動かしますもはや使用されていないので、これは値渡しとなり、コピーが作成されます。

これは、可能であれば、argリストの中にオブジェクトを構築することがパフォーマンスの向上であることを意味します。誰でも確認できますか？

Answer 1

このペアはpush_backへの呼び出しで作成されるため、コピーされずに移動されますが、正しいですか？

一時的なペアが構築され、rvalue参照引数push_backにバインドされます。その後、その引数からvectorに移動されます。

vp.push_back({s,i}); 

は同等です：

vp.push_back(std::pair<std::string, int>{s, i}); 

とほぼ同等である：

std::pair<std::string, int> p{s, i}; 
vp.push_back(std::move(p)); 

あなたはは右辺値参照のオーバーロードを使用して、一時的なを必要としません。

あなたの2番目のコード例は：

pair<string,int> p{s,i}; 
vp.push_back(p); 

はい、ではなく動くのコピーを行います。いずれにしても、まだsをペアにコピーしています（一時的またはp）。だから、より良いアプローチは、同様に一時的にsを移動するには、次のようになります。

vp.push_back({std::move(s), i}); 

あるいはそもそも中間ペアを持っていないために：

vp.emplace_back(std::move(s), i); 

ところということ。これは、私には意味します可能な限り、argリストの中にオブジェクトを構築することはパフォーマンスの向上です。誰でも確認できますか？を移動

は、パフォーマンスの向上ですが、あなたは常にstd::move()関数に引数ができます - それは一時的である必要はありません。 argリスト内での構築は、移動元オブジェクトのデストラクタが呼び出されたときに影響します。

Answer 2

あなたは正しいですか？

argリスト内のオブジェクトを作成することが改善されています。

ただし、しばらくお待ちください。 intは移動できません。それはコピーです。一般的にはstd::stringが移動され、これが安全なものになりますmalloc/newコール。

このすべてが可読性に影響しない場合は、それを行ってください。そうでなければ、ほとんどの場合、それは私が信じるような大きな改善ではありません。私は一つでも多くの「道」

vp.emplace_back(std::move(s), i); 

更新を示唆している

std::pair<std::string,int> p{s,i}; 
vp.push_back(std::move(p)); 

もう一つの「道」を示唆している

- それは問題で削除されたように私はcin一部を削除しました。