Xcode 4.2 с раскадровкой: программа получила сигнал: «EXC_BAD_ACCESS»

Я реализую простое приложение с раскадровкой, в котором есть кнопка из «обычного» представления в табличное представление:

введите здесь описание изображенияБольшая картина здесь.

Сначала я поместил представление навигации в раскадровку и поместил кнопку в корневой вид (представление «Добро пожаловать!»). Затем я поместил табличный вид в раскадровку и связал обе страницы с помощью кнопки «Местоположения» в раскадровке (до тех пор я не пишите любой код).До этого все работало.Но потом я создал LocationsViewController для TableView,потому что хотел заполнить ячейку табличного представления данными.

LocationsViewController.m:

#import "LocationsViewController.h"



@interface LocationsViewController ()
{
    NSArray* locations; //list of cities in our table
}
@end 






@implementation LocationsViewController


- (id)initWithStyle:(UITableViewStyle)style
{
    self = [super initWithStyle:style];
    if (self) {
        // Custom initialization
    }
    return self;
}

- (void)didReceiveMemoryWarning
{
    // Releases the view if it doesn't have a superview.
    [super didReceiveMemoryWarning];

    // Release any cached data, images, etc that aren't in use.
}

#pragma mark - View lifecycle

- (void)viewDidLoad
{
    [super viewDidLoad];

    // Uncomment the following line to preserve selection between presentations.
    // self.clearsSelectionOnViewWillAppear = NO;

    // Uncomment the following line to display an Edit button in the navigation bar for this view controller.
    // self.navigationItem.rightBarButtonItem = self.editButtonItem;

    locations = [NSArray arrayWithObjects:@"New Delhi", @"Durban", @"Islamabad", @"Johannesburg", @"Kathmandu", @"Dhaka", @"Paris", @"Rome", @"Colorado Springs", @"Rio de Janeiro", @"Beijing", @"Canberra", @"Malaga", @"Ottawa", @"Santiago de Chile", nil];

}

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

- (void)viewWillAppear:(BOOL)animated
{
    [super viewWillAppear:animated];
}

- (void)viewDidAppear:(BOOL)animated
{
    [super viewDidAppear:animated];
}

- (void)viewWillDisappear:(BOOL)animated
{
    [super viewWillDisappear:animated];
}

- (void)viewDidDisappear:(BOOL)animated
{
    [super viewDidDisappear:animated];
}

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

#pragma mark - Table view data source

- (NSInteger)numberOfSectionsInTableView:(UITableView *)tableView
{
#warning Potentially incomplete method implementation.
    // Return the number of sections.
    return 1;
}

- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section
{
#warning Incomplete method implementation.
    // Return the number of rows in the section.
    return [locations count];
}

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath
{
    static NSString *CellIdentifier = @"Cell";

    UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier];
    if (cell == nil) {
        cell = [[UITableViewCell alloc] initWithStyle:
                UITableViewCellStyleDefault reuseIdentifier:CellIdentifier];
    }

    // Configure the cell...

    [cell.textLabel setText:[locations objectAtIndex:indexPath.row]];
    return cell;
}

/*
 // Override to support conditional editing of the table view.
 - (BOOL)tableView:(UITableView *)tableView canEditRowAtIndexPath:(NSIndexPath *)indexPath
 {
 // Return NO if you do not want the specified item to be editable.
 return YES;
 }
 */

/*
 // Override to support editing the table view.
 - (void)tableView:(UITableView *)tableView commitEditingStyle:(UITableViewCellEditingStyle)editingStyle forRowAtIndexPath:(NSIndexPath *)indexPath
 {
 if (editingStyle == UITableViewCellEditingStyleDelete) {
 // Delete the row from the data source
 [tableView deleteRowsAtIndexPaths:[NSArray arrayWithObject:indexPath] withRowAnimation:UITableViewRowAnimationFade];
 }   
 else if (editingStyle == UITableViewCellEditingStyleInsert) {
 // Create a new instance of the appropriate class, insert it into the array, and add a new row to the table view
 }   
 }
 */

