優化簡單的方法

Question

所以基本上我有一個例子，我創建了兩種方法，一種是從包含單位價格的類中返回全價，另一種是返回價格後貼現的方法。

public int getFullPrice(Product product){ 
     int pricePerUnit = product.getPricePerUnit(); 
     int fullPrice = this.quantity * pricePerUnit; 
     return fullPrice; 
    } 
    public int priceAfterDiscount(Product product){ 
     int pricePerUnit = product.getPricePerUnit(); 
     int fullPrice = this.quantity * pricePerUnit; 
     return fullPrice - this.discountRate; 
    } 

我不知道這是否會是更好的做法是創建可以傳遞到第二個方法，或者這是否是不好的做法，因爲雖然我可重用代碼，如果第二種方法的第一種方法中的變量被執行了，它將不得不通過第一種方法之前？

public int getFullPrice(Product product){ 
     int pricePerUnit = product.getPricePerUnit(); 
     int fullPrice = this.quantity * pricePerUnit; 
     return fullPrice; 
    } 
    public int priceAfterDiscount(int fullPrice){ 
     return fullPrice - this.discountRate; 
    } 

我不是100％確定它是否從第一種方法中獲得fullPrice。或者我採取的方法是否不合理。我知道這樣做肯定會有一個更簡單的方式，而不需要重複代碼

Answer 1

依賴副作用的代碼行爲，特別是以前執行的代碼的副作用幾乎總是一個壞主意。

如果在兩個公共方法之間共享代碼，更好的方法是將公共代碼重構爲私有或受保護的方法。

在這種情況下，折扣後的價格執行完全相同的全價計算計算，因此先調用它，然後發佈流程以減少重複的代碼。 （如果我明白了）：

public int getFullPrice(Product product){ 
    int pricePerUnit = product.getPricePerUnit(); 
    int fullPrice = this.quantity * pricePerUnit; 
    return fullPrice; 
} 

public int priceAfterDiscount(Product product){ 
    return getFullPrice(product) - this.discountRate; 
} 

Answer 2

這個代替了嗎？

public int getFullPrice(Product product){ 
    int pricePerUnit = product.getPricePerUnit(); 
    return this.quantity * pricePerUnit; 
} 

public int priceAfterDiscount(Product product){ 
    return getFullPrice(product) - this.discountRate; 
}