Assertion failure when using UISearchDisplayController in UITableViewController Assertion failure when using UISearchDisplayController in UITableViewController ios ios

Assertion failure when using UISearchDisplayController in UITableViewController


Try using self.tableView instead of tableView in dequeueReusableCellWithIdentifier:

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath{    UITableViewCell *cell = [self.tableView dequeueReusableCellWithIdentifier:@"BreedCell"];    //Create PetBreed Object and return corresponding breed from corresponding array    PetBreed *petBreed = nil;    if(tableView == self.searchDisplayController.searchResultsTableView)        petBreed = [_filteredBreedsArray objectAtIndex:indexPath.row];    else        petBreed = [_breedsArray objectAtIndex:indexPath.row];    cell.accessoryType  = UITableViewCellAccessoryDisclosureIndicator;    cell.textLabel.text = petBreed.name;    return cell;}

This code works pretty well

Note

If you have custom height cells, do not use

[self.tableView dequeueReusableCellWithIdentifier:CellIdentifier forIndexPath:indexPath];

Use this instead

[self.tableView dequeueReusableCellWithIdentifier:CellIdentifier];


The reason why it worked great on first run but then crashed if you exited the results table and went back in for another search is because the Search Display Controller is loading a new UITableView each time you enter search mode.

By search mode I mean, you've tapped the textfield and you've began to type, at which point a table view is generated to display results, exiting this mode it achieved by hitting the cancel button. When you tap the textfield the second time and begin typing again - this is entering "search mode" for the second time.

So in order to avoid the crash you should register the cell class for the table view to use in the searchDisplayController:didLoadSearchResultsTableView: delegate method (from UISearchDisplayDelegate) of instead of in your controllers viewDidLoad method.

As follows:

- (void)searchDisplayController:(UISearchDisplayController *)controller didLoadSearchResultsTableView:(UITableView *)tableView{    [tableView registerClass:[DPContentTableCell class] forCellReuseIdentifier:cellIdentifier];    [tableView registerClass:[DPEmptyContentTableCell class] forCellReuseIdentifier:emptyCellIdentifier];}

This caught me by surprise because on iOS 7... the table view is being reused. So you can register the class in viewDidLoad if you prefer. For legacy sakes, I'll keep my registration in the delegate method I mentioned.


After searching, 'tableView' of cellForRowAtIndexPath method seems not an instance of the Table that you define. So, you can use an instance of a table that defines the cell. Instead of:

UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:@"Cell" forIndexPath:indexPath];

Use:

UITableViewCell *cell = [self.tableView dequeueReusableCellWithIdentifier:@"Cell" forIndexPath:indexPath];

(Do not use the tableView of cellForRowAtIndexPath method, use self.tableView.)