2016-04-08 12 views
1

私はある量の予約を表すエンティティを持っています。このプロパティが負であるエンティティも存在し、予約の可用性がないことを示します。論理は、正の量の特性値を有する予約量を減少させる。したがって、私は次の文を持っています:コード内で重要な単項マイナスを目立たせるには?

long quantityToDecrease = -reservation.getQuantity(); 

私のコードの読者がそれを認識するように、どうすれば単項マイナスを目立たせることができますか?

+2

'-reservation.getQuantity();'を返すだけの適切な名前のメソッドを作り、それを代わりに使用します。私。 'quantityToDecrease = notAvailable(予約);'。 – Kayaman

+0

いつでもコメントを追加できます。 –

+0

その前にゼロ、コメント、またはその両方を置きます。独立した方法を実装するための根気のないアドバイスに従わないでください。それはコードレビューで削除する最初の方法です。いずれにしても、問題をシフトさせるだけです。 – EJP

答えて

1

私はその周りにインラインコメントや空行と一緒に行きました:

// note the unary minus 
long quantityToDecrease = -reservation.getQuantity(); 

しかし、それはコードをよりビジネス環境のように読みますので、私はまた、@Kayaman魅力的で提案された解決策を見つけるのですか論理が記述される。

-1

はそれが本当にあなたが別々のメソッドを作ることができますし、多分別々のクラスの中で目立つようにするには:

long quantityToDecrease = negate(reservation.getQuantity());

それはキルオーバーのように見えるかもしれないが、私はこの方法を使用すると、読者の注目を集めるだろうと思います最高の。あなたは多くの場所でこれを使用する場合

また、多分あなたは、あなたがあなたの否定メソッドを定義することができ、その方向で独自NumberUtilクラスか何かを書くことができ、それが何かのように作る:。

long quantityToDecrease = NumberUtil.negate(reservation.getQuantity());

関連する問題