上限を設定してもカウントアップが止まらない

Question

前提・実現したいこと

pickerviewのList2の配列でカウントアップの上限の回数を選んで、スタートするとカウントがList2で選んだ回数で止まるようにしたい

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

コードを一通り書いてみたが、セットした回数では止まらずカウントが続いた状態となった

該当のソースコード

import UIKit

class TimerViewController: UIViewController,UIPickerViewDelegate, UIPickerViewDataSource {
   
   @IBOutlet weak var timeCountLabel: UILabel!
   @IBOutlet weak var PickerView: UIPickerView!
    
    var timer: Timer?
    var interval = 0.0
    var count: Int = 0
    var upperlimit = 0
    //カウントアップの間隔
    let list1:[Double] = [0.5,1.0,1.5,2.0,2.5,3.0,3.5,4.0,4.5,5.0,5.5,6.0,7.5,8.0,8.5,9.0,9.5,10.0]
    //カウントの上限
    let list2:[Int] = [2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25,26,27,28,29,30]
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        PickerView.delegate = self
        PickerView.dataSource = self
        
        PickerView.selectRow(0, inComponent: 0, animated: false)
        PickerView.selectRow(1, inComponent: 1, animated: false)
        
        // 表示を初期化
        clear()
      
    }
    
    func pickerView(_ pickerView: UIPickerView, didSelectRow row: Int, inComponent component: Int)
    {
        print(component)
        if component == 0 {
            interval = list1[row]
            
        }
    }
    
    // ドラムロールの列数
    func numberOfComponents(in pickerView: UIPickerView) -> Int {
        return 2
    }
    
    // ドラムロールの行数
    func pickerView(_ pickerView: UIPickerView, numberOfRowsInComponent component: Int) -> Int {
        switch component {
        case 0:
            return list1.count
        case 1:
            return list2.count
        default:
           return 0
        
            
        }
    }
    
    // ドラムロールの各タイトル
    func pickerView(_ pickerView: UIPickerView, titleForRow row: Int, forComponent component: Int) -> String? {
        switch component {
            
        case 0:
            return String(format: "%.1f秒", Double(list1[row]))
        case 1:
            return String(format: "%d回", Int(list2[row]))
        default:
            return "error"
        
        }
    }
    
    @IBAction func start() {
        // 起動中のタイマーを無効化
        timer?.invalidate()
        
        

        timer = Timer.scheduledTimer(withTimeInterval: self.interval, repeats: true) { (timer) in
            
            print("counter")
            self.count = self.count + 1
            
            DispatchQueue.main.async {
                self.timeCountLabel.text = String(self.count)
            }
            
    func pickerView(_ pickerView: UIPickerView, didSelectRow row: Int, inComponent component: Int)
            {
                print(component)
                self.upperlimit = self.list2[row]
                
         if self.count == Int(self.list2[row]) {
            timer.invalidate()
                }
           }
        }
    }
    
    @IBAction func stop() {
        // 起動中のタイマーを無効化(ただし、count変数の中身とラベルの表示はそのまま)
        timer?.invalidate()
    }
    
    @IBAction func clear() {
        // 起動中のタイマーを無効化し、count変数とラベルの表示を初期化
        timer?.invalidate()
        count = 0
        self.timeCountLabel.text = String(self.count)
    }
    
    
}

。

Answer 1

実験してみたのでこれで大丈夫かと思います。
問題は、piclerView(:didSelectRowAt)が二つあって、使われていない方に必要なコードが書かれていたことでした。

import UIKit

class TestTimerViewController: UIViewController, UIPickerViewDelegate, UIPickerViewDataSource {
    @IBOutlet weak var timeCountLabel: UILabel!
    @IBOutlet weak var PickerView: UIPickerView!
    
