NCMBから取得したデータを読み込む関数にバグがあります。

Question

import UIKit
import NCMB

class ViewController: UIViewController, UICollectionViewDataSource, UICollectionViewDelegate{
    
    
    @IBOutlet var timescheduleCollectionView: UICollectionView!
    var subjectArray = [NCMBObject]()
    
    override func viewDidLoad() {
        super.viewDidLoad()
        timescheduleCollectionView.dataSource = self
        timescheduleCollectionView.delegate = self
        
        let nib = UINib(nibName: "subjectsCollectionViewCell", bundle: Bundle.main)
        timescheduleCollectionView.register(nib, forCellWithReuseIdentifier: "Cell")
        
        let layout = UICollectionViewFlowLayout()
        layout.itemSize.height = self.view.bounds.height / 9
        layout.itemSize.width = self.view.bounds.width / 7
        layout.minimumInteritemSpacing = 0
        layout.minimumLineSpacing = 0
        timescheduleCollectionView.collectionViewLayout = layout
    }
    
    
    override func viewWillAppear(_ animated: Bool) {
        super.viewWillAppear(animated)
        load()
    }
    
    
    func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {
        self.performSegue(withIdentifier: "register", sender: nil)
    }
    
    func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
        return 42
    }
    
    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        let cell = timescheduleCollectionView.dequeueReusableCell(withReuseIdentifier: "Cell", for: indexPath) as! subjectsCollectionViewCell
        print(indexPath.row)
        return cell
    }
    
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if segue.identifier == "register"{
            let AddSubjectsViewController = segue.destination as! AddSubjectsViewController
            let selectedIndex = timescheduleCollectionView.indexPathsForSelectedItems!
            AddSubjectsViewController.selectedPath = selectedIndex[0][1]
        }
    }
    
    func load() {
        let query = NCMBQuery(className: "Subjects")
        query?.findObjectsInBackground({(result, error) in
            if error != nil {
                print(error!)
            }
            else {
                self.subjectArray = result as! [NCMBObject]
                self.timescheduleCollectionView.reloadData()
            }
        })
    }
}

時間割を作成する工程で、collectionviewとNCMBのかけ合わせで作ろとしています。
CollectionViewのセルをタップすると授業名と教室名を登録する画面に遷移し、登録したデータをNCMBに保存し、それを時間割で表示したいと思っています。今、その表示する工程でエラーが発生しています。
内容は以下の通りです。

55行目のfunc load()関数はNCMBに保存してあるデータを取得し、subjectArrayに格納して、CollectionViewをreloadData()で再読み込みをする関数なのですが、時間割を登録したら時間割のページに戻るたびに、データが読み込み、つまり時間割が表示されるようにしたいので、viewWillAppearにload()関数を置くことによって、実現しています。
しかし、load()関数を置くことによって、検証のために設置してある43行目のprint(indexPath.row)が二重に表示されてしまう結果が得られてしまっています。そのせいで、load()関数でCollectionViewを読み込みされていないsubjectArray(つまり空の配列)が出来てしまい、データを表示することが出来ません。解決策をよろしくお願いします。

Answer 1

しかし、load()関数を置くことによって、検証のために設置してある43行目のprint(indexPath.row)が二重に表示されてしまう結果が得られてしまっています。そのせいで、load()関数でCollectionViewを読み込みされていないsubjectArray(つまり空の配列)が出来てしまい、データを表示することが出来ません。

cellForItemAt のdelegateメソッドは、必要に応じてTableViewから何度も呼び出されることがあります。何度呼び出されても指定されたindexPathのセルを返す処理が実施できればいいです。

今のコードで一番大きな問題は、 numberOfItemsInSection のdelegateメソッドが return 42 を返していて、データを読み込んでない状態でも42個のセルを表示しようとしていることです。

そして、 cellForItemAt のdelegateメソッドの中でセルにデータを表示する処理が何もないため、42個のセルには何も表示されていないということのように見えます。 subjectArray[indexPath.row] の中に入っているデータをセルに設定する処理をここに書く必要があると思いますが、それを書いたらエラーが発生するので一旦削除しているということであれば、そのエラーの原因はデータを読み込んでいないのに42個のセルを表示しようしている(subjectArrayに42個のデータが入っていると思って処理している)ためです。

numberOfItemsInSection のdelegateメソッドは、 return subjectArray.count を返却し、cellForItemAt のdelegateメソッドの中で subjectArray[indexPath.row] の中に入っているデータをセルに設定すれば、NCMBから読み込んだデータを正しくテーブルに表示できそうに思います。

Comments

[yumax]

ありがとうございます。