【Xcode,Swift】Buttonをタップしたら、Thread 1: signal SIGABRTが出てきた

Question

swift4
xcode9

ボタンをタップしたら、突然Thread 1: signal SIGABRTが出ました。
全部のOutletとActionの紐付けを解除し、もう1度つなぎなおしました。
変数名も一部前と変更しました。
しかし、解決しません。

import UIKit

class ResultViewController: UIViewController {
    
    override func viewDidLoad() {
        super.viewDidLoad()

        // Do any additional setup after loading the view.
        displayresult()
        result()
    }

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        // Dispose of any resources that can be recreated.
    }
    
    @IBOutlet weak var PLAYERLabel: UILabel!
    
    @IBOutlet weak var CPUULabel: UILabel!
    
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if segue.identifier == "WIN" {
            let gameover_instance = segue.destination as! GameOverViewController
            self.score = 1
            gameover_instance.Score += self.score
        }
        
    }
    
    
    //勝敗の判定を表す変数
    var winorlose = ""
    
    //スコアを保持する変数
    var score = 0
    
    //ChoiceHandViewControllerから渡される値を格納する変数
    var Player_Hand = ""
    var CPU_Hand = ""
    
    //両者の手を表示
    func displayresult() {
        //プレイヤーの手を表示
        PLAYERLabel.text = Player_Hand
        //CPUの手を表示
        CPUULabel.text = CPU_Hand
    }
    
    //勝敗を判定する処理
    func result() {
        if Player_Hand == CPU_Hand {
            winorlose = "lose"
            print("bbbb")
        } else {
            winorlose = "win"
            print("aaaa")
        }
    }
    
    
    //プレイヤーが勝ちなら前画面に戻る、負けならゲーム終了
    @IBAction func ChangeViewButton(_ sender: Any) {
        if winorlose == "win" {
            performSegue(withIdentifier: "WIN", sender: nil)
        } else if winorlose == "lose" {
            performSegue(withIdentifier: "LOSE", sender: nil)
        }
    }
    

    /*
    // 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.
    }
    */

}

import UIKit

class GameOverViewController: UIViewController {

    override func viewDidLoad() {
        super.viewDidLoad()

        // Do any additional setup after loading the view.
        
        display_score()
    }

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        // Dispose of any resources that can be recreated.
    }
    
    @IBOutlet weak var ScoreLabel: UILabel!
    
    
    //ResultViewControllerで渡されたscoreを格納する変数
    var Score = 0
    
    //勝った回数を表示
    func display_score() {
        ScoreLabel.text = "(Score)"
    }
    

    /*
    // 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.
    }
    */

}

下部がMainStoryBoard、コンソール、エラー箇所の画像です。
どこが原因なのか教えていただけないでしょうか？
よろしくお願いします。

「追記」

上記のエラーコードがこちらです。

Could not cast value of type 'MyAcchiMuiteHoi.ChoiceHandViewController' 
(0x108a8ba50) to 'MyAcchiMuiteHoi.GameOverViewController' (0x108a8b990). 

2018-08-22 09:38:55.508647+0900 MyAcchiMuiteHoi[8071:235142]
 Could not cast value of type 'MyAcchiMuiteHoi.ChoiceHandViewController' 
(0x108a8ba50) to 'MyAcchiMuiteHoi.GameOverViewController' (0x108a8b990). (lldb)

Comments

[fuzzball]

let gameover_instance = segue.destination as! GameOverViewController の上に print(segue.destination) を追加して出力を教えて下さい。

[nekokichi]

<MyAcchiMuiteHoi.ChoiceHandViewController: 0x7fcf1dc29210> <MyAcchiMuiteHoi.ChoiceHandViewController: 0x7fcf1dc29210> Could not cast value of type 'MyAcchiMuiteHoi.ChoiceHandViewController' (0x10864da78) to 'MyAcchiMuiteHoi.GameOverViewController' (0x10864d9b8). 2018-08-22 09:36:59.398950+0900 MyAcchiMuiteHoi[7979:233301] Could not cast value of type 'MyAcchiMuiteHoi.ChoiceHandViewController' (0x10864da78) to 'MyAcchiMuiteHoi.GameOverViewController' (0x10864d9b8). (lldb)

