Discussion :

[midemarc]

Bonjour,
Je veux récupérer un accessibilityIdentifier de chacune de mes cellules de UICollectionView.
Pour cela, j'effectue les traitements suivants :
-dans la fonction cellForItemAtIndexPath, Je crée une cellule "custom" (héritant de UICollectionViewCell pour lui ajouter une imageView) avec dequeueReusableCellWithReuseIdentifier

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
- (MyCollectionViewCell *)collectionView:(UICollectionView *)collectionView cellForItemAtIndexPath:(NSIndexPath *)indexPath{
static NSString *identifier = @"MyCell";        
MyCollectionViewCell *cell;
cell = [collectionView dequeueReusableCellWithReuseIdentifier:identifier forIndexPath:indexPath];

J'ajoute ensuite une ImageView dans ma cellule et je lui donne un identifiant grâce à accessibilityIdentifier

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
imageView.image = [UIImage imageNamed: name];
[imageView.image setAccessibilityIdentifier:name];
cell.imageView = currentImageView;

Quand j'essaie d'accéder à la collectionView existante, je rencontre un problème innatendu :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

NSArray *tableauDesCellules = [self.collectionView visibleCells];

J'obtiens dans tableauDesCellules un tableau de 0 objets, comme si collectionView était vide. Quand j'essaie d'itérer à travers chacune des cellules visibles par contre, plutôt que de sortir de ma boucle immédiatement, je me retrouve avec des cellules d'index désordonnés. Ainsi :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
for(MyCollectionViewCell *cell in self.collectionView.visibleCells)
{
    NSIndexPath *indexPath = [self.collectionView indexPathForCell:cell];
    NSLog(@"%d",indexPath.item);
}

me retourne :

2014-06-30 14:06:16.702 [5685:60b] 38
2014-06-30 14:06:16.702 [5685:60b] 17
2014-06-30 14:06:16.702 [5685:60b] 21
2014-06-30 14:06:16.703 [5685:60b] 0
2014-06-30 14:06:16.703 [5685:60b] 25
2014-06-30 14:06:16.703 [5685:60b] 4
2014-06-30 14:06:16.704 [5685:60b] 29
2014-06-30 14:06:16.704 [5685:60b] 8
2014-06-30 14:06:16.704 [5685:60b] 33
…

Du coup, mes accessibilityIdentifiers ne sont pas ceux attendus. Ce problème me bloque depuis des jours et je ne vois pas comment m'en sortir. Toute aide sera grandement appréciée.

PS : ma fonction cellForItemAtIndexPath effectue une initialisation avec une couleur aléatoire dans chaque cellule. Dois-je y voir un lien ? Si c'est le cas, comment détecter qu'une cellule existe déjà et récupérer son contenu ?

[LeBzul]

Salut,
J'imagine que tu passe un Array à UICollectionView lors de l'initialisation;
Tu as essayé de récupérer tes valeurs directement dans cet Array ? Logiquement la position dans l'array est le même que dans la CollectionView.

[midemarc]

Non, j'ai défini une classe dérivant de UICollectionViewController en temps que dataSource et implémenté les méthodes associées numberOfItemsInSection et cellForItemAtIndexPath. Du coup, quand ma UICollectionView se charge, ma méthode cellForItemAtIndexPath est appelée autant de fois que nécessaire pour remplir la colleciton view. Du coup pas d'Array et c'est moins simple.
Tu pense que c'est une mauvaise pratique ?

[LeBzul]

Je ne pense pas que ce soit une mauvaise pratique, mais je n'utilise pas celle la.
Pourquoi as tu besoin de l'accessibilityIdentifier de ton image et à quel moment ?
Il y a peut être une autre solution pour arriver au but final.

La methode cellForItemAtIndexPath ne dessine pas une fois toutes les cellules; Elle dessine que celle visible à l'écran, donc si on joue avec le scroll la methode cellForItemAtIndexPath sera appelé autant de fois que les celulles passent de non visible à visible.

[midemarc]

J'ai besoin d'un nombre fixe de cellules, j'ai dimensionné cellules et collectionview de façon à ce que chacune des cellules soit présente dans cette dernière sans scroll.
En conséquent, la fonction cellForItemAtIndexPath est appelée pour chacune de mes cellules dès le chargement de la scène.
Mon objectif est de récupérer un élément aléatoire parmi ceux présent dans la collection view. Pour cela, je pensais parcourir la collectionView et noter chacun des éléments différents présents dans la liste. L'accessibilityIdentifier est un moyen pour moi de détecter sur quel élément je tombe dans la liste puisque mes éléments sont des imageView.

Puisque toute mes cellules sont visibles en même temps, je pensais pouvoir itérer à travers cette collection grâce au

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
for(MyCollectionViewCell *cell in self.collectionView.visibleCells)
{
    NSIndexPath *indexPath = [self.collectionView indexPathForCell:cell];
    NSLog(@"%d",indexPath.item);
}

mais l'ordre des cellules que j'obtiens est aberrant, c'est ce que montre le log que j'ai affiché.

[midemarc]

Il existe un bug sur l'attribut visibleCell d'une collectionView : http://openradar.appspot.com/radar?id=2855401
Les cellules sont renvoyées dans le désordre, mais il n'y a pas de perte pour autant !
En fait, je n'ai pas de raison de vouloir parcourir la collectionView dans l'ordre, ça me semblait simplement la seule manière logique, et je ne voyais pas comment faire autrement. Pour l'instant, j'ai implémenté une solution où j'incrémente des compteurs dans mon for et vérifie à la fin ce que j'ai obtenu :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
int beige,rouge, bleu, jaune, vert;
beige=0;
rouge=0;
bleu=0;
vert=0;
jaune=0;
int row=0;
		
NSArray *tableauDesCellules = [self.collectionView visibleCells];

NSLog(@"iteration through cells");
	
for(MyCollectionViewCell *cell in self.collectionView.visibleCells)
{
        //NSIndexPath *indexPath = [self.collectionView indexPathForCell:cell];
        //NSLog(@"indexPath : %d",indexPath.item);

        int level = [preferences integerForKey:@"level"];
		

		nom_image = [[cell getImageView].image accessibilityIdentifier];

		if([nom_image isEqual:@"beige"])
			beige++;
		else if([nom_image isEqual:@"vert"])
			vert++;
		else if([nom_image isEqual:@"jaune"])
			jaune++;
		else if([nom_image isEqual:@"bleu"])
			bleu++;
		else if([nom_image isEqual:@"rouge"])
			rouge++;
                row++;
}

	NSLog(@"Nombre de :");
	NSLog(@"beige : %d",beige);
	NSLog(@"jaune : %d",jaune);
	NSLog(@"bleu : %d",bleu);
	NSLog(@"rouge : %d",rouge);
	NSLog(@"vert : %d",vert);
    NSLog(@"%d" ,row);

Ce n'est pas hyper propre pour autant. Je pense peut-être implémenter une solution avec des tags sur chacune de mes cellules, et itérer parmi les tags. Je reviendrai vers vous dans ce cas.

[LeBzul]

C'est cool si tu as trouvé une solution à ton problème.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
	if([nom_image isEqual:@"beige"])
			beige++;
		else if([nom_image isEqual:@"vert"])
			vert++;
		else if([nom_image isEqual:@"jaune"])
			jaune++;
		else if([nom_image isEqual:@"bleu"])
			bleu++;
		else if([nom_image isEqual:@"rouge"])
			rouge++;

Pour améliorer cette partie, tu devrais utiliser un NSMutableDictionary en utilisant comme clef le nom_image et en valeur le nombre d'itération. En gros, si la clef n'existe pas tu l'ajoutes au dictionary avec la valeur 1, sinon tu fais +1 a la valeur déjà présente.
C'est simple à mettre en place et le jour ou tu voudras ajouter d'autres couleurs, ca fonctionnera sans rajouter de code et ca feras disparaître les quantités de if/else à la chaines.

[foetus]

Encore plus simple mettre un tag (propriété de UIView) croissant sur les cellules:
cellule 1 -> tag X + 0
cellule 2 -> tag X + 1
cellule 3 -> tag X + 2
...
cellule n -> tag X + (n - 1)

Et avec un petit test (0 >= tag - X > n), et un simple tableau short all_counters[n], tu peux faire all_counters[tag - X]++.

Peut-être un petit enum { n = 50; };