選手のリストで最高の得点を得ようとしています

Question

public class Player 
{ 
    public string Name { get; private set; } 
    public string Firstname { get; private set; } 
    public Ploeg Team { get; private set; } 
    public int Goal { get; set; } 
} 
public class GlobalCollection 
{ 
    public List<Ploeg> PloegCollection { get; private set; } 
    public List<Speler> SpelerCollection { get; private set; } 
} 
public GlobalCollection() 
{ 
     SpelerCollection.Add(new Speler("Stanciu", "Nicolae", p1 , 0)); 
     SpelerCollection.Add(new Speler("Massimo", "Bruno", p1 , 0)); 
     SpelerCollection.Add(new Speler("Hanni", "Sofiane", p1 , 0)); 
     SpelerCollection.Add(new Speler("Teodorczyk", "Lukasz", p1, 0)); 
} 

私はゴール+1を設定するボタンを持っています。

private void goalButton_Click(object sender, EventArgs e) 
    { 
     if (ploeg1ListBox.SelectedIndex >= 1) 
     { 
      Player pl = (Player)team1ListBox.SelectedItem; 
      pl.player++; 
      GoalForm goal = new GoalForm(); //winform with picture 
      goal.ShowDialog(); 
     } 

ここでは、最高のゴールのスコアラーをクリックすると表示されます。私はMAXVALUEを使用してみましたので、..に

foreach(Speler sp in data.SpelerCollection) 
    { 
     for (int counter = 0; counter > data.SpelerCollection.Count;counter++) 
     { 

を私は私の小さなプログラムのための右のコードを見つけることができないよう、あなたは私の男を助けることができますか？

Grtsないのはなぜ

Answer 1

：それを行うの

var maxGoals = SpelerCollection.Max(s => s.Goal); 
Speler maxScorer = SpelerCollection.Where(s => s.Goal == maxGoals).First(); 
// rest of your logic ... 
// you should handle the case, when more than one 
// player have scored the same amount of goals. 
// It would be better to get a collection back and then 
// display the result depending on the number of players returned 

Answer 2

一つの方法：あなたは、LINQを使用している場合

var playerWithHighestGoalRank = data.SpelerCollection 
             .OrderByDescending(player => player.Goal) 
             .First();