L'appel de setCollectionViewLayout:animation ne recharge pas UICollectionView

Je suis à l'aide d'un UICollectionView avec deux modèles personnalisés qui sont sous-classé de UICollectionViewFlowLayout

Quand ma vue se charge tout d'abord je l'ai mis à une vue par défaut à l'aide de: 

[self.collectionView setCollectionViewLayout:self.tableViewLayout];

Ce appels: cellForItemAtIndexPath - donc pas de problème là. C'est dans viewWillAppear méthode.

Puis-je utiliser un UIButton pour modifier la mise en page de la vue actuelle de la prochaine mise en page comme ceci: 

-(void)changeViewLayoutButtonPressed
{
    self.changeLayout = !self.changeLayout;


    if (self.changeLayout){

        [self.tradeFeedCollectionView setCollectionViewLayout:self.grideLayout animated:YES];

    }

    else {


        [self.tradeFeedCollectionView setCollectionViewLayout:self.tableViewLayout animated:YES];

    }
}

Après cet appel - numberOfItemsInSection est appelé. J'ai vérifié dans le débogueur que le retour de compte n'est pas zéro.

Après cet appel, rien d'autre n'est appelé et mon UICollectionView modifie la mise en page mais depuis cellForItemAtIndexPath n'est pas nommé - les cellules ne sont pas correctement paramétré.

si j'ai mis [self.collectionView roloadData] à l'intérieur de numberOfItemsInSection - alors cellForItemAtIndexPath est appelé et les cellules sont de l'installation.

Je ne devrais pas besoin de faire un appel manuel de recharger les données dans numberOfItemsInSection. Quelqu'un peut me dire ce qui se passe?

Modifier

J'ai oublié de mentionner que je suis à l'aide de deux UINibs pour les deux cellules différentes, comme ils ont besoin de légèrement différentes dispositions sont appliquées. Serait-ce un problème?
Merci!

Edit 2

Droite, donc j'ai eu à travailler presque comme je veux. Voici tout le code utilisé pour l'affichage de la collection et comment je modifier ses dispositions.

Les Dispositions Personnalisées

Ce sont des sous-classé de UICollectionViewFlowLayout - La raison pour laquelle je sous-classé est tout simplement en raison de la face de mon UICollectionView besoins à l'apparence et au comportement très proche de la Pomme de pré-faites la mise en page. Juste différentes tailles de cellule, cependant.

TableViewLayout
- Pas de code dans le fichier d'en-tête.
- BBTradeFeedTableViewLayout.m

@implementation BBTradeFeedTableViewLayout

-(id)init
{
    self = [super init];

    if (self){

        self.itemSize = CGSizeMake(320, 80);
        self.minimumLineSpacing = 0.1f;
    }

    return self;
}
@end

Gride Disposition
- Pas de code dans le fichier d'en-tête.
- BBTradeFeedGridViewLayout.m

   @implementation BBTradeFeedGridViewLayout

-(id)init
{
    self = [super init];

    if (self){

        self.itemSize = CGSizeMake(159, 200);
        self.minimumInteritemSpacing = 0.1f;
        self.minimumLineSpacing = 0.1f;
    }
    return self; 
}
@end

Puis dans le ViewController qui a mon UICollectionView - je déclarer les deux modèles personnalisés comme: 

@property (strong, nonatomic) BBTradeFeedTableViewLayout *tableViewLayout;
@property (strong, nonatomic) BBTradeFeedGridViewLayout *grideLayout;

Je puis d'enregistrer les deux .xib fichiers pour le collectionView:

  [self.tradeFeedCollectionView registerNib:[UINib nibWithNibName:@"BBItemTableViewCell" bundle:nil] forCellWithReuseIdentifier:@"TableItemCell"];
        [self.tradeFeedCollectionView registerNib:[UINib nibWithNibName:@"BBItemGridViewCell" bundle:nil] forCellWithReuseIdentifier:@"GridItemCell"];

