画面遷移ができません。カウントダウンアプリを作成しています。

エラーは出ていないのですが、画面遷移ができず、開始ボタンを押すとそのまま固まってしまいます。

直接的な原因は、NextViewControllerで

    override func viewDidLoad() {
        super.viewDidLoad()
        runTimer()
        timer.invalidate() // <-ココが問題
        count = 0
        timerLabel.text = timeString(time: TimeInterval(getTime))
    }

という具合に、タイマーをinvalidate（無効にする、という意味）しているからだと思います。

画面遷移のやり方を教えていただきたいです。

提示していただいたソースを見ると、

       let storyboard: UIStoryboard = self.storyboard!
        let timer = storyboard.instantiateViewController(withIdentifier: "toNextViewController") as! NextViewController

        timer.getTime = timeTotal

        self.present(timer, animated: true, completion: nil)

        total()

        self.performSegue(withIdentifier: "toNextViewController", sender: nil)

とありますが、

• StoryBoardで作成したViewControllerのインスタンス化
• Segueでの画面遷移

の2つを混同して使われているように思えます。

元記事ではSegueを使っていませんので、前者に統一するのがいいかと思います。

また、値の渡し方がこれで合っているのかもわかりません。そのことに関してもアドバイスが欲しいです。

• StoryBoardで作成したViewControllerのインスタンス化

で値渡しをするのであれば、この方法で問題ないかと思います。Segueを使う場合にはまた異なってきます。

その他、配列の初期化方法や、PickerViewに渡すデータの扱い方など、先々問題に繋がりそうな部分もありましたので、コメントを入れてみました。

もしわからない点があればコメントいただくか、内容によっては新たにご質問いただければと思います。

import UIKit

class ViewController: UIViewController, UIPickerViewDelegate, UIPickerViewDataSource{

    @IBOutlet weak var start: UIButton!
    @IBOutlet weak var time: UIPickerView!

    // MARK: 変更
    // Int(arrayLiteral: ) はコンパイラが使う内部向け表現なので使わない
    // また、今回の目的だと一次元配列で済むため、次のように書き換える
    // let data = [[Int](arrayLiteral: 15,30,45,60,90,120)]
    let data = [15, 30, 45, 60, 90, 120]

    var pickerdata:String!
    // MARK: 変更
    // Int!型のように、!で宣言するオプショナル型の利用は極力避ける。
    // 今回の場合は単純にゼロで初期化すれば良い。
    //var timeTotal:Int!
    var timeTotal = 0

    @IBAction func start(_ sender: Any) {

        let storyboard: UIStoryboard = self.storyboard!
        let timer = storyboard.instantiateViewController(withIdentifier: "toNextViewController") as! NextViewController

        // MARK:
        // もし、timeTotalがオプショナル型で宣言されていたら、この時点で実行時エラーになる可能性が高い
        timer.getTime = timeTotal
        self.present(timer, animated: true, completion: nil)

        // MARK: 削除
        // present()で表示したのであれば、下記の行は不要
        //total()
        //self.performSegue(withIdentifier: "toNextViewController", sender: nil)

    }

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

        time.dataSource = self
        time.delegate = self

        // MARK: 追加
        // timeTotal の初期値を900（秒）にする。
        // 一番最初からピッカービューを動かさずに画面を遷移させても、初期値が代入されないことへの対応
        timeTotal = 15 * 60
    }

    func numberOfComponents(in pickerView: UIPickerView) -> Int {
        // MARK: 変更
        // PickerView に使うのは1列だけなので、今回は決め打ちで値を返す
        //return data.count
        return 1
    }

    func pickerView(_ pickerView: UIPickerView, numberOfRowsInComponent component: Int) -> Int {
        // MARK: 変更
        // 今回使うのは一次元配列なので、配列の要素数を .count を使って返す
        //return data[component].count
        return data.count
    }


    func pickerView(_ pickerView: UIPickerView, titleForRow row: Int, forComponent component: Int) -> String? {
        // MARK: 変更
        //return String(data[component][row])
        return String(data[row])
    }

    func pickerView(_ pickerView: UIPickerView, didSelectRow row: Int, inComponent component: Int) {
        total()
    }

    //    pickerの数字をたすメソッド
    func total() {
        // MARK: 変更
        // 一次元配列になったため計算方法を変更
        //timeTotal = data[0][time.selectedRow(inComponent: 0)] * 60
        timeTotal = data[time.selectedRow(inComponent: 0)] * 60
    }

}

import UIKit

class NextViewController: UIViewController {

    var timer = Timer()
    var count = 0
    var getTime:Int!

    @IBOutlet weak var timerLabel: UILabel!

    override func viewDidLoad() {
        super.viewDidLoad()
        runTimer()
        // MARK: 削除
        // invalidate()はタイマを止めるメソッドのため、記述するとタイマは動かずカウントダウンしない。
        //timer.invalidate()
        count = 0
        timerLabel.text = timeString(time: TimeInterval(getTime))
    }

    // カウントダウンをする関数
    @objc func updateTimer() -> Int {

        count += 1
        //時間　＝　pickerで設定する時間　ー　count
        // MARK: 変更
        // getTimeはオプショナル型だが、!で宣言されたオプショナル型なので、アクセス時のアンラップは不要
        //let remainCount = getTime! - count
        let remainCount = getTime - count
        timerLabel.text = timeString(time: TimeInterval(remainCount))

        //            timerLabel.text = "\(remainCount)"

        //        0秒になったら止まる
        if remainCount == 0 {
            timer.invalidate()
            timerLabel.text = "00:00:00"
        }
        return remainCount

    }

    //タイマーを動かす関数
    func runTimer() {
        // MARK: コメント
        // scheduledTimerは呼び出された直後から指定された関数の呼び出しを始める。
        timer = Timer.scheduledTimer(timeInterval: 1, target: self, selector: #selector(self.updateTimer), userInfo: nil, repeats: true)
    }

    //00:00:00に変える処理
    func timeString(time: TimeInterval) -> String {
        let hour = Int(time) / 3600
        let minutes = Int(time) / 60 % 60
        let second = Int(time) % 60

        return String(format: "%02d:%02d:%02d", hour, minutes, second)
    }

    // MARK: 削除
    // viewDidLoad()で同じような処理を行なっているので、どちらか一方に記述する
//    override func viewDidAppear(_ animated: Bool) {
//
//        timerLabel.text = timeString(time: TimeInterval(getTime))
//        print(getTime ?? "this is nil")
//
//    }

}