TableView IndexPath Speicherproblem

Lude · 20. November 2009, 09:46

Hallo, ich habe folgendes Problem.
Ich möchte eine TableView zum abhacken erstellen. Das klappt auch alles sehr gut. Nun habe ich aber ein Problem mit folgender Methode:

Quellcode

-(void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath*) newIndexPath{
if (![[tableView cellForRowAtIndexPath:newIndexPath] accessoryType] == UITableViewCellAccessoryCheckmark){
[[tableView cellForRowAtIndexPath:newIndexPath] setAccessoryType:UITableViewCellAccessoryCheckmark];
}

Diese Methode checkt nicht nur, die zelle am indexPath sondern auch andere einträge. Scoll ich in meinem Table weiter nach unten sind auch dort zellen gecheckt.
Es hat den anschein als ob 10zellen in den speicher geladen werden. check ich nun die erse zeile ist 1,11,21... gecheckt. check ich die 2 zelle sind 2,12,22 geckeckt usw...
Ich denke ich mach irgendetwas mit der Speicherverwaltung falsch.
Kann mir jemand weiterhelfen?

Pascal · 20. November 2009, 10:08

Die Tabelle kann 10 Zeilen aufs mal darstellen (bei Default-Höhe). Da die Tabellenzellen recycled werden (reusable...) wird, wenn du nach unten scrollst, die oben rausgescrollte Zelle für die nächste von unten verwendet, und wenn diese gecheckt war ist nun diese auch gecheckt, darum dein 10er Interval.

Lösung: in der tableCellFor... Delegate Methode den accessory-view immer entsprechend setzen.

Lude · 20. November 2009, 10:26

Original von Pascal
Die Tabelle kann 10 Zeilen aufs mal darstellen (bei Default-Höhe). Da die Tabellenzellen recycled werden (reusable...) wird, wenn du nach unten scrollst, die oben rausgescrollte Zelle für die nächste von unten verwendet, und wenn diese gecheckt war ist nun diese auch gecheckt, darum dein 10er Interval.

Lösung: in der tableCellFor... Delegate Methode den accessory-view immer entsprechend setzen.

so was in der Art hatte ich mir gedacht, nur leider kann ich mit deinem Lösungsansatz nichts anfangen, sorry bin echt noch totaler anfänger.
Wärst du so nett und könntest die Lösung noch etwas ausführen.

Pascal · 20. November 2009, 10:51

Klar.

Du hast ja in deiner Tableview-Delegate die Methode cellForRowAtIndexPath: (o.ä., bin grad am iPhone unterwegs). Wie sieht die aus? Dort musst du den Accessory-View setzen, entweder als checkmark oder eben nicht.

Lude · 20. November 2009, 10:57

das ist ja auch das komische, hier tritt der gleiche Effekt auf, die sieht so aus:

Quellcode

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {
NSInteger row = [indexPath row];
NSInteger section = [indexPath section];
static NSNumberFormatter *numberFormatter = nil;
if (numberFormatter == nil) {
numberFormatter = [[NSNumberFormatter alloc] init];
[numberFormatter setNumberStyle:NSNumberFormatterNoStyle];
}
static NSString *CellIdentifier = @"Cell";
UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier];
if (cell == nil) {
cell = [[[UITableViewCell alloc] initWithStyle:UITableViewCellStyleSubtitle reuseIdentifier:CellIdentifier] autorelease];
UITableViewCellStyleSubtitle;
}
// Get the event corresponding to the current index path and configure the table view cell.
Nahrung *nahrung = (Nahrung *)[[gruppenArray objectAtIndex:section]objectAtIndex:row];
cell.textLabel.adjustsFontSizeToFitWidth;
cell.textLabel.text = [nahrung name];
NSNumberFormatter *formatter = [[NSNumberFormatter new] autorelease];
cell.detailTextLabel.text =[NSString stringWithFormat:@"Fettpunkte: %@ ",[formatter stringFromNumber:nahrung.fettpunkte ]];
if (eventController) {
for (int i = 0; i < [eventController.nahungSelectedArray count]; i++){
Nahrung *nahrungSel = (Nahrung *)[eventController.nahungSelectedArray objectAtIndex:i];
if ([nahrungSel.name isEqualToString:nahrung.name]){
cell.accessoryType = UITableViewCellAccessoryCheckmark;
}
}
}
return cell;
}

Alles anzeigen

