ううFirestoreによるデータの表示がうまくいきません

初めてFirebaseを用いてSNS系アプリを作ろうとしてます。今回はテストなので検索窓から全てのユーザーのアカウントをtableviewで表示させ、tableview cellをタップすると画面遷移してユーザーのプロフィールページにとび、名前やプロフィールを表示させようとしています。

問題点
Stringでユーザー名を画面遷移時に渡しており、print()を使って画面遷移先のviewdidloadで渡されたユーザー名を確認すると正常に渡されていることが確認されました。しかし、それを用いてnameLabel.text に代入して画面遷移先のユーザーの名前を表示するラベルには表示されません。

試したこと
画面遷移時にユーザー固有のIDも一緒に渡しCloud Firestoreから固有のIDをもとにデータを取り出そうと考えたのですがこれもうまくいきませんでした。

下のコードのこの部分で条件分岐の全てがelseに流れたのにおかしさを感じましたが原因がわかりませんでした。
よろしくお願いします。

db.collection("users").getDocuments() { (querySnapshot, error) in
            if let error = error {
                print(error)
            } else {
                for document in querySnapshot!.documents {

                    if document.documentID == self.passedUserId{
                        print("yayayaya")
                        print(self.passedUserId)
                    }else{
                        print(self.passedUserId)
                        print("nononono")
                    }
                    print("(document.documentID) => (document.data())")
                }
            }
        }

画面遷移はSearchViewController.swift→DetailViewController.swiftです

DetailViewController.swift

import UIKit
import NCMB
import SCLAlertView
import FirebaseFirestore
import FirebaseAuth
import FirebaseStorage

class DetailViewController: UIViewController, UITextViewDelegate {


    @IBOutlet var nameLabel: UILabel!
    var passedUserName = String()
    var passedUserId = String()
    
    
    override func viewDidLoad() {
        nameLabel.text = String(passedUserName)
        print(passedUserName)
        print(passedUserId)
        
        let db = Firestore.firestore()
        
        db.collection("users").getDocuments() { (querySnapshot, error) in
            if let error = error {
                print(error)
            } else {
                for document in querySnapshot!.documents {
                    
                    if document.documentID == self.passedUserId{
                        print("yayayaya")
                        print(self.passedUserId)
                    }else{
                        print(self.passedUserId)
                        print("nononono")
                    }
                    print("(document.documentID) => (document.data())")
                }
            }
        }
        
        
        /*
        let document = db.collection("users")
        document.getDocuments { (snapshot, error) in
            if error != nil{
                print(error)
            }else{
                print(snapshot?.documents)
            }
        }
        */
        
        /*
        let document = db.collection("users").document(passedUserId)
        document.getDocument { (snapshot, error) in
            if error != nil {
                print(error)
            }else{
                print("ああああああああ")
            }
        }
        */
        
        
    }
    
    override func viewWillAppear(_ animated: Bool) {
        nameLabel.text = passedUserName
    }
    
    
    func loadDetail(){
        
    }
}

修正後
SearchViewController.swift

import UIKit
import NCMB
import Kingfisher
import PKHUD
import FirebaseAuth
import FirebaseFirestore

class SearchViewController: UIViewController, UITableViewDelegate, UITableViewDataSource{

    var allUsers = [String]()
    var allUsersId = [String]()
    
    var selectedUser = String()
    var selectedUserId = String()
    
    @IBOutlet var searchUserTableView: UITableView!
    
    let db = Firestore.firestore()
   
    
    var yay = ["月曜日", "火曜日", "水曜日"]
    
    

    override func viewDidLoad() {
        super.viewDidLoad()
        print("1")
        searchUserTableView.dataSource = self
        searchUserTableView.delegate = self
        searchUserTableView.tableFooterView = UIView()
        
        
        
       loadUsers()
        
    }
    
    override func viewWillAppear(_ animated: Bool) {
        super.viewWillAppear(animated)
    }
    
