Сбой приложения — проблема с основными данными или управлением памятью?

#iphone #objective-c #memory #core-data

#iPhone #objective-c #память #ядро-данные

Вопрос:

Сообщение консоли:

Пришлось использовать изображение, потому что оно неправильно форматировалось в сообщении в виде текста.

полученное предупреждение о уровне памяти 2 появилось до сбоя приложения.

ошибка появляется в этой строке — cell.textLabel.text = tempRoutine.name;

ссылка на полноразмерное изображение (http://www.box.net/shared/static/7igj3r4trh.png )

введите описание изображения здесь

ViewController:

 @implementation RoutineTableViewController

@synthesize tableView;
@synthesize eventsArray;
@synthesize entered;
@synthesize managedObjectContext;

#pragma mark - View lifecycle

- (void)viewDidLoad
{
    if (managedObjectContext == nil) 
    { 
        managedObjectContext = [(CurlAppDelegate *)[[UIApplication sharedApplication] delegate] managedObjectContext]; 
    }

    NSFetchRequest *request = [[NSFetchRequest alloc] init];
    NSEntityDescription *entity = [NSEntityDescription entityForName:@"Routine" inManagedObjectContext:managedObjectContext];
    [request setEntity:entity];

    NSError *error = nil;
    NSMutableArray *mutableFetchResults = [[managedObjectContext executeFetchRequest:request error:amp;error] mutableCopy];
    if (mutableFetchResults == nil) {
        // Handle the error.
    }
    [self setEventsArray:mutableFetchResults];
    [mutableFetchResults release];
    [request release];

    UIBarButtonItem * addButton = [[UIBarButtonItem alloc] initWithBarButtonSystemItem:UIBarButtonSystemItemAdd target:self action:@selector(showPrompt)];
    [self.navigationItem setLeftBarButtonItem:addButton];
    [addButton release];

    UIBarButtonItem *editButton = [[UIBarButtonItem alloc]initWithTitle:@"Edit" style:UIBarButtonItemStyleBordered target:self action:@selector(toggleEdit)];
    self.navigationItem.rightBarButtonItem = editButton;
    [editButton release];

    [super viewDidLoad];
}

- (void)viewDidUnload
{
    self.eventsArray = nil;
    [super viewDidUnload];
}

-(void)toggleEdit
{
    [self.tableView setEditing: !self.tableView.editing animated:YES];

    if (self.tableView.editing)
        [self.navigationItem.rightBarButtonItem setTitle:@"Done"];
    else
        [self.navigationItem.rightBarButtonItem setTitle:@"Edit"];
}

- (void)dealloc
{
    [managedObjectContext release];
    [eventsArray release];
    [entered release];
    [super dealloc];
}

- (void)didReceiveMemoryWarning
{
    [super didReceiveMemoryWarning];
}

#pragma mark -
#pragma mark Add an event

-(void)addEvent
{    
    Routine *routine = (Routine *)[NSEntityDescription insertNewObjectForEntityForName:@"Routine" inManagedObjectContext:managedObjectContext];

    routine.name=entered;

    NSError *error = nil;
    if (![managedObjectContext save:amp;error]) {
        // Handle the error.
    }
    NSLog(@"%@", error);

    [eventsArray insertObject:routine atIndex:0];

    NSIndexPath *indexPath = [NSIndexPath indexPathForRow:0 inSection:0];

    [self.tableView insertRowsAtIndexPaths:[NSArray arrayWithObject:indexPath] withRowAnimation:UITableViewRowAnimationFade];

    [self.tableView scrollToRowAtIndexPath:[NSIndexPath indexPathForRow:0 inSection:0] atScrollPosition:UITableViewScrollPositionTop animated:YES];
}

-(void)showPrompt
{
    AlertPrompt *prompt = [AlertPrompt alloc];
    prompt = [prompt initWithTitle:@"Add Workout Day" message:@"n n Please enter title for workout day" delegate:self cancelButtonTitle:@"Cancel" okButtonTitle:@"Add"];
    [prompt show];
    [prompt release];
}

- (void)alertView:(UIAlertView *)alertView willDismissWithButtonIndex:(NSInteger)buttonIndex
{


    if (buttonIndex != [alertView cancelButtonIndex])
    {
        entered = [(AlertPrompt *)alertView enteredText];

        if(eventsArray amp;amp; entered)
        {
            Routine *tempRoutine = (Routine *)[NSEntityDescription insertNewObjectForEntityForName:@"Routine" inManagedObjectContext:managedObjectContext]; 
            tempRoutine.name = entered;
           // Routine *tempRoutine = [[Routine alloc]init];
            //tempRoutine.name = entered;
            [eventsArray addObject:tempRoutine];
            [tempRoutine release];
            [tableView reloadData];
            [self addEvent];
        }
        /*
        if(eventsArray amp;amp; entered)
        {
            [eventsArray addObject:entered];
            [tableView reloadData];
            [self addEvent];
        }
        */
    }
}

#pragma mark - Table view data source

- (NSInteger)numberOfSectionsInTableView:(UITableView *)tableView
{
    return 1;
}

- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section
{
    return [eventsArray count];
}

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

    // Dequeue or create a new cell.

    UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier];

    if (cell == nil)
    {
        cell = [[[UITableViewCell alloc] initWithStyle:UITableViewCellStyleSubtitle reuseIdentifier:CellIdentifier] autorelease];
    }

    Routine *tempRoutine = (Routine *)[eventsArray objectAtIndex:indexPath.row];
    cell.textLabel.text = tempRoutine.name;

    cell.accessoryType = UITableViewCellAccessoryDisclosureIndicator;

    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;
}

