機能が実行されていないか正しく呼び出されていません

Question

変数vTotalPaymentは機能しません。これは、私が何を間違えているのか分からないにもかかわらず、私がどのように機能を呼び出すかによると考えています。私はこれもswitch文の悪用の可能性があることを認識しています。

function begin() { 
 
    var vTicketType = prompt('Please enter the type of ticket'); 
 
    var vTicketQty = Number(prompt('Please enter amount required')); 
 

 
    document.write('Ticket Type: ' + vTicketType + '<br />'); 
 
    document.write('Ticket Qty: ' + vTicketQty + '<br />'); 
 

 
    var vTicketPrice = calcPrice(vTicketType); 
 
    var vTotalPayment = calcTotal(vTicketQty, vTicketPrice); 
 

 
    document.write('Total Payment: ' + vTotalPayment); 
 

 
} 
 
begin(); 
 

 
function calcPrice(Prices) { 
 
    var Price = 0; 
 
    switch (vTicketType) { 
 
    case 'A': 
 
     Price = 100; 
 
     break; 
 
    case 'B': 
 
     Price = 75; 
 
     break; 
 
    case 'C': 
 
     Price = 50; 
 
     break; 
 
    default: 
 
     Price = -1; 
 
    } 
 
    return (Price); 
 
} 
 

 

 
function calcTotal(Payments) { 
 
    var Payment = (vTicketQty * vTicketPrice); 
 
    return (Payment); 
 
}

Answer 1

あなただけcalcTotalを機能させる一つの引数を渡すともそれを使用していません。あなたのcalcTotalを以下のコードに置き換え、まだ動作していない場合はコメントでお知らせください。

function calcTotal(vTicketQty, vTicketPrice) { 
    return (vTicketQty * vTicketPrice); 
} 

Answer 2

メソッドのパラメータが間違っています。パラメーターの数や間違った名前。

代わりのcalcPrice(Prices)私はそれはそれはおそらくあなたは、次の操作を行うことができcalcTotal(vTicketQty, vTicketPrice)

function begin() { 
 
    var vTicketType = prompt('Please enter the type of ticket'); 
 
    var vTicketQty = Number(prompt('Please enter amount required')); 
 

 
    document.write('Ticket Type: ' + vTicketType + '<br />'); 
 
    document.write('Ticket Qty: ' + vTicketQty + '<br />'); 
 

 
    var vTicketPrice = calcPrice(vTicketType); 
 
    var vTotalPayment = calcTotal(vTicketQty, vTicketPrice); 
 

 
    document.write('Total Payment: ' + vTotalPayment); 
 

 
} 
 
begin(); 
 

 
function calcPrice(vTicketType) { 
 
    var Price = 0; 
 
    switch (vTicketType) { 
 
     case 'A': 
 
      Price = 100; 
 
      break; 
 
     case 'B': 
 
      Price = 75; 
 
      break; 
 
     case 'C': 
 
      Price = 50; 
 
      break; 
 
     default: 
 
      Price = -1; 
 
    } 
 
    return (Price); 
 
} 
 

 

 
function calcTotal(vTicketQty, vTicketPrice) { 
 
    var Payment = (vTicketQty * vTicketPrice); 
 
    return (Payment); 
 
}

Answer 3

あるべきcalcPrice(vTicketType)、代わりのcalcTotal(Payments)されるべきだと思う：

(function begin() { 
 
    var vTicketType = prompt('Please enter the type of ticket'); 
 
    var vTicketQty = Number(prompt('Please enter amount required')); 
 

 
    document.write('Ticket Type: ' + vTicketType + '<br />'); 
 
    document.write('Ticket Qty: ' + vTicketQty + '<br />'); 
 

 
    var vTicketPrice = calcPrice(vTicketType); 
 
    var vTotalPayment = calcTotal(vTicketQty, vTicketPrice); 
 

 
    document.write('Total Payment: ' + vTotalPayment); 
 

 
})(); 
 

 
function calcPrice(vTicketType) { 
 
    var Price = 0; 
 
    switch (vTicketType) { 
 
     case 'A': 
 
      Price = 100; 
 
      break; 
 
     case 'B': 
 
      Price = 75; 
 
      break; 
 
     case 'C': 
 
      Price = 50; 
 
      break; 
 
     default: 
 
      Price = -1; 
 
    } 
 
    return Price; 
 
} 
 

 

 
function calcTotal(vTicketQty, vTicketPrice) { 
 
    var Payment = (vTicketQty * vTicketPrice); 
 
    return Payment; 
 
}

ベストプラクティスについては

あなたは、あなたが2つのパラメータを使用している関数を呼び出しているが、あなたの関数定義であなただけの1つのパラメータを持っている場合は、あなたが支払いを返すか、価格