reloadData not working for UICollectionView in iOS7

Question

So basically i have search method "searchTableList" and after i get the result i want, i want to reload the uicollectionview

- (void)searchBar:(UISearchBar *)searchBar textDidChange:(NSString *)searchText {
   // NSLog(@"Text change - %d",isSearching);

    //Remove all objects first.
    [filteredContentList removeAllObjects];

    if([searchText length] != 0) {
        isSearching = YES;
        [self searchTableList];
    }
    else {
        isSearching = NO;
    }

    [self.collectionView reloadData];


}

However after "[self.collectionView reloadData]" Nothing happens !

After using this, it crashes all the time !

[self.collectionView reloadItemsAtIndexPaths:[self.collectionView indexPathsForVisibleItems]];
[self.collectionView reloadData];

What can i do, Thanks alot!!! :D

Error message is:

    2014-04-16 18:24:35.684 SampleProject1[59602:60b] the item width must be less that the width of the UICollectionView minus the section insets left and right values.
2014-04-16 18:24:37.709 SampleProject1[59602:60b] *** Assertion failure in -[UICollectionView _endItemAnimations], /SourceCache/UIKit_Sim/UIKit-2935.137/UICollectionView.m:3688
2014-04-16 18:24:37.713 SampleProject1[59602:60b] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'attempt to insert item 0 into section 0, but there are only 0 items in section 0 after the update'

Fixed It:

Basically i was returning the count of an unused array which always had a lenght of 0 instead of the relevant array !

So i had to change This:

if (isSearching) {

    return [searchResult count];

} else {
    return [self.tracks count];
}

To: if (isSearching) {

        return [filteredContentList count];

    } else {
        return [self.tracks count];
    }

Then after this i removed all the reloadData jargons and replaced it with:

[self.collectionView reloadData];

Thanks alot guys !!

If it crashes, what is the error message? – Marc Apr 16 '14 at 17:20 — Marc, Apr 16 '14 at 17:20
@MarcMosby just updated the code and the error message :) – ipalibowhyte Apr 16 '14 at 17:27 — ipalibowhyte, Apr 16 '14 at 17:27
Google your last error message. Only the reason text. – Marc Apr 16 '14 at 17:30 — Marc, Apr 16 '14 at 17:30
Is the delegate assigned properly? – Nate Flink Apr 16 '14 at 17:40 — Nate Flink, Apr 16 '14 at 17:40

score 4 · Accepted Answer · edited Jan 24 '16 at 07:41

4

After writing reloadItemsAtIndexPaths no need of writing again reloadData.
Reload the respective collection View row using main thread.

Code Snippet:

[[NSOperationQueue  mainQueue]  addOperationWithBlock:^{

    // Reload the respective collection view row using the main thread.

    [self.collectionView reloadItemsAtIndexPaths:[self.collectionView indexPathsForVisibleItems]];

}];

edited Jan 24 '16 at 07:41

Bista

7,869
3
27
55

answered Apr 16 '14 at 17:56

Jayaprada

944
8
11

What is the crash log ?Is it the same as previous? – Jayaprada Apr 16 '14 at 18:13
Try this http://stackoverflow.com/questions/16341778/uicollectionviewflowlayout-not-invalidating-right-on-orientation-change – Jayaprada Apr 16 '14 at 18:44
Thanks alot brother ! finally fixed. After going through your solution i found out that i was returning 0 this was because i was using an unused array. How stupid of me :) Thanks alot for the time though !! – ipalibowhyte Apr 16 '14 at 18:58
@Pizzy213codes, I can be your sister :). M female :) – Jayaprada Apr 16 '14 at 18:59
how do we do that in swift 3? – nyxee Jul 21 '17 at 06:55

score 0 · Answer 2 · answered Jan 21 '17 at 16:07

Check for any [collectionView reloadData] commands in your willDisplayCell... method. I had this in there and it worked once, but when I needed to reloadData again with a new dataset it just wouldn't trigger any more willDisplayCell calls.

It was all on the main thread, that wasn't the problem. More like an overflow problem.

reloadData not working for UICollectionView in iOS7

2 Answers2