-(void)tableView:(UITableView *)tableView commitEditingStyle:(UITableViewCellEditingStyle)editingStyle forRowAtIndexPath:(NSIndexPath *)indexPath
 {

     if (editingStyle == UITableViewCellEditingStyleDelete) {

         // Delete the managed object at the given index path.
         NSManagedObject *eventToDelete = [eventsArray objectAtIndex:indexPath.row];
         [managedObjectContext deleteObject:eventToDelete];

         // Update the array and table view.
         [eventsArray removeObjectAtIndex:indexPath.row];
         [tableView deleteRowsAtIndexPaths:[NSArray arrayWithObject:indexPath] withRowAnimation:YES];

         // Commit the change.
         NSError *error = nil;
         if (![managedObjectContext save:amp;error]) {
             // Handle the error.
         }
     }
 }
  

Комментарии:

1. Привет, Фейсал, ты проверяешь, какое значение получено из tempRoutine. имя?

2. Конечно, я сделаю NSLog и проверю прямо сейчас.

3. Хорошо, и посмотрите, какое значение, приходите и дайте мне знать.

4. Я установил NSLog после того, как пользователь нажал кнопку добавить, я получил 2011-04-12 02:13:14.275 Curl[133:707] tempRoutine.name is: 1977840 . И приложение все еще зависает после добавления нескольких ячеек. Также дублируются ячейки и т.д.

5. Здравствуйте, пожалуйста, также проверьте значение all objects в eventsArray. И проверьте, какое значение выдает tempRoutine для каждого index. tempRoutine. имя — это целое число или строка?

Ответ №1:

Когда вы создаете свои подпрограммы, вы создаете их с помощью insertNewObjectForEntityForName:inManagedObjectContext: . Затем вы их освобождаете. Но insertNewObjectForEntityForName:inManagedObjectContext: не возвращает объект, которым вы владеете, в соответствии с правилами управления памятью или документацией метода, поэтому вам не следует выпускать.

Комментарии:

1. хорошо, так где я могу его выпустить? и я должен удалить код выпуска?

2. @Faisal: В alertView:willDismissWithButtonIndex: вы должны удалить строку [tempRoutine release] .

3. спасибо, что исправляет сбой, я думаю, я только что добавил большинство ячеек, которые я сделал, обычно это завершается сбоем через 2. но двойное добавление все еще там. я заметил, что он добавляет одну ячейку в первую позицию и ту же ячейку в последнюю позицию. есть мысли?

4. Я думаю, что у меня получилось, у меня был дублированный код insertobject как в addevent, так и в методах оповещения dissmiss

5. Хорошо, итак, я избавился от дубликатов ячеек при добавлении новых ячеек, удалив код obejct вставки из addEvent метода. Однако, когда я перезапускаю приложение, оно дублирует все ячейки. Есть предложения?

Ответ №2:

привет,

В вашем табличном представлении отображается слишком много данных одновременно?

Я думаю, что это единственная проблема с вашим кодом. Если одновременно вызывается более 50-60 строк и вы просматриваете их, прокручивая вверх и вниз, это может вызвать предупреждение о памяти.

Лучший способ использовать отложенную загрузку, назвать количество raw, например, вы можете загружать только 20 raw за раз, а затем загружать другие строки по мере прокрутки пользователем вниз.

Вот хорошее обсуждение того же самого. Вы не используете изображения, но некоторые изображения могут привести к сбою приложения.

Надеюсь, это решит вашу проблему.

Комментарии:

1. На самом деле в таблице всего 2-3 строки данных. Изначально их 0, пока пользователь не добавит свои собственные.