Не могу найти причину сбоя моего приложения - PullRequest
Новая
       36

Не могу найти причину сбоя моего приложения

0 голосов
/ 10 сентября 2011

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

Я думаю, это может быть утечка памяти.

Вот мой код. Если кто-то видит что-то, что я пропустил, или, возможно, угадаю, по какой причине, пожалуйста, сообщите мне. Ничего не отображается в отладчике. 

@synthesize colorTextField, pickerView, pickerToolbar;

- (id)initWithNibName:(NSString *)nibNameOrNil bundle:(NSBundle *)nibBundleOrNil
{
    self = [super initWithNibName:nibNameOrNil bundle:nibBundleOrNil];
    if (self) {
        // Custom initialization
    }
    return self;
}

- (void)dealloc
{
    [colorTextField release];
    [pickerView release];
    [pickerToolbar release];
    [super dealloc];

}

- (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.
}


- (void)viewDidLoad
{
    [super viewDidLoad];

    colorsArray = [[NSMutableArray alloc] init];
    [colorsArray addObject:@"Red"];
    [colorsArray addObject:@"Orange"];
    [colorsArray addObject:@"Yellow"];
    [colorsArray addObject:@"Green"];
    [colorsArray addObject:@"Blue"];
    [colorsArray addObject:@"Indigo"];
    [colorsArray addObject:@"Violet"];
}

- (void)setColor
{    
     actionSheet = [[UIActionSheet alloc] initWithTitle:nil delegate:nil cancelButtonTitle:nil destructiveButtonTitle:nil otherButtonTitles:nil];   

     [actionSheet setActionSheetStyle:UIActionSheetStyleBlackTranslucent];

     CGRect pickerFrame = CGRectMake(0, 40, 0, 0);  
     pickerView = [[UIPickerView alloc] initWithFrame:pickerFrame];  
     pickerView.showsSelectionIndicator = YES;  
     pickerView.dataSource = self;  
     pickerView.delegate = self;  

     [actionSheet addSubview:pickerView];
     [pickerView release];

     pickerToolbar = [[UIToolbar alloc] initWithFrame:CGRectMake(0, 0, actionSheet.bounds.size.width, 44)];
    [pickerToolbar setBarStyle:UIBarStyleBlack];
    [pickerToolbar sizeToFit];

    UIBarButtonItem* doneButton = [[UIBarButtonItem alloc] initWithTitle:@"Done" style:UIBarButtonItemStyleBordered target:self action:@selector(pickerDoneClicked)];

     [pickerToolbar setItems:[NSArray arrayWithObject:doneButton] animated:NO];
     [doneButton release];

     [actionSheet addSubview:pickerToolbar];
     [pickerToolbar release];

     [actionSheet showInView:self.view];

     [actionSheet setBounds:CGRectMake(0, 0, 320, 485)];  
}

- (void)pickerDoneClicked
{    
   [actionSheet dismissWithClickedButtonIndex:0 animated:YES];
    [actionSheet release];
}

- (BOOL)textFieldShouldBeginEditing:(UITextField *)textField
{
    [self setColor];
    return NO;
}


- (NSInteger)numberOfComponentsInPickerView:(UIPickerView *)thePickerView {

     return 1;
 }

 - (NSInteger)pickerView:(UIPickerView *)pickerView numberOfRowsInComponent:(NSInteger)component
 {
     return [colorsArray count];
 }

 - (NSString *)pickerView:(UIPickerView *)thePickerView titleForRow:(NSInteger)row forComponent:(NSInteger)component {
    return [colorsArray objectAtIndex:row];
 }

 - (void)pickerView:(UIPickerView *)thePickerView didSelectRow:(NSInteger)row inComponent:(NSInteger)component {

     NSLog(@"Selected Color: %@. Index of selected color: %i", [colorsArray objectAtIndex:row], row);
    [colorTextField setText:(NSString *)[colorsArray objectAtIndex:row]];
}

- (void)viewDidUnload
{
    [self setColorTextField:nil];
    [super viewDidUnload];
}

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

 @end

Ответы [ 2 ]

2 голосов
/ 10 сентября 2011

Вы дважды отпускаете pickerView и pickerToolbar, один раз в setColor и снова в dealloc.

Хорошим шаблоном при освобождении представлений, удерживаемых в иваре, является установка ивара вноль после освобождения вида, это предотвратит двойной выпуск.

0 голосов
/ 10 сентября 2011

Я думаю, что вы выпустили некоторые из ваших вещей Mor Ethan однажды EG:

в сделке у вас есть два [релиз pickerView]; [pickerToolbar release];

которые уже выпущены например: [actionSheet addSubview: pickerView]; [pickerView release];

так что мне нечего выпускать, думает, поэтому удалите дубликаты релизов ??

...