优化简单的方法

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; 
}