c# - Ifステートメントの合理化c＃

Question

私はこのステートメントを使おうとしています

status_label.Text = err.Message + " || " + err.InnerException == null ? " " : err.InnerException.Message;

基本的に、nullではないためにInnerExceptionがある場合はそれを表示し、nullの場合は何も出力しません。

これは私がそれを持つことができると思うのと同じくらい合理化されています

 status_label.Text = err.Message;
 if (err.InnerException != null)
     status_label.Text += " || " + err.InnerException.Message;

乾杯。

score 3 · Accepted Answer

代わりにこれを使用してください：

status_label.Text =  err.InnerException == null 
    ? err.Message
    : err.Message + " || " + err.InnerException.Message;

score 1 · Accepted Answer

三項演算子の周りの括弧がありません

status_label.Text = err.Message + " || " + (err.InnerException == null ? " " : err.InnerException.Message);

または、|| が必要ない場合 null innerexception の場合

status_label.Text = err.Message + (err.InnerException == null ? "" : " || " + err.InnerException.Message);

score 0 · Accepted Answer

これでうまくいくはずです：

status_label.Text = string.Format("{0}{1}", err.Message, err.InnerException != null ? string.Format(" || {0}", err.InnerException.Message) : string.Empty);

score 0 · Accepted Answer

まず、ifステートメントはこれでなければなりません

status_label.Text = err.Message;
 if (err.InnerException != null)
     status_label.Text += " || " + err.InnerException.Message;

これよりも合理化するために、条件付きおよび/またはnull合体を利用できますが、それは完全に読みやすいです。

ただし、必要に応じてこれを行うこともできます。

status_label.Text = string.Concat(err.Message, err.InnerException == null ? string.Empty : " || " + err.InnerException.Message);

それはもう簡潔ですか？あまり。

score 0 · Accepted Answer

完全な例外が印刷されても構わない場合は、次のようにします。

status_label.Text = err.ToString();
status_label.Text += err.InnerException;

(これにより、各例外の前に「System Exception:」などが表示されることに注意してください。)

それ以外の場合、コードはそのままで十分に合理化されているように見えます。(コンパクトさよりも読みやすさの方が望ましい場合があります。)

c# - Ifステートメントの合理化c＃

5 に答える 5

Related

Reference