swift4のTableViewでの画面遷移について

swift4でTableViewCellの画面遷移を行いたい

Xcodeを用いてアプリケーション開発を行なっています。現在、Buttonにsegueを実装して画面遷移を行なった場合は画面遷移後のlabelに文字列は送れて表示ができているのですが、TableViewのタッチイベントにsegueを実装して画面遷移を行うとうまく表示されません。

該当のソースコード

ViewContriller
1import UIKit
2
3class ViewController: UIViewController, UITableViewDelegate, UITableViewDataSource{ 
4    var selectedText:String = ""
5    
6    let TODO = ["aaaa", "bbbb", "cccc"] 
7    
8    
9    
10    @IBAction func nextButton(_ sender: Any) {
11        selectedText="test1"
12        self.performSegue(withIdentifier: "nextButton", sender: nil)
13    }
14    override func viewDidLoad() {
15        super.viewDidLoad()
16    }
17
18    
19    override func didReceiveMemoryWarning() {
20        super.didReceiveMemoryWarning()
21    }
22
23    
24    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
25        return TODO.count
26    }
27
28    
29    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
30        
31        let cell: UITableViewCell = tableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath)
32        
33        cell.textLabel!.text = TODO[indexPath.row]
34        return cell
35    }
36    
37    func tableView(_ table: UITableView,didSelectRowAt indexPath: IndexPath) {
38        selectedText=TODO[indexPath.row]
39        self.performSegue(withIdentifier: "next", sender: nil)
40    }
41    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
42        if (segue.identifier == "next") {
43            let nextVC = segue.destination as! secondViewController
44            nextVC.text = selectedText
45        }
46        if (segue.identifier == "nextButton") {
47            let nextVC = segue.destination as! secondViewController
48            nextVC.text = selectedText
49        }
50    }
51    
52}
53

secondViewController
1import UIKit
2
3class secondViewController: UIViewController {
4    var text = ""
5    @IBOutlet weak var label: UILabel!
6    override func viewDidLoad() {
7        super.viewDidLoad()
8        print(text)
9        label.text = text
10    }
11    
12
13}
14

エラーログ

printで出力した文字列は、buttonをタップした場合はtest1と二度表示されるのですが、TableViewをタップした場合では一度目の出力が空文字となってしまい、うまく渡せません。二度目の出力はTODO配列の中身をうまく出力してくれます。また、button,TableView共にタップした際以下のようなエラーログが出力されています。

Warning: Attempt to present <test.secondViewController: 0x7fa10d617f00> on <test.ViewController: 0x7fa10d703ac0> whose view is not in the window hierarchy!

解決したい点

うまくTableViewにおける値渡しを行いたい
なぜsecondViewControllerのViewDidLoad()は二度呼び出されるのか

よろしくお願いします。

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

回答1件

ベストアンサー

何点か確認していただく必要があると思います。
1). tableViewのdelegateの接続ができているか
2). segueの接続の仕方が正しいのか
の2点です。

1.確認までに。
コードにはtableView.delegate = selfの記載がないので、
おそらくstoryboardで行っているということですよね。
遷移・データの受け渡しもできているので、
きちんと設定できているはずですよね。

2.が特に怪しいと思っています。
storyboard上で、segueの接続を行っていると思いますが、、、
下の図の方法A(UIButtonやTableViewCellから直接Ctrlでドラッグ)で行っていませんか？
その場合は、ViewControllerにコードを記載しなくても、自動で遷移してくれます。
(ただし、その分、値(selectedTextやTODO)などのデータの受け渡しを指定できません)

MicotoTisakiさんは、おそらく上記＋コードで遷移を記載しているので、
二重に処理が行われprintがダブって表示されているのではないでしょうか？

解決するには、方法Aではなく、方法Bで繋ぐ必要があります。
遷移する時に何かしら実行したい時や、値を渡したい場合など、方法Bを用います。
まず現在あるsegueの矢印は削除してください。
次に○で囲った、Viewの上にある、３つのアイコンのうち、一番左の
黄色い◉からCtrlを押しながら、目的のViewまで繋ぐです。
もちろんsegue identifierを設定してくださいね。

これで解決できると思います。