Puis, j'ai un UIButton qui modifie la mise en page: 

    -(void)changeViewLayoutButtonPressed
{

    if (!self.gridLayoutActive){

        self.gridLayoutActive = YES;
        [self.tradeFeedCollectionView setCollectionViewLayout:self.grideLayout animated:YES];
        self.lastLayoutUsed = @"GridLayout";

    }

    else {

        self.gridLayoutActive = NO;

        [self.tradeFeedCollectionView setCollectionViewLayout:self.tableViewLayout animated:YES];
        self.lastLayoutUsed = @"TableLayOut";
    }
    [self.tradeFeedCollectionView reloadItemsAtIndexPaths:[self.tradeFeedCollectionView indexPathsForVisibleItems]];

}

Enfin - quand un appel de service web se fait - il appelle [self.tradeFeedCollectionView reloadData];

Donc il ma UICollectionView obtient reloaded et la dernière méthode: 

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

{   static NSString *tableCellIdentifier = @"TableItemCell";
    static NSString *gridCellIdentifier = @"GridItemCell";

    if (indexPath.item < [self.tradeSearchArray count]){

    if (self.gridLayoutActive == NO){

        BBItemTableViewCell *tableItemCell = [collectionView dequeueReusableCellWithReuseIdentifier:tableCellIdentifier forIndexPath:indexPath];

    if ([self.tradeSearchArray count] > 0){

        self.toolBarButtomItem.title = [NSString stringWithFormat:@"%d Results", self.searchResult.searchResults];

        tableItemCell.gridView = NO;
        tableItemCell.backgroundColor = [UIColor whiteColor];
        tableItemCell.item = self.tradeSearchArray[indexPath.row];

    }
        return tableItemCell;
}else

    {

        BBItemTableViewCell *gridItemCell= [collectionView dequeueReusableCellWithReuseIdentifier:gridCellIdentifier forIndexPath:indexPath];

    if ([self.tradeSearchArray count] > 0){

        self.toolBarButtomItem.title = [NSString stringWithFormat:@"%d Results", self.searchResult.searchResults];


        gridItemCell.gridView = YES;
        gridItemCell.backgroundColor = [UIColor whiteColor];
        gridItemCell.item = self.tradeSearchArray[indexPath.row];

    }

    return gridItemCell;
    }

Le problème avec le code ci-dessus

Le code ci-dessus fonctionne. Cependant, les cellules n'animez pas bien, et il semble étrange, presque comme si une actualisation qui s'est passé. Qui est en raison de l'appel:

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

Mais c'est la seule façon que je pouvais obtenir pour être proche de ce que je veux.

  • Je trouve cette question très longtemps.
  • Quoi de mal à cela?
InformationsquelleAutor Robert J. Clegg | 2014-03-18
  1. 20

    -setCollectionViewLayout: ou -setCollectionViewLayout:animated: ne causera pas de vos UICollectionView recharger ses données.

    • Si vous voulez changer votre style de cellule ou de mettre à jour les données, appel [self.collectionView reloadData] et UICollectionViewDataSource protocole méthodes seront appelées.

    • Si vous souhaitez modifier votre UICollectionView la mise en page à une autre, appelez -setCollectionViewLayout:. Ou si vous voulez mettre à jour votre UICollectionView mise en page, il suffit d'appeler [self.collectionView.collectionViewLayout invalidateLayout].

    De la cellule de mise en page et les données sont deux choses différentes dans UICollectionView.

    Mise à jour

    vous devriez vous débarrasser de toutes les données obsolètes, probablement par -reloadData. Et calculer le nouveau cadres de chaque cellule dans votre UICollectionViewLayout sous-classes. Remplacer - (UICollectionViewLayoutAttributes *)layoutAttributesForItemAtIndexPath:(NSIndexPath *)indexPath et - (NSArray *)layoutAttributesForElementsInRect:(CGRect)rect.

    Vous décidez de la nouvelle contentOffset. Par exemple, vous pouvez garder le indexPath de certains visible de la cellule et de décider pour faire défiler jusqu'à que indexPath ou setContentOffset: à la frame.origin de la nouvelle cellule dans le même indexPath après votre mise en page a changé.

    • Je veux changer la cellule de mise en page de mon tableView type de grille de mise en page. Cependant, à moins de me demander manuellement le uicollectionview pour recharger - les cellules sont laissées dans leur ancien état. La seule solution que j'ai trouvé était à l'aide de cette méthode: [self.tradeFeedCollectionView reloadItemsAtIndexPaths:[self.tradeFeedCollectionView indexPathsForVisibleItems]]; - mais je puis lâche l'animation et ça à l'air bizarre. J'aimerais résoudre ce problème. Je vais même augmentation de la prime. 😀
    • Pourriez-vous mettre un peu de code?
    • Je vais mettre à jour la question avec le code que j'ai actuellement
    • J'ai mis à jour ma question. Comme je l'ai dit plus tôt - ce code fonctionne. Cependant, les cellules ne sont pas animer correctement. Si je retire la solution de contournement - ils s'animent, et pourtant je suis de gauche avec sortie de données de la cellule et il semble foiré.
    • Recharger la cellule sera la cause de la cellule précédente instance d'être jetés. Avez-vous essayé de recharger les cellules de données avant de changer de mise en page dans votre -(void)changeViewLayoutButtonPressed?
    • Oui. Si je l'appelle: [self.tradeFeedCollectionView reloadData}; avant que je change la mise en page - anime correctement que si je suis en haut de la UICollectionView. Si je défile vers le bas un peu et de changer la mise en page, la UICollectionView disparaît pendant une demi-seconde et réinitialise l'affichage vers le haut de UICollectionView. Si je l'appelle: [self.tradeFeedCollectionView reloadItemsAtIndexPaths:[self.tradeFeedCollectionView indexPathsForVisibleItems]]; avant-je changer la mise en page - les cellules aller complètement foiré jusqu'à ce que je commence à défiler, puis ils semblent normaux.
    • Aussi à ajouter: L'effet, je suis à la recherche est ce que ebay ont dans leur application iOS. Ils varient entre une table de type cellule à une cellule de la grille avec une très belle animation en douceur qui vous tient à votre position actuelle. –
    • Vous pouvez essayer de reloadData d'abord, et ensuite de changer votre configuration après 0,2 s. Le code va comme: [self.tradeFeedCollectionView reloadData]; dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t)(0.2 * NSEC_PER_SEC)), dispatch_get_main_queue(), ^{ if (!auto.gridLayoutActive){ ...} });
    • Belle idée. Cependant, cela me donne le même problème que l'utilisation de l'auto.tradeFeedCollectionView reloadItemsAtIndexPaths:[self.tradeFeedCollectionView indexPathsForVisibleItems]] avant de changer la mise en page. Alors dès que je commence à défiler les cellules commencent à apparaître comme normal. Un tel comportement étrange! J'ai aussi augmenté le temps de 0,2 à 1 seconde. Ne fait aucune différence.
    • Ça c'est sûr. [self.tradeFeedCollectionView reloadItemsAtIndexPaths:[self.tradeFeedCollectionView indexPathsForVisibleItems]] seulement mettre à jour les données des éléments visibles. La vieille, mal configuré cellules venir Quand vous faites défiler, mais votre mise en page est changé à ce point.
    • Si je commence à défiler, puis les cellules s'afficher correctement. Ses avant que je ne commence à défiler que le problème est présent. Aussi, l'animation est foutu. Cela ne devrait pas être si difficile. :-(. Remercie de votre aide si!
    • Je suis confus. Tu veux dire les cellules d'affichage lorsque vous utilisez -reloadData mais les choses sont différentes lorsque vous utilisez -reloadItemsAtIndexPaths?
    • Lorsque j'utilise une recharge de données sans l'envoi de code que vous avez fourni des cellules d'affichage de l'amende. Ils animent seulement si je suis en haut de UICollectionView. Si j'ai défile un peu - pas d'animation - UICollectionView montre le bon cellules encore - mais au niveau de l'index.la ligne 0 - pas et l'index.ligne //où je défile vers. Si j'utilise reloadItemsAtIntexPath de nouveau, sans la méthode dispatch - je n'ai pas d'animation, mais les cellules apparaissent correctement et je suis conservées au même endroit, j'ai défilé pour. Si je l'appelle reloadAtIndexPath AVANT que je change la mise en page - puis-je obtenir un comportement inattendu.
    • Lorsque les cellules sont en désordre et je commence à défiler, ils commencent à apparaître correctement lorsque de nouvelles cellules viennent sur l'écran et les vieilles cellules qui est allé hors de l'écran revenir affiche correctement. Si j'utilise votre méthode dispatch - C'est le même comportement que l'utilisation de reloadItemsAtIndexPath avant de changer la mise en page.
    • J'ai pas encore de croire que dans [self.collectionView reloadItemsAtIndexPaths:[self.collectionView indexPathsForVisibleItems]]. Vos modèles sont différents, le itemSizes sont très variables. Et actuellement visible éléments peuvent être hors de l'écran et les éléments invisibles peut-être venir, en raison de votre mise en page des stratégies. C'est bien que -reloadData permet de réinitialiser le contentOffset. Je pense que c'est une mise en question et il est de votre UICollectionViewLayoutla responsabilité de comprendre la façon de traiter avec le contentOffset. Après tout, le précédent est compensée à vide de sens lors de vos données à jour et de la taille de la cellule sont le changement.
    • Je pense que vous êtes sur quelque chose. Comment pourrais-je le compte pour le contentOffSet? C'est la première fois que je suis en utilisant UICollectionView et une mise en page personnalisée.
    • À mon humble avis, vous devriez vous débarrasser de toutes les données obsolètes, probablement par -reloadData. Et calculer le nouveau cadres de chaque cellule dans votre UICollectionViewLayout sous-classes. Remplacer - (UICollectionViewLayoutAttributes *)layoutAttributesForItemAtIndexPath:(NSIndexPath *)indexPath et - (NSArray *)layoutAttributesForElementsInRect:(CGRect)rect. Vous décidez de la nouvelle contentOffset. Par exemple, vous pouvez garder le indexPath de certains visible de la cellule et de décider pour faire défiler jusqu'à que indexPath ou setContentOffset à la frame.orgin de la nouvelle cellule dans le même indexPath après votre mise en page a changé.
    • Je pense que vous avez raison. Je vais avoir à gérer ça moi-même. Bien que je n'ai jamais utilisé ces méthodes et ne sais pas comment je pourrais même aller à leur mise en œuvre. Je vais voir si je peux pas trouver des exemples en ligne, et comment d'autres l'ont fait. Vous m'avez fait bénéficier le plus de l'aide et je l'apprécie vraiment. Mise à jour de votre réponse avec le dernier commentaire, si vous voulez, et je vais le marquer comme réponse. J'espère que je vais arriver avec ces méthodes. Merci encore mec.

    InformationsquelleAutor liuyaodong
  2. 16

    Appelez simplement reloadData avant vous passez votre mise en page (la performBatchUpdates appel n'est pas obligatoire):

    [self.collectionView reloadData];
[self.collectionView.collectionViewLayout invalidateLayout];
[self.collectionView setCollectionViewLayout:yourLayout animated:YES];

    Dans Swift:

    self.collectionView?.reloadData()
self.collectionView?.collectionViewLayout.invalidateLayout()
self.collectionView?.setCollectionViewLayout(yourLayout, animated: true)
    • Merci pour l'aide. Votre code, tandis qu'il travaille, génère des problèmes d'implantation des avertissements dans ma console. Le premier code que j'ai posté que la solution fonctionne toujours. Outre les mises en garde, à l'aide de votre code, le résultat final est exactement le même. Merci quand même!
    • Bizarre... je ne vois pas d'avertissement sur iOS 8 ou 9. J'ai été confronté à quelques mises en garde lors de l'utilisation performBatchUpdates (en essayant de charger la collecte de la disposition de l'affichage des données lors de la mise en page est verrouillée) et j'ai donc trouvé que seulement mettre reloadData() avant le changement de disposition a fait le tour (sans que cette étrange mise en garde). En tout cas merci pour votre thread, il me permet de gagner beaucoup de temps 🙂
    • J'ai dû mettre reloadData avant performBatchUpdates et la setCollectionViewLayout dans le performBatchUpdates achèvement bloc.
    InformationsquelleAutor Artheyn
  3. 1

    Vous devriez appeler [self.tradeFeedCollectionView invalidateLayout]; avant 

    if (self.changeLayout){
    [self.tradeFeedCollectionView setCollectionViewLayout:self.grideLayout animated:YES];
}
else {
    [self.tradeFeedCollectionView setCollectionViewLayout:self.tableViewLayout animated:YES];
}
    • Merci mais l'auto.tradeFeedCollectionView ne répond pas à invalidateLayout.
    • Peut-être que soi.tradeFeedCollectionView n'est pas votre UICollectionView. Essayez ensuite: [self.collectionView invalidateLayout]
    • tradeFeedCollectionView est une instance de UICollectionView. déclaré comme ceci: @property (forte, nonatomic) IBOutlet UICollectionView *tradeFeedCollectionView; Edit: j'ai remarqué les changements de vue et les cellules sont configurés de manière incorrecte. Jusqu'à ce que je commence à défiler - alors ils sont configurés de manière incorrecte. Je me demande pourquoi est-ce?
    • tradeFoodViewController répond à invalidateIntrinsicContentSize - cependant, il n';t recharger les cellules correctement. À moins que je ne commence à défiler.
    • Édité le sollicité avec les infos que j'ai oublié d'ajouter
    • Aussi à ajouter: L'effet, je suis à la recherche est ce que ebay ont dans leur application iOS. Ils varient entre une table de type cellule à une cellule de la grille avec une très belle animation en douceur qui vous tient à votre position actuelle.

    InformationsquelleAutor Lolloz89
  4. 0

    Vous ne devriez avoir à appeler cette méthode:

    [self.collectionView setCollectionViewLayout:layout animated:YES];

    J'ai eu un bug similaire, mais il s'est avéré que mon sous-vues de l'cellules ne sont pas de programme d'installation avec Mise en page Automatique. La cellule a été redimensionnement correctement, mais mon sous-vues n'étaient pas, tant il ressemblait à la taille de la cellule n'a jamais changé. Vérifier dans votre code rapidement par la mise en clipsToBounds = OUI sur votre cellulaire.contentView.

    InformationsquelleAutor Michael Rose
  5. 0

    Ici, c'est le truc simple si vous souhaitez mettre à jour les dispositions de l'Utiliser partout où vous êtes à la mise à jour de la collectionView:

    [self.collectionView removeFromSuperview]; //First remove the Collection View

//Relocate the view with layouts
UICollectionViewFlowLayout *layout=[[UICollectionViewFlowLayout alloc] init];
layout.scrollDirection = UICollectionViewScrollDirectionHorizontal;
layout.minimumInteritemSpacing = 10;
layout.minimumLineSpacing = 10;

 self.collectionView = [[UICollectionView alloc] initWithFrame:CGRectMake(0, 0, self.view.frame.width,100) collectionViewLayout:layout];
[self.collectionView setDataSource:self];
[self.collectionView setDelegate:self];
[self.collectionView registerClass:[UICollectionViewCell class] forCellWithReuseIdentifier:@"cellIdentifier"];
[self.collectionView reloadData];

[self.view addSubview:self.collectionView];
    InformationsquelleAutor Shivam Singh
  6. 0

    (Posté solution sur le nom de l'auteur de la question).

    J'ai enfin trouvé une belle solution simple et.

    L'marqué réponse est bonne, que je devais l'appeler, -reloadData sur mon UICollectionView - Toutefois, cela détruit l'animation lors de la commutation entre les deux cellules.

    Après avoir pris des suggestions de l'marqué réponse - j'ai juste retravaillé mon UIButton "méthode", qui modifie les cellules

    J'ai trouvé que si je l'appelle: 

    [self.collectionview performBatchUpdates:^{  } completion:^(BOOL finished) {

}];

    Selon la WDDC vidéo - ce qui anime les changements fonctionne avec -reloadData méthode:

    Exemple: 

    [self.collectionView reloadData]; 
    [self.collectionView performBatchUpdates:^{
        [self.collectionView.collectionViewLayout invalidateLayout];
        [self.collectionView setCollectionViewLayout:self.grideLayout animated:YES];
    } completion:^(BOOL finished) {

    }];
}
    InformationsquelleAutor
  7. 0

    J'ai trouvé que c'facilement et agréablement l'animation des cellules de genre et la mise en page en même temps, il est préférable de reloadSections (au lieu de reloadData), suivie par la définition d'une nouvelle mise en page avec une animation:

    [self.collectionView reloadSections:[NSIndexSet indexSetWithIndex:0]];
[self.collectionView setCollectionViewLayout:newLayout animated:animated];
    InformationsquelleAutor Rivera