引数に基づいて変数を設定する規則はありますか？

Question

は、私はフラグか何かで機能を持っていると言う：

void foo (Param p1, Param p2, bool setVariable) 
{ 
    //if setVariable is true, set some bool var to true, else false 
} 

は、以下の他の上の1つの強い好みはありますか？

if (setVariable) 
    _someClassVariable = true; 
else 
    _someClassVariable = false; 

または

_someClassVariable = setVariable; 

明らかに第二は少ないタイピングですが、最初は読みやすくとして私を打ちます。どちらが優先されますか？

Answer 1

私は通常、2番目を好むでしょう。私にとって、最初は強力な警告であり、コードを書いた人は誰でも最高の状態でほとんど能力がないということです。

私はまた、boolをパラメータとして渡すことをお勧めします。まれに、foo(true);とfoo(false);が本当にどういう意味なのかがすぐ分かりません。 foo(do_this);対foo(do_that);

Answer 2

私は2番目に行くでしょう、それは読みやすいものです。可読性は、両方のオプションの選択よりも変数名のほうが多くなります。変数とパラメータの名前が適切な場合、代入は自然になります。

bool conditionHolds() { 
    if (condition) 
     return true; 
    else 
     return false; 
} 

（そして、あなたはそれをやって検討しているならば、それを再考してください）

Answer 3

私は二番目のを好むでしょう：これは、あなたがしないだろう関数からブール値を返すに似ています。タイピングが少なくてもあなたの議論が十分でない場合は、同僚の意見も考慮してください。

if (condition) 
    return true; 
else 
    return false; 

のようなものをプロダクションコードで見れば私は個人的に笑います。 （変数がboolであり、これを使用してキャストメカニズムを実装していない場合）。

Answer 4

後者のほうがはるかに優れています。そうでなければ、コードとコンパイラ生成コードの両方で複雑さが増します（恐らくは最適化されていますが）。

私はまた、先頭のアンダースコア表記を避けるでしょう、それらの名前のいくつかは、標準ライブラリとコンパイラのために予約されています。