repeat文がうまく動作しません。

Question

前提・実現したいこと

タップする度に、画像①を表示→画像②を表示→画像を消す→画像①を表示（繰り返し）
というコードを書きたいのですが、現状、いきなり画像②が表示され、画像が消えた後繰り返し処理もされません。

何がいけないのでしょうか？？
ご教授のほどお願い致します。

質問の内容
①繰り返し処理が行われない。
②画像②から表示されてしまう。

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

特になし

該当のソースコード

import UIKit

class ViewController: UIViewController {

override func viewDidLoad() {
    super.viewDidLoad()
    // Do any additional setup after loading the view.
}

@IBOutlet weak var coin: UIImageView!
var coinCount = 0


@IBAction func tapCoin(_ sender: Any) {
    
    repeat {
    
    
    if coinCount == 0 {
        coin.image = UIImage(named: "myImage")
        
    } else if coinCount == 1 {
        coin.image = UIImage(named: "myImage2")
        
    } else if coinCount == 2 {
        coin.removeFromSuperview()
    }
    
    coinCount = coinCount + 1
        
    } while(coinCount < 2)
    
}

}

swift
Xcode11.4

単純なミスかもしれませんが、よろしくお願いいたします。

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

ここにより詳細な情報を記載してください。

import UIKit

class ViewController: UIViewController {

    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view.
    }

    @IBOutlet weak var coin: UIImageView!
    var coinCount = 0
    
    
    @IBAction func tapCoin(_ sender: Any) {
        
        if coinCount == 0 {                                    ↓エラーが出ます。
            coin.image = UIImage(named: "myImage")　　__　=Thread 1: Fatal error:__ 
        __ Unexpectedly found nil while implicitly unwrapping an Optional value__
            
        } else if coinCount == 1 {
            coin.image = UIImage(named: "myImage2")
            
        } else if coinCount == 2 {
            coin.removeFromSuperview()
        }
        
        coinCount = coinCount + 1
        coinCount = coinCount % 3
            

    }
    
}

Comments

[TsukubaDepot]

ソースコードは```で括ってください。
たとえば、

```swift
let a = 1
```

とすれば、他の質問のようにソースコードを整形した形で表示できます。

Answer 1

//画像の表示を非表示を繰り返すコードは、以下のコードになります。勉強になりました。

import UIKit

class ViewController: UIViewController {

override func viewDidLoad() {
    super.viewDidLoad()
    // Do any additional setup after loading the view.
}

@IBOutlet weak var coin: UIImageView!
var coinCount = 0


@IBAction func tapCoin(_ sender: Any) {
    
    if coinCount == 0 {
        coin.image = UIImage(named: "myImage")
        
    } else if coinCount == 1 {
        coin.image = UIImage(named: "myImage2")
        
    } else if coinCount == 2 {
        coin.image = nil
    }
    
    coinCount = coinCount + 1
    coinCount = coinCount % 3
        

}

}

Answer 2

次のようにすれば動きます。

@IBAction func tapCoin(_ sender: Any) {
    if coinCount == 0 {
        coin.image = UIImage(named: "myImage")
    } else if coinCount == 1 {
        coin.image = UIImage(named: "myImage2")
    } else if coinCount == 2 {
        coin.removeFromSuperview()
    }

    coinCount = coinCount + 1
}

repeat-while文は不要です。これを削除するだけで動くはずです。

オリジナルのコード通りにrepeat-while文で括ってしまうと、一度ボタンを押しただけでボタンを更新するすべての処理を実行してしまいます。

行いたいことは、「ボタンを押すたびに、押した回数に応じて画像を変更する」ということだと思います。

この場合、「繰り返して押す」という処理は人間がやっているわけなので、コードとして実装すべきところは、押された後の処理だけです。

Comments

[magiee]

TsukubaDepotさん、回答ありがとうございます。
とても分かりやすく。repeat文を理解することができました。

さらに質問をしても申し訳ないのですが、
coinCountが２までいった後、再度coinCounを０に戻して、
同じことを繰り返し行いたい場合は、どのようにすればよろしいのでしょうか？
アドバイスをいただけると大変嬉しいです。

[TsukubaDepot]

たとえば、0, 1, 2, 0, 1, 2と繰り返すのであれば、
coinCount = coinCount + 1
coinCount = coinCount % 3
とやってみてください。
% 演算子は剰余（割り算のあまり）を求める演算子です。
どのような整数であっても、3で割ったあまりは 0, 1, 2 の3通りですから、このように記述することで 0-2 の繰り返しが得られます。

[magiee]

回答ありがとうございます。
coinCount = coinCount % 3　を追記してみましたが、エラーが出ました。
coinCount == 2 から　０に戻るタイミングで、
Thread 1: Fatal error: Unexpectedly found nil while implicitly unwrapping an Optional value
という、エラーが coin.image = UIImage(named: "myImage")　の横に表示されました。

また、デバックエリアのほうでは、以下の文が出ました。
sender UITapGestureRecognizer 0x00000002809bc800
self tapImage.ViewController 0x000000014d4016f0

何か、間違っているのでしょうか？

[TsukubaDepot]

そのエラーが出る理由は、3回目のタップの処理で
　coin.removeFromSuperview()
を行っているからです。
ここを、
coin.image = nil
とすれば動くと思います。