1

cellのaccessoryで付けることができるDetail(iの丸マーク)を押すと、画面遷移できる仕組みにしたいのですが、どのように関連付けをしたり、コードを書けばいいのかわかりません。

画像の説明をここに入力

画像の説明をここに入力

関連付けはしてみたものの、マークを押しても全く動きません。


↓変更後

import UIKit

class ViewController: UIViewController, UITableViewDataSource, TestDelegate {

@IBOutlet weak var testTableView: UITableView!


//表示データ
var dataList = ["","","","","","","",""]

//データを返すメソッド(スクロールなどでページを更新する必要が出るたびに呼び出される)
func tableView(_ tableView:UITableView, cellForRowAt indexPath:IndexPath) -> UITableViewCell {
    let cell = tableView.dequeueReusableCell(withIdentifier: "TestCell", for:indexPath) as UITableViewCell
    cell.textLabel?.text = dataList[indexPath.row]

    return cell
}

//データの個数を返すメソッド
func tableView(_ tableView:UITableView, numberOfRowsInSection section:Int) -> Int {
    return dataList.count
}


//削除ボタン
func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCellEditingStyle, forRowAt indexPath: IndexPath) {
    if editingStyle == .delete {
    dataList.remove(at: indexPath.row)
    testTableView.deleteRows(at: [indexPath], with: .fade)
    }
}


func TableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
    let cell = tableView.dequeueReusableCell(withIdentifier: "Cell", for: indexPath)
    cell.textLabel?.text = "No. \(indexPath.row)"
    // 偶数行にDetailボタンを表示する。
    if indexPath.row & 0x01 == 1 {
        cell.accessoryType = .detailButton
    }

    return cell
}

// UITableViewDelegate
// アクセサリボタンをタップしたら……
func tableView(_ tableView: UITableView, accessoryButtonTappedForRowWith indexPath: IndexPath) {
    // セグエGoNextを実行する。
    performSegue(withIdentifier: "GoNext", sender: nil)
}


override func viewDidLoad() {
    super.viewDidLoad()
    // Do any additional setup after loading the view, typically from a nib.

   //自作セルをテーブルビューに登録する。
    let testXib = UINib(nibName:"TestTableViewCell", bundle:nil)
    testTableView.register(testXib, forCellReuseIdentifier:"TestCell")
}

//デリゲートメソッド
func textFieldDidEndEditing(cell: TestTableViewCell, value:String) {
    //変更されたセルのインデックスを取得する。
    let index = testTableView.indexPathForRow(at: cell.convert(cell.bounds.origin, to:testTableView))

    //データを変更する。
    dataList[index!.row] = value
    print(dataList)
}

override func didReceiveMemoryWarning() {
    super.didReceiveMemoryWarning()
    // Dispose of any resources that can be recreated.
}


}

@nagonsoftware さんの教えていただいた通りにやったつもりなのですが、iの丸を押しても画面遷移できません。
上に載せたコードは、ViewControllerのものです。
TableViewのCellはTextFieldをタップするだけで簡単に入力できる仕組みにしています。
この仕組みのコードを書いているから、画面遷移できないのかなと思い、色々コードを書く順番を並び替えたりしたのですが、ダメでした。。
何が原因で画面遷移ができないのか、さっぱりわかっていません。

1
  • 問題点は2点。1点目。ViewControllerクラスの宣言部に、UITableViewDelegateがありません。ついでにUITextFieldDelegateも抜けています。class ViewController: UIViewController, UITableViewDataSource, UITableViewDelegate, UITextFieldDelegateとしてください(TestDelegateは、何か意味不明なので、削除しています)。2点目。TableView DataSourceのメソッドtableView(_:cellForRowAt:)が二つあります。そのままではコンパイルエラーになるので、一方を大文字のTableで始めているようですが、そのためUITableViewDataSourceのメソッドとみなされません。実装を統合して、ひとつにしてください。全然「教えていただいた通りにやっ」てませんよ。 Commented 2017年4月29日 21:01

2 件の回答 2

0

UIStoryboardSegueによる画面遷移を、Storyboardの編集でなく、プログラムで実行するには、どうすればいいか?
質問者さんが求めているものは、上の質問に帰結することに、まず気づくことが大事です。
セグエのみならず、Storyboard上の編集は、すべてプログラムコードで記述できる。それが基本です。UIStoryboardUIStoryboardSegueUIViewControllerこれら3つのリファレンスを読めば、どういうプロパティ、メソッドを使えば、セグエを編集できるかを学習できます。
UITableViewDelegateのメソッドtableView(_:accessoryButtonTappedForRowWith:)を使うと、セルのアクセサリをタップした時の処理を記述できます。ここに、セグエを実行して、画面遷移するコードを記述すると、DetailButtonをタップして、画面遷移することができます。


