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

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

ただいまの
回答率

88.10%

1画面中に複数collectionViewを設置した時にクラッシュします。

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 2,380

score 7

前提・実現したいこと

swift3,xcode8.2.1にて作業をしています。

以下こちらの記事を参考させていただきました。
https://teratail.com/questions/61968
http://docs.fabo.io/swift/uikit/054_customuicollectionview.html
文字数の関係で必要ないだろうと思われるところは略しています。

1画面中に3つのcollectionViewをscrollViewに横並びに設置して
先月、今月、来月の順で表示ができるカレンダーを作っています。
実際には常に今月のみが表示されていて、その他前後の月はスワイプすることで表示されます。
来月にスワイプで切り替わったきったときに今月を表示させつつ各月を次の月にreloadDateさせてます。
先月の場合はその逆です。

発生している問題・エラーメッセージ

スワイプ時の月によってクラッシュします。
月によって行数が変わるのですが、適切な数字を入れてもなぜか該当月にクラッシュします。

2017-02-02 13:12:20.628 CalendarScrollApp[13353:574951] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'UICollectionView received layout attributes for a cell with an index path that does not exist: <NSIndexPath: 0xc000000004600116> {length = 2, path = 1 - 35}'
(lldb)


といったメッセージがコンソールに表示されます。

該当のソースコード

基本的にこちらの記事を参考させていただきました。
https://teratail.com/questions/61968
http://docs.fabo.io/swift/uikit/054_customuicollectionview.html
文字数の関係で必要ないだろうと思われるところは略しています。

上記以外の試みとして、
scrollViewを設置しその中にcollectionViewを置いています。
スワイプ処理以下のようにしています。

    func scrollViewDidEndDecelerating(_ scrollView: UIScrollView) {
        let w = UIScreen.main.bounds.width
        let x = scrollView.contentOffset.x
        var frag: Int! = 0

        switch x {
        case 0..<w:
            frag = 1
            break
        case (w * 2)..<(w * 3):
            frag = 2
            break
        default:
            frag = 0
        }

        if frag == 1 {
            selectedDatePrev = dateManagerPrev.prevMonth(date: selectedDatePrev as Date) as NSDate
            selectedDate = dateManagerTo.prevMonth(date: selectedDate as Date) as NSDate
            selectedDateNext = dateManagerNext.prevMonth(date: selectedDateNext as Date) as NSDate
        } else if frag == 2 {
            selectedDatePrev = dateManagerPrev.nextMonth(date: selectedDatePrev as Date) as NSDate
            selectedDate = dateManagerTo.nextMonth(date: selectedDate as Date) as NSDate
            selectedDateNext = dateManagerNext.nextMonth(date: selectedDateNext as Date) as NSDate
        }
        if frag != 0 {
            calenderCollectionViewPrev.reloadData()
            calenderCollectionViewTo.reloadData()
            calenderCollectionViewNext.reloadData()
            headerLabel.text = changeHeaderTitle(date: selectedDate as Date)
            headerLabel.sizeToFit()
            headerLabel.center = CGPoint(x: headerView.center.x, y: headerView.center.y/2)

            scrollView.contentOffset.x = w
        }
    }

試したこと

ステップ実行でnumberOfItemsInSectionでcell数を渡すタイミングでクラッシュしているところまではわかったのですが、、、それ以上がどうしてもわかりません。

    func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
        if collectionView == calenderCollectionViewTo {
            return dateManagerTo.daysAcquisition()
        } else if collectionView == calenderCollectionViewNext {
            return dateManagerNext.daysAcquisition()
        } else if collectionView == calenderCollectionViewPrev {
            return dateManagerPrev.daysAcquisition()
        } else {
            return 0
        }
    }


collectionViewをインスタンス化した順番によって変わりますがいずれかをreturnした時にクラッシュします。

数字の渡し方が悪いのか全く見当違いのことをしているのか、
どうかアドバイスを頂けますと幸いです。
よろしくお願いします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • hidegoro

    2017/02/02 15:37 編集

    func numberOfSections(in collectionView: UICollectionView) -> Int { return 1 }となります。

    キャンセル

  • fuzzball

    2017/02/02 15:38

    DateManagerは全く同じで変更していないということでしょうか?

    キャンセル

  • hidegoro

    2017/02/02 15:53

    ベストアンサーのDateManager.siwftをそのまま使っていますが、conversionDateFormatのformatter.dateFormat = "d"をformatter.dateFormat = "M-d"にしている以外はそのまま使っています。

    キャンセル

回答 1

checkベストアンサー

+1

 その1

index pathが存在しないというエラーです。
path = 1 - 35のrowは「日」でしょうか?35日というのはおかしいですよね。
CollectionView側ではなく、CollectionViewに値を渡している側に問題があると思います。

 その2

numberOfSections(in:)で1を返しているのであれば、sectionは0にしかならないはずですが1になっています。
やはり、CollectionView側ではなく、「値を渡している側」に問題があると思います。

 その3

月の切り替えのところを、下記のように変更するとどうなるでしょうか?(キャストは外しているのでエラーが出たら追加して下さい)

if frag == 1 {
    dateManagerPrev.prevMonth(date: dateManagerPrev.selectedDate)
    dateManagerTo.prevMonth(date: dateManagerTo.selectedDate)
    dateManagerNext.prevMonth(date: dateManagerNext.selectedDate)
} else if frag == 2 {
    dateManagerPrev.nextMonth(date: dateManagerPrev.selectedDate)
    dateManagerTo.nextMonth(date: dateManagerTo.selectedDate)
    dateManagerNext.nextMonth(date: dateManagerNext.selectedDate)
}

 その4

現在、3つのカレンダーでUICollectionViewFlowLayoutを使い回していますが、それぞれ別のインスタンスを作成して下さい。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/02/03 12:08

    よろしくお願いします。https://github.com/nagahide/CalendarScrollApp

    キャンセル

  • 2017/02/03 17:33

    その4を追記しました。

    キャンセル

  • 2017/02/04 15:24

    完璧です。クラッシュせずに機能しました。UICollectionViewFlowLayoutは使い回しではいけないのですね。なぜなのか疑問が残りますがこれはまた独自に調べてみます。ありがとうございました。

    キャンセル

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

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

関連した質問

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