obwohl die if-abfrage (f ([nahrungSel.name isEqualToString:nahrung.name]) nicht zutrifft sind einträge gehackt.
darauß schließe ich, dass es eine delegate oder so was für den Accessory geben muss der AccessoryCheckmark für einen bestimmten index path gespeichter hat.

j-ckruse · 20. November 2009, 11:07

Hi,

ich bin gerade nicht an meinem Mac, aber teste mal das verhalten, wenn du ein "else" einbaust und da drin einen anderen accessoryType festlegst.

Wie gesagt, ich kann es gerade selber leider nicht testen.

Lude · 20. November 2009, 11:15

Original von j-ckruse
Hi,

ich bin gerade nicht an meinem Mac, aber teste mal das verhalten, wenn du ein "else" einbaust und da drin einen anderen accessoryType festlegst.

Wie gesagt, ich kann es gerade selber leider nicht testen.

ich weiß zwar nicht warum, aber es geht! super!!!! 1000 Dank!

Pascal · 20. November 2009, 11:35

Original von Lude
ich weiß zwar nicht warum, aber es geht! super!!!! 1000 Dank!

Es geht aus dem Grund welchen ich dir versucht habe zu erklären. Jede 10. Zelle hat den accessory Type gesetzt weil die Zelle recycelt wird und du die Chechbox vorhin nur gesetzt, aber nicht entfernt hast wenn nötig. Das ELSE tut das nun.

Edit:
Du solltest dein Modell nochmal überdenken, diese Schleife wo du checkst ob die Checkmark gesetzt werden soll ist Ressourcenfressend. Eine property ist bestimmt sauberer.

Edit2:
Wieso machst du einen numberFormatter welchen du nicht benutzst? Das adjustsFontsize tut ebenfalls nichts. Und wenn du deine Loop behalten willst, nimm doch wenigstens Fast-Enumeration.

Lude · 20. November 2009, 18:27

Original von Pascal

Original von Lude
ich weiß zwar nicht warum, aber es geht! super!!!! 1000 Dank!

Es geht aus dem Grund welchen ich dir versucht habe zu erklären. Jede 10. Zelle hat den accessory Type gesetzt weil die Zelle recycelt wird und du die Chechbox vorhin nur gesetzt, aber nicht entfernt hast wenn nötig. Das ELSE tut das nun.

Edit:
Du solltest dein Modell nochmal überdenken, diese Schleife wo du checkst ob die Checkmark gesetzt werden soll ist Ressourcenfressend. Eine property ist bestimmt sauberer.

Edit2:
Wieso machst du einen numberFormatter welchen du nicht benutzst? Das adjustsFontsize tut ebenfalls nichts. Und wenn du deine Loop behalten willst, nimm doch wenigstens Fast-Enumeration.

also ich hab es mal versucht deinen Vorschlag umzusetzen. Bin jedoch kläglich gescheitert.
Meine Idee war so, ich erzeuge einen neuen datentyp ChecktNahrung der von Nahrung erbt.

Quellcode

#import "Nahrung.h"
@interface ChecktNahrung : Nahrung {
Boolean checkt;
}
@property Boolean checkt;
@end

aber irgendwas mache ich hier falsch. Möchte ich checkt setzen:

nahrung = [nahrungAr objectAtIndex:i];
nahrung.checkt = FALSE[/PHP]

Quellcode

ChecktNahrung *nahrung = [[ChecktNahrung alloc] init];
nahrung = [nahrungAr objectAtIndex:i];
nahrung.checkt = FALSE

stürtzt das Programm mit folgender Fehlermeldung ab:
2009-11-20 18:24:59.206 Locations[18232:207] Failed to call designated initializer on NSManagedObject class 'ChecktNahrung'
2009-11-20 18:25:00.455 Locations[18232:207] *** -[NSManagedObject setCheckt:]: unrecognized selector sent to instance 0x4b58610
2009-11-20 18:25:00.456 Locations[18232:207] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '*** -[NSManagedObject setCheckt:]: unrecognized selector sent to instance 0x4b58610'

Pascal · 20. November 2009, 18:59

Ich lege dir nahe, einführende Literatur zu lesen. Du wirst sehen, programmieren in Obj-C wird dir danach 1000x mehr Freude bereiten:
• Der Datentyp heisst "BOOL", nicht "Boolean"
• Es heisst "NO", nicht "FALSE"
• Du hast wahrscheinlich das synthesize vergessen
Und das ist jetzt erst der Kommentar zu deinem Header-Code für "ChecktNahrung".

Also: Du sollst ja keine Subklasse erstellen, sondern lediglich deiner Nahrung (welche ja sicher Properties wie "name" hat) eine weitere property "checked" hinzufügen. Das machst du so:

Nahrung.h

Quellcode

...
@interface Nahrung : DeineSuperklasse_wahrscheinlichNSManagedObject_lautFehlermeldung
{
...
BOOL checked;
...
}
...
@property (nonatomic, assign) BOOL checked;
...
@end

Alles anzeigen

Nahrung.m

Quellcode

...
@implementation Nahrung
// deinde anderen Properties
@synthesize checked;
...
// alle deine Methoden
...
@end

Soweit so gut. Wenn nun jemand eine Tabellenzelle antappt, dann machst du das wohl in der tableView:didSelectRowAtIndexPath:

Quellcode

- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath
{
// hier bitte noch error checking
Nahrung *selectedNahrung = [nahrungsArray objectAtIndex:indexPath.row]; // angenommen du hast einen 1-dimensionalen array
selectedNahrung.checked = !selectedNahrung.checked;
}

Wenn du nun nur die ausgewählte Nahrung willst, dann gehst du einfach deinen nahrungsArray durch und steckst jede Nahrung, welche checked == YES hat, in einen neuen mutable Array und gibst den zurück.

Viel Spass

Lude · 23. November 2009, 09:10

puuh, hab das jetzt alles so gemacht.

Bekomme aber sobald ich auf die cheked propertie zugreife immer noch diese Fehlermeldung:
2009-11-23 09:03:24.588 Locations[2395:207] *** -[NSManagedObject checked]: unrecognized selector sent to instance 0x4d5f310
2009-11-23 09:03:24.589 Locations[2395:207] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '*** -[NSManagedObject checked]: unrecognized selector sent to instance 0x4d5f310'

Quellcode

@interface Nahrung : NSManagedObject
{
BOOL checked;
}
@property (nonatomic, retain) NSString * gruppe;
@property (nonatomic, retain) NSNumber * fettpunkte;
@property (nonatomic, retain) NSString * name;
@property (nonatomic, retain) NSString * portionierung;
@property (nonatomic, retain) NSSet * selectierteNahrungen;
@property (nonatomic, assign) BOOL checked;
@end

Alles anzeigen

Quellcode

#import "Nahrung.h"
#import "SelectierteNahrung.h"
@implementation Nahrung
@dynamic gruppe;
@dynamic fettpunkte;
@dynamic name;
@dynamic portionierung;
@dynamic selectierteNahrungen;
@synthesize checked;
@end

Alles anzeigen

Quellcode

for (int i = 0; i < [nahrungAr count]; i++) {
Nahrung *nahrung = [nahrungAr objectAtIndex:i];
nahrung.checked = NO;

Aber echt mal DANKE für deine geduld und mühen.

Pascal · 23. November 2009, 11:33

Anscheinend hast du keine Nahrungs-Objekte in deinem Array "nahrungAr". Mach mal folgendes, gleich vor dieser for-Schleife, und zeig was da kommt:

Quellcode

NSLog(@"Object 0: %@", [nahrungAr objectAtIndex:0]);

Lude · 23. November 2009, 12:01

doch da sollten laut debugger schon welche drin sein.

2009-11-23 12:00:07.466 Locations[2889:207] Object 0: <NSManagedObject: 0x4d15820> (entity: Nahrung; id: 0x4d12870 <x-coredata://4CE83F68-FB10-4AA4-84DD-14C80838243C/Nahrung/p1> ; data: <fault>)

Pascal · 23. November 2009, 12:07

Das sind aber Objekte vom Typ "NSManagedObject", nicht von deiner "Nahrung"sklasse drin.

Lude · 24. November 2009, 08:42

Original von Pascal
Das sind aber Objekte vom Typ "NSManagedObject", nicht von deiner "Nahrung"sklasse drin.

aber hier:

Nahrung *nahrung = [nahrungAr objectAtIndex:i];

erstell ich doch dann welche Objekte vom Typ Nahrung, oder nicht?
und danach wird ja der Fehler geworfen.

Pascal · 24. November 2009, 08:48

Nein, du erstellst dort ja nichts, du holst nur ein Objekt aus dem Array und "behauptest", es wären Nahrungs-Objekte.