[swift] 4*4のまるばつゲームの不具合について

Question

前提・実現したいこと

４つ揃えると勝ちになるゲームを作成中です。

今、生じている不具合の内容です。

・まるを４つ揃えても、勝ちにならない。
・ばつを４つ揃えた後に引き続きプレイできる。

これを
・まるorばつを４つ揃えた時点でボタンを押せないようにしたい。

発生している問題・エラーメッセージ

エラーメッセージ

該当のソースコード

import UIKit

class ViewController: UIViewController {
    
    var activePlayer = 1 //cross
    var gameState = [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]
    
    let winningCombinations = [[0,1,2,3],[4,5,6,7],[8,9,10,11],[12,13,14,15],[0,4,8,12],[1,5,9,12],[2,6,10,14],[3,7,11,15]]
    
    var gameIsActive = true
    
    @IBOutlet weak var label: UILabel!
    
    
    
    @IBAction func action(_ sender: AnyObject)
    {
        if (gameState[sender.tag-1] == 0 && gameIsActive == true)
        {
            gameState[sender.tag-1] = activePlayer
            
            if (activePlayer == 1)
            {
                sender.setImage(UIImage(named: "Cross.png"),for: UIControl.State())
                activePlayer = 2
            }
            else {
                sender.setImage(UIImage(named: "Nought.png"),for: UIControl.State())
                activePlayer = 1
            }
        }
        
        for combination in winningCombinations
        {
            if gameState[combination[0]] != 0 &&
                // 0番目と1番目が同じコマか判断
                gameState[combination[0]] == gameState[combination[1]] &&
                // 1番目と2番目が同じコマか判断
                gameState[combination[1]] == gameState[combination[2]] &&
                // 2番目と3番目が同じコマか判断
                gameState[combination[2]] == gameState[combination[3]] {
                gameIsActive = false
                
                if gameState[combination[0]] == 1
                {
                    label.text = "✖️の勝ち！"
                }
                else
                {
                    label.text = "○の勝ち！"
                }
                
                playAgainButton.isHidden = false
                label.isHidden = false
            }
        }
        
        gameIsActive = false
        for i in gameState {
            if i == 0 {
                gameIsActive = true
                break
            }
        }
        if gameIsActive == false {
            label.text = "引き分け！"
            label.isHidden = false
            playAgainButton.isHidden = false
        }
    }
    
    @IBOutlet weak var playAgainButton: UIButton!
    @IBAction func playAgain(_ sender: AnyObject) {
        
        gameState = [0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0]
        gameIsActive = true
        activePlayer = 1
        
        playAgainButton.isHidden = true
        label.isHidden = true
        
        for i in 1...16 {
            let button = view.viewWithTag(i) as! UIButton
            button.setImage(nil, for: UIControl.State())
        }
    }
    
    override func viewDidLoad() {
        super.viewDidLoad()
    }
    
    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
    }
}

補足情報（FW/ツールのバージョンなど）

Answer 1

        for combination in winningCombinations {
            if gameState[combination[0]] != 0 && gameState[combination[0]] ==
                gameState[combination[1]] && gameState[combination[1]] ==
                gameState[combination[2]] && gameState[combination[2]] ==
                gameState[combination[3]] {
                gameIsActive = false
                
                if gameState[combination[0]] == 1 {
                    label.text = "✖️の勝ち！"
                } else {
                    label.text = "○の勝ち！"
                }
                
                playAgainButton.isHidden = false
                label.isHidden = false
                
                // この時点で勝負が決まっているので、return などで終了させる
            }
        }
        
        gameIsActive = false

        for i in gameState {
            if i == 0 {
                gameIsActive = true
                break
            }
        }

上記のロジックだと、勝負がついた結果 gameIsActive が false になったにもかかわらず、次の for-in 文で gameIsActive が true になってしまう可能性があります。

コード中のコメント通り、勝負が決まった時点で関数から抜けるなどすれば、期待通りの動きになるはずです。

    let winningCombinations = [[0,1,2,3],[4,5,6,7],[8,9,10,11],[12,13,14,15],[0,4,8,12],[1,5,9,12],[2,6,10,14],[3,7,11,15]]

上記の表ですが、一部間違いがあるようです。

[1,5,9,12] ではなく、[1,5,9,13]ではないでしょうか。

Comments

[isekiryu]

// この時点で勝負が決まっているので、return などで終了させる
return gameIsActive = false
と入れてみた結果、うまく行きました。２つ目の記事ありがとうございます。

[TsukubaDepot]

コンパイルは通っているかもしれませんが、文法的に間違っています。
@IBAction func action(_ sender: AnyObject)

と、action という関数は戻り値なしで宣言されていますので

return

だけで成立します。

[isekiryu]

そうなんですね。変更します。ありがとうございました。