if / elseまたはブール式によるブール代入のどちらがより保守性が高いですか？

どちらがより保守性が高いと考えられますか？

if (a == b) c = true; else c = false;

または

 c = (a == b);

Code Completeで調べてみましたが、答えが見つかりません。

最初のほうが読みやすいと思います（文字通り大声で読むことができます）。2番目の方法は確かに理にかなっており、コードを削減しますが、C＃開発者にとってそれが保守可能であるかどうかはわかりません（たとえば、このイディオムはPythonでもっと見られると思います）。

2番目のオプションの方が優れています。

コードの意図を曖昧にして保守性を損なう巧妙なプログラミングショートカットに注意する明確な理由があります。ですから、私はあなたに質問をしたことを責めません。

しかし、私はc = (a == b);巧妙なトリックの例とは考えていません。それは単純な概念の単純な表現です。できるだけ簡単です。

最初の例の適切な「保守可能な」フォーマット（不足しているブレースと1行のコンストラクトはなく、賢いショートカットだと思います）は、次のコードを生成します。

if (a == b)
{
    c = true; 
}
else 
{
    c = false;
}

私の経験では、このような冗長でエラーを起こしやすい方法で単純なブールロジックを記述することは、「簡単な」コードの兆候です。このコードベースでより複雑なロジックがどのように処理されているのか不思議に思うでしょう。

まず、2つのフォームが同等ではないことを理解してください。

if (a == b) c = true;

cがにa等しい場合はtrueに設定されb、等しくない場合、その値は既にあるもののままになります。

c = (a == b);

cがにa等しい場合はtrueに設定されb、等しくない場合はfalseに設定されます。

最初のフォームのスタイルで、2番目のフォームに相当するものが必要な場合は、次のように記述する必要があります。

if (a == b) {
  c = true;
} else c = false;

これで、2つのうちどちらが読みやすく、保守しやすく、何かが変更された場合にバグが発生する可能性が低くなります。2番目のフォームに固執します。

最初のフォームの方が読みやすいことに同意しません-1行に2つのステートメントがあるのは確かに慣用的なC＃ではifなく、ブレースを使用せずにステートメントを作成することはお勧めできません。

第二に、2番目のフォームの保守性が低いことがわかりません。保守するものがありません。それは関係の簡単な声明だaとb、それはもはや単純に表現することができませんでした。

2番目の形式を好むもう1つの理由cは、単一のステートメントで宣言して割り当てることができるということです。

bool c = (a == b);

変数を変更すると、簡単にエラーが発生する可能性があるため、回避します。ifステートメントを使用するには、変数を条件の前に宣言してから変更する必要があります。

「より保守しやすい」ことは非常に主観的です。

私は通常、コード削減よりも読みやすさと意図を好みます。縮小フォームを使用して、8文字の入力文字を保存すると思います。

言語を取り巻く言語と文化を取り入れることは、私の意見では「読みやすさ」の特徴です。

結果として得られるバイトコードを最適化するためにコードを削減するためにパフォーマンスが原因となる場合がありますが、プロファイリング後に慎重に行う必要があります。

二番目。これは、その、あまりの繰り返し（DRY）を持ち、何が起こっているかを理解することが容易であるcかどうかの値に保持しているaことがb同じです。

私見、さらに良いでしょう

c = a == b

私が書くように

• 1 + 2 + 3 の代わりに ((1 + 2) + 3)
• 5 + 3 * 7 の代わりに (5 + (3 * 7))

明らかに、些細な不必要なコードは美徳ではありません。散らかっています。

投票者、私の修正された答えの何が悪いのか詳しく説明してください。

はい、c = (a == b);読みにくい場合があります（さらに悪いことに、StyleCopは不必要な括弧について文句を言います）が、私はまだのシンプルさが好きですa == b。したがって、ここで私はときの両方を使用することが好きなものであるaとb同じです。

private static bool NoPeriod
{
    get
    {
        return this.MyWaveLength == this.HerWaveLength;
    }
}

そして、あなたは次のことができますthis.c = this.NoPeriod：

this.c = this.MyWavelength == this.HerWaveLength;