[nekokichi]

print(segue.destination)がない場合が、こちらです。Could not cast value of type 'MyAcchiMuiteHoi.ChoiceHandViewController' (0x108a8ba50) to 'MyAcchiMuiteHoi.GameOverViewController' (0x108a8b990). 2018-08-22 09:38:55.508647+0900 MyAcchiMuiteHoi[8071:235142] Could not cast value of type 'MyAcchiMuiteHoi.ChoiceHandViewController' (0x108a8ba50) to 'MyAcchiMuiteHoi.GameOverViewController' (0x108a8b990). (lldb)

[fuzzball]

質問内に追記して下さい。（改行位置が分かりにくいです）

[fuzzball]

``` で囲ってもらえますか‥。

[fuzzball]

エラーを見ると、ChoiceHandViewControllerに遷移しようとしていますが？（print追加前のログで十分です。print(segue.destination)を追加したときのログは不要なので削除して下さい）

[nekokichi]

はい、もしwinorlose変数の値が"WIN"（Player_HandとCPU_Handが一致しない場合）なら、ChoiceHandViewControllerに遷移するようになってます。

[nekokichi]

今シミュレータで試したのですが、GameOverVIewControllerへ遷移する場合は正常に動作します。おそらく、ChoiceViewControllerへ遷移する際の処理に問題があるかと思います。

Answer 1

//ResultViewController / prepare(for:sender:)
let gameover_instance = segue.destination as! GameOverViewController

segue.destinationには遷移先ViewControllerのインスタンスが入っていますので、今回の場合はChoiceHandViewControllerが入っています。それをGameOverViewControllerにキャストしようとして失敗し落ちています。

そもそも、この時点でGameOverViewControllerのインスタンスは生成されていません。
処理の流れを見直して下さい。

Comments

[nekokichi]

確かに、segue.identifier == "WIN"（ChoiceHandViewController）へ遷移するはずなのに、違うsegueが選択されてました。

ご指摘いただいた箇所をコメントアウトしてみると、正常に動作しました。

しかし、このままだと、ChoiceHandViewControllerのscoreの値をGameViewControllerに渡せません。

ご指摘いただいた箇所は、Player_HandとCPU_Handが一致しない時にscoreに1が加算され、そのままChoiceHandViewControllerに移行して再び両者の手が一致しない時に、scoreが初期化されてしまうのです。

なので、初期化される前に、GameOverViewControllerのScoreにscoreを渡そうとしたのです。

[nekokichi]

いえ、少々お待ちください。
もうすぐ直せそうなので、しばらくお待ちください。

[nekokichi]

Result..Controllerのscore変数をGame..Controllerだけでやり取りするのではなく、

Result..Controllerで勝敗を判定して勝ったらscoreに1を追加し、
Choice..ControllerのSCORE変数に渡して、再びResult..Controllerに遷移する際に、SCORE変数の値をscore変数に渡し、
もし勝敗が負けの場合、Result..Controllerのscore変数をGame..ControllerのScore変数に渡しました。

つまり、Result..Controllerで取得したscoreの値を"WIN"の遷移先と共有し、最終的にLOSEの遷移先へと渡せばよかったみたいです。

ここまで付き合っていただきありがとうございました。
なんども助けていただき助かりました。

[fuzzball]

少々まどろっこしいですが、とりあえずはそれでいいと思います。

スコア変数剥き出しなのもアレなので、スコア管理クラスを作ってViewController間で持ち回るとか、持ち回らずにシングルトンにするとか。（シングルトンに抵抗がなければ）

[nekokichi]

おっしゃる通り、score,Score,SCOREと３つも変数を作り、いちいち値を渡すのは非効率だと思います。
できれば、まとめて管理するとか、シンプルな処理で実現したいです。
一応解決したので、これで終了します。
ありがとうございました。