選択されたtableViewCellに表示されているfirestoreデータのdocumentIDを取得したい

Question

知りたいこと

選択されたtableViewCellに表示されているfirestoreデータのdocumentIDを取得する方法を知りたいです。

やろうとしていることの全体像

選択したtableViewcellを"delete"ボタンをタップして削除したいので、

"delete"ボタンをタップ
→ 選択しているセルのindexPathを取得
→ 取得したindexPathに格納しているfirebaseのfirestoreデータを削除
→ tableViewをリロード

という流れで行おうと考えております。

現状

「取得したindexPathに格納しているfirebaseのfirestoreデータを削除」の段階のデータを削除する時に、documentIDを指定しなければならないようで ( https://firebase.google.com/docs/firestore/manage-data/delete-data?hl=ja )、そのIDの取得方法がわからず詰まっています。

@IBAction func tapDeleteButton(_ sender: Any) { //check mark がついているセルを削除
        
        //選択されているセルのindexPath.rowを取得
        func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
            let selectedCell = tableView.cellForRow(at: indexPath)
           
            // (削除する)選択されたセルに入っているdraftDataのdocumentIDを取得
            let documents = documentsnapshot.documents  //ここでエラー
            for i in 0 ..< documents.count {
                let dictData = documents[i].data()
                let documentID = documents[i].documentID
                print("Document ID (documentID)")
            }

            // forestoreの方のデータを削除 ("DC"は公式のをコピペしたままのものです。)
            db.collection("drafts").document("DC").delete() { err in
                if let err = err {
                    print("Error removing document: (err)")
                } else {
                    print("Document successfully removed!")
                }
            } 
        }
        // tableview(draftData)をリロード
        self.tableView.reloadData()  
    }

エラーコード

Use of unresolved identifier 'documentsnapshot'; did you mean 'DocumentSnapshot'?

documentIDは、自動生成の.addメソッドを使用しています。
https://firebase.google.com/docs/firestore/manage-data/add-data

データの取得は、tableViewCellに表示させる前にtimestampで上から新着順に表示するようにしています。

func getDraftsData() {
        guard let uid = user?.uid else { return }
        db.collection("drafts").whereField("uid", isEqualTo: uid).getDocuments() { (querysnapshot, err) in
            if let err = err {
                print("error getting documents: (err)")
            } else {
                let personalDraftData = querysnapshot!.documents.sorted {personalDraftData,provisionalDraftData in
                    guard let personalDraftsData = personalDraftData.data()["timestamp"] as? Timestamp,
                        let provisionalDraftsData = provisionalDraftData.data()["timestamp"] as? Timestamp else { return false }
                    return (personalDraftsData.dateValue() > provisionalDraftsData.dateValue())
                }
                self.draftData = personalDraftData
                self.tableView.reloadData()
            }
        }
    }

どのようにしたらdocumentIDを取得できるか教えていただきたいです。
よろしくお願いします。

補足

ご回答いただきました内容にコードを書き換えて実行した結果です。
設定したブレイクポイントではブレイクしてくれず、ボタンアクションが完了した最後の "}" でブレイクされます。
また、シミュレータの動きとしては、delete ボタンタップ後は選択されていたセルのチェックマークが外れるだけで、firestoreのコンソール内のデータを確認しても削除されておりません。

Comments

[takabosoft]

> db.collection("drafts").document("DC").delete()
で言うところの"DC"を、cellが指し示す何かしらの文字列に変えたいという質問ですよね？

たぶんcellを作っているときはdrafts[indexPath.row]にアクセスして構築しているんだと思いますが、
drafts[indexPath.row]の中にそれらしきドキュメント名は入っていないのでしょうか？

[退会済みユーザー]

>"DC"を、cellが指し示す何かしらの文字列に変えたいという質問ですよね？
そうです！

>たぶんcellを作っているときはdrafts[indexPath.row]にアクセスして構築しているんだと思いますが、
drafts[indexPath.row]の中にそれらしきドキュメント名は入っていないのでしょうか？
申し訳ありません、どのメソッドをみたらわかるでしょうか？？orz

