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

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

ただいまの
回答率

90.32%

  • Swift

    7694questions

    Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

  • Xcode

    4353questions

    Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。

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

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 1,091

hidegoro

score 1

前提・実現したいこと

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/02 15:16 編集

    ご回答ありがとうございます。カレンダーのマス目の数を返すようになっています。例として2017/2なら5週あるので5x7=35になります。ですので28,35,42しかありません。

    キャンセル

  • 2017/02/02 16:22

    その2を書きました。

    キャンセル

  • 2017/02/02 16:29

    本来のコードではnumberOfSections(in:)は2を渡しています。一つ目のセクションは曜日を表示し、二つ目のセクションに日付(月部分)を表示させています。実際にクラッシュしている箇所は試したことで表示していますコードのreturn dateManagerTo.daysAcquisition()部分でクラッシュしています。

    キャンセル

  • 2017/02/02 16:39

    daysAcquisition()の中のどこでクラッシュしているのでしょうか?

    キャンセル

  • 2017/02/02 16:43

    returnで返した時にクラッシュします。

    キャンセル

  • 2017/02/02 16:58

    daysAcquisition()のreturnでしょうか?numberOfItems()のreturnでしょうか?

    キャンセル

  • 2017/02/02 17:11

    その3を書きました。

    キャンセル

  • 2017/02/02 17:20

    numberOfItems()のreturnです。試したことでも書いています。カレンダーをスワイプする都度reloadDataするわけですが、月によってマスの数が変わるのでその月に応じた数を返しているつもりなのですが、3月から4月に変わる時に35から42になるため数を入れ直しつつもクラッシュするのがわからないのです。数が違ってクラッシュなのか?collectionViewが見えてない状態で値を変えているからクラッシュしているのか?その辺のご回答をいただきたかったです。長々お付き合い感謝します。よろしくお願いします。

    キャンセル

  • 2017/02/02 17:43

    その3書いてますので試して下さい。(見落とされてそうな気がしたので念のため)

    キャンセル

  • 2017/02/02 19:35

    ご連絡遅くなり申し訳ありません。コードありがとうございます。なぜか症状が変わりました。ビルドして初期状態は1,2,3月が表示されていて、位置は2月になっているのですが、前までは2から3月に変更する時にクラッシュしていたのが、4から5月になる時にクラッシュするようです。若い月は2016.12月から11月に変わる時です。クラッシュ箇所は一緒です。

    キャンセル

  • 2017/02/03 00:01

    selectedDateは、各DateManager内にありますので、新たに用意する必要はありません。ViewController内のselectedDateと、DateManager内のselectedDateが不整合を起こしているのではないか?と怪しんでいます。

    ViewController内に定義しているselectedDateは削除して、DateManager経由でアクセスするようにしてみて下さい。(私のコードを参考にしてください)

    そろそろ断片的なコードでは対応が厳しくなってきていますので、全コードの公開を検討して下さい。

    キャンセル

  • 2017/02/03 10:06

    おはようごいざます。ViewController内のselectedDateを全てDateManager内のselectedDateに置き換えてみたのですが症状は変わりません。全コードを公開する場合ははやはりgit等がいいのでしょうか?

    キャンセル

  • 2017/02/03 10:33

    githubでも、適当なオンラインストレージでも、何でもいいですよ。
    (ソースコード内の個人情報に気を付けて下さいね)

    キャンセル

  • 2017/02/03 10:56

    ありがとうございます。それではgithubに公開します。しばらくお待ちください。

    キャンセル

  • 2017/02/03 12:08

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

    キャンセル

  • 2017/02/03 17:33

    その4を追記しました。

    キャンセル

  • 2017/02/04 15:24

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

    キャンセル

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

  • Swift

    7694questions

    Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

  • Xcode

    4353questions

    Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。