Firestoreのデータを表示しているtableViewのセルをタップしたとき、そのセルのドキュメントIDを取得したい。

Question

前提・実現したいこと

Firestoreのデータを表示しているtableViewのセルをタップしたとき、そのセルのドキュメントIDを取得したい。

実現したいことは以下の通りです。
1,タップしたセルのドキュメントIDを取得
2,DetaiviewControllerにそのIDを渡す。
3,DetaiviewControllerにあるgetActionボタンを押したらそのIDの投稿をマイページに保存する

プログラミング始めたてで質問の仕方が適切でないかもしれませんが、回答していただけると嬉しいです。

発生している問題・エラーメッセージ

タップしたセルのドキュメントIDが取得できません。

該当のソースコード

 override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "Cell", for: indexPath) as! TimeLineCell
        cell.timeLineModel = self.timeLines[indexPath.row]

        return cell
    }
    
    override func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        
        let selectedCell = tableView.cellForRow(at: indexPath)
        musicName = timeLines[indexPath.row].musicName
        groupName = timeLines[indexPath.row].groupName
        constitution = timeLines[indexPath.row].constitution
    
        performSegue(withIdentifier: "detail", sender: nil)
    }
    
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        
        if segue.identifier == "detail" {
            
            let detailViewController = segue.destination as! DetailViewController
            
            detailViewController.musicName = musicName
            detailViewController.groupName = groupName
            detailViewController.constitution = constitution
        }

import Foundation
import Firebase

class TimeLineModel {
    
    var musicName:String = ""
    var groupName:String = ""
    var constitution:String = ""
    var ref = Firestore.firestore().collection("timeLine")
    
    
    init(musicName:String,groupName:String,constitution:String) {
        
        self.musicName = musicName
        self.groupName = groupName
        self.constitution = constitution
    }
    
    init(document:QueryDocumentSnapshot) {
        
        if let value = document.data() as? [String:Any] {
            
            musicName = value["musicName"] as! String
            groupName = value["groupName"] as! String
            constitution = value["constitution"] as! String
        }
    }
    
    func toContents() -> [String:Any] {
        return ["musicName":musicName,"groupName":groupName,"constitution":constitution]
    }
    
    func save() {
        ref.addDocument(data: toContents())
    }

試したこと

https://teratail.com/questions/203222
この記事の方法を試しましたが　draftData　が何を指すのかわからず理解で解決できませんでした。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[TsukubaDepot]

問題の本質がFirestoreに関係してくるとなると判断できない部分もありますが、すくなくとも

tableView(_ :cellForRowAt:)
では、Cell に定義してある timeLineModel に値を渡していますが、タップされた時のメソッド

tableView(_:didSelectRowAt:)
では、タップされたセルのインスタンをを次の方法で得ているものの、
let selectedCell = tableView.cellForRow(at: indexPath)

値の代入は上記の selectedCell を使わずに
musicName = timeLines[indexPath.row].musicName
groupName = timeLines[indexPath.row].groupName
constitution = timeLines[indexPath.row].constitution

と、timeLines を使ってやっているようですが、そのあたりの整合性はどのようになっているのでしょうか。

[Ogikubo]

ご指摘ありがとうございます。
教材のコードを写している部分もあり、恥ずかしながら自分でもよく理解できていなかったですが、渡されている値は正しかったです、、、
selectedCellを使ったやり方をもしよろしければ教えていただきたいです。

[Ogikubo]

tableViewに並んでいる順番とtimeLinesという配列の中の投稿の順番が同じで、timeLines[indexPath.row]と書くことでタップされた投稿の情報を取得できると考えています。

[TsukubaDepot]

指摘したいことは、tableView(_:indexPath:)　でカスタムセル内のプロパティ（変数）にtimeLineModel を代入していますが、その他の場所で一切使えわれていないということです。

2020/12/09 20:25 　のコメントでもあるように、配列で保持しているデータとセルのデータの順番が一致しているのであれば、あえてセルにデータを持たせる意味は少ないのではないでしょうか。

ちなみに、それを簡潔に解決されているのが、hameji001さんのご回答です。
セルには情報を持たせず、indexPath を使ってアクセスすべきデータを管理しています。

[Ogikubo]

なるほど、、senderを使ってindexpathの情報ごと送ればその必要はなかったですね！
ありがとうございます！
hajimeさんの回答を参考にまた考え直して見たいと思います！
ご指摘本当にありがとうございました！

[TsukubaDepot]

sender で送る以外に、下記のプロパティを使って得ることも可能です（非選択状態に戻すと使えないので、使う順番は考える必要がありますが）。

https://developer.apple.com/documentation/uikit/uitableview/1615000-indexpathforselectedrow

[Ogikubo]

この方法でも選択したセルの情報を得ることが出来るんですね。
他にも応用できそうな情報ありがとうございます、とてもためになります！
親切な対応ありがとうございます！

Answer 1

自分も似たようなことをしています。

簡単な方法はdocumentIDをTimeLineModelに持たせることです。

class TimeLineModelにdocumentID(String型)用にメンバ変数を設定し、
init(document:QueryDocumentSnapshot)で
document.documentIDを取得し、そのメンバ変数に保存すればいいと思います。

あと、classよりstructの方が堅牢なシステムになるので、いいみたいです。

また、TableViewControllerではメンバ変数に
timeLines: [TimeLineModel]を保持しているようなので、
TimeLineModelごと次のDetailViewControllerに渡すように変えるといいと思います。
(DetaiViewControllerにTimeLineModel保持用のメンバ変数を用意する)

func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
    let selectedTimeLine = timeLines[indexPath.row]
    performSegue(withIdentifier: "detail", sender: selectedTimeLine)
    // ↑ senderに選択された行のtimeLineを入れる
}

override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
    if segue.identifier == "detail" {
        let detailVC = segue.destination as! DetailViewController
        detailVC.timeLine = sender as! TimeLineModel
        // ↑ここでsenderからTimeLineModelを取り出し、次のViewにセット
    }
}

Comments

[Ogikubo]

ご回答本当にありがとうございます！！早速試させていただきました！
追加で少し質問をしたいのですが、、
tabelViewControllerで受け取ったTimeLineModelにある変数（musicNameなど）を取得するにはどうしたら良いでしょうか？

[hameji001]

それはとても簡単です。
たとえば、 DetailViewControllerに pushedTimeLine: TimeLineModel をメンバ変数に用意したとします。そのmusicNameやgroupNameを取得するには、pushedTimeLine.musicNameと記述すれば、String型のデータをそのまま利用できます。
nextView ?

[Ogikubo]

ありがとうございます！できました！
丁寧に教えてくださり本当にありがとうございました！