ジャンケンアプリの作成

Question

前提・実現したいこと

ジャンケンアプリの作成になります。
✊✌️✋のいずれかのUIButtonを押すと、cpのimageViewが✊✌️✋の順番に画像がグルグルし、2秒後にlabelが勝敗判定（win, draw, lose）に変化します。同時にグルグルしていたimageViewがcpが出した手の画像に切り替わります。ちなみにcpの手はランダムです。

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

2回目以降のジャンケン結果がおかしくなる。
自分の押した手とcpの出力された手の正しい勝敗がlabelにprintされない。また、2回目以降imageViewのグルグルがなくなる。

該当のソースコード

import UIKit

final class ViewController: UIViewController {

    @IBOutlet weak var cpHand: UIImageView!
    @IBOutlet weak var janken: UILabel!
    @IBOutlet weak var selectbutton: UIButton!
    @IBOutlet weak var scissorsbutton: UIButton!
    @IBOutlet weak var paperbutton: UIButton!
    @IBOutlet weak var rockbutton: UIButton!
    
       var timer: Timer?
       var nowIndex:Int = 0
    
     enum hand: Int  {
           case rock = 0
           case scissors = 1
           case paper = 2
     }
    
     var cpHandImage:[UIImage] = [
            UIImage(named: "rockpic")!,
            UIImage(named: "scissorspic")!,
            UIImage(named: "paperpic")!
     ]
    
     override func viewDidLoad() {
          super.viewDidLoad()
          // Do any additional setup after loading the view.
     }
    
        @IBAction func selectbutton(_ sender: UIButton) {
        
           let Button = Int(sender.tag)
           guard let youHand = hand(rawValue: Button) else {
                return
           }
           guard let cpHand = hand(rawValue: Int.random(in: 0...2)) else {
                return
           }
        
               if timer == nil {
                   timer = Timer.scheduledTimer(timeInterval: 0.1, target: self, selector: #selector(changeImage), userInfo: nil, repeats: true)
               }
             DispatchQueue.main.asyncAfter(deadline: .now() + 1.99) {
                       self.timer?.invalidate()
                         self.janken(youHand: youHand, cpHand: cpHand)
                         self.result(youHand: youHand, cpHand: cpHand)
                         self.timer = nil
             }
            }
        
        @objc func changeImage() {
               nowIndex += 1
               if (nowIndex == cpHandImage.count) {
                   nowIndex = 0
               }
               cpHand.image = cpHandImage[nowIndex]
        }
        
          func janken(youHand: hand, cpHand: hand) {
            if youHand.rawValue == cpHand.rawValue {
                janken.text = "you draw"
            }else if (youHand.rawValue + 1)%3 == cpHand.rawValue {
                janken.text = "you win!"
            }else {
                janken.text = "you lose"
            }
          }
    
          func result(youHand: hand, cpHand: hand) {
                     if cpHand.rawValue == 0 {
                         self.cpHand.image = UIImage(named: "rockpic")
                     } else if cpHand.rawValue == 1 {
                         self.cpHand.image = UIImage(named: "scissorspic")
                     } else if cpHand.rawValue == 2 {
                         self.cpHand.image = UIImage(named: "paperpic")
                     }
          }
}

Comments

[TsukubaDepot]

掲載されているコードが崩れているようです。
このままだと見にくいですし、仮にコピーしても正しく動かすのに手間がかかりますので、整形しなおしていただけますでしょうか。

[Narikura]

申し訳ございません。修正いたしました。よろしくお願いいたします。

[Narikura]

ご回答いただきありがとうございます。ご回答のように実装いたしました（質問ページのcodeも伴って変更させていただきました）。しかしながら、✊✌️✋全てのボタンが✊ボタンとして処理されてしまっているようです。改善策を教えていただくことは可能でしょうか。自分としてはsender.tagで割り振られているものだと思っておりました。
追記：　これまでバラバラに宣言されていたbuttonを
　　　　　　　　　@IBOutlet var selectbutton: [UIButton]!
　　　でまとめましたが、変化はありませんでした。

[TsukubaDepot]

試しに変更後のコードで動かしてみましたが、私の環境だときちんと動いています。
もちろん、ボタンは sender.tag で割り振っていることに気付いていましたので、StoryBoard で各ボタンに0, 1, 2 のタグを割り振ってあります。

ちなみに、私の回答に対するご質問ですので、できれば回答直下のコメント欄にコメントいただければと思います。

[Narikura]

私が勘違いをしておりました。何かの拍子でtagが全て０になっていたようです。ありがとうございました。

コメントにつきましても以後気をつけます。不慣れなものですみませんでした。

Answer 1

    if (timer == nil) {
        timer = Timer.scheduledTimer(timeInterval: 0.1, target: self, selector: #selector(changeImage), userInfo: nil, repeats: true)
    }

ここで timerが nil か否かをチェックしていますが、2秒経過後に再度 timer を nil に戻していないため、実質このブロック内は初回のみしか実行されません。

たとえば、このような感じに変更すると期待したように動くかと思います。

import UIKit

final class ViewController: UIViewController {
    // 中略

    // MARK: 安全のため ?　のオプショナル型で宣言する
    //var timer: Timer!
    var timer: Timer?

    // 中略
    
    @IBAction func selectbutton(_ sender: UIButton) {
        
        let Button = Int(sender.tag)
        guard let youHand = hand(rawValue: Button) else {
            return
        }
        guard let cpHand = hand(rawValue: Int.random(in: 0...2)) else {
            return
        }
        
        // MARK: DispatchQueue に入れる
        //      Timer.scheduledTimer(withTimeInterval: 2.0, repeats: false) {_ in
        //          self.janken(youHand: youHand, cpHand: cpHand);
        //          self.result(youHand: youHand, cpHand: cpHand)
        //      }
        
        // MARK: 一般的に Swift の if では () は付けない
        //if (timer == nil) {
        if timer == nil {
            timer = Timer.scheduledTimer(timeInterval: 0.1, target: self, selector: #selector(changeImage), userInfo: nil, repeats: true)
        }
        
        DispatchQueue.main.asyncAfter(deadline: .now() + 1.99) {
            // MARK: timer は ? のオプショナル型なので、オプショナルチェーン（ドットの前が?）となる
            //self.timer.invalidate()
            self.timer?.invalidate()
            //　MARK: ラベルの更新なども DispatchQueue に入れてしまう
            self.janken(youHand: youHand, cpHand: cpHand)
            self.result(youHand: youHand, cpHand: cpHand)
            // MARK: timer を nil にする
            self.timer = nil
        }
    }

    // 後略
}

!付きのオプショナル型に再度 nil を代入することは可能なのですが、バグの温床になりかねないので、? 付きのオプショナル型として宣言した方がいいかと思います。

それに伴い、timer のメソッドにアクセスするためには、オプショナルチェーン（Optional Chaining）を使う必要がありますので、ご確認ください。

• Optional Chaining

また、2秒後の処理は一つの DispatchQueue にまとめた方が見通しが立ちやすいと思います。

ところで、現在の実装だと、ボタンを押した後結果が出るまでの間もボタンを押すことが可能となっています。このままだと問題が発生する可能性がありますので、一度ボタンを押したら結果を押すまでボタンを操作できないようにしたほうがいいかと思います。

• 【Swift5】Buttonの表示/非表示・有効/無効を切り替えてみる

Comments

[Narikura]

解決いたしました。２度にわたる質問にも答えていただきありがとうございました。とても分かりやすく手厚いご回答でした。