Повторное использование ячеек UICollectionView - некоторые ячейки не выбраны

Question

В UICollectionView у меня есть пользовательский UICollectionViewCellClass, где prepareForReuse переопределяется для сотрудников по умолчанию.

У меня есть NSMutableArray, содержащий NSIndexPaths от didSelectItemAtIndexPath:.

В cellForItemAtIndexPath: Я переформатировал выделенные ячейки так, чтобы они были выбраны.

-(UICollectionViewCell *)collectionView:(UICollectionView *)collectionView cellForItemAtIndexPath:(NSIndexPath *)indexPath{ 

ButtonCell *cell = [collectionView dequeueReusableCellWithReuseIdentifier:@"ButtonCell" forIndexPath:indexPath]; 

NSString *title = self.ingredientsBook.names[indexPath.item]; 

cell.label.text = title; 

if ([self isSelectedIndexPath:indexPath]){ 

    cell.backgroundColor = [UIColor whiteColor]; 
    cell.label.textColor = [UIColor blueColor]; 
} 
return cell; 

} 

-(void)collectionView:(UICollectionView *)collectionView didSelectItemAtIndexPath:(NSIndexPath *)indexPath{ 

self.searchButton.enabled = YES; 

ButtonCell *cell = (ButtonCell *)[collectionView cellForItemAtIndexPath:indexPath]; 
[selectedCellIndexPaths addObject:indexPath]; 
NSLog(@"%@", selectedCellIndexPaths); 
cell.backgroundColor = [UIColor whiteColor]; 
cell.label.textColor = [UIColor blueColor]; 

NSString *name = self.ingredientsBook.names[indexPath.item]; 
[self.selectedIngredientNames addObject:name]; 


} 

Проблема в том, что при нажатии первой ячейки невозможно выбрать 16-й или 17-й. Или, если я коснусь первых трех, невозможно выбрать три последних. didSelectItemAtIndexPath не называется.

Я чувствую, что это должно быть что-то очень простое, но я не вижу его прямо сейчас.

Я попытался поместить NSLogs в shouldSelectItemAtIndexPath, чтобы понять, был вызван этот метод и метод вообще не вызван. Это происходит, когда между выбранным и проблемным существует расстояние 16 ячеек.

Вот другие методы источника данных и isSelectedIndexPath:

-(BOOL)isSelectedIndexPath:(NSIndexPath *)indexPath{ 

for (NSIndexPath *test in selectedCellIndexPaths){ 
    if (test == indexPath){ 
     return YES; 
    } 
} 

return NO; 
} 

-(NSInteger)collectionView:(UICollectionView *)collectionView numberOfItemsInSection:(NSInteger)section{ 

return [self.ingredientsBook.names count]; 
} 

-(NSInteger)numberOfSectionsInCollectionView:(UICollectionView *)collectionView{ 

return 1; 
} 

-(BOOL)collectionView:(UICollectionView *)collectionView shouldSelectItemAtIndexPath:(NSIndexPath *)indexPath{ 

NSLog(@"%@", indexPath); 

return YES; 
} 


-(void)collectionView:(UICollectionView *)collectionView didDeselectItemAtIndexPath:(NSIndexPath *)indexPath{ 

self.searchButton.enabled = ([[collectionView indexPathsForSelectedItems] count] > 0); 

ButtonCell *cell = (ButtonCell *)[collectionView cellForItemAtIndexPath:indexPath]; 
cell.backgroundColor = [UIColor blackColor]; 
cell.label.textColor = [UIColor whiteColor]; 

[selectedCellIndexPaths removeObject:indexPath]; 
NSString *name = self.ingredientsBook.names[indexPath.item]; 
[self.selectedIngredientNames removeObject:name]; 



} 

Answer 1

я нашел две проблемы. Метод prepareForReuse, казалось, искажал вещи, поэтому я просто удалил его. Основная проблема, однако, заключалась в том, как вы реализовали isSelectedIndexPath :. Как только он находит первый выбранный элемент при прохождении элементов, он возвращает YES и выходит из цикла. То, что вы хотите сделать, это просто проверить, если indexPath содержится в массиве selectedCellIndexPaths:

-(BOOL)isSelectedIndexPath:(NSIndexPath *)indexPath{ 

    if ([selectedCellIndexPaths containsObject:indexPath]) { 
     return YES; 
    }else{ 
     return NO; 
    } 
} 

Или, если вы предпочитаете использовать более лаконичный синтаксис, вы можете заменить если-нибудь блок с:

return ([selectedCellIndexPaths containsObject:indexPath])? YES : NO; 

Answer 2

Недавно я столкнулся с тем же вопросом с моим приложением. Выбор UICollectionViewCell работал правильно до iOS 8.3, впоследствии я начал видеть странное поведение. Ячейки, которые на самом деле не были выбраны, будут отобраны, другие ячейки, казалось бы, случайно не могут быть выбраны.

я имел как пользовательские setSelected и prepareForResuse методы, реализованную на подклассе UICollectionViewCell как таковые:

-(void)setSelected:(BOOL)selected 
{ 
    [super setSelected:selected]; 

    if (selected) 
    { 
     [[self selectedIndicator] setHidden:NO]; 
    } 
    else 
    { 
     [[self selectedIndicator] setHidden:YES]; 
    } 
} 

-(void)prepareForReuse 
{ 
    [[self imageView] setImage:nil]; 
} 

Метод prepareForReuse просто сбросить вид изображения в пользовательской ячейке.

В моем методе prepareForReuse я не звонил [super prepareForReuse] (который в соответствии с документацией ничего не делает по умолчанию). Когда я добавил вызов [super prepareForReuse], весь выбор работал так, как предполагалось. Хотя Apple заявляет, что реализация по умолчанию ничего не делает, они также рекомендуют вызывать супер. Следуя этой рекомендации, я решил проблему.

Answer 3

В iOS 10 я обнаружил, что программная очистка UICollectionView с включенным множественным выбором была ошибкой, когда была включена предварительная выборка.И, конечно, предварительная выборка включена по умолчанию в iOS 10.