コードの簡潔さや読みやすさを好みますか？[閉まっている]

言語のショートカットを使用して、コードをより簡潔にすることができます。

たとえば、三項およびヌルの合体演算子はコードの量を減らすことができますが、間違いなく読みやすさを損ないます：

C＃の場合：

Person newGuy = new Person();
if (boss == null) {
    newGuy.Boss = GetDefaultBoss();
} else {
    newGuy.Boss = boss;
}

機能的に同等です：

Person newGuy = new Person();
newGuy.Boss = boss ?? GetDefaultBoss();

しかし、明らかにもっと冗長です。

簡潔さと読みやすさの点では、どこで線を引きますか？

両方。

あなたの最初の例は確かにより冗長で、間違いなくより明確です...しかし、それはまた、1行ではなく5行をスキャンすることを必要とします。さらに悪いことに、それはその目的を強調しません-に値を割り当てますnewGuy.Boss。

あなたの2番目の例は、null合体演算子に慣れていない場合は1秒かかりますが、その目的については間違いなく、値のソースを探してより大きなルーチンをスキャンしている場合、これを簡単に選択できるようにしてください。

今、これを対比してください：

if (boss == null) {
    newGuy.Boss = GetDefaultBoss();
    newGuy.IsTemp = true;
    newGuy.AddTask("orientation");
} else {
    newGuy.Boss = boss;
    newGuy.IsTemp = false;
}

... with：

newGuy.Boss = boss ?? GetDefaultBoss();
newGuy.IsTemp = boss == null;
if ( boss == null ) newGuy.AddTask("orientation");

後者の例もやはりはるかに短いですが、同じテストによってトリガーされたタスクが明確に見えるようにすることで、目的がわかりにくくなりました。ここでは、前者の冗長性が正当化されていると感じています。

両方とも良い目標ですが、どちらかを選択せざるを得ない場合は、常に可読性を維持します。

私はあなたの例が読みやすさと簡潔さの両方を改善すると主張します。ただし、以下を考慮してください。

if( a > b )
{
    foo = bar
}
else
{
    if( c.isThing() ){
        foo = somethingElse;
    }
    else{
        foo = someFurtherThing.GetFoo();
    }
}

とは対照的に

foo = a > b ? bar ?? whatever.something : c.isThing() ? somethingElse : someFurtherThing.GetFoo();

後者は簡潔ですが、読み通すのは困難です。前者は冗長ですが、論理の流れは明確です。

最終的に、簡潔さは、画面にもっと収まる能力以外の目的にはあまり役立ちません。読みやすさはデバッグを容易にするため、一般的に推奨されます。

原則として、簡潔さのために読みやすさを犠牲にすることは決してありませんが、その主題に関する知識の不足を別のプログラマーに基づいて読みやすさを判断することはありません。

簡潔さと読みやすさは反対ではありません。この答えのように、短いほうが読みやすい場合があります。

私は読みやすさを好みますが、それは時々簡潔なコードを使用することを意味します。（つまり、より大きな条件ブロック内の比較的単純な条件の3項です。）

基本的に、理解するのが不必要に難しい場合は、しないでください。

コードは最初に書かれたよりも頻繁に変更されるため、簡潔さと矛盾する場合は、可読性が最初になります。一方：

1. 構文ノイズと定型コードは、意図をあいまいにして読みやすさを損なうことがよくあります。簡潔なコードの方が読みやすい場合もあります。たとえば、ラムダ関数またはデリゲート/ファーストクラス関数と、シングルメソッドインターフェイスを実装するシングルメソッドクラスを考えてください。

2. 最小の共通分母のみを知っているやっかいな能力を持つコードモンキーではなく、言語とその独自の/高度な機能を十分に理解しているかなり経験豊富なプログラマーにとって、コードの読みやすさに基づいて読みやすさを評価する必要があります。

私がまだ言及していないと思われる一面：あなたの目標は何ですか？

仕事のセキュリティだけが気になるなら、他のすべてよりも簡潔さとコンパクトさを求めてください。コードへのコメントもスキップします。

クールな新しいプロジェクトで作業しているときにコードを簡単に他の人に引き渡すことができるようにしたい場合は、読みやすさ、明快さ、しっかりしたコメントを求めてください。

_{^{注：上記は個人に関するものではありません、@ Damovisa。2つのポジションの間で選択する人のためです。}}

冗長バージョンには利点として1つあります。

より多くの行があり、ほとんどのデバッガーは行指向です！式の途中にブレークポイントを設定することは非常に困難ですが、通常はブロックステートメント内に設定するのは簡単です。

言い換えれば、デバッガーをいつ起動させるかをエディターで確認したいのはboss == nullどれですか？

（つまり、私は??-演算子が好きだと言った）

読みやすさは最初に来る必要があり、長期的にはほとんどの人が既存のコードの変更または拡張に最も時間を費やします-読みやすさは保守性の大部分です。

とはいえ、簡潔さは読みやすさに貢献できるものです。たとえば、あなたの質問では、2番目のスニペットはより読みやすく、簡潔です。