21

このコーディング方法の利点がわかりません。私が今取り組んでいるプロジェクトは、これらのステートメントでいっぱいなので、1 つの間違いではありません。

もう一つの例:

return getNumberOfBooks() > 5 ? true : false;

そしてもう一つ:

return isRed() ? true : false;
4

4 に答える 4

49

これを行う理由はまったくありません。

これは冗長であり、コードが読みにくくなります。

以下ははるかに読みやすいです。

return ( getNumberOfBooks() > 5 );

return isRed();
于 2012-06-08T18:55:35.160 に答える
5

あなたは初心者プログラマーと一緒にプロジェクトに取り組んでいるようです。どこからでもリファクタリングを開始します。

return getNumberOfBooks() > 5 ? true : false;
return isRed() ? true : false;

次の方法で変更します。

return getNumberOfBooks() > 5;
return isRed();
于 2012-06-08T19:03:20.103 に答える
2

明らかに、これは不必要な冗長性です。

Eclipseでそうすることをお勧めします:

  1. regexp "return (something) ? true : false" に一致するものをすべて見つけて、something
  2. regexp "return (something) ? false : true" with !(something) に一致するものすべて

この正規表現の作成は、このプロジェクトを作成した人に委任できます:D

于 2012-06-08T19:23:59.270 に答える
2

三項演算子はコードをほとんど判読不能にしますが、これはスマートなコーディング方法だと考えられていますが、本当に必要でない限り推奨されるべきではありません

于 2012-06-08T18:59:06.467 に答える