はじめに
Rubyでトランプゲーム「戦争」を実装する課題に取り組んだ際、ループ処理のロジックで複数回ハマりました。
この記事では「なぜそう書いてはいけないか」という原因と対策をセットで残しておきます。
トランプゲームの戦争のルール概要
- 52枚のカードを2人に26枚ずつ配る
- 強さの順：A > K > Q > J > 10 > ... > 2
- お互いに手札の一番上を出して比較、強い方がすべての場札を獲得する
- 引き分けの場合、もう1枚出して勝負を続ける（決着がつくまで繰り返す）
- 誰かの手札が0枚になったらゲーム終了
この「引き分けのたびにカードを積み上げる」処理がループの肝になります。
ハマりポイント① ループ変数を使わないまま書いてしまう
問題のコード（例）
for i in players
if name == "player1"
puts "Yes"
end
end
何が起きるか
ループ変数 i を条件式の中で一切使っていないため、players の要素数だけ puts "Yes" が繰り返し実行される。
# players が2要素なら
Yes
Yes
原因
ループを書いた目的（「コレクションの中から条件に合うものを探す」）と、実際のコード（「ループ変数を使わずに固定条件を繰り返す」）がズレている。
対策
ループが本当に必要かどうかを問い直す。要素が含まれているか調べるだけなら include? で十分。
# ループ不要
if players.include?("player1")
puts "Yes"
end
ループを使う場合は、必ずループ変数を条件式の中で活用する。
for player in players
if player == "player1" # ← 変数 player を使う
puts "Yes"
end
end
ハマりポイント② field と played を分けずに管理する
問題のコード（例）
field = []
loop do
# 両プレイヤーがカードを出す
field << player1.draw
field << player2.draw
if field[0].strength > field[1].strength # ← インデックスがズレる！
player1.robcards(field)
field = []
elsif field[0].strength < field[1].strength
player2.robcards(field)
field = []
else
puts "引き分けです。"
# field はリセットしない → 次のループで field が増えていく
end
end
何が起きるか
引き分けが続くと field に以下のようにカードが蓄積する。
1回目：field = [p1のカード, p2のカード]
2回目：field = [p1のカード, p2のカード, p1のカード, p2のカード]
3回目：field = [p1のカード, p2のカード, p1のカード, p2のカード, p1のカード, p2のカード]
field[0] と field[1] は常に「最初に出したカード」を指してしまい、今回出したカードで比較ができない。
原因
「場に蓄積するカード（field）」と「今回の比較に使うカード（played）」を同じ変数で管理しようとしたため。
対策
役割を分けて2つの変数で管理する。
field = [] # loop の外で初期化（引き分けカードの蓄積用）
loop do
played = [] # loop の中で初期化（今回出したカードの比較用）
played << player1.draw
played << player2.draw
field.concat(played) # 今回のカードを field に追加
if played[0].strength > played[1].strength # ← played で比較
player1.robcards(field)
field = [] # 勝負がついた時だけリセット
elsif played[0].strength < played[1].strength
player2.robcards(field)
field = []
else
puts "引き分けです。"
# field はリセットしない（次のループへ）
end
end
| 変数 | 役割 | 初期化タイミング |
|---|---|---|
field |
引き分け時も含めた全場札 |
loop の外で一度だけ |
played |
今回のターンで出したカード（比較用） |
loop の中で毎回 |
ハマりポイント③ concat の参照問題（dup を使う）
問題のコード（例）
def robcards(field)
@cards.concat(field)
field.clear # field をクリアしたら @cards の中身も消える！
end
何が起きるか
concat は参照を追加するため、field と @cards の末尾が同じオブジェクトを指す状態になる。
field.clear を呼ぶと @cards 内の要素も消えてしまう。
原因
concat はオブジェクトへの参照を追加する。元の配列とコピー先の配列が同じオブジェクトを共有している。
対策
dup で別オブジェクトとしてコピーしてから concat する。
def robcards(field)
@cards.concat(field.dup.shuffle) # dup で独立したコピーを作る
end
また、勝者がもらったカードを shuffle してから手札に加えることで、同じパターンが繰り返されて無限ループになるリスクも防げる。
ハマりポイント④ 集計変数を puts の中で足し算してしまう
問題のコード（例）
total = 0
for i in numbers
puts total += i # ← ループのたびに出力してしまう
end
何が起きるか
ループのたびに途中経過が出力される。最終的な合計だけ出したい場合には使えない。
対策
集計とアウトプットを分ける。
total = 0
for i in numbers
total += i # 集計だけ
end
puts total # ループの後に出力
まとめ：ループを書く前のチェックリスト
| チェック項目 | NG例 | OK例 |
|---|---|---|
| ループ変数を使っているか |
for i in arr でも i を使わない |
条件式で必ず i を参照する |
| ループが本当に必要か | 要素の有無確認に for を使う |
include? などを使う |
| 比較用・蓄積用の変数を分けているか |
field だけで両方兼ねる |
field（蓄積）と played（比較）を分ける |
concat 後に元配列を変更していないか |
concat 後に field.clear する |
field.dup でコピーしてから concat
|
| 集計とアウトプットを分けているか | ループ内で puts total += i
|
ループ後に puts total
|
おわりに
ループ処理でバグが起きる多くのケースは、「ループが何をしているか」と「自分がやりたいこと」のズレから発生します。
コードを書く前に「このループは何を繰り返しているのか」「変数の役割はひとつに絞れているか」を言葉にする習慣が大切だと実感しました。
※ この記事はClaude(AI)を使って執筆・整理しました。