画面遷移を繰り返すとエラーが発生する

Question

###前提・実現したいこと
UITableViewでセルを選択して画面遷移すると選択したセルに該当する星座の地平座標（現在時刻・現在位置から見たときの方位角と高度）を算出し、スマホを空にかざすと画面に表示された矢印がその方向を示してくれるというようなアプリを作っています。

遷移後の画面から星座選択の画面に戻り、また星座を選択して画面遷移する、ということを1、２回繰り返しているとアプリが落ちて以下のようなエラーが出ていました。

何度も画面を行ったり来たりしてきちんと矢印の図を表示するところまで出来るようにしたいです。

UITableViewで選択した星座の地平座標を算出するために日時と位置情報を取得し、さらに加速度センサーと方位センサー（電子コンパス）を使用しています。

文字数制限のためソースコードを大幅に省略していてわかりにくいと思いますがアドバイス頂けると助かります。

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

unexpectedly found nil while unwrapping an Optional value

見てみると、スマホの傾き（a）がnilになっていました。

###SecondViewController.swift（星座選択画面）

import UIKit

class SecondViewController: UIViewController, UITableViewDelegate, UITableViewDataSource {
    
    @IBOutlet weak var Label2: UILabel!
    
    @IBOutlet weak var tableView: UITableView!
    
    // Tableで使用する配列を設定する
    private let myItems: NSArray = ["いて座", "うお座", "おうし座", "おおいぬ座", "おおぐま座", "おとめ座", "おひつじ座", "オリオン座", "カシオペア座", "かに座", "こいぬ座", "こぐま座", "こと座", "さそり座", "しし座", "てんびん座", "はくちょう座", "ふたご座", "みずがめ座", "やぎ座", "わし座"]

    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        Label2.text = "探したい星座を選ぼう！"
    }
    
    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        // Dispose of any resources that can be recreated.
    }
    

    override func viewWillAppear(_ animated: Bool) {
        super.viewWillAppear(animated)
        
        if let indexPathForSelectedRow = tableView.indexPathForSelectedRow {
            tableView.deselectRow(at: indexPathForSelectedRow, animated: true)
        }
    }
    
    override func viewDidAppear(_ animated: Bool) {
        super.viewDidAppear(animated)
        if let indexPathForSelectedRow = tableView.indexPathForSelectedRow {
            tableView.deselectRow(at: indexPathForSelectedRow, animated: true)
        }
    }


    // Cellの総数を返す.
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return myItems.count
    }
    
    // Cellに値を設定する
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        // 再利用するCellを取得する.
        let cell = tableView.dequeueReusableCell(withIdentifier: "MyCell", for: indexPath as IndexPath)
        
        // Cellに値を設定する.
        cell.textLabel!.text = "\(myItems[indexPath.row])"
        
        return cell
    }
    
    // Cellが選択された際に呼び出される
    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        print("Num: \(indexPath.row)")
        print("Value: \(myItems[indexPath.row])")
        
        let storyboard: UIStoryboard = self.storyboard!
        let nextView = storyboard.instantiateViewController(withIdentifier: "third") as! ThirdViewController
        
        let seizaNum = indexPath.row
        
        let appDelegate: AppDelegate = UIApplication.shared.delegate as! AppDelegate
        appDelegate.seizaNum = seizaNum
        
        self.present(nextView, animated: true, completion: nil)
    }
 
    @IBAction func backToSecond(segue: UIStoryboardSegue) {}
    
    /*
    // MARK: - Navigation

    // In a storyboard-based application, you will often want to do a little preparation before navigation
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        // Get the new view controller using segue.destinationViewController.
        // Pass the selected object to the new view controller.
    }
    */
}

###ThirdViewController.swift（画面遷移後）

import UIKit
import CoreMotion
import CoreLocation


class ThirdViewController: UIViewController, CLLocationManagerDelegate  {
    
    // 座標(0,0)の位置にUIImageViewを幅300,高さ300で作成する
    let myImageView = UIImageView(frame: CGRect(x:0, y:0, width:300, height:300))
    
    var myMotionManager: CMMotionManager!
    var locationMgr: CLLocationManager!
    
    var a: Double!
    var MJD2: Double!
    var δ: Double!
    var α: Double!
    var deg_A2: Double!
    var deg_h: Double!
    var difY: Double!