    var timer: Timer?
    var interval = 0.0
    var count: Int = 0
    var upperlimit = 0
    //カウントアップの間隔
    let list1:[Double] = [0.5,1.0,1.5,2.0,2.5,3.0,3.5,4.0,4.5,5.0,5.5,6.0,7.5,8.0,8.5,9.0,9.5,10.0]
    //カウントの上限
    let list2:[Int] = [2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25,26,27,28,29,30]
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        PickerView.delegate = self
        PickerView.dataSource = self
        
        PickerView.selectRow(0, inComponent: 0, animated: true)
        interval = list1[0]
        PickerView.selectRow(1, inComponent: 1, animated: true)
        upperlimit = list2[1]
        // 表示を初期化
        clear()
        
    }
    
    
    func pickerView(_ pickerView: UIPickerView, didSelectRow row: Int, inComponent component: Int)
    {
        print("didSelectRow: (component), (row)")
        if component == 0 {
            interval = list1[row]
        } else {    // component == 1
            upperlimit = list2[row]
        }
    }

    // ドラムロールの列数
    func numberOfComponents(in pickerView: UIPickerView) -> Int {
        return 2
    }
    
    // ドラムロールの行数
    func pickerView(_ pickerView: UIPickerView, numberOfRowsInComponent component: Int) -> Int {
        switch component {
        case 0:
            return list1.count
        case 1:
            return list2.count
        default:
            return 0
        }
    }
    
    // ドラムロールの各タイトル
    func pickerView(_ pickerView: UIPickerView, titleForRow row: Int, forComponent component: Int) -> String? {
        switch component {
        case 0:
            return String(format: "%.1f秒", Double(list1[row]))
        case 1:
            return String(format: "%d回", Int(list2[row]))
        default:
            return "error"
        }
    }
    
    deinit {
        print("deinit")
    }
    
    @IBAction func start() {
        // 起動中のタイマーを無効化
        timer?.invalidate()
        
        print("Timer start. Up to (upperlimit).")
        
        timer = Timer.scheduledTimer(withTimeInterval: self.interval, repeats: true) { (timer) in
            
            print("counter:(self.count), limit:(self.upperlimit)")
            self.count = self.count + 1
            
            DispatchQueue.main.async {
                self.timeCountLabel.text = String(self.count)
            }
            
            if self.count >= self.upperlimit {
                timer.invalidate()
            }
        }
    }
    
    @IBAction func stop() {
        // 起動中のタイマーを無効化(ただし、count変数の中身とラベルの表示はそのまま)
        timer?.invalidate()
    }
    
    @IBAction func clear() {
        // 起動中のタイマーを無効化し、count変数とラベルの表示を初期化
        timer?.invalidate()
        count = 0
        self.timeCountLabel.text = String(self.count)
    }
    
}

Comments

[marogura]

やっと動きました！本当にありがとうございます。。。
eytyetさんには感謝しかないです。

[eytyet]

最初からきちんとした回答を提示できず、すいませんでした。動くと嬉しいですよね。私も嬉しいです。

Answer 2

テストしてないので動かなかったらごめんなさい。

    @IBAction func start() {
        // 起動中のタイマーを無効化
        timer?.invalidate()

        timer = Timer.scheduledTimer(withTimeInterval: self.interval, repeats: true) { [unowned self] (timer) in

            print("counter")
            self.count = self.count + 1

            DispatchQueue.main.async {
                self.timeCountLabel.text = String(self.count)
            }

            if self.count >= self.upperlimit {
                timer.invalidate()
           }
        }

    func pickerView(_ pickerView: UIPickerView, didSelectRow row: Int, inComponent component: Int) {
        print(component)
        self.upperlimit = self.list2[row]
    }

Comments

[marogura]

ダメでした。。。

[eytyet]

すいません。バグがあったので直してみました。どうでしょうか。

[marogura]

なぜかカウントが１で止まるようになりました。。

[eytyet]

ちょっと実験してから回答修正しますね。すいません。

[eytyet]

実験してみたら、循環参照による解放できない問題は起こりませんでした。何を間違えたかなぁ...