質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.60%

セルをタップした時にそのセルの情報をセグエした先のViewControllerで表示したいのですが

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 1,626
退会済みユーザー

退会済みユーザー

現在swiftを使って簡単な日記アプリを作っています。
投稿された日記をUserDefaultsを使って保存し、一覧をTableViewのセルに表示するところまでは実装できたのですが、セルをタップした時に値がセグエ先のViewControllerに渡せていません。
AppDelegateを使って値を渡そうとすると、最初のタップがnilになり、次のタップの時に先ほどタップしたセルの情報がセグエ先のViewControllerに表示されてしまいます。
プロトコルを作成して値を渡したほうが良いのでしょうか?
また、セルの情報を別のViewControllerで表示する方法の主流は何なのでしょうか?

見づらいかもしれませんが、私が書いたコードも記述しておきます。
Appdelegate

class AppDelegate: UIResponder, UIApplicationDelegate {
    var diaryContent:String?
    var diaryDate:String?
}

//セルがタップされた時のメソッド
    override func tableView(tableView: UITableView, didSelectRowAtIndexPath indexPath: NSIndexPath) {
        performSegueWithIdentifier("ToDetailDiaryViewController", sender: nil)
        let diary = diaries[indexPath.row]
        diaryContent = diary.content
        diaryDate = diary.date
    }
override func prepareForSegue(segue: UIStoryboardSegue, sender: AnyObject?) {
        if segue.identifier == "ToDetailDiaryViewController" {
            let destinationViewController = segue.destinationViewController as! UINavigationController
            let detailDiryViewController = destinationViewController.topViewController as! DetailDiaryViewController
            detailDiryViewController.diary = self.currentSelectedDiary
            var appDelegate:AppDelegate = UIApplication.sharedApplication().delegate as! AppDelegate
            appDelegate.diaryContent = diaryContent
            appDelegate.diaryDate = diaryDate
        } else {
            let destinationViewController = segue.destinationViewController as! UINavigationController
            let newDiaryViewController = destinationViewController.topViewController as! NewDiaryViewController
            newDiaryViewController.delegate = self
        }
    }

var appDelegate: AppDelegate = UIApplication.sharedApplication().delegate as! AppDelegate
        var diaryContent = appDelegate.diaryContent
        var diaryDate = appDelegate.diaryDate
        
        diaryTextView.text = diaryContent
        self.navigationItem.title = "\(diaryDate)"
        println(diaryDate)
        println(diaryContent)
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • Stripe

    2015/07/10 21:28

    「セグエ先のViewController」のコードも書いてください。

    キャンセル

回答 2

checkベストアンサー

0

ご呈示のコードを最小限で修正するなら、
  var diaryContent //メンバ変数
  var diaryDate

//セルがタップされた時のメソッド
    override func tableView(tableView: UITableView, didSelectRowAtIndexPath indexPath: NSIndexPath) {
        let diary = diaries[indexPath.row]
        self.diaryContent = diary.content
        self.diaryDate = diary.date
        performSegueWithIdentifier("ToDetailDiaryViewController", sender: nil)
    }
override func prepareForSegue(segue: UIStoryboardSegue, sender: AnyObject?) {
        if segue.identifier == "ToDetailDiaryViewController" {
            let destinationViewController = segue.destinationViewController as! UINavigationController
            let detailDiryViewController = destinationViewController.topViewController as! DetailDiaryViewController
            detailDiryViewController.diary = self.currentSelectedDiary
            detailDiryViewController.diaryContent = self.diaryContent //遷移先のコントローラーに格納用のプロパティを用意しておく
            detailDiryViewController.diaryDate = self.diaryDate
        } else {
            let destinationViewController = segue.destinationViewController as! UINavigationController
            let newDiaryViewController = destinationViewController.topViewController as! NewDiaryViewController
            newDiaryViewController.delegate = self
        }
    }
こんな感じでしょうか。
手打ちなのでビルドが通らなければ申し訳無いですが、やりたい事はご推察いただけると思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

そのコードですと、prepareForSegueが呼ばれた時点では、appDelegateのメンバ変数にまだ値が入っていない状態になってしまいます。
早急にコードを直すとしたら

    override func tableView(tableView: UITableView, didSelectRowAtIndexPath indexPath: NSIndexPath) {
        let diary = diaries[indexPath.row]
        var appDelegate:AppDelegate = UIApplication.sharedApplication().delegate as! AppDelegate
        appDelegate.diaryContent = diary.content
        appDelegate.diaryDate = diary.date
        performSegueWithIdentifier("ToDetailDiaryViewController", sender: nil)
    }
とすれば良いと思われますが、そもそもdiary~変数はappDelegateに定義せずとも、呼び出し元のメンバ変数にしておけば良いと思います。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/07/11 11:34

    ご回答ありがとうございます!
    返信が遅くなってしまい、すみません。

    diary~変数を呼び出し元のメンバ変数にする場合は、どこに記述したらよいのでしょうか?

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.60%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る