Firestoreと連携するクラスをインスタンス化しようとすると「 Missing argument for parameter 'document' in call」のエラーが出る

Question

環境

言語：Swift 5.4
開発環境；xcode 12.5
クラウドデータベース：Cloud Firestore

やりたいこと

Swift×FirestoreでToDoリストアプリを作っています。

最終的には、マルチデバイスでどのデバイスとも同期できるようにしたいと思い、
下記のページを参考にFirestoreと連携するTableViewのアプリを作成しています。

【Firebase×Swift】データ取得とTableView表示（後編）〜取得データの変換とセル表示〜

ToDoのクラスとしてMyTodoクラスを作り、MyTodoクラスからインスタンス（myTodo）を作り、インスタンスの配列を処理することでToDoの追加・削除等の機能を作っていきたいと考えています。

困っていること

MyTodoクラスを元に、タスクを追加するためのインスタンスmyTodoを作成しようとすると、xcodeで

let myTodo = MyTodo()

の箇所で下記のエラーが出てビルドできないです。

Missing argument for parameter 'document' in call

MyTodo() をMyTodo（document: QueryDocumentSnapshot）としたりしましたがエラーは消えず、ハマってしまいこれ以上進めない状況です。対処法等、分かりましたら教えていただけると助かります。
なお、CloudFirestoreとクライアントでデータの送受信ができるところまでは確認できています。

MyTodoクラスの初期化（init）のところがいけないのでしょうか。

コード

import UIKit
import Firebase
import FirebaseFirestoreSwift


class ViewController: UIViewController, UITableViewDataSource, UITableViewDelegate {

    //*************************************************
    //ToDoのテキストを格納する配列の定義
    //*************************************************
    var todoList = [MyTodo]()
    let db = Firestore.firestore()

    //*************************************************
    //UITableViewのoutlet定義
    //*************************************************
    @IBOutlet weak var tableView: UITableView!

    //*************************************************
    //画面の初回ロード時の処理
    //*************************************************
    override func viewDidLoad() {
        super.viewDidLoad()
        print("ログ_viewdidload：start")


        //++++++++++++++++++++++++++++++++++++++++++++
        //Firestoreからドキュメントのデータを取得して、todoListのデータ型に変換し配列の各要素に入れる
        //++++++++++++++++++++++++++++++++++++++++++++
        //Firestoreからドキュメントのデータ取得。
        print("ログ_viewdidload：Getting documents is start")
        db.collection("todo").getDocuments() { (querySnapshot, err) in
            if let err = err {
                print("ログ_viewdidload：Error getting documents(err)")
            } else {
                print("ログ_viewdidload：Getting documents is success!")
                for document in querySnapshot!.documents{
                    print("ログ_viewdidload：documentの中身を確認(document.data())")
                }
            }

            self.todoList = querySnapshot!.documents.map {
                document in let data = MyTodo(document: document)
                return data
            }
        }
    }

    //*************************************************
    // +ボタンをタップした時の処理（タスク入力オブジェクトの作成・処理）
    //*************************************************

    @IBAction func tapAddButton(_ sender: Any) {
        
        // タスク入力オブジェクトとなるアラートダイアログを生成
        let alertController = UIAlertController(
            title: "ToDo追加",
            message: "ToDoを入力してください",
            preferredStyle: UIAlertController.Style.alert
        )

        // テキストエリアを追加
        alertController.addTextField(configurationHandler: nil)

        //*************************************************
        //OKボタンの配置、ボタン押下時の処理
        //*************************************************
        // OKボタンを追加
        let okAction = UIAlertAction(
            title: "OK",
            style: UIAlertAction.Style.default
        ){
            (action: UIAlertAction) in
            // OKボタンがタップされたときの処理（ToDoの配列に入力値を挿入。テーブルの先頭行に挿入する）
            if let textField = alertController.textFields?.first {

                let myTodo = MyTodo()    //<--ココ！　MyTodoクラスをインスタンス化してmyTodo定数を生成したい
                myTodo.todoTitle = textField.text!
                self.todoList.insert(myTodo, at: 0)
                
                // テーブルに行を追加
                self.tableView.insertRows(
                    at: [IndexPath(row: 0, section: 0)],
                    with: UITableView.RowAnimation.right
                )
                
                //追加したToDoについて、Firestoreのtodoコレクションに新規のtodoドキュメントを追加
                let ref: DocumentReference? = nil
                self.db.collection("todo").addDocument(data: [
                    "todoTitle": myTodo.todoTitle!,
                    "todoDone": false
                ]) { err in
                    if let err = err {
                        print("Error adding document: (err)")
                    } else {
                        print("Document added with ID: (ref!.documentID)")
                    }
                }
            }
        }

        // OKボタンがタップされたときの処理
        alertController.addAction(okAction)

        //*************************************************
        //キャンセルボタンの配置、ボタン押下時の処理
        //*************************************************
        // CANCELボタンがタップされたときの処理
        let cancelButton = UIAlertAction(
            title: "CANCEL",
            style: UIAlertAction.Style.cancel,
            handler: nil
        )

        // CANCELボタンを追加
        alertController.addAction(cancelButton)
        // アラートダイアログを表示
        present(alertController, animated: true, completion: nil)
    }


