cellForItemAtのメソッドが呼ばれない原因は？

Question

前提・実現したいこと

collectionViewのcellを表示させるだけのアプリを作っています。
しかし、cellForItemAtのメソッドが呼ばれず、cellを生成することができません。
もし原因がわかる方がいましたら、教えていただきたいです。
よろしくお願いします。

該当のソースコード

import UIKit

class HomeController: UICollectionViewController {

    override func viewDidLoad() {
        super.viewDidLoad()
        navigationItem.title = "Home"
        collectionView.backgroundColor = UIColor.white
        collectionView.register(UICollectionViewCell.self, forCellWithReuseIdentifier: "cell")
        
    }

    override func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
        return 5
    }
    
    override func numberOfSections(in collectionView: UICollectionView) -> Int {
        return 1
    }
    
    
    override func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "cell", for: indexPath)
        cell.backgroundColor = UIColor.orange
        return cell
    }
}

import UIKit

@UIApplicationMain
class AppDelegate: UIResponder, UIApplicationDelegate {

    var window: UIWindow?


    func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?) -> Bool {
        
        window = UIWindow(frame: UIScreen.main.bounds)
        window?.makeKeyAndVisible()
        let layout = UICollectionViewLayout()
        window?.rootViewController = UINavigationController(rootViewController: HomeController(collectionViewLayout: layout))
        return true
    }

    func applicationWillResignActive(_ application: UIApplication) {
        // Sent when the application is about to move from active to inactive state. This can occur for certain types of temporary interruptions (such as an incoming phone call or SMS message) or when the user quits the application and it begins the transition to the background state.
        // Use this method to pause ongoing tasks, disable timers, and invalidate graphics rendering callbacks. Games should use this method to pause the game.
    }

    func applicationDidEnterBackground(_ application: UIApplication) {
        // Use this method to release shared resources, save user data, invalidate timers, and store enough application state information to restore your application to its current state in case it is terminated later.
        // If your application supports background execution, this method is called instead of applicationWillTerminate: when the user quits.
    }

    func applicationWillEnterForeground(_ application: UIApplication) {
        // Called as part of the transition from the background to the active state; here you can undo many of the changes made on entering the background.
    }

    func applicationDidBecomeActive(_ application: UIApplication) {
        // Restart any tasks that were paused (or not yet started) while the application was inactive. If the application was previously in the background, optionally refresh the user interface.
    }

    func applicationWillTerminate(_ application: UIApplication) {
        // Called when the application is about to terminate. Save data if appropriate. See also applicationDidEnterBackground:.
    }


}

試したこと

・ブレークポイントを貼ってcellForRowAtが呼ばれているか調べた。

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

Swift4/Xcode10.0

// 2018/10/24 情報を追記しました！

Answer 1

質問に掲載されているHomeControllerとAppDelegateを
そのまま実装して試してみたところ、質問通りの現象が発生するので
少し調べてみました。

すると、numberOfSectionsやnumberOfItemsInSectionは
呼ばれるのに、cellForItemAtだけ呼ばれませんでした。

ということはセルを表示するようにレイアウトできていないのだろうと思って
改めてコードを見たところ、AppDelegateで作成しているコレクションビューレイアウトが

let layout = UICollectionViewLayout()

となっていました。
初期状態のUICollectionViewLayoutを作っただけなので、
具体的なレイアウトが何も設定されておらず、セルを表示する場所がなかったということだと思います。

とりあえずこれを

let layout = UICollectionViewFlowLayout()

にしたら、表示されるようになりました。

参考にしたと言っていたYoutubeはどうしていたのだろうと思って、ざっと見たところ、7分30秒あたりのコードでは、やはりUICollectionViewFlowLayoutを使っていました。

Comments

[kawano108]

回答ありがとうございます！！
TakeOneさんのおっしゃる通り実装して見たら無事セルが表示されました！
今回のセルが表示されなかったのはcollectionViewのレイアウトを設定する記述がなかったからなんですね。
Youtubeを見返しても気づきませんでした。笑
ただ今回ハマったことでUICollectionViewLayoutを弄るにはUICollectionViewFlowLayoutが必要だと分かりました。
原因を突き止めて頂いたTakeOneさん、並びに各方面からアドバイスして頂いた皆様、ご協力ありがとうございました！

Answer 2

以下はXcode 9での話でした

質問のコードではエラーが出る箇所があります。（以下の二行）

collectionView.backgroundColor = UIColor.white
collectionView.register(UICollectionViewCell.self, forCellWithReuseIdentifier: "cell")

