UITableView的執行委託時

Question

的代碼崩潰低於：

@interface PreferenceViewController : UIViewController <UITableViewDelegate, UITableViewDataSource> { 
    UITableView * preference; 
    UISegmentedControl * dimension; 
    NSArray * timeslot; 
} 

@property (nonatomic, retain) IBOutlet UITableView * preference; 
@property (nonatomic, retain) IBOutlet UISegmentedControl * dimension; 
@property (nonatomic, retain) NSArray * timeslot; 

@end 

- (void)viewDidLoad 
{ 
    preference.delegate = self; 
    preference.dataSource = self; 
    timeslot = [NSArray arrayWithObjects:@"7:00-8:00", @"8:00-9:00", @"9:00-10:00", @"10:00-11:00", @"11:00-12:00", @"12:00-13:00", @"13:00-14:00", @"14:00-15:00", @"15:00-16:00", @"16:00-17:00", nil]; 
    NSLog(@"size of array is %d", [timeslot count]); 
    [super viewDidLoad]; 
    // Do any additional setup after loading the view from its nib. 
} 

- (void)viewDidUnload 
{ 
    [super viewDidUnload]; 
    // Release any retained subviews of the main view. 
    // e.g. self.myOutlet = nil; 
} 

- (BOOL)shouldAutorotateToInterfaceOrientation:(UIInterfaceOrientation)interfaceOrientation 
{ 
    // Return YES for supported orientations 
    return (interfaceOrientation == UIInterfaceOrientationPortrait); 
} 

#pragma mark - Table View delegate 
- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath 
{ 

} 

- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section 
{ 
    if (dimension.selectedSegmentIndex == 0){ 
     return [timeslot count]; 
    }else 
     return 15; 
} 

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath { 
    //NSLog(@"TESTING"); 
    static NSString *CellIdentifier = @"Cell"; 
    UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier]; 
    if (cell == nil) { 
     cell = [[[UITableViewCell alloc] initWithFrame:CGRectZero reuseIdentifier:CellIdentifier] autorelease]; 
    } 
    // this is the MAGIC line! 

    if (dimension.selectedSegmentIndex == 0) 
     cell.text = [timeslot objectAtIndex:indexPath.row]; 

    return cell; 
} 

想知道是什麼引起的崩潰？它發生在numberOfRowsInSection委託和調用時隙數..

Answer 1

timeslot = [NSArray arrayWithObjects:@"7:00-8:00", @"8:00-9:00", @"9:00-10:00", @"10:00-11:00", @"11:00-12:00", @"12:00-13:00", @"13:00-14:00", @"14:00-15:00", @"15:00-16:00", @"16:00-17:00", nil]; 

這條線應該有-retain。

timeslot = [[NSArray arrayWithObjects:@"7:00-8:00", @"8:00-9:00", @"9:00-10:00", @"10:00-11:00", @"11:00-12:00", @"12:00-13:00", @"13:00-14:00", @"14:00-15:00", @"15:00-16:00", @"16:00-17:00", nil] retain]; 

UPD -autorelease不是一個好主意。您只能自動釋放/釋放您擁有的對象。當您使用-arrayWithObjects時，您的對象由本地AutoreleasePool擁有。

這是簡單的說法。每當你創建一個對象而沒有-alloc或-copy方法它是自動釋放，這意味着它將有引用計數1，直到消息範圍的結束。技術上autorelease池業務是異步，所以它可能會崩潰或可能不會，但你應該認爲這個對象是殭屍在這一點上。所以你必須使用-retain爲了保存這個對象用於其他方法的使用。如果您明確使用-retain，則必須在完成此對象時明確使用-release。那麼爲什麼不馬上使用[[NSArray alloc] initWithObjects:...]。這裏更合適。你也可以在這裏使用保留屬性，但是當你沒有時，你不想暴露內部變量。

Answer 2

在viewDidLoad，要指定新創建（和自動釋放）的NSArray的伊娃，而不是財產，這意味着它是不保留你。因此，當它到達tableView:numberOfRowsInSection:時，該數組已被處理掉，並且您得到EXC_BAD_ACCESS或類似的崩潰。

因此，我通常將我的ivars _name而不是name命名爲匹配屬性。這確實意味着合成線必須是更復雜的@synthesize name=_name，但它可以防止意外分配給伊娃而不是屬性。