Нужна помощь в упрощении моего кода при оценке покера

Question

В настоящее время я создаю оценщика покера, который сравнивает руку с несколькими игроками и определяет, кто победитель. Однако метод фактического сравнения и тай-брейка оказывается намного более трудным, чем следовало бы. Есть ли способ упростить мой код? Поскольку я не размещаю весь исходный код, я обсужу функциональность некоторых методов, вызванных внутри:

int HandEval (Card [] hand) берет массив карт и возвращает int, который символизирует оценку (т. е. три вида будут иметь более высокий балл, чем одна пара)

int GetHighCard (Card [] hand) берет массив карт и возвращает ранг высокой карты, используемой для тай-ломания (т. е. . в случае тройка, ранг одного из этих трех видов будут возвращены)

public void FindWinner() 
    { 
     int maxScore = 0; 
     int maxRank = 0; 
     List<String> potentialWinners = new List<string>(); 
     List<Card[]> candHand = new List<Card[]>(); 

     Dictionary<string, Card[]> deckTable = new Dictionary<string, Card[]>(); 
     Dictionary<string, int> scoreTable = new Dictionary<string, int>(); 
     Dictionary<string, int> highTable = new Dictionary<string, int>(); 

     //place each player to deckTable which holds the player name and their hand 
     //place each player to scoreTable which holds the player name and the score of their hand 
     for (int i = 0; i < players.Count; i++) 
     { 
      deckTable.Add(players[i].GetName(), players[i].GetSorted()); 
      scoreTable.Add(players[i].GetName(), HandEval(players[i].GetSorted())); 

     } 

     //display the player name and the score of their hand 
     foreach (KeyValuePair<string, int> pair in scoreTable) 
     { 
      Console.WriteLine("{0}, {1}", pair.Key, +pair.Value); 
     } 

     //find the max score of the current game 
     foreach (var kvp in scoreTable) 
     { 
      if (kvp.Value > maxScore) 
      { 
       maxScore = kvp.Value; 
      } 
     } 

     //display the max score 
     Console.WriteLine("The maximum score is " + maxScore); 

     //for all players with the max score, add them to the potential winners list 
     foreach (var kvp in scoreTable) 
     { 
      if (kvp.Value == maxScore) 
      { 
       potentialWinners.Add(kvp.Key); 
      } 
     } 

     //if there are more than one potential winner, run the tie-break checks 
     if (potentialWinners.Count > 1) 
     { 
      Console.WriteLine("Potential winners include: "); 
      for (int i = 0; i < potentialWinners.Count(); i++) 
      { 
       Console.WriteLine("" + potentialWinners[i] + ""); 
      } 

      //add the name of the potential winners and the rank of the high card as a key value pair to highTable 
      for (int i = 0; i < potentialWinners.Count(); i++) 
      { 
       if (deckTable.ContainsKey(potentialWinners[i])) 
       { 
        Card[] cHand = deckTable[potentialWinners[i]]; 
        highTable.Add(potentialWinners[i], GetHighCard(cHand)); 
       } 
      } 

      Console.WriteLine("Displaying potential winners with their high card rank."); 
      foreach (KeyValuePair<string, int> pair in highTable) 
      { 
       Console.WriteLine("{0}, {1}", pair.Key, pair.Value); 
      } 

      //find the max rank of high card from all potential winners 
      foreach (var kvp in highTable) 
      { 
       if (kvp.Value > maxRank) 
       { 
        maxRank = kvp.Value; 
       } 
      } 

      Console.WriteLine("The final winner after tie-breaking is"); 

      //display the winner(s) with the highest rank of card 
      foreach (var kvp in highTable) 
      { 
       if (kvp.Value == maxRank) 
       { 
        Console.WriteLine("" + kvp.Key + ""); 
       } 
      } 
     } 
     //if there is only one potential winner, display the name 
     else 
     { 
      Console.WriteLine("The final winner is"); 
      Console.WriteLine(potentialWinners[0]); 
     } 
    } 
} 

Любая помощь или советы будут весьма признателен!

Answer 1

for (int i = 0; i < potentialWinners.Count(); i++) 
{ 
    Console.WriteLine("" + potentialWinners[i] + ""); 
} 

//add the name of the potential winners and the rank of the high card as a key value pair to highTable 


for (int i = 0; i < potentialWinners.Count(); i++) 
{ 
    if (deckTable.ContainsKey(potentialWinners[i])) 
    { 
     Card[] cHand = deckTable[potentialWinners[i]]; 
     highTable.Add(potentialWinners[i], GetHighCard(cHand)); 
    } 
} 

Вы, в основном вызывая то же самое для цикла 2 раза, вы могли бы принести

Console.WriteLine("" + potentialWinners[i] + ""); 

в цикл после него ... также PLS удалить ("") из вашей консоли WriteLine нет его делать в принципе ничего, но делает ваш код медленнее

то же самое, как говорят, для этих петель

foreach (var kvp in highTable) 

и

foreach (var kvp in scoreTable) 

простые вещи, чтобы сделать ваш код smaler и более эффективным

Приветствия

Answer 2

Ваша самая большая ошибка состоит в том, чтобы игнорировать какой язык вы используете. Ваш стиль необходим, а не объектно-ориентированный.

Что вы хотите сделать, это создать классы для обработки таких вещей, как cardHand или scoreTable. Затем вы даете им методы, такие как getBestHand. Часть кода, в котором распечатывается материал должен выглядеть как этот псевдо-код:

печати «Победитель является» + table.getBestHand.getPlayer.name

, то есть без каких-либо для каждого цикла или тому подобного в этой части кода. Вы хотите разделить логику и вывод. Обычно это хороший знак, когда одна функция или метод имеет размер не более половины экрана (конечно, исключения). Возьмите все, что вы могли бы описать одним словом, но имеет более пяти строк кода и сделать из него метод.

Для этого изолированного примера это, конечно же, даст больше кода, но я полагаю, что даже в игре в покер, если у вас уже есть соответствующие классы, вы будете платить. Если не в количестве кода, то, по крайней мере, в читаемости и ремонтопригодности.