Обновите collectionView на viewdidload после получения UserDefaults

Question

У меня есть представление коллекции, и вы можете выбирать элементы в нем и включать и выключать их, изменяя цвет фона.Ячейки включаются / выключаются благодаря логическому значению, указанному в стрелке, которую я сделал для всех ячеек.Я сохранил значение bool, но когда я пытаюсь записать их обратно в массив и использовать collectionView.reloadData(), приложение вылетает.Мой код collectionView:

extension OLLViewController: UICollectionViewDataSource, UICollectionViewDelegate {

func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {  //set the amount of items in the CollectionView to the amount of items in the OLLData dictionary
    return OLLData.OLLCasesList.count

}

func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {  //set each cell to a different mamber of the dict.
    let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "OLLCell", for: indexPath) as! OLLCell
    cell.imageView.backgroundColor = OLLData.OLLCasesList[indexPath.item]._isSelected ? UIColor.orange : UIColor.clear //change colour if selected

    let image = OLLData.OLLCasesList[indexPath.item]._imageName

    cell.label.text = image
    cell.imageView.image = UIImage(named: image)

    let savedIsSelected = defaults.bool(forKey: Key.isSelected)

    OLLData.OLLCasesList[indexPath.item]._isSelected = savedIsSelected
    //collectionView.reloadData() //when uncommented it crashes the app

    return cell
}

func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath)  { //detect if case selected and reload CollectionView
    let caseName = OLLData.OLLCasesList[indexPath.item]._imageName
    print(caseName, OLLData.OLLCasesList[indexPath.item]._isSelected)
    OLLData.OLLCasesList[indexPath.item]._isSelected = !OLLData.OLLCasesList[indexPath.item]._isSelected
    defaults.set(OLLData.OLLCasesList[indexPath.item]._isSelected, forKey: Key.isSelected)
    collectionView.reloadItems(at:[indexPath])

    collectionView.reloadData()

    if OLLData.OLLCasesList[indexPath.item]._isSelected == true { //if the item is selected, add to selectedCases array
        selectedCases.append(OLLData.OLLCasesList[indexPath.item]._id)
        selectedCaseNames.append(OLLData.OLLCasesList[indexPath.item]._imageName)
        print(selectedCases, selectedCaseNames) //debugging
        numberOfSelectedCases.text = String(selectedCases.count)
    }
    else if OLLData.OLLCasesList[indexPath.item]._isSelected == false { //remove from selectedCases array
        selectedCases.removeAll(where: { $0 == OLLData.OLLCasesList[indexPath.item]._id })
        selectedCaseNames.removeAll(where: { $0 == OLLData.OLLCasesList[indexPath.item]._imageName })
        print(selectedCases, selectedCaseNames) //debugging
        numberOfSelectedCases.text = String(selectedCases.count)
    }
}

._isSelected - это логическое значение, которое указывает, является ли ячейка «переключенной».

Буду признателен за любые идеи.

Answer 1

Прежде всего, раскомментирование этой строки создаст бесконечный цикл.cellForRowAt происходит потому, что представление коллекции перезагружается, поэтому вызывать обновление во время обновления представления коллекции нецелесообразно.

Поэтому ваша проблема в том, что вы не знаете, как отображать выбранные ячейки в представлении коллекции., верно?

Вот функция, которая запускается непосредственно перед тем, как представление коллекции собирается отобразить ячейку:

func collectionView(_ collectionView: UICollectionView, 
                    willDisplay cell: UICollectionViewCell, 
                    forItemAt indexPath: IndexPath)
{
    <#code#>
}

Внутри этой функции вы должны:

1. Приведите cell в вашу OLLCell (безопасно, если вы хотите быть тщательным)
2. Посмотрите на свои данные и посмотрите, должна ли быть выбрана ячейка OLLData.OLLCasesList[indexPath.item]._isSelected
3. Спросите у вашей приведенной ячейкиизменить его цвет / интерфейс / внешний вид в соответствии с вашим ._isSelected логическим

Шаг 3 имеет ОЧЕНЬ важное предупреждение.Вы должны изменить пользовательский интерфейс, когда ._isSelected ложно И когда это правда.Поскольку представление коллекции повторно использует ячейки, старое состояние пользовательского интерфейса будет случайным образом повторяться.Поэтому установка его каждый раз является хорошим способом обеспечить желаемое поведение.

Вот пример:

func collectionView(_ collectionView: UICollectionView, 
                    willDisplay cell: UICollectionViewCell, 
                    forItemAt indexPath: IndexPath)
{
    //Cast the vanilla cell into your custom cell so you have access 
    //to OLLCell's specific functions and properties.
    //Also make sure the indexPath falls in the indices of your data 
    if let myCastedCell = cell as? OLLCell,
       0 ..< OLLData.OLLCasesList.count ~= indexPath.item 
    {
        myCastedCell.imageView.backgroundColor = OLLData
            .OLLCasesList[indexPath.item]._isSelected 
                ? UIColor.orange 
                : UIColor.clear 
    }
}