プログラムで自分自身を繰り返さないようにするための最適化

Question

私はこの本当にシンプルなピザ注文プログラムを書いたが、私は自分自身を繰り返しているような気がする。最適化のヒント？

コード：

var orderCount = 0; 
 

 
function takeOrder(crustType, topping) { 
 
    console.log('Order:' + crustType + ' ' + 'pizza' + ' ' + topping); 
 
    orderCount = orderCount + 1; 
 
} 
 

 
function getSubTotal(itemCount) { 
 
    return itemCount * 7.5; 
 
} 
 

 
function calculateVat(VAT) { 
 
    return VAT/10 * 2; 
 
} 
 

 
// List the orders 
 
takeOrder('Thin Crust Pizza', 'with bacon'); 
 
takeOrder('Fat Crust Pizza', 'with pepporoni'); 
 
takeOrder('Medium Crust Pizza', 'with Vegi'); 
 
takeOrder('Medium Crust Pizza', 'with steak'); 
 
takeOrder('Medium Crust Pizza', 'with Sausage'); 
 
// List the Total Minus VAT 
 
console.log(getSubTotal(orderCount)); 
 

 
//Stores the cost with and With out VAT 
 
var totalNoVAT = getSubTotal(orderCount); 
 
var totalYesVAT = getSubTotal(orderCount) + calculateVat(totalNoVAT); 
 

 
//Log the cost of the VAT and Total Order Cost with VAT 
 
console.log('The VAT on that order would be' + ' ' + calculateVat(totalNoVAT)); 
 
console.log('This give you a grand total of' + ' ' + totalYesVAT);

Answer 1

あなたorderCountがグローバルであることから、あなたが機能getSubTotalでそれを渡す必要はありません。また、getSubTotal（）値を変数に保存します。それを何度も何度も呼び出す必要はありません。

Answer 2

これはちょっとしたプログラムです。

ピザコンストラクタを使用してJavaScriptコンストラクタを学習することを検討してください。

function order(size, topping, price, customer, address) { 
    this.Pizza = { 
     Size: size; 
     Toppings: toppings; 
    }, 
    this.Price = price; 
    this.Customer = customer; 
    this.DeliveryAddress = address; 
} 

var order_one = new order ('12"', 'Pepperoni', 12.99, 'Josh', '1, Typical Street, Typicaltown');