引数の値が変更できず、スロットが止まらない。

Question

MacOS Catalina 10.15.4
Xcode 11.5 beta
Swift version 4

簡単なスロットアプリを作っています。
1秒毎にスロットが回って7つある画像を順番に表示し、ストップボタンを押せば止まるような仕組みを想定しています。
以下のようにcheckをBool値で設定し、trueの間は回り続け、ストップボタンを押せばflaseに変わって止まるようにしたいです。

import UIKit

class ViewController: UIViewController {

    @IBOutlet weak var image1: UIImageView!
    var check1 = true
    var picarray:[UIImage] = []


    override func viewDidLoad() {
        super.viewDidLoad()

        // UIImage インスタンスの生成
        let pic1:UIImage = UIImage(named:"pic1")!
        let pic2:UIImage = UIImage(named:"pic2")!
        let pic3:UIImage = UIImage(named:"pic3")!
        let pic4:UIImage = UIImage(named:"pic4")!
        let pic5:UIImage = UIImage(named:"pic5")!
        let pic6:UIImage = UIImage(named:"pic6")!
        let pic7:UIImage = UIImage(named:"pic7")!
        
        picarray = [pic1, pic2, pic3, pic4, pic5, pic6, pic7]
        
        
        // UIImageView 初期化
        image1.image = pic1

       move(image: image1, check: check1)
}

    func move(image: UIImageView, check: Bool){
        if check == true{
       　if n >= 7{ n = 0 }
        image.image = picarray[n]
        n += 1
        
        DispatchQueue.main.asyncAfter(deadline: .now() + 1) {
            
            self.move(image: image, check: check)
            }
        }
    }
    
    @IBAction func stop1(_ sender: Any) {
       check1 = false
    }

ただ、ボタンを押してcheck1 = falseにしても次の1秒には```true

この記述だと```move()```に渡した```check1```は```true```のまま値は変わらないのでしょうか。
(試しに```func move()```中のcheckを引数ではなく```check1```に変えると、ボタンを押せばスロットが期待通り止まります。)

ストップボタンは他にも複数あり、できたら汎用性の高い一つの```func```で表したいです。

初心者でコードが汚く恐縮ですが、記述の仕方に誤りなどありましたらご教示頂けると幸いです。
お手数ですが、宜しくお願い致します。

Answer 1

move(image: image1, check: check1)

関数の引数はコピー渡しですので、ここで渡されたcheck1と、関数の仮引数であるcheckはもはや別の実体（インスタンス）となってしまいます。

したがって、

 @IBAction func stop1(_ sender: Any) {
       check1 = false
    }

ここでcheck1の値を変更しても、その値は｀move()`という関数の中には反映されません。

したがって

試しにfunc move()中のcheckを引数ではなくcheck1に変えると、ボタンを押せばスロットが期待通り止まります。

のように、check1を直接参照するようにすれば期待通りの動きとなります。

inoutを使えば参照渡しにすることは可能ですが管理が面倒になりそうなので、別のロジックを考えた方がいいかもしれません（想像だけですが、クラスや構造体を作って1つのスロットを1つのインスタンスにすれば管理は色々楽になるかもしれません）。

Comments

[Dave]

早速ご回答頂きありがとうございます。
値型と参照型の違い…というものでしょうか。
コピーされるので元の値を変更しても反映されないんですね。勉強になります。
動かすだけなら関数だけでなんとかなりそうですが、これを機にクラスや構造体を試してみようと思います。