    override func viewDidLoad() {
        super.viewDidLoad()
        
        let appDelegate: AppDelegate = UIApplication.shared.delegate as! AppDelegate
        let seizaNum = appDelegate.seizaNum
        
        // 探したい星の赤緯（δ[度]）、赤経（α[時]）
        // いて座
        if seizaNum == 0 {
            self.δ = -25.0
            self.α = 19.0
            
            Label3.text = "いて座を探しています..."
        }
        // うお座
        else if seizaNum == 1 {
            self.δ = 15.0
            self.α = 1.0
            
            Label3.text = "うお座を探しています..."
        }
        /*
        省略
        */
        
        
        // 現在時刻を取得
        let myDate: Date = Date()
        
        // カレンダーを取得
        let myCalendar: NSCalendar = NSCalendar(calendarIdentifier: NSCalendar.Identifier.gregorian)!
        
        /*
        省略
        */     
        
        // 準ユリウス日（MJD2）を求める
        let Y = Double(myComponetns.year!)
        let M = Double(myComponetns.month!)
        let D = Double(myComponetns.day!)
        let hour = Double(myComponetns.hour!)
        /*
        省略
        */
        
        locationMgr = CLLocationManager()

        locationMgr.delegate = self
        
        // 位置情報の精度を設定
        locationMgr.desiredAccuracy = kCLLocationAccuracyBest
        locationMgr.distanceFilter = 100
        
        locationMgr.startUpdatingLocation()
        
        
        // 何度動いたら角度の変更が通知されるかを指定する（この場合動くたび）
        locationMgr.headingFilter = kCLHeadingFilterNone
        
        // デバイスのどの方向を北にするかを設定する（この場合デバイスの上部）
        locationMgr.headingOrientation = CLDeviceOrientation.portrait
        
        // センサーからの通知を受け取れるようにする
        locationMgr.startUpdatingHeading()

        myMotionManager = CMMotionManager()
        
        // 更新周期を設定
        myMotionManager.accelerometerUpdateInterval = 0.1
        
        // 加速度の取得を開始
        myMotionManager.startAccelerometerUpdates(to: OperationQueue.main, withHandler: {(accelerometerData, error) in
            if let e = error {
                print(e.localizedDescription)
                return
            }
            guard let data = accelerometerData else {
                return
            }
            
            // 取得した加速度を傾き（ラジアン値）に変換
            let a_radian = -atan(data.acceleration.y/sqrt(pow(data.acceleration.x,2) + pow(data.acceleration.z,2)))
            
            // ラジアンを度に変換
            self.a = fabs(a_radian*(180 / M_PI) - 90.0) // スマホの傾きをaとする。立てた状態のスマホを直通する軸を０度にする（-90°）。絶対値をとる。
        })
    }

    
    // 位置情報が更新された場合の処理
    func locationManager(_ manager: CLLocationManager, didUpdateLocations locations: [CLLocation]) {
        
        // 観測地の緯度（Ψ[度]）、経度（λ[時]）
        let Ψ = manager.location!.coordinate.latitude
        let λ = manager.location!.coordinate.longitude
        let λ2 = -λ / 15   // 東経は-、西経は+。 度を時に変換する
        
        // グリニッジ恒星時（θG[時]）を求める
        // 地方恒星時（θ[時]）を求める
        // 時角（H[度]）を求める
        // 度をラジアンに変換する
        /*
        省略
        */
        
        // 方位角（deg_A2）と高度（h）を求める
        let tanA = coshsinA / coshcosA
        let A = atan(tanA)
        let deg_A = A * (180.0 / M_PI)   // ラジアンを度に変換する
        
        // 象限ごとの場合分け
        if coshsinA > 0 && coshcosA > 0 {   // sinAは+、cosAも+ ⇨第１象限
            self.deg_A2 = deg_A
            
            // 高度（h）
            let A2 = self.deg_A2 * (M_PI / 180.0)   // 度をラジアンに変換する
            let cosA = cos(A2)
            let tanh = sinh / (coshcosA / cosA)
            let h = atan(tanh)
            self.deg_h = h * (180.0 / M_PI)   // ラジアンを度に変換する
        }
        /*
        省略
        */
    }

    
    // コンパスの角度に変化があった場合はlocationManagerメソッドが起動する
    func locationManager(_ manager: CLLocationManager, didUpdateHeading newHeading: CLHeading) {
        
        // デバイスの現在の向きを変数（myHoui）で定義する
        let myHoui: Double = newHeading.magneticHeading
        
        // ある一点の高度（h）とデバイスの傾き（a）を比較する
        self.difY = self.deg_h - self.a

        switch self.a {
        // デバイスの傾き（a）が0~45度のとき
        case 0...45:
            // 探したい星の方位角（deg_A2）が180°より大きいとき
            if self.deg_A2 > 180.0 {
                if myHoui > (self.deg_A2 - 180.0) && myHoui <= (self.deg_A2 - 5.0) {
                    if self.difY > 5.0 {
                        let myImage = UIImage(named: "myarrow45.png")
                        myImageView.image = myImage
                        myImageView.layer.position = CGPoint(x: self.view.bounds.width/2, y: 250.0)
                        self.view.addSubview(myImageView)
                    }
                    /*
                    省略
                    */
        default:
            break
        }
    }

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        // Dispose of any resources that can be recreated.
    }
    
    /*
    // MARK: - Navigation

    // In a storyboard-based application, you will often want to do a little preparation before navigation
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        // Get the new view controller using segue.destinationViewController.
        // Pass the selected object to the new view controller.
    }
    */
}

###補足情報(言語/FW/ツール等のバージョンなど)
Swift3.0
Xcode8.0

Answer 1

MotionManagerのハンドラの中でaに値が代入される前に、locationManager(_:didUpdateHeading:)が呼び出されているのではないでしょうか？printを追加するなどして確認してみて下さい。

もしそうであれば、locationManager(_:didUpdateHeading:)の最初にnilチェックを行えば良いと思います。

func locationManager(_ manager: CLLocationManager, didUpdateHeading newHeading: CLHeading) {
    guard let a = self.a else {
        return //a=nilなので何もせずに戻る
    }
    //以下、傾きは a でアクセス出来る
    :
    // ある一点の高度（h）とデバイスの傾き（a）を比較する
    self.difY = self.deg_h - a
    :

こんな感じ？（動作確認はしていません）