NSMutableArrayの重複オブジェクトの値を変更しますか？

Question

私のBlackjackアプリケーションでは、NSMutableArrayである手の値を取得する以下のメソッドがあります。ここで問題となるのは、2つのエースが手に入っているときは12でなければなりませんが、エースを11とカウントするため22になり、結果としてlowValueが返されます。

forループで既にAceが見つかっているかどうかを確認し、別のものを見つけたら、次のAceを11ではなく1にすることができます。

ありがとうございます！

int getHandValue(NSMutableArray *hand) { 
    int lowValue = 0; 
    int highValue = 0; 

    for (KCCard *aCard in hand) { 
      if (aCard.value == Ace) { 
       lowValue+= 1; 
       highValue+= 11; 
      } else if (aCard.value == Jack || aCard.value == Queen || aCard.value == King) { 
       lowValue += 10; 
       highValue += 10; 
      } else { 
       lowValue += aCard.value; 
       highValue += aCard.value; 
      } 
    } 

    return (highValue > 21) ? lowValue : highValue;  
} 

Answer 1

int getHandValue(NSMutableArray *hand) { 
    int lowValue = 0; 
    int highValue = 0; 
    BOOL isFoundAce = NO; 

for (KCCard *aCard in hand) { 
     if (aCard.value == Ace) { 
      if (isFoundAce) { 
       lowValue+= 1; 
       highValue+= 1; 
      } 
      else { 
       lowValue+= 1; 
       highValue+= 11; 
       isFoundAce= YES; 
      } 

     } else if (aCard.value == Jack || aCard.value == Queen || aCard.value == King) { 
      lowValue += 10; 
      highValue += 10; 
     } else { 
      lowValue += aCard.value; 
      highValue += aCard.value; 
     } 
} 

return (highValue > 21) ? lowValue : highValue;  
} 

Answer 2

おそらく、最初にNOに設定するforループの前にブール値を追加できます。 Aceが見つかったら、ブール値をYESに設定した後でforループを終了することができます。別のAce & &ブール値== YESになると、それに応じてケースを処理できます。例をzsxwingから冗長コードなし

Answer 3

私の例：

int getHandValue(NSMutableArray *hand) { 
    int cardValue = 0; 
    int aceCount = 0; 
    for (KCCard *aCard in hand) { 
     if (aCard.value == Ace) { 
      aceCount++; 
      cardValue += 11; 
     } else if (aCard.value == Jack || aCard.value == Queen || aCard.value == King) { 
      cardValue += 10; 
     } else { 
      cardValue += aCard.value; 
     } 
    } 
    while ((cardValue > 21) && (aceCount > 0)) { 
     cardValue -= 10; 
     aceCount--; 
    } 
    return cardValue; 
}