画面遷移先のクラスに値を渡したい

画面遷移先のクラス（MusicPostViewController）のquestionPostLabelといラベルのtext にこの遷移元のクラス（MusicViewController）のquestionLabelのテキストを渡したいです。

segueで渡そうと思い@IBAction func toMusicpost()で以下のように書いたのですが、Thread 1: Fatal error: Unexpectedly found nil while implicitly unwrapping an Optional valueというエラーが出てしまいます。
これは予期せぬエラーが見つかったということなのでas! Stringと書いてみましたが変化なしでした。。
接続されていることも確認しましたが、どういった対処が必要でしょうか。

何かヒント頂けると幸いです。

class MusicViewController: UIViewController ,IndicatorInfoProvider {
    
    //ここがボタンのタイトルに利用されます
    var itemInfo: IndicatorInfo = "音楽"
    var qestionlabel :String = ""
    @IBOutlet weak var questionLabel: UILabel!
    @IBOutlet weak var label: UILabel!
    override func viewDidLoad() {
        super.viewDidLoad()
        questionLabel.text = "オススメの曲は？"
    }
    
    //必須
    func indicatorInfo(for pagerTabStripController: PagerTabStripViewController) -> IndicatorInfo {
        return itemInfo
    }
    @IBAction    func toMusicpost() {
        
        performSegue(withIdentifier: "toMusicPost", sender: nil)
        
    }
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if segue.identifier == "toMusicPost"{
            let musicPostViewControllrt = segue.destination as! MusicPostViewController
           //ここでThread 1: Fatal error: Unexpectedly found nil while implicitly unwrapping an Optional valueのエラー 
            musicPostViewControllrt.questionPostLabel.text = questionLabel.text as! String
        }
        
    }
}

解決
遷移先(MusicPostViewController)のviewdidloadにquestionPostLabel.text = questionlabelと表示しnilが入らないようにして、そのquestionlabelに遷移元で表示したquestionable(=questionLabel.text)を代入することで成功しました！
回答者の皆様本当にありがとうございました！！

遷移元

class MusicViewController: UIViewController ,IndicatorInfoProvider {
    
    //ここがボタンのタイトルに利用されます
    var itemInfo: IndicatorInfo = "音楽"
    var questionlabel:String?
    @IBOutlet weak var questionLabel: UILabel!
    @IBOutlet weak var label: UILabel!
    override func viewDidLoad() {
        super.viewDidLoad()
        questionLabel.text = "オススメの曲は？"
    }
    
    //必須
    func indicatorInfo(for pagerTabStripController: PagerTabStripViewController) -> IndicatorInfo {
        return itemInfo
    }
    @IBAction    func toMusicpost() {
　　　　　//questionlabelにquestionLabel.text代入
        questionlabel = questionLabel.text
        performSegue(withIdentifier: "toMusicPost", sender: nil)
        
    }
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if segue.identifier == "toMusicPost"{
            let musicPostViewControllr = segue.destination as! MusicPostViewController
            
            musicPostViewControllr.questionlabel = questionlabel
        }
        
    }
}

遷移先

class MusicPostViewController: UIViewController,UINavigationControllerDelegate, UIImagePickerControllerDelegate, UITextViewDelegate {
    
    let placeholderImage = UIImage(named: "photo-placeholder")
    let musicViewController = MusicViewController()
    
    var resizedImage: UIImage!
　　//questionlabe保持
    var questionlabel:String?
    @IBOutlet weak var questionPostLabel: UILabel!
    @IBOutlet var postImageView: UIImageView!
    
    @IBOutlet var postTextView: UITextView!
    
    @IBOutlet var postButton: UIBarButtonItem!
    
    override func viewDidLoad() {
        super.viewDidLoad()
        postTextView.delegate = self
　　　　　//初期値にquestionlabelが入る
        questionPostLabel.text = questionlabel
        
    }

takabosoft

2019/08/27 05:14 編集

うーん、これってコンパイル通りますかね？ MusicPostViewControllerのソースも載せてください。

sussan0416

2019/08/27 05:17

MusicPostViewController の実装も追加してもらえると、回答しやすくなります。また、可能であれば、エラーが発生したときのスクリーンショットもあると、より判断しやすいかなと思います。

ROKIDOG

2019/08/27 07:56

sussan0416さんご指摘いただきありがとうございます！判断しやすい文になるようこれから活かしていきます！アドバイスありがとうございました！！

行動規範の内容に同意します

回答1件

ベストアンサー

prepareForSegueの段階では、遷移先のViewControllerのviewは初期化されていないため、IBOutletなどで接続しているオブジェクトはnilになっています。
初期化前のnilの状態にアクセスしているためクラッシュが発生しています。

解決法としては遷移先のViewControllerに文字列を保持するプロパティを追加して、遷移先のViewControllerのviewDidLoadなどでそれをセットしてあげる方法などがあります。

swift
1
2// 遷移元ViewController
3
4class MusicViewController: UIViewController {
5  ...
6  override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
7    if segue.identifier == "toMusicPost"{
8      let musicPostViewControllrt = segue.destination as! MusicPostViewController
9      musicPostViewControllrt.question = questionLabel.text as! String
10    }
11  }
12  ...
13}
14
15// 遷移先ViewController
16
17class MusicPostViewController: UIViewController {
18  ...
19  @IBOutlet weak var questionPostLabel: UILabel!
20  var question: String?
21
22  override func viewDidLoad() {
23    super.viewDidLoad()
24    questionPostLabel.text = question
25    ...
26  }
27  ...
28}

投稿2019/08/27 05:13

simorgh3196

総合スコア157

takabosoft

2019/08/27 05:21

十中八九これだと私も思いますが、質問者さんのコードだと > musicPostViewControllrt.questionPostLabel.text = ... ではなく、 > musicPostViewControllrt.questionPostLabel = ... と書かれているので遷移先のoutletの問題ではないのかなとも思っているところです。（それかソースの載せ間違えか・・・）