iOS ipad UITableView array -> crash when drog down the tableView

[kimti]

IOS 4.2

create project and select "Split View-based Application"

define the array in RootViewController.h file:

NSMutableArray *a;

init the array in viewDidLoad method

a = [NSMutableArray arrayWithObjects:@"1",@"2",@"3",@"4",@"5",nil];

implementation method:

-(UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *) indexPath{
static NSString *CellIdentifier = @"CellIdentifier";
UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier];
if(cell == nil){
cell = [[[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier] autorelease];
cell.accessoryType = UITableViewCellAccessoryNone;
}
if(indexPath.row < [a count]){
cell.textLabel.text = [a objectAtIndex:indexPath.row];
}
return cell;
}

---------------------------------------
Solution: insert follow code after init the array a:

[a retain];

thx admanimal.

 

[mheidt]

try a -1 in the objectAtIndex of "a"

 

[admanimal]

Did you retain a? The code as posted results in a potentially being invalid as soon as viewDidLoad returns.

 

[kimti]

try a -1 in the objectAtIndex of "a"

thx ,but i have follow:

if(indexPath.row < [a count]){
      cell.textLabel.text = [a objectAtIndex:indexPath.row];
}

Did you retain a? The code as posted results in a potentially being invalid as soon as viewDidLoad returns.

i not retain a ,i noly follow:

a = [NSMutableArray arrayWithObjects:@"1",@"2",@"3",@"4",@"5",nil];

 

[admanimal]

You should probably read up on memory management then. But for now, add

[a retain];

after you call arrayWithObjects and see what happens.

 

[kimti]

You should probably read up on memory management then. But for now, add

[a retain];

after you call arrayWithObjects and see what happens.

oh yes man,you are right.
much thank you .

 

[admanimal]

You probably did it the correct way in your own code if it is working, but the solution you added to the original post is incorrect. The retain message should go after you initialize the array.