C++ 11の範囲ベースのforループ効率 "のconstオート＆i" が "自動I" C++ 11では

Question

対、私はそうのようないくつかのコンテナを反復処理することができます

for(auto i : vec){ 
    std::cout << i << std::endl; 
} 

しかし、私がいることを知っていますこの不 - 不、私は唯一の印刷にvecの値を必要とするので、 - （EDIT：の各要素）のコピーになりvecを、ので、代わりに私が行うことができます：

for(auto &i : vec){ 
    std::cout << i << std::endl; 
} 

は、しかし、私はvecの値が変更されていないとのconst - 正しさを遵守んことを確認するので、私は行うことができます。

for(const auto &i : vec){ 
    std::cout << i << std::endl; 
} 

だから私の質問は：私だけで見に必要がある場合（EDIT：の各要素）の余分なコピーを持たないために最後のループ（const auto &i）が常に好ましいとは思わないでしょうvec？

私はこの中でこの変更を検討している私が開発しているプログラムを持っています。それは効率が重要であるからです（私が最初の場所でC++を使用している理由）。

Answer 1

はい。引数を読んだだけの場合は、同じ理由でパラメータconst&を作成します。

T  // I'm copying this 
T&  // I'm modifying this 
const T& // I'm reading this 

これは「デフォルト」です。 Tが基本タイプ（内蔵）の場合、コピーはエイリアシングよりも安価であるため、一般的にはconst T（参照なし）に戻って読み込みます。

---

私は効率が

1. それに重要であることから、私は全体でこの変更を行う検討しているここで開発していたプログラムを持っている盲目の抜本的な変更を加えないでください。 。速くて壊れたプログラムよりも働くプログラムが優れています。
2. ループを反復処理しても、おそらく大きな違いはありません。あなたは理由のためにループしているのですか？あなたのループのボディは、はるかに可能性が高いです。
3. 効率が重要である場合は、実際遅いのではなく、は遅いかもしれない部分を推測である、あなたのプログラムの部分を見つけるためにプロファイラを使用します。なぜあなたの推測が間違っているのか＃2を見てください。

Answer 2

ベクターに文字列が含まれているとします。長い文字列。 5000の長い文字列。それらを不必要にコピーすると、非常に効率の悪いforループが作成されます。

あなたのコードがあなたの意図に従っていることを確認してください。ループの内部にコピーが必要ない場合は、コピーを作成しないでください。

上記のように参照番号&を使用するか、イテレータを使用してください。