/*
 // Override to support rearranging the table view.
 - (void)tableView:(UITableView *)tableView moveRowAtIndexPath:(NSIndexPath *)fromIndexPath toIndexPath:(NSIndexPath *)toIndexPath
 {
 }
 */

/*
 // Override to support conditional rearranging of the table view.
 - (BOOL)tableView:(UITableView *)tableView canMoveRowAtIndexPath:(NSIndexPath *)indexPath
 {
 // Return NO if you do not want the item to be re-orderable.
 return YES;
 }
 */

#pragma mark - Table view delegate

- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath
{
    // Navigation logic may go here. Create and push another view controller.
    /*
     <#DetailViewController#> *detailViewController = [[<#DetailViewController#> alloc] initWithNibName:@"<#Nib name#>" bundle:nil];
     // ...
     // Pass the selected object to the new view controller.
     [self.navigationController pushViewController:detailViewController animated:YES];
     [detailViewController release];
     */
}

@end

LocationsViewController.h:

#import <UIKit/UIKit.h>

@interface LocationsViewController : UITableViewController

@end

Поскольку я объявил этот класс как CustomClass для TableView, нажатие кнопки больше не работает. При запуске приложения на симуляторе iPhone и нажатии кнопки приложение зависает, и я получаю сообщение: Program received signal: “EXC_BAD_ACCESS”.

Я читал, что проблема заключается в ошибке управления памятью в коде. Я включил Zombie-Thing и получил сообщение в точке return [locations count]; с консольным сообщением [__NSArrayI count]: message sent to deallocated instance 0x6869460. Значит, он где-то освобождается? Как решить проблему? Любая помощь приветствуется!


person Kirinriki    schedule 05.06.2012    source источник


Ответы (2)


Одна из возможностей - использовать свойство

in .h

@property (copy, nonatomic) NSArray *locations;

in .m

@synthesize locations;
- (void)viewDidLoad
{
    [super viewDidLoad];
    self.locations = [NSArray arrayWithObjects:@"New Delhi", @"Durban", @"Islamabad", @"Johannesburg", @"Kathmandu", @"Dhaka", @"Paris", @"Rome", @"Colorado Springs", @"Rio de Janeiro", @"Beijing", @"Canberra", @"Malaga", @"Ottawa", @"Santiago de Chile", nil];

}
- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section
{
    return [self.locations count];
}
person Pfitz    schedule 05.06.2012
comment
ах здорово! Оно работает! Спасибо. (Я понимаю, мне еще нужно больше узнать о свойствах) - person Kirinriki; 06.06.2012

Первый:

locations = [NSArray arrayWithObjects:@"New Delhi", @"Durban", @"Islamabad", @"Johannesburg", @"Kathmandu", @"Dhaka", @"Paris", @"Rome", @"Colorado Springs", @"Rio de Janeiro", @"Beijing", @"Canberra", @"Malaga", @"Ottawa", @"Santiago de Chile", nil];

[NSArray arrayWithObjects...] возвращает автоматически выпущенный объект, и поскольку вы устанавливаете его непосредственно в свою переменную экземпляра, вам необходимо сохранить его:

locations = [[NSArray arrayWithObjects:@"New Delhi", @"Durban", @"Islamabad", @"Johannesburg", @"Kathmandu", @"Dhaka", @"Paris", @"Rome", @"Colorado Springs", @"Rio de Janeiro", @"Beijing", @"Canberra", @"Malaga", @"Ottawa", @"Santiago de Chile", nil] retain];

Примечание. Рекомендуется использовать свойства вместо переменных экземпляра.

Второй:

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

В соответствии с правилами управления памятью alloc init возвращает сохраненный объект, так что вы просочитесь сюда. Измените его на:

cell = [[[UITableViewCell alloc] initWithStyle:
                UITableViewCellStyleDefault reuseIdentifier:CellIdentifier] autorelease];
person graver    schedule 05.06.2012
comment
здорово, тоже работает! Я до сих пор не так много знаю о свойствах, возможно, поэтому я не использовал его до сих пор. Изучу свойства больше, спасибо. - person Kirinriki; 06.06.2012
comment
Да, вы должны, даже рекомендуется использовать их вместо переменных экземпляра... - person graver; 06.06.2012