    //*************************************************
    // テーブルの行数返却
    //*************************************************
    // テーブルの行数を返却する


    func tableView(
        _ tableView: UITableView,
        numberOfRowsInSection section: Int
    ) -> Int {
        // Todoの配列の長さを返却する
        print("ログ_tableView_numberOfRowsInSection：配列の長さは(todoList.count)")

        return todoList.count
    }

    //*************************************************
    // テーブルのセルの表示（テーブルの行ごとにセルを返却）
    //*************************************************
    // テーブルの行ごとのセルを返却する
    func tableView(
        _ tableView: UITableView,
        cellForRowAt indexPath: IndexPath
    ) -> UITableViewCell {
        // Storyboardで指定したtodoCell識別子を利用して再利用可能なセルを取得する
        let cell = tableView.dequeueReusableCell(
            withIdentifier: "todoCell",
            for: indexPath
        )
        // 行番号に合ったToDoの情報を取得
        let myTodo = todoList[indexPath.row]
        print("ログ_tableView_cellForRowAt：myTodoの中身は(todoList[indexPath.row])")
        print("ログ_tableView_cellForRowAt：行番号に合ったToDoは(String(describing: myTodo.todoTitle))")

        // セルのラベルにToDoのタイトルをセット
        cell.textLabel?.text = myTodo.todoTitle


        // セルのチェックマーク状態をセット
        if myTodo.todoDone {
            // チェックあり
            cell.accessoryType = UITableViewCell.AccessoryType.checkmark
        } else {
            // チェックなし
            cell.accessoryType = UITableViewCell.AccessoryType.none
        }

        return cell
    }



    //*************************************************
    // セルをタップした時の処理（チェックを入れたり外したり）
    //*************************************************
    // セルをタップしたときの処理
    func tableView(
        _ tableView: UITableView,
        didSelectRowAt indexPath: IndexPath
    ) {
        let myTodo = todoList[indexPath.row]

        if myTodo.todoDone {
            // 完了済みの場合は未完了に変更
            myTodo.todoDone = false
        } else {
            // 未完の場合は完了済みに変更
            myTodo.todoDone = true
        }

        // セルの状態を変更（特定の行だけリロードする）
        tableView.reloadRows(
            at: [indexPath],
            with: UITableView.RowAnimation.none
        )

        let ref: DocumentReference? = nil   //Firestoreでドキュメントを指定するため、タップしたドキュメントIDを取得
        myTodo.todoId = ref!.documentID

        //フラグを変更したToDoについて、Firestoreのtodoコレクションの該当するtodoドキュメントのフラグを変更
        db.collection("todo").document(myTodo.todoId!).updateData([
            "todoDone": myTodo.todoDone,
        ]) { err in
            if let err = err {
                print("Error updating document: (err)")
            } else {
                print("Document successfully updated")
            }
        }

    }
}
//*************************************************
// MyTodoクラスを作成
//*************************************************

class MyTodo: NSObject{
    var todoId: String?
    var todoTitle: String?
    var todoDone: Bool = false

    //取得したデータをFirestoreのデータ型に当てはめていくための型（MyTodoのデータ型）
    init(document: QueryDocumentSnapshot) {
        self.todoId = document.documentID
        self.todoTitle = document.data()["todoTitle"] as? String
        self.todoDone = document.data()["todoDone"] as! Bool
    }
}

Comments

[hoshi-takanori]

MyTodo は引数がない init を持たないのでエラーになります。
init() {
// 中身は空で良い
}
を追加すると動くと思いますが、todoId を持たない MyTodo ができるのは良くない気が…。