Answer 1

//選択されたindexPathの配列
var selectedIndexPaths: [IndexPath] = [
@IBAction func tapDeleteButton(_ sender: Any) { //check mark がついているセルを削除

        //選択されているセルのindexPath.rowを取得
       
        for indexPath in selectedIndexPaths{

             let dictData = draftData[indexPath.row].data()
             let documentID = draftData[indexPath.row].documentID
             print("Document ID (documentID)")

             db.collection("drafts").document(documentID).delete() { err in
                if let err = err {
                    print("Error removing document: (err)")
                } else {
                    print("Document successfully removed!")
                    self.tableView.reloadData()
                }
            } 
        }
    }

 func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
            let selectedCell = tableView.cellForRow(at: indexPath)


            //タップされた時の処理
　　　　　　　if selectedIndexPaths.contains(indexPath){
                for (i, ip) in slectedIndexPaths.enumerated(){
                   if ip == indexPath{
                      selectedCell.backgroundColor = .clear
                      selectedIndexPaths.remove(at: i)
                      break
                   }
                }
            }else{
                selectedIndexPaths.append(indexPath)
                //セルが選択されていることを示す何らかの処理, 例えば赤くするとか
                selectedCell.backgroundColor = .red
            }

            tableView.reloadData()
            

            
        }

Comments

[退会済みユーザー]

ありがとうございます。
試してみます！

[退会済みユーザー]

すみません。やっぱり中が走らないです。。

[fathy]

もう少し具体的に教えてください。
あとコードの画面のスクリーンショットを送ってください

[退会済みユーザー]

すみません、スクリーンショットをこの回答に貼り付ける方法がわからないのですが、どこに貼ったらよろしいでしょうか？

