コード全体を正しくインデントし直すとすぐわかるはずなのですが、あなたのviewDidLoad()
はtimerUpdate()
メソッドの定義の中に完全に含まれてしまっています。
@objc func timerUpdate() {
//ここはtimerUpdate()メソッドの中
//...
func viewDidLoad() {
super.viewDidLoad()
myTimer = Timer.scheduledTimer(timeInterval: 1.0, target: self, selector: #selector(ViewController.timerUpdate), userInfo: nil, repeats: true)
}
//ここもまだtimerUpdate()メソッドの中
}
Swiftは言語仕様として、このようなネストされた関数定義を許しているのですが、内側のネストされた関数定義はインスタンスメソッドとはみなされず、外側のメソッドの中だけで有効なローカル関数になります。
従ってあなたのコードには、本来の(インスタンスメソッドとしての)viewDidLoad()
が存在しないと判断され、(ネストされた)内側のviewDidLoad()
も呼ばれることはありません。
(この辺り、コンパイラ技術的には「使用されていないローカル関数がある」警告を出すのはそれほど難しくないはずなのですが…。)
とにかく、結果としてあなたのmyTimer
には正しくスケジューリングされたTimer
は設定されておらず、(どんな挙動をするのかドキュメント化されていないイニシャライザinit()
で作られた)「何の役にも立たない」Timer
インスタンスが設定された状態になっています。当然のようにあなたがスケジュールしようとした動作は何も起こりません。
このような誤りが見つけにくくなっている原因としては、あなたのプロパティmyTimer
の宣言が典型的なバッドプラクティスのコードになっていることが挙げられます。
var myTimer = Timer()
Timer()
で得られるインスタンスがどのようなものか、はっきりあなたが理解していない限り、このようなプロパティ宣言は 絶対に するべきではありません。(残念ながらこのような非常に悪い書き方が、検索結果の上位に出てくることがしばしばあります。)
var myTimer: Timer?
のように宣言しておけば、初期化が終わっているはずのmyTimer
がnil
のままであることに早めに気付けるでしょう。
そこら辺も含めて、あなたのコードを書き直すとこんな感じになります。
import UIKit
class ViewController: UIViewController {
@IBOutlet weak var countLabel: UILabel!
var timeCount = 100
var myTimer: Timer? //### 変数を「何の役にも立たないインスタンス」で初期化するのは絶対に避けるべき
@objc func timerUpdate() {
timeCount = timeCount - 1
countLabel.text = String(timeCount)
if timeCount == 0 {
myTimer?.invalidate()
myTimer = nil //### invalidate()したTimerは再利用できないので、nilにしてしまう
}
}
override func viewDidLoad() { //### 正しいviewDidLoad()ならoverrideが無いとエラーになる
super.viewDidLoad()
//### targetにselfを指定したら、#selectorの中身もself.で始める
myTimer = Timer.scheduledTimer(timeInterval: 1.0, target: self, selector: #selector(self.timerUpdate), userInfo: nil, repeats: true)
}
}
そのほかにチェックすべき点、私の現在のオススメの書き方など###
入りのコメントで示しています。内容をお確かめの上おためしください。