CollectionViewのdataSourceメソッドが発火しない

Question

UICollectionViewのdataSourceメソッドが発火しません。
storyboardは使っておらず、コードのみの実装です。

class ViewController: UIViewController {

    private var contentCollectionView = UICollectionView(frame: .zero, collectionViewLayout: UICollectionViewLayout())

    override func viewDidLoad() {
        super.viewDidLoad()

        contentCollectionView.dataSource = self
        contentCollectionView.register(ContentCollectionViewCell.self, forCellWithReuseIdentifier: "Cell")
        
        let layout = UICollectionViewFlowLayout()
        layout.itemSize = CGSize(width: screenWidth * 0.4, height: 60)
        contentCollectionView = UICollectionView(frame: .zero, collectionViewLayout: layout)

        contentCollectionView.backgroundColor = .yellow
        self.view.addSubview(contentCollectionView)
        contentCollectionView.translatesAutoresizingMaskIntoConstraints = false
        contentCollectionView.topAnchor.constraint(equalTo: view.topAnchor, constant: 0).isActive = true
        contentCollectionView.leftAnchor.constraint(equalTo: view.leftAnchor, constant: 0).isActive = true
        contentCollectionView.rightAnchor.constraint(equalTo: view.rightAnchor, constant: 0).isActive = true
        contentCollectionView.bottomAnchor.constraint(equalTo: view.bottomAnchor, constant: 0).isActive = true
    }
}

extension ViewController: UICollectionViewDataSource {
    func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
        return 6
    }
    
    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        print("dataSorce method")
        let cell = contentCollectionView.dequeueReusableCell(withReuseIdentifier: "Cell", for: indexPath)
        return cell
    }
}

class ContentCollectionViewCell: UICollectionViewCell {
    
    let titleLabel = UILabel()
    let numberLabel = UILabel()
    
    override func awakeFromNib() {
        super.awakeFromNib()
    }
    
    override init(frame: CGRect) {
        super.init(frame: frame)
        print("cell init")

        setupView()
    }
    
    required init?(coder: NSCoder) {
        fatalError("init(coder:) has not been implemented")
    }
    
    func setupView() {
        titleLabel.text = "default"
        self.contentView.addSubview(titleLabel)
        titleLabel.translatesAutoresizingMaskIntoConstraints = false
        titleLabel.centerXAnchor.constraint(equalTo: self.contentView.centerXAnchor, constant: 0).isActive = true
        titleLabel.topAnchor.constraint(equalTo: self.contentView.topAnchor, constant: 10).isActive = true
        
        numberLabel.text = "default"
        self.contentView.addSubview(numberLabel)
        numberLabel.translatesAutoresizingMaskIntoConstraints = false
        numberLabel.topAnchor.constraint(equalTo: titleLabel.bottomAnchor, constant: 3).isActive = true
        numberLabel.leftAnchor.constraint(equalTo: self.contentView.leftAnchor, constant: 0).isActive = true
    }
}

・cellForItemAt内のprintと、カスタムセルのinit内のprintは実行されていない。
・dequeueReusableCell(withReuseIdentifier: "Cell", for: indexPath)の
identifierを登録していない適当な文字列に変更してもエラーがおきない。

どこか見落としている箇所はありますか？

Answer 1

現在のコードだと

class ViewController: UIViewController {

    private var contentCollectionView = UICollectionView(frame: .zero, collectionViewLayout: UICollectionViewLayout())

上記の部分で UICollactionView のインスタンスを作成し、viewDidLoad()で

    override func viewDidLoad() {
        super.viewDidLoad()

        contentCollectionView.dataSource = self
        contentCollectionView.register(ContentCollectionViewCell.self, forCellWithReuseIdentifier: "Cell")

        let layout = UICollectionViewFlowLayout()
        layout.itemSize = CGSize(width: screenWidth * 0.4, height: 60)

Cell の登録や Cell のサイズを設定したあと、

        contentCollectionView = UICollectionView(frame: .zero, collectionViewLayout: layout)

再度 UICollectionView のインスタンスを作り直しているため、dataSource や register(_:forCellWithReuseIdentifier:) の情報がリセットされてしまっていることが原因かと思います。

対策は色々あるかと思いますが、ひとつはクロージャと lazy キーワードを使い、

class ViewController: UIViewController {
    private lazy var contentCollectionView: UICollectionView = {
        var layout = UICollectionViewFlowLayout()
        layout.itemSize = CGSize(width: self.view.frame.width * 0.4, height: 60)
        return UICollectionView(frame: .zero, collectionViewLayout: layout)
    }()
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        contentCollectionView.dataSource = self
        contentCollectionView.register(ContentCollectionViewCell.self, forCellWithReuseIdentifier: "Cell")

のような感じにする方法があるかと思います。

あるいは、とりあえず適当な引数を与えて一度インスタンスを作り、その後順番を追って上書きする方法もあります。

class ViewController: UIViewController {
    private var contentCollectionView = UICollectionView(frame: .zero, collectionViewLayout: UICollectionViewLayout())
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        let layout = UICollectionViewFlowLayout()
        layout.itemSize = CGSize(width: self.view.frame.width * 0.4, height: 60)
        contentCollectionView = UICollectionView(frame: .zero, collectionViewLayout: layout)
        
        contentCollectionView.dataSource = self
        contentCollectionView.register(ContentCollectionViewCell.self, forCellWithReuseIdentifier: "Cell")

残りの手段としては、contentCollectionView をオプショナル型で宣言する方法もありますが、それなりに危険性がありますので、その内容を把握されて使うのがよろしいかと思います。

    private var contentCollectionView:UICollectionView!

    override func viewDidLoad() {
        super.viewDidLoad()
        
        let layout = UICollectionViewFlowLayout()
        layout.itemSize = CGSize(width: self.view.frame.width * 0.4, height: 60)
        contentCollectionView = UICollectionView(frame: .zero, collectionViewLayout: layout)
        
        contentCollectionView.dataSource = self
        contentCollectionView.register(ContentCollectionViewCell.self, forCellWithReuseIdentifier: "Cell")

Comments

[HiakruKuroda]

なるほど！質問のコードで上書きできていると勘違いしていました。
無事実装できました。
回答ありがとうございます！