ここを修正するだけでセルは表示されました。
エラーが出てないのであれば、質問に書かれていないことが原因でセルが表示されない（cellForRowAtが呼ばれない）と思われますので、情報の追記をお願いします。

Comments

[fuzzball]

viewDidLoad()内のcollectionViewが何者なのかが分かれば解決すると思います。

[kawano108]

ご回答頂きありがとうございます！
僕の方ではエラーは出ていませんでした。
なので、他に原因があるかと思います。
AppDelegateを含めた情報を追記しますので、もしよろしければもう一度見ていただけないでしょうか。
よろしくお願いします。

> viewDidLoad()内のcollectionViewが何者なのかが分かれば解決すると思います。

viewDidLoad()内のcollectionViewはcollectionViewのインスタンスだと認識してます。
だからcollectionViewクラスのプロパティであるbackgroundColorやメソッドであるregisterを呼ぶことができます。ということは分かってるんですが、それが今回のようにcellForItemAtメソッドが呼ばれない理由とどう繋がるのか分かりません。
無知ですみませんが、もう少し追加の説明を頂けると幸いです。

[fuzzball]

すみません、私のミスでした。
UICollectionViewControllerのプロパティのcollectionViewは、Xcode9までは UICollectionView? だったのに、Xcode10から UICollectionView! になってました。
なので、今のコードのままでエラーにはなりません。（ということはコードに問題は無いということです）

質問ですが、
・viewDidLoad()は呼ばれてますか？
・「cellForItem(at:)は呼ばれているけどセルが表示されない」のではなく「cellForItem(at:)が呼ばれていない」で間違いないでしょうか？

[kosanai]

動かない理由が特にないんですが
collectionViewがOptionalじゃないのがよくわかりませんね、コンパイルエラーになると思うんですが（Swiftバージョンの違い？）
動画でも
collectionView?.register(UICollectionViewCell.self, forCellWithReuseIdentifier: "cell")
となっています
collectionView?としたらどうなりますか？

[fuzzball]

>>kosanaiさん
いや、! もオプショナルです。（Implicitly Unwrapped Optionalというやつ）なので ! とか ? とか付けなくても勝手にアンラップされます。Swiftというか、SDKの違いになるのかな？

>>kawano108さん
kosanaiさんのコメントに書かれている2018/10/25 00:23のコメントを見ました。
Storyboardは使ってないんですね。（追記されたAppDelegateのコードをちゃんと見てませんでした）
さきほど付いたTakeOneさんの回答で解決しそうですね。

[kosanai]

失礼しました、スレッド間違えました（上の方に書こうと思ってこちらに書いてしまいました）
Xcode10からcollectionViewの取扱が変わっていたんですね（1つ上のコメントを読んでいませんでした）

Answer 3

間違いが5,6点くらいあります
まずは簡単なチュートリアルをGoogleや参考書やQiitaなどで探して同じように組んでみてください

ちゃんと書くとこうなります

class HomeController: UIViewController {

    @IBOutlet weak var homeCollectionView: UICollectionView!

    override func viewDidLoad() {
        super.viewDidLoad()
        navigationItem.title = "Home"

        homeCollectionView.delegate = self
        homeCollectionView.datasource = self

    }

    func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
        return 5
    }

    func numberOfSections(in collectionView: UICollectionView) -> Int {
        return 1
    }


    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "cell", for: indexPath)
        cell.backgroundColor = UIColor.orange
        return cell
    }
}

Comments

[fuzzball]

質問のコードは UICollectionViewController を継承しています。

[kosanai]

すいません勘違いしました
わからないですねこれ。

[kawano108]

ご回答いただきありがとうございます！
上記の例はストーリーボードを使ってUIViewControllerにCollectionViewを乗せて作った形ですね！
試しに同じように作ったら動きました！
ありがとうございます！
けど今回はCollectionViewControllerを使って、ソースコードのみで作りたいんです。
その理由は以下の動画を参考にしてるからです。

https://www.youtube.com/watch?v=3Xv1mJvwXok&list=PL0dzCUj1L5JGKdVUtA5xds1zcyzsz7HLj

なので、回答頂いたことは大変嬉しいんですが、もし良ければCollectionViewControllerを使う場合のアドバイスを頂けると幸いです。

Answer 4

cellForItemAtはUICollectionViewDataSourceプロトコルのメソッドですが、
UICollectionViewDataSource delegateが設定されていますか？
xib/StoryBoardで設定していないのであれば、コード上でHomeControllerをUICollectionViewDataSourceから派生する様にして、dataSourceに設定してあげる必要があります。

Comments

[fuzzball]

UICollectionViewControllerなので、何もしなくてもdelegateとdataSourceは繋がっています。