＞中が走らない
とは、
＠IBAction { } が走っていない、と言うことでして
db.collection("drafts").document(documentID).delete() { err in
の行と
if selectedIndexPaths.contains(indexPath){
の行にブレイクポイントを設定して行いました。

[fathy]

質問を編集して追記してください。

cellを選択していることは確認できていますか？
タップした時に赤くなりますか？

[退会済みユーザー]

かしこまりました。少々お待ちくださいませ。

先にご質問の回答をさせていただきます。
>cellを選択していることは確認できていますか？
はい、できております。

>タップした時に赤くなりますか？
下記のエラーが出ていたのと、私のアプリでは色を変えることはしない(デフォルトでグレーになっている)ためコメントアウトして実行しておりました orz
申し訳ありません。必要であればエラーを直してからもう一度試します。

Value of optional type 'UITableViewCell?' must be unwrapped to refer to member 'backgroundColor' of wrapped base type 'UITableViewCell'

[fathy]

スクリーンショット拝見致しました。
@IBAction func tap〜の中から，
func tableView(_ tableView: UITableView〜を取り出してください。
関数内で関数を定義している状態になっているため，
tableView(tableView:didTap:indexPath)関数が呼ばれていません。

回答内容をよく見て修正してください。

[退会済みユーザー]

大変申し訳ありません、ご指摘ありがとうございますorz
@IBAction の関数内から取り出して修正してビルド致しました。

現在、すでに別箇所に
func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath)
メソッドが存在していて、そのメソッド内に書いていたメソッド
( checkMark(checkMarkStatus: true) のみ )と合わせてビルドしたところ、
"check all" と "unselect all" での選択は可能なのですが、１つ１つ選択をする動きがおかしくなったので、修正を試みております。

ご回答、誠にありがとうございます!!!

[fathy]

よかったです。
よろしければ解決済にしてください。

[退会済みユーザー]

すみません、まだ解決できていないので解決しましたらそのようにするつもりです。。。

Answer 2

let documents = documentsnapshot.documents  //ここでエラー
            for i in 0 ..< documents.count {
                let dictData = documents[i].data()
                let documentID = documents[i].documentID
                print("Document ID (documentID)")
            }
// forestoreの方のデータを削除 ("DC"は公式のをコピペしたままのものです。)
            db.collection("drafts").document("DC").delete() { err in
                if let err = err {
                    print("Error removing document: (err)")
                } else {
                    print("Document successfully removed!")
                }
            } 

の部分を

            var deleteID = ""
            for i in 0 ..< draftData.count {
                let dictData = documents[i].data()
                let documentID = documents[i].documentID
                print("Document ID (documentID)")

                //section　は一つと仮定しています。
                if i == indexPath.row{
                   deleteID = documentID
                   break
                }
            }


            db.collection("drafts").document(deleteID).delete() { err in
                if let err = err {
                    print("Error removing document: (err)")
                } else {
                    print("Document successfully removed!")
                    self.tableView.reloadData()
                }
            } 

やっていることは，そのセルのindexPathから表示されているセルのdocumentを取得しています。

Comments

[退会済みユーザー]

ご回答ありがとうございます。ご回答くださった方法で試してみたところ、
let dictData = documents[i].data()
let documentID = documents[i].documentID

の documents のところでエラーがでました↓
Use of unresolved identifier 'documents'; did you mean 'UIDocument'?

ので、documents→draftData に変更して実装したところ、
全てのドキュメントidとデータを全取得してきます。そしてデータは削除されませんでした。

[fathy]

すみません。間違えました。
documentsを全てdraftDataに置き換えてください。

[fathy]

あごめんなさい。やったんですね。

[fathy]

var deleteID = ""
for i in 0 ..< draftData.count {
let dictData = draftData[i].data()
let documentID = draftData[i].documentID
print("Document ID (documentID)")

//section　は一つと仮定しています。
if i == indexPath.row{
deleteID = documentID
　　　　　　print("deleteID: (deleteID)")
print("indexPath: (indexPath)")
break
}
}

などとしてifの部分が通っているか確かめていただけますか？

[退会済みユーザー]

先ほどご報告していなかったのですが、まずif文の行にブレイクポイントを設定していまいたがその場所では停止せず、でした。また、if文内に print("@@@@@@@(deleteID)") (@は私が見やすいようにです)を書いていましたが通っていなかったです。

[退会済みユーザー]

ちなみに停止したのは
self.tableView.reloadData()
の文でした。

[fathy]

すみません。
質問文の最初のコードに関数の定義位置の問題に気づいていませんでした。
修正点を改めて回答させて頂きたいのですが，1点確認したいのが，選択したあとdeletebutton を押したタイミングで削除を実行するのか，deletebuttonを押したあとでcell選択時に削除を実行するのかどちらでしょうか。

[退会済みユーザー]

こちらこそ申し訳ありません。
おそらく前者の「選択したあとdeletebutton を押したタイミングで削除を実行する」だと思います(後者のほうがよく理解できませんでした。)
選択するのはedit ボタンを編集モードにしてから行います。
選択のところのコードを補足として記載させていただきますm(_ _ )m

[退会済みユーザー]

補足です。
編集モードでセルを選択して、削除ボタンで削除を実行したいです。
(同じ編集モードで、ボタンタップで一括選択・一括選択解除もできるようにしています)

```
override func viewDidLoad() {
super.viewDidLoad()

//複数選択できるようにする
tableView.allowsMultipleSelectionDuringEditing = true
}
```

```
@IBAction func tapCloseButton(_ sender: Any) {
if closeButton.image == nil && closeButton.title == "Check all" {
for row in 0..<draftData.count {
let indexPath = IndexPath(row: row, section: 0)
tableView.selectRow(at: indexPath, animated: true, scrollPosition: .none)
checkMark(checkMarkStatus: true)
print(indexPath)
}
} else if closeButton.image == nil && closeButton.title == "Unselect all" {
for row in 0..<draftData.count {
let indexPath = IndexPath(row: row, section: 0)
tableView.deselectRow(at: indexPath, animated: true)
checkMark(checkMarkStatus: true)
}
} else {
self.dismiss(animated: true, completion: nil)
}
}
```