루프 성능 내 루프

Question

내가 실제로 게시하는 유일한 이유는 내 코드 성능이이 코드 블록의 영향을 받고 있기 때문에 다른 곳에서 foreach 루프를 사용하기 때문입니다.

다른 사람이 성능에 도움이 될 수있는 다른 제안이나 코드의 다른 결함을 지적했는지 궁금합니다. 당신은 그냥 내림차순 사전에서 (다른 위치에) 상위 5 개 점수를 표시하고 다른 사전에서 플레이어의 목록에 보내는 것보다시피

public override void DisplayScore() 
    { 
     byte MessageLocation = 0; 
     foreach (var kvp in PlayerScores.OrderByDescending((s => s.Value))) 
     { 
      if (MessageLocation == 5) 
       break; 

      if (MessageLocation == PlayerScores.Count) 
       break; 

      foreach (var player in PlayerList.Values) 
      { 
       SendMessage(MessageLocation, "My text"); 
      } 
      Score++; 
     } 
    } 

.

Answer 1

대신 당신이 선수로 전송 다음 목록에 점수를 추가하고하여 일부 성능을 얻을 수있다 루프 중첩는 예 :

//List containing player and score 

foreach (var kvp in PlayerScores.OrderByDescending((s => s.Value))) 
{ 
     //Add scores to list  
} 

foreach (var player in PlayerList.Values) 
{ 
     //Send scores to players 
} 

Answer 2

내가 이중 루프가 문제라고 생각하지 않습니다. LINQ 쿼리 PlayerScores.OrderByDescending((s => s.Value))을 확인하는 것이 좋습니다. 점수의 수에 따라 순서에 시간이 걸릴 수 있으며 특히 값이 사전에서 오는 경우 시간이 걸릴 수 있습니다. 독해의 내부 구조는 키와 값을 통해 열거하는 데 비용이 많이 듭니다.

당신은 그것을 볼 수있는 다음 코드 (약간 개선) 및 Visual Studio 2015, 그것을 테스트 할 수 있습니다, 얼마나 오랫동안 단일 실행 단계를 수행하십시오

public override void DisplayScore() 
{ 

    var scores = PlayerScores.OrderByDescending(s => s.Value).Take(5).ToArray(); 

    foreach (var kvp in scores) 
    { 
     foreach (var player in PlayerList.Values) 
     { 
      SendMessage(MessageLocation, "My text"); 
     } 
    } 
}