CellForRowAtIndex leaks

Fortrackz · 13. Mai 2016, 15:56

Hey Leute,

was ist denn an diesem Code falsch? Wenn ich den mit Instruments tracke bekomme ich Leaks.

Quellcode

- (UICollectionViewCell *)collectionView:(UICollectionView *)collectionView cellForItemAtIndexPath:(NSIndexPath *)indexPath
{
NewsCollectionViewCell *cell = [collectionView dequeueReusableCellWithReuseIdentifier:reuseIdentifier forIndexPath:indexPath];
if(cell == nil)
cell = [[NSBundle mainBundle] loadNibNamed:@"NewsCollectionViewCell" owner:self options:0][0];
cell.mainLabel.text = [news[indexPath.row] valueForKey:@"pagetitle"];
cell.subtitleLabel.text = ([((NSString *)[news[indexPath.row] valueForKey:@"introtext"]) isEqualToString:@""]) ? [StringHelper stringByStrippingHTML:((NSString *)[news[indexPath.row] valueForKey:@"content"])] : ((NSString *)[news[indexPath.row] valueForKey:@"introtext"]);
cell.dateLabel.text = [NSString stringWithFormat:@"%@",[news[indexPath.row] valueForKey:@"publishedon"]];
cell.backgroundColor = [[UIColor whiteColor] colorWithAlphaComponent:0.9];
return cell;
}

Alles anzeigen

nussratte · 13. Mai 2016, 16:02

ist 5 und 6 nicht überflüssig?
du registrierst ja die Cell und dequeue gibt dir immer eine entsprechende Cell zurück

macmoonshine · 13. Mai 2016, 19:24

Wie groß ist das Loch und wird es bei jedem Aufruf größer?

Fortrackz · 17. Mai 2016, 07:58

[Blockierte Grafik: http://abload.de/img/bildschirmfoto2016-05ajudn.png]

Werde daraus jetzt nicht schlau. Aber es ist MonDienstag morgen...

sauerzapf · 18. Mai 2016, 10:29

Was ist denn Dein reuseIdentifier und hat die neue Zelle den selben?

Fortrackz · 19. Mai 2016, 10:04

Jawohl hat den selben. "NewsCollectionViewCell"

nussratte · 19. Mai 2016, 11:31

nussratte schrieb:

ist 5 und 6 nicht überflüssig?
du registrierst ja die Cell und dequeue gibt dir immer eine entsprechende Cell zurück

a) das sollte wie gesagt auch weg können, du bekommst da immer eine korrekte Cell zurück, in das If dürfte er niemals reinspringen
b) warum benutzt du valueForKey? und nicht objectForKey?
c) nur mal als Tipp: solltest du deinen Code etwas aufräumen, schon alleine deine News als Variable rauszuziehen macht das ganze um einiges lesbarer
wenn du

Quellcode

[news[indexPath.row]

in

Quellcode

NSDictionary *oneNews = [news[indexPath.row];

rausziehen würdest, wären alle nachfolgenden Zeilen verständlicher
zu mal du dann auch noch einen Schritt weitergehen kannst, wenn du denn dein (NSString *) brauchst, machst du das auch davor, dann machst du das auch nur einmal

d) was du "hinter" ? und : dem label zuweisen möchtest, würde ich wahrscheinlich auch in variablen schieben, dann wird deine Zuweisung nochmal deutlich übersichtlicher und im debugger kannst du dann auch einfacher sehen was da drin ist