2016-04-08 18 views
1

Bazı miktarların rezervasyonunu temsil eden bir varlığım var. Ayrıca, bu mülkün negatif olduğu ve rezervasyon için müsait olmadığına işaret eden varlıklar da bulunacaktır. Mantık daha sonra pozitif miktar özellik değeri ile rezervasyon miktarını azaltacaktır. Bu nedenle bu açıklamayı var: Benim kod okuyucu bunun farkında olacak şekilde tekli eksi öne çıkacak hale nasılÖnemli bir tekil eksi kodda nasıl göze çarpar?

long quantityToDecrease = -reservation.getQuantity(); 

?

+2

Sadece '-reservation.getQuantity() işlevini döndürecek iyi bir yöntem hazırlayın ve bunu kullanın. Yani quantityToDecrease = notAvailable (rezervasyon); '. – Kayaman

+0

Her zaman yorum ekleyebilirsiniz. –

+0

Önüne bir sıfır veya bir yorum veya her ikisini de koyun. Ayrı bir yöntem uygulamak için inane tavsiyesine uymayın: Bu, kod incelemesinde kaldırdığım ilk şey olurdu. Her durumda, sadece sorunu değiştirir. – EJP

cevap

1

Ben onun etrafında satır içi açıklama ve boş hatları ile gitti:

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

Ama kod busines gibi daha okutan çünkü ben de @Kayaman çekici önerdiği çözüm bulmak do mantık anlatılmaktadır.

-1

bu gerçekten ayrı yöntem yapabilir ve belki ayrı sınıfta dikkat çekici hale getirmek:

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

da öldürmenin üzerinde benzeyebilir ama bu yolu okuyucuların dikkatini çekmek olacağını düşünüyorum en iyisi. . Eğer yerlerde bir sürü bu kullanırsanız

Ayrıca belki daki inkâr yöntemini tanımlayabilirsiniz o yönde, kendi NumberUtil sınıf veya bir şeyler yazmak gibi o şey yapma edebilirsiniz:

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