Problem mit eigener UITableViewCell

And-Wolf · 10. Mai 2011, 21:17

Hallo,

ich hätte keine Pause mit Objective C machen sollen. Aber Abschlussprüfung geht nunmal vor...

Ich verzweifel gerade an folgendem:

Ich habe eine TableView indem ich die Cells selber gestalte.

Dazu möchte ich jeder Cell einen Button in der Größe der Cell (höhe -1) hinzufügen.
Auf den Button möchte ich oben Links angefangen ein Label einfügen.

Das Funtkioniert auch wunderbar.
Aber sobald ich einmal nach Unten gescrollt habe hat der mir der Unterste Objekt ganz Oben reingeschrieben und hat scheinbar versetzt alle Cells nochmal neu gezeichnet.

Desweiterne bekomme ich kein Event mehr auf die Cell.

Woran kann das liegen.

Hier noch mein Code für die eigene Cell:

Quellcode

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath
{
static NSString *CellIdentifier = @"Cell";
UITableViewCell *cell = [self.tableView dequeueReusableCellWithIdentifier:CellIdentifier];
if (cell == nil) {
cell = [[[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:CellIdentifier] autorelease];
}
// Configure the cell...
UIButton *button = [UIButton buttonWithType:UIButtonTypeCustom];
[[button layer]setCornerRadius:12.0f];
[[button layer]setMasksToBounds:YES];
button.frame = CGRectMake(cell.frame.origin.x, cell.frame.origin.y, cell.frame.size.width, [[self.localroom.DateList objectAtIndex:indexPath.row]factor] * 2 -1);
button.backgroundColor = [UIColor colorWithRed:(227.0/255.0) green:(0.0/255.0) blue:(74.0/225.0) alpha:1.0];
UILabel *label = [[UILabel alloc]initWithFrame:CGRectMake(5.0, 5.0, 300.0, 50.0)];
label.text = [[self.localroom.DateList objectAtIndex:indexPath.row] title];
label.text = [label.text stringByAppendingString:@"\n"];
label.text = [label.text stringByAppendingString:[[self.localroom.DateList objectAtIndex:indexPath.row]time]];
label.backgroundColor = [UIColor colorWithRed:(227.0/255.0) green:(0.0/255.0) blue:(74.0/225.0) alpha:1.0];
[button addSubview:label];
[label release];
[cell addSubview:button];
//Wenn ich den Button hier freigebe habe ich den später keinen Text in der Cell.
//Noch nee Lösung ausarbeiten...
//[button release];
return cell;
}

Alles anzeigen

Danke und Gruß
Andreas

longW · 10. Mai 2011, 21:23

And-Wolf schrieb:

Hallo,

ich hätte keine Pause mit Objective C machen sollen. Aber Abschlussprüfung geht nunmal vor...

Ich verzweifel gerade an folgendem:

Ich habe eine TableView indem ich die Cells selber gestalte.

Dazu möchte ich jeder Cell einen Button in der Größe der Cell (höhe -1) hinzufügen.
Auf den Button möchte ich oben Links angefangen ein Label einfügen.

Das Funtkioniert auch wunderbar.
Aber sobald ich einmal nach Unten gescrollt habe hat der mir der Unterste Objekt ganz Oben reingeschrieben und hat scheinbar versetzt alle Cells nochmal neu gezeichnet.

Desweiterne bekomme ich kein Event mehr auf die Cell.

Woran kann das liegen.

Hier noch mein Code für die eigene Cell:

Quellcode

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath

{

static NSString *CellIdentifier = @"Cell";

UITableViewCell *cell = [self.tableView dequeueReusableCellWithIdentifier:CellIdentifier];

if (cell == nil) {

cell = [[[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:CellIdentifier] autorelease];

}

// Configure the cell...

UIButton *button = [UIButton buttonWithType:UIButtonTypeCustom];

[[button layer]setCornerRadius:12.0f];

[[button layer]setMasksToBounds:YES];

button.frame = CGRectMake(cell.frame.origin.x, cell.frame.origin.y, cell.frame.size.width, [[self.localroom.DateList objectAtIndex:indexPath.row]factor] * 2 -1);

button.backgroundColor = [UIColor colorWithRed:(227.0/255.0) green:(0.0/255.0) blue:(74.0/225.0) alpha:1.0];

UILabel *label = [[UILabel alloc]initWithFrame:CGRectMake(5.0, 5.0, 300.0, 50.0)];

label.text = [[self.localroom.DateList objectAtIndex:indexPath.row] title];

label.text = [label.text stringByAppendingString:@"\n"];

label.text = [label.text stringByAppendingString:[[self.localroom.DateList objectAtIndex:indexPath.row]time]];

label.backgroundColor = [UIColor colorWithRed:(227.0/255.0) green:(0.0/255.0) blue:(74.0/225.0) alpha:1.0];

[button addSubview:label];

[label release];

[cell addSubview:button];

//Wenn ich den Button hier freigebe habe ich den später keinen Text in der Cell.

//Noch nee Lösung ausarbeiten...

//[button release];

return cell;

}

Alles anzeigen

Danke und Gruß
Andreas

Implementier mal:

Quellcode

- (CGFloat)tableView:(UITableView *)tableView heightForRowAtIndexPath:(NSIndexPath *)indexPath

And-Wolf · 10. Mai 2011, 21:27

Habe ich mit drin.
Sieht so aus:

Quellcode

- (CGFloat)tableView:(UITableView *)tableView heightForRowAtIndexPath:(NSIndexPath *)indexPath {
return [[self.localroom.DateList objectAtIndex:indexPath.row]factor] * 2;
}

Aber hat doch nichts mit meinem Problem zu tun.
Oder sehe ich das falsch?

AR.DDev · 10. Mai 2011, 21:28

Verschiebe mal das alles in
if (cell == nil)

macmoonshine · 10. Mai 2011, 21:43

AR.DDev schrieb:

Verschiebe mal das alles in
if (cell == nil)

Daran liegt es. Wenn eine Zelle wiederverwendet wird, fügst Du ihr erneut einen Button und ein Label hinzu, obwohl sie die bereits besitzt. Du solltest außerdem Tags verwenden, um dieses Views wiederzufinden. Also z. B.

Quellcode

UILabel *label ;
...
if(cell == nil) {
...
label = [[UILabel alloc] initWithFrame:...];
label = 100;
[cell.contentView addSubview:label];
[label autorelease];
}
else {
label = (UILabel *)[cell.contentView viewWithTag:100];
}
label.text = ...;

Alles anzeigen

Außerdem solltest Du alle Subviews der Zelle zu deren Content-View hinzufügen.

And-Wolf · 10. Mai 2011, 22:10

Hallo,

danke für die schnellen Antworten

Das mit den Tags verstehe ich noch nicht ganz.

Muss jedes Tag das ich irgendwie irgendwann mal erzeuge eine eindeutige ID haben?

Muss ich in dem else Zweig der nil anweisung alles nochmal bauen nur mit den schon erzeugten Tags?

Gruß
Andreas

AR.DDev · 10. Mai 2011, 22:16

Eben nicht, sonst werden der Zelle bei jedem Scrollen X Label hinzugefügt.
Was du aber im Else-Zweig setzen solltest wäre der Text. Um dann dort (und ggf. auch an anderer Stelle) an das Label zu kommen verwendest du Tags. Am Besten die Zeilennummer.
Im Prinzip was macmoonshine schon gesagt hat:

Quellcode

if (cell == nil) {
[label setTag:[indexPath row]];
} else {
label = (UILabel *)[[cell contentView] viewWithTag:[indexPath row]];
}

Ein Tag muss einen Integer-Wert haben.

And-Wolf · 10. Mai 2011, 22:19

Danke

Habe es gerade auf den 2ten Blick auch verstanden und Umgesetzt.

Läuft perfekt

Nun muss ich nurnoch rausfinden warum ich keine Celle mehr auswählen kann

Gruß
Andreas

ramo · 10. Mai 2011, 22:46

AR.DDev schrieb:

Eben nicht, sonst werden der Zelle bei jedem Scrollen X Label hinzugefügt.
Was du aber im Else-Zweig setzen solltest wäre der Text. Um dann dort (und ggf. auch an anderer Stelle) an das Label zu kommen verwendest du Tags. Am Besten die Zeilennummer.
Im Prinzip was macmoonshine schon gesagt hat:

Quellcode

if (cell == nil) {

[label setTag:[indexPath row]];

} else {

label = (UILabel *)[[cell contentView] viewWithTag:[indexPath row]];

}

Ein Tag muss einen Integer-Wert haben.

@macmoonshine hat es schon angedeutet: [label setTag:[indexPath row]+100]; denn Tag=0 haben alle Elemente, kann Probleme machen !

longW · 11. Mai 2011, 00:04

ramo schrieb:

AR.DDev schrieb:

Eben nicht, sonst werden der Zelle bei jedem Scrollen X Label hinzugefügt.
Was du aber im Else-Zweig setzen solltest wäre der Text. Um dann dort (und ggf. auch an anderer Stelle) an das Label zu kommen verwendest du Tags. Am Besten die Zeilennummer.
Im Prinzip was macmoonshine schon gesagt hat:

Quellcode

if (cell == nil) {

[label setTag:[indexPath row]];

} else {

label = (UILabel *)[[cell contentView] viewWithTag: [indexPath row]];

}

Ein Tag muss einen Integer-Wert haben.

@macmoonshine hat es schon angedeutet: [label setTag:[indexPath row]+100]; denn Tag=0 haben alle Elemente, kann Probleme machen !

Ganz verstehe ich nicht, warum ihr dem Anhängsel noch den Index zuordnet, ob mit oder ohne konstantem Wert spielt keine Rolle.

Weil eine Sammlung von Views immer mal wieder durchforstet werden muss, sind 'tags' eingeführt worden.
Ein"tag" dient zum Identifizieren innerhalb der Hierarchie, und weil eine Zelle
('cell' oder Zelle, aber nicht Celle, das ist eine Stadt und ein Kreis in Niedersachsen)
eine ebensolche Hierarchie besitzt, benutzt man das denn ebenso.
Dafür hat jede besondere Art, beispielsweise eine Text, eine bestimmte Konstante als 'tag',
die man dann ruckzuck mit einer speziellen Methode wieder finden kann:

Aus der Doku:

viewWithTag:
Returns the view whose tag property contains the specified integer value.

- (UIView *)viewWithTag: (NSInteger)tag
Parameters
tag
The tag value to search for.
Return Value
The view in the receiver’s hierarchy whose tag property matches the value in the tag parameter.

Discussion
This method searches the current view and all of its subviews for the specified view.

Man kann das natürlich so machen, vieles geht. Das läuft dann aber diametral zum Konzept der Datenquelle und der Delegation.

macmoonshine · 11. Mai 2011, 07:38

Tags werden bei TableView-Zellen meistens für zwei Anwendungsfälle eingesetzt:

Über das Tag lassen sich bestimmte Subviews über die Methode viewWithTag: wiederfinden. Hierbei muss der Tagwert konstant gewählt werden und am besten ungleich 0. Das ist mein Beispiel oben.
Mit dem Tag lässt sich die Zeilennummer in Action-Methoden für Buttons o. Ä. rekonstruieren. Dabei bekommt der Tag des Button (bzw. Controls) die Zeilennummer der Zelle (von mir aus auch plus 100 ;-). Das ist das Beispiel von ramo.

Was aber wichtig ist: Beides auf einmal geht nicht. Wenn der Tagwert variabel ist, kann ich viewWithTag: in tableView:cellForRowAtIndexPath: nicht dazu verwenden, den Button zu finden. Und wenn der Tagwert bei allen Buttons gleich ist, kann ich damit nicht die Zeilennummer ermitteln.

Glücklicherweise lassen sich aber beide Anwendungsfälle auch noch über andere Wege lösen

And-Wolf · 12. Mai 2011, 10:49

Hallo,

die Beschriftung passt nun und gerät nicht durcheinander.
Aber bei einem Scrollen passen nicht mehr die größen der cells.

Irgendwie kommt da alles durcheinander.
Die eine Cell wird über die andere gezeichnet.

Oder der Button den ich hinzufügen füllt nicht die cell aus.

Hier mein aktueller Code.
Evtl. könnt ihr mir ja nochmal helfen:

Quellcode

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath
{
static NSString *CellIdentifier = @"Cell";
UITableViewCell *cell = [self.tableView dequeueReusableCellWithIdentifier:CellIdentifier];
UITextView *label;
UIButton *button;
if (cell == nil)
{
cell = [[[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:CellIdentifier] autorelease];
button = [UIButton buttonWithType:UIButtonTypeCustom];
[[button layer]setCornerRadius:12.0f];
[[button layer]setMasksToBounds:YES];
button.frame = CGRectMake(cell.frame.origin.x +1, cell.frame.origin.y +1, cell.frame.size.width - 2, ([[self.localroom.DateList objectAtIndex:indexPath.row]factor] *2) -2);
button.backgroundColor = [UIColor colorWithRed:(227.0/255.0) green:(0.0/255.0) blue:(74.0/225.0) alpha:1.0];
label = [[UITextView alloc]initWithFrame:CGRectMake(5.0, 5.0, button.frame.size.width, button.frame.size.height)];
label.backgroundColor = [UIColor colorWithRed:(227.0/255.0) green:(0.0/255.0) blue:(74.0/225.0) alpha:1.0];
label.scrollEnabled = NO;
label.editable = NO;
label.userInteractionEnabled = YES;
label.tag = 100;
}
else
{
label = (UITextView*)[cell.contentView viewWithTag:100];
button = (UIButton*)[cell.contentView viewWithTag:200];
}
label.text = [[self.localroom.DateList objectAtIndex:indexPath.row]title];
label.text = [label.text stringByAppendingString:@"\n"];
label.text = [label.text stringByAppendingString:[[self.localroom.DateList objectAtIndex:indexPath.row]time]];
label.text = [label.text stringByReplacingOccurrencesOfString:@"oe" withString:@"ö"];
label.text = [label.text stringByReplacingOccurrencesOfString:@"ae" withString:@"ä"];
[button addSubview:label];
[cell.contentView addSubview:button];
cell.contentView.userInteractionEnabled = NO;
return cell;
}
- (CGFloat)tableView:(UITableView *)tableView heightForRowAtIndexPath:(NSIndexPath *)indexPath {
return ([[self.localroom.DateList objectAtIndex:indexPath.row]factor] *2);
}

Alles anzeigen

Danke & Gruß
Andreas

macmoonshine · 12. Mai 2011, 11:06

Quellcode

[button addSubview:label];
[cell.contentView addSubview:button];

Das muss doch in den If-Block.

Du solltest mal die String-Verarbeitung für label.text überarbeiten. Das kannst Du mit stringWithFormat: wesentlich vereinfachen.

BTW: Bist Du Dir sicher, dass niemals Soest, Moers oder Agendaende in Deiner Liste stehen?

And-Wolf · 12. Mai 2011, 11:17

Hallo,

habe ich mit drin.
Nun wird es schonmal nichtmehr durcheinander geschmissen.
Aber es wird an einigen Stellen immernoch eine Cell über die andere gezeichnet und zu klein gezeichnet.

habt ihr da noch eine Idee?

Ich denke das er vorher immer die alte View wieder übermalt hat und deswegen alles wirsch aussah?

Ja ich bin mir sicher.
Werde mir aber die Stringverarbeitung nochmal anschauen.

danke

macmoonshine · 12. Mai 2011, 11:58

And-Wolf schrieb:

Aber es wird an einigen Stellen immernoch eine Cell über die andere gezeichnet und zu klein gezeichnet.

habt ihr da noch eine Idee?

Überprüfe mal die Höhe der Zellen. Wahrscheinlich liegt es daran.