    func loadUsers(){
        db.collection("users").getDocuments(){ (querySnapshot, err) in
            if let err = err {
                print(err)
            } else {
                for document in querySnapshot!.documents{
                    //print(document.documentID)
                    let idForDocument = document.documentID
                    
                    self.allUsersId.append(idForDocument)
                    
                    print(self.allUsersId)
                    
                    let docRef = self.db.collection("users").document(idForDocument)
                    docRef.getDocument { (snapshot, error) in
                        if error != nil{
                            print(error)
                        }else{
                            //print(snapshot?.data())
                            let userName = snapshot?.get("userName") as! String
                            
                            
                            print(userName)
                            
                            self.allUsers.append(userName)
                            print("aaaaaaaaaaaaaaaaa")
                            print(self.allUsers)
                            print("2")
                            self.searchUserTableView.reloadData()
                        }
                    }
             
                }
                
                
            }
            
        }
        
        
    }
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        print("3")
        return allUsers.count
        
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        
        let cell = tableView.dequeueReusableCell(withIdentifier: "Cell")!
        cell.textLabel?.text = allUsers[indexPath.row]
        print("4")
        cell.tag = indexPath.row
        return cell
    }
    
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if segue.identifier == "toDetail"{
            let detailViewController = segue.destination as! DetailViewController
            detailViewController.passedUserName = selectedUser
            detailViewController.passedUserId = selectedUserId
            
            print("11111")
        }
    }
    
    
    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        let selectedUser = allUsers[indexPath.row]
        let selectedUserId = allUsersId[indexPath.row]
        
        
    
        print(selectedUser)
        print(selectedUserId)
        
        print("222222")
        performSegue(withIdentifier: "toDetail", sender: nil)
        print("333333")
        tableView.deselectRow(at: indexPath, animated:  true)
        print("444444")
    }

行動規範の内容に同意します

回答1件

コードの流れから、TableView Cell から　DetailViewController へ Segue を繋いでいるように見受けられますが、その場合の処理の流れは

prepare(for:sender:) が呼び出される
tableView(_:didSelectRowAt:) が呼び出される

という順番になるため、1. ではの文字列が渡される結果になってしまいます。

つまり、2. で値を渡すための変数に値を代入（選択したセルに関連する値を代入）していますが、その処理の前に遷移のための処理が行われてしまっている、ということです。

したがって、TableView Cell から Segue を繋ぐのではなく、SearchViewController から DetailViewController へ Segue を繋ぎ、identifier を設定し、tableView(_:didSelectRowAt:)の適切な場所からperformSegue(withIdentifier:sender:) を呼び出すような流れにする必要があるかと思います。

ちなみに、両方の ViewController とも、overrideした関数内で親となる関数を呼び出していませんが、これは何らかの理由があるのでしょうか。

今回はこれが原因というわけではありませんが、特段の理由がなければ

Swift
1    override func viewDidLoad() {
2        //処理
3    }
4
5    override func viewWillAppear(_ animated: Bool) {
6        //処理
7    }
8

ではなく、

Swift
1    override func viewDidLoad() {
2        super.viewDidLoad()
3        // 処理
4    }
5    
6    override func viewWillAppear(_ animated: Bool) {
7        super.viewWillAppear(animated)
8        //処理        
9    }

と、親関数を呼び出してあげない限り、予想外の結果を招くことがあるかと思いますのでご確認ください。

投稿2020/09/11 23:03

TsukubaDepot

総合スコア5086

退会済みユーザー

2020/09/12 01:44

SearchViewController.swiftの修正後のコードを追加したました。222222→11111→333333→444444 のような挙動を示しました。そして依然DetailViewControllerのnameLabel.textに値が入りません。また、SearchViewController から DetailViewController へ Segue を繋ぎ、identifier を設定しました。 overrideに関しては特に意図はありませんでした。一度消してしまったので、補完機能を使って書いた際に表示されなかったものだと思われます。

TsukubaDepot

2020/09/12 03:08

DetailViewControlle の viewDidLoad() にある print(passedUserName) print(passedUserId) ただ、そのまま値を表示しても、どのメソッドから出力されているかわかりにくくなりますので、 print(#function, passedUserId) や、 print(#file, passedUserId) ように、関数（メソッド）名やファイル名を合わせて出力させた方がわかりやすいかもしれません。ここの部分では、渡された値は表示されていますでしょうか。

退会済みユーザー

2020/09/12 07:12

print(#function, passedUserName) print(#function, passedUserId) の場合は viewDidLoad() viewDidLoad() となります。 print(#file, passedUserName) print(#file, passedUserId) の場合は Desktop/ios_app/SOBIT1/SOBIT1/DetailViewController.swift Desktop/ios_app/SOBIT1/SOBIT1/DetailViewController.swift と返されます。

行動規範の内容に同意します