方法は二回を介して実行されている

Question

class Myclass 
{ 
    public string Driver1() 
    { 
     string a = ""; 

     Console.Write("Please enter drivers name: "); 
     a = Console.ReadLine(); 

     return a; 
    } 
    public int numberOfTrips() 
    { 
     int a = 0; 
     { 
      Console.Write("Enter the number of trips: "); 
      a = Convert.ToInt32(Console.ReadLine()); 
     } 
     return a; 
    } 
    public List<float> Payments() 
    { 
     List<float> a = new List<float>(); 
     float input; 

     for (int i = 0; i<numberOfTrips(); i++) 
     { 
      Console.Write("Enter payment {0}: ", (1 + i)); 
      input = float.Parse(Console.ReadLine()); 
      Console.WriteLine("Payment added"); 
      a.Add(input); 
     } 

     return a; 
    } 
} 
class Program 
{ 
    static void Main(string[] args) 
    { 
     Myclass a = new Myclass(); 

     string name = a.Driver1(); 
     int trip = a.numberOfTrips(); 
     float total = a.Payments().Sum(); 

     Console.WriteLine("\nDriver: {0}\n" + "Number of trips: {1}\n" + "Total payment: {2}\n", name, trip, total); 
    } 
} 

私が午前問題は、それがforループ含む方法に到達する前に、「公共のint numberOfTripsは（）」メソッドを二回実行されていることです。私はこれが、ループを止めるべきときを指定するためにforループ内で使用しているという事実と関係していると思います。私はこれを間違って行ったと推測しています。どうすればこの問題を修正できますか？私は、ユーザーが何回支払いを要求するかを設定できるようにする必要があります。

何か助けていただければ幸いです。

Answer 1

だけPaymentsへのパラメータとしてnumberOfTripsから番号を渡します、私はこれを試してみましたが、それが働いている

Myclass a = new Myclass(); 

string name = a.Driver1(); 
int trip = a.numberOfTrips(); 
float total = a.Payments(trip).Sum(); 

Answer 2

Main（）とPayments（）でnumberOfTrips（）を呼び出す代わりに、MyClassにインスタンス変数または静的変数を作成してみてください。次に、すべての支払いが計算された後、その変数からの旅行の数を取得できます。

Answer 3

これは間違いありません。最初に実行されるのは、 '旅行'変数を設定するためのMainです。 2回目の実行は、forループ宣言内のPaymentsにあります。