CODEWITHSUNDEEP
iosswiftuicollectionview
Dominik
Dominik

Reputation: 419

Multiple collectionviews in one Viewcontroller causes Index out of range error

I try to use three collectionviews in one Viewcontroller. I parse the data like the following method shows: At the bottom i add the data depending on the position to the right list (this part works)

 func getEventData(eventIDs: [String], plz: String, positiona: Int){


        for eventId in eventIDs {
        let ref = Database.database().reference().child("Events").child(plz).child(eventId)
         ref.observe(.value, with: { snapshot in
            let item = snapshot.value as? [String: AnyObject]
            let eventName = item?["name"] as! String
            let date = item?["date"] as! String
            let lat = item?["lat"] as! String
            let lng = item?["lng"] as! String
            let infos = item?["additionalInfos"] as! String
            let position = item?["position"] as! String
            let ts = item?["ts"] as! Int
            let createdBy = item?["createdBy"] as! String

         let timestamp = NSDate().timeIntervalSince1970
            if (ts > Int(timestamp)) {
                let eo = EventObject(eventID: eventId, eventName: 
   eventName, info: infos, createdBy: createdBy, date: date, lat: lat, lng: lng, position: position, ts: ts)
                if positiona == 0{
                    self.acceptedEvents.append(eo)
                    self.acceptedEventscv.reloadData()

                }else if positiona == 1{
                    self.myEvents.append(eo)
                    self.myEventscv.reloadData()

                }else if positiona == 2{
                   self.storedEvents.append(eo)
                self.storedEventscv.reloadData()
                }
            }

         })
        }
    }

In my NumbersofItemsInSection method i did the following which works as well:

func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
    if collectionView == self.acceptedEventscv{
    return acceptedEvents.count
    }else if collectionView == self.storedEventscv{
        return storedEvents.count
    }else if collectionView == self.myEventscv{
        return myEvents.count
    }else{
        return 0
    }
    }

and in my CellForRowAtItem method i tried the following

func collectionView(_ collectionView: UICollectionView, 
 cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
 if collectionView == self.acceptedEventscv{
 let cell = 
  collectionView.dequeueReusableCell(withReuseIdentifier: 
"acceptedEventsCell", for: indexPath) as! 
 acceptedEventsCollectionViewCell
 let eo = acceptedEvents[indexPath.row]
  cell.eventName.text = eo.eventName

  let items = eo.date!.components(separatedBy: " ")//Here replase 
  space with your value and result is Array.
 //Direct line of code
 //let items = "This is my String".components(separatedBy: " ")
 let date = items[0]
let time = items[1]
cell.date.text = date
cell.time.text = time

getUserNameAge(label: cell.usernameAge)
return cell
}else {
  let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "savedEventsCell", for: indexPath) as! savedEventsCollectionViewCell
  let eo = acceptedEvents[indexPath.row]
  cell.eventName.text = eo.eventName

  let items = eo.date!.components(separatedBy: " ")//Here replase space with your value and result is Array.
  //Direct line of code
  //let items = "This is my String".components(separatedBy: " ")
  let date = items[0]
  let time = items[1]
  cell.date.text = date
  cell.time.text = time


  getUserNameAge(label: cell.usernameAge)

  return cell
}

}

The problem is if i have more items in the seccond CollectionView, i always get this error:

Thread 1: Fatal error: Index out of range

at this line of code: 

let eo = acceptedEvents[indexPath.row]

Upvotes: 0

Views: 39

Answers (1)

Shehata Gamal
Shehata Gamal

Reputation: 100533

cellForItemAt also must be like

if collectionView == self.acceptedEventscv{
  let item = acceptedEvents[indexPath.row]
   ----
  return cell
}else if collectionView == self.storedEventscv{
    let item = storedEvents[indexPath.row]   
    ----
    return cell
}else {
     let item = myEvents[indexPath.row] 
     ----
     return cell  
}

what happens now in your case is that you access the same array acceptedEvents in both cases where the returned count in numberOfItemsInSection may be different

Upvotes: 1

Related Questions