変数に複数のクラス呼び出しを格納する方が速いのですか？

Question

あなたはこのような方法がある場合：

float method(myClass foo) 
{ 
    return foo.PrivateVar() + foo.PrivateVar(); 
} 

が、代わりに、これを行うために/常に速いほうがよい？：

float method(myClass foo) 
{ 
    float temp = foo.PrivateVar(); 
    return temp + temp; 
} 

を、私はあなたがfoo.PrivateVar()のようなコールを置くことになっていない知っていますたとえば、forループでは、実際に値を1回だけ使用する必要がある場合、何度も評価するためです（場合によっては

for (int i = 0; i < foo.PrivateInt(); i++) 
{ 
    //iterate through stuff with 'i' 
} 

これより最初の例のようなコードを2番目の例に変更することを前提にしていますが、コンパイラよりも賢くならないように人々に言われました！そしてそれは通話を非常によくインラインにすることができます。

私は何もプロファイルしたくありません。これについては、いくつかの簡単なルールが必要です。私は仕事のアプリケーションのデモを書いています。誰かがコードを見てルーキーミスをすることを望んでいません。

Answer 1

これは完全にPrivateVar()の処理内容と定義されている場所によって異なります。コンパイラがPrivateVar()のコードにアクセスでき、その関数を呼び出すことによって副作用がないことを保証することができます。あなたの2番目のコード例でやったことがあります。

forループとまったく同じです。だから、一度しか評価されていないことを確かめたいのであれば、それは非常に高価な関数なので（明示的に副作用が発生してもそれが厄介でないことを保証することも意味する）

PrivateVar()がゲッターに過ぎない場合は、コンパイラがCSEを実行しなくても、パフォーマンスの違いはすべてのケースの99.9999％で問題にはなりません。

編集：CSEはCommon Subexpression eliminiationの略であり、正確にはそれを表しています;）wikiページには単純な乗算の例が示されていますが、関数呼び出しのような大規模なコード構造に対してもこれを行うことができます。

a = b++ * c + g; 
d = b++ * c * d; 

し、それを変更する：

tmp = b++ * c; 
a = tmp + g; 
d = tmp * d; 

我々は一度だけのコードを評価するにはつまり、このコードのためにCSEをやって、セマンティクスを変更しないことを保証する必要があるすべての場合において

明らかに違法になるでしょう（関数呼び出しについては、明らかにもう少し複雑ですが、同じ原則です）。