[koshira27]

回答ありがとうございます。

init(document: QueryDocumentSnapshot) {
self.todoId = document.documentID
self.todoTitle = document.data()["todoTitle"] as? String
self.todoDone = document.data()["todoDone"] as! Bool
}

上記部分を頂いたコメントの通り下記のようにするとビルドされるまではいくようになりました。

init() {
// 中身は空で良い
}

ただし、上記にすると、ビルドはできるがFirestoreからとってきたデータがtableViewに表示されなくなってしまいます。

質問に記載した参照ページによると、init(document: の処理は、Firestoreからとってきたドキュメントをテーブルビューに表示できるデータ型（クラスの型））へ変換する処理とのことです。
実際にinit() {}に変更してコンソールでログを確認して、Firestoreからdocumentを取り込めても、
todoList配列への配置でデータが取り込まれなくなっています。
（反対に、init(document: とするとコンソールのログでデータが取り込まれていることが確認できます）

実は頂いた修正は既に試していました。試したのですが、init() {}だとビルドはできるがtableViewに表示されず、init(document: だとエラーがでてビルドできず。どちらも作りたいものが作れず、私の知識ではにっちもさっちもいかない状況になっているのが今の状況です。

おそらく、init(){}の箇所でデータ型の変換をするのではなく、
別の場所でデータ型の変換ができればビルドが通り、データ変換もできるようになると推測しています。
（ただしその具体的な方法がわからない・・・）。それとも何か根本的に間違っているのか・・・・やっぱり分からず・・・。

その他コメント頂いたtodoIdを持たないMyTodoができる件については、
とりあえず動くものができてからデータモデルを整備していこうと考えています。

[hoshi-takanori]

init(document: QueryDocumentSnapshot) はそのままで、引数なしの init() を追加しましたか？

[koshira27]

追加のコメントありがとうございます。

init(document: QueryDocumentSnapshot) はそのままで、引数なしの init() を追加したところ。ビルドできて追加したデータがtabeViewに表示されるようになりました！

私の中で、クラスにinit（イニシャライザ）は一つしか設定できないと思い込んでいたのですが、複数のイニシャライザを定義できるんですね。複数のイニシャライザを定義することについて少しネットで調べたところ、イニシャライザを定義した後はインスタンス生成時はそれぞれinitメソッドで定義した引数と一致する型と値を指定する必要があるのですね。

4日くらいハマっていたので大分助かりました。ありがとうございます！

[koshira27]

hoshi-takanoriさんをベストアンサーにしたいと思いますので、回答欄に最初のアドバイス等記載いただけると幸いです。

Answer 1

一つのクラス (や構造体) には引数の異なる複数のイニシャライザを定義できますので、既存の init(document: QueryDocumentSnapshot) はそのままで、引数なしの init() を追加すると良いでしょう。

この際、let myTodo = MyTodo() として生成したものは todoId や todoTitle の型がオプショナルであるため、初期値が nil になることに注意してください。

 class MyTodo: NSObject{
     var todoId: String?
     var todoTitle: String?
     var todoDone: Bool = false
 
+    init() {
+    }
 
     //取得したデータをFirestoreのデータ型に当てはめていくための型（MyTodoのデータ型）
     init(document: QueryDocumentSnapshot) {
         self.todoId = document.documentID
         self.todoTitle = document.data()["todoTitle"] as? String
         self.todoDone = document.data()["todoDone"] as! Bool
     }
 }

ちなみに、新規生成したドキュメントの id を取得する方法はこんな感です。また、テーブルに行を追加するのは、Firestore の処理が成功してからの方が良いような。
参考: swift - How to get document id after creation Firestore ios? - Stack Overflow

-                let ref: DocumentReference? = nil
-                self.db.collection("todo").addDocument(data: [
+                let ref = self.db.collection("todo").document()
+                print(ref.documentID)
+                ref.setData([
                     "todoTitle": myTodo.todoTitle!,
                     "todoDone": false
                 ]) { err in // 以下同様

Comments

[koshira27]

ありがとうございました！
複数のイニシャライザを定義できることについて勉強になりました。

新規生成したドキュメントの id を取得することについてもありがとうございます。本課題が解決してちょうど修正に取り組み始めたところなのでとても助かります。頂いたコメントとサンプルコードを参考に取り組んでみたいと思います。