Leak caught by tools performance - iphone

Question

After running the application with (Run performance tool -> leak), the tool found a memory leak.
I run app, view appears and press button for a tableview no leaks.
tableView appears and if I don't touch anything no leaks.
If I select the first cell and leave it alone, 3 to 4 seconds later leak.
Here is my code:
didSelectRowAtIndexPath function is commented out.
(UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {
static NSString *CellIdentifier = #"Cell";
UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier];
if (cell == nil)
{
cell = [[[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefaultreuseIdentifier:CellIdentifier] autorelease];
}
// Configure the cell...
UITextField *FirstField = [[UITextField alloc] initWithFrame:CGRectMake(10, 10, 130, 25)];
FirstField.delegate = self;
FirstField.tag = indexPath.row;
[cell.contentView addSubview:FirstField];
FirstField.returnKeyType = UIReturnKeyNext;
[FirstField release];
return cell;
}

Answer 1

You don't need to create the textfield everytime..Since you are setting a tag for the textfield, you try to get the textfield with tag.
UITextField *FirstField = (UITextField *)[cell.contentView viewWithTag:indexPath.row];
if it is nil, then you create the textfield. This may not be a solution for your problem..But it is definitely optimize the memory usage.

Answer 2

Wow I can't believe I didn't see that! #Gomathi doesn't know that that is the memory leak.
Follow, even if the cell already exists you are creating a new UITextField and adding it to the cell. But that does not replace the old TextField because it's not a property it's a subview. You need to set the TextField like so :
(UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {
static NSString *CellIdentifier = #"Cell";
UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier];
if (cell == nil){
//Configure the cell
cell = [[[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefaultreuseIdentifier:CellIdentifier] autorelease];
UITextField *firstField = [[UITextField alloc] initWithFrame:CGRectMake(10, 10, 130, 25)];
firstTextField.tag = 25;// any number will do
firstField.delegate = self;
firstField.returnKeyType = UIReturnKeyNext;
[cell.contentView addSubview:firstField];
[firstField release];
}
UITextField *firstField = (UITextField *)[cell.contentView viewWithTag:25];
firstField.text = #"row specific content";
return cell;
}
+1 Gomathi you had the seed of the answer.