Storyboard上で、セグエをオブジェクト間に結ぶとき、ボタンとViewController、テーブルセルとViewControllerの間に結ぶことが多いと思いますが、本件では、ViewControllerとViewController間を結んでください。
画像の説明をここに入力
セグエを結んだら、セグエのIdentifierに、任意の文字列をつけます。つけないと、プログラムからセグエを識別することができません。(ここではGoNextとします)
画像の説明をここに入力
そして、テーブルに関する処理をする、以下のコードを記述します。

import UIKit

class ViewController: UIViewController, UITableViewDelegate, UITableViewDataSource {

    // UITableViewDataSource
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return 12
    }

    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "Cell", for: indexPath)
        cell.textLabel?.text = "No. \(indexPath.row)"
        // 偶数行にDetailボタンを表示する。
        if indexPath.row & 0x01 == 1 {
            cell.accessoryType = .detailButton
        } else {
            cell.accessoryType = .none
        }

        return cell
    }

    // UITableViewDelegate
    // アクセサリボタンをタップしたら……
    func tableView(_ tableView: UITableView, accessoryButtonTappedForRowWith indexPath: IndexPath) {
        // セグエGoNextを実行する。
        performSegue(withIdentifier: "GoNext", sender: nil)
    }

    // セグエ実行時の処理は、ここに記述。
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        // Do Anything
    }

}

UIViewControllerのメソッドperformSegue(withIdentifier:sender:)が、セグエをプログラムで実行する、具体的なメソッドです。


質問と直接関係ないことで、気づいたことがあるので、アドバイスします。
添付なさったStoryboardのスクリーンショットを拝見しましたが、A View Controllerと、B View Controllerの間に、相互にセグエを結んでいますね?実は、これはやってはいけない典型的な例です。これをやってしまうと、

インスタンスa(A View Controller)→ インスタンスb(B View Controller)→ インスタンスa'(A View Controller)→ インスタンスb'(B View Controller)→ インスタンスa''(A View Controller)→ インスタンスb''(B View Controller)→ ……

このように、新しいインスタンスを作りながら画面遷移していき、けっして元のインスタンスに戻ることがありません。意図しない挙動になりますし、メモリの圧迫、ひいてはメモリリークの原因になります。
セグエ遷移元に戻るには、Unwind Segueを使ってください。これも、ネット検索すれば、いくつも情報がヒットします。「サンプルコードがObjective-Cだから」という言い訳で、検索をサボってはいけません。(現在のところ)Swiftのコードであっても、ブリッジでObjective-CのAPIを呼んでいるので、言語に関係なく、概要はまったく同じです。

2
  • 前回の質問でも回答をいただき本当に助かっております。ありがとうございます。教えていただいた通りにやってみたのですが、動かなくて、、、質問の内容の下に画面遷移できない詳細を書き足したので、読んでいただけたらと思います。よろしくお願いします!
    – Hana
    Commented 2017年4月29日 15:28
  • ありがとうございます。無事実行できました!
    – Hana
    Commented 2017年5月2日 14:18
0

Storyboard上でセグエを正しく設定できていれば、コードは書かなくても普通に遷移できるようになります。セグエの設定手順を見直して、正しく設定し直すか、それでダメであればTable Viewの方が正しくPrototype Cellを表示できていない可能性があります。

  1. まずは普通にアクセサリーを設定したPrototype Cellから遷移先にcontrol-ドラッグします。

control-ドラッグ

  1. ドラッグを終了した時に表示されるポップアップメニューの中でAccessory Actionの中から好きな遷移スタイルを選びます。

Accessory Action

Xcode8.3.2でビルドして、Detailのⓘから遷移先に指定したViewControllerに確実に遷移できるのは確認しました。実際には遷移先になんらかの情報を渡す必要があるでしょうから、その分のコードは追加する必要があるでしょうが、それはなくても「全く動きません」と言うことはないはずです。

上記の手順に従ってやり直してもダメなら、そもそもTable View側で正しくPrototype Cellを表示していない可能性がありますので、その辺の情報を詳しく提示していただかないとなんとも言えません。コメントを付けるかご質問を編集して情報を追記してください。


なお、右のView Controllerから左のTable Viewを含むView Controllerに向けたセグエが存在するようですが、これだと「元の画面に戻る」のではなく、「新しい画面を別に作って新しい画面の方に遷移する」と言う動作になります。それは意図した動作ではないが直し方がわからないと言ったことがあれば、別質問としてご投稿ください。

1
  • ありがとうございます。丁寧でとてもわかりやすかったです!
    – Hana
    Commented 2017年5月2日 14:19

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。