iOS UITableViewCell and UIButton issue

[final1cut]

Hi
I have a custom UITableViewCell and in there except from other elements i want to draw two buttons which once either pressed the one animates a transformation on the button and the other changes picture(actual actions dont matter that much).What happens is that except for the cell in which the button was pressed i see the effect in other cells too.
Also if i use an animation i see the transformed button drawn behind the normal one..one on top of the other
to be more precise heres a sample of my code:

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

static NSString *CellIdentifier = @"Cell";
	AnswerView * cell = (AnswerView *)[tableView dequeueReusableCellWithIdentifier:CellIdentifier];
	UIButton * thumbsUp;
	if(cell == nil)
	{
		cell = [[[AnswerView alloc] initWithFrame:CGRectZero reuseIdentifier:CellIdentifier] autorelease];
		button= [[[UIButton  alloc] initWithFrame:CGRectMake(x, y, w, 
															   h)]autorelease];
		button.tag = 2;
		[button addTarget:self action:@selector(clicked:) forControlEvents:UIControlEventTouchUpInside];
	}
	else
		button = (UIButton *)[cell.contentView viewWithTag:2];

if(/* some logic involving indexPath.row */)
 [button setFrame: ] 
else
[button setFrame:

...
[cell.contentView addSubView:button];
return cell;
}

Is this a completely the wrong way to go ?

 

[johnnyjibbs]

Since you have subclassed the cell, why don't you stick the buttons and all the cell layout into your AnswerView class? That way, you'll just be adding and removing AnswerView cell objects rather than keeping the buttons and cells seperately.

At the moment, I think you are running into problems when the cells are reused and you may also have a memory leak, which is why the buttons are building up on top of each other.

 

[final1cut]

thnks for the reply...
i noticed a pattern in how the button is wrongly drawn in other cells and its every other 2 cells.Does this has to do with how the cells are reused?

 

[PhoneyDeveloper]

You need to addSubview the button inside the block of code where you create the cell and button. Your code is adding the subview every time, which isn't correct when reusing the cell.