質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.91%

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

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 312

score 13

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を登録していない適当な文字列に変更してもエラーがおきない。

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

現在のコードだと

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")

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/09/29 15:29

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.91%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る