Где / Если освободить свойства в UITableViewCell - PullRequest
0 голосов
/ 28 августа 2011

РЕДАКТИРОВАТЬ: Хорошо, поэтому я выяснил, как исправить мою первоначальную проблему, но я не уверен, что это лучший способ.

Мой новый вопрос, скажем, у меня есть подкласс UITableViewCell со следующим объявлением свойства в заголовке:

@property (nonatomic, retain) IBOutlet UILabel *levelLabel;

Это связано с IB.Можно ли не выпускать это в dealloc и вообще не выпускать?Это единственный способ понять, как заставить его работать, не выдавая ошибку exc_bad_access.Раньше он вызывал dealloc, когда ячейка tableview исчезла с экрана, но тогда она все еще была нужна.куда мне выпустить материал, или он позаботится об этом для меня?

Оригинальное название: Утечка памяти в UITableView и exc_bad_access Хорошо, я запутался.Я следил вместе с этим учебным пособием онлайн, делая пользовательские UITableViewCells.Я сделал один, и я сделал все, как сказал мне учебник.Мой подкласс UITableViewCell содержит 3 UILabels и 3 UIButton, и все они определены как свойства и связаны в IB.Мне нужно, чтобы они были доступны для класса, потому что мне нужно знать, когда нажимаются кнопки, и иметь возможность изменять текст.Когда я запускаю приложение, я начинаю прокручивать, и через несколько секунд оно вылетает, с exc_bad_access в main (нет вывода в консоль).Но когда я запускаю приложение на инструментах с NSZombieEnabled, оно вообще не падает, а работает нормально.Однако, поскольку инструменты показывают вам распределение, я вижу, что они очень быстро растут, особенно когда я прокручиваю.Я не знаю, все ли это выделения или они освобождаются, но все же это кажется слишком быстрым.

Вот PointCoordinatesCell.h (моя пользовательская ячейка):

#import <UIKit/UIKit.h>

@interface PointCoordinatesCell : UITableViewCell


@property (nonatomic, retain) IBOutlet UILabel *levelLabelLabel;
@property (nonatomic, retain) IBOutlet UILabel *levelLabel;
@property (nonatomic, retain) IBOutlet UILabel *levelDescriptionLabel;
@property (nonatomic, retain) IBOutlet UIButton *beginningButton;
@property (nonatomic, retain) IBOutlet UIButton *developingButton;
@property (nonatomic, retain) IBOutlet UIButton *secureButton;

@end

PointCoordinatesCell.m:

#import "PointCoordinatesCell.h"

@implementation PointCoordinatesCell
@synthesize levelLabel, levelLabelLabel, levelDescriptionLabel, beginningButton, developingButton, secureButton;

- (void)dealloc{
    [super dealloc];
    [levelLabel release];
    [levelLabelLabel release];
    [levelDescriptionLabel release];
    [beginningButton release];
    [developingButton release];
    [secureButton release];
}

- (id)initWithStyle:(UITableViewCellStyle)style reuseIdentifier:(NSString *)reuseIdentifier
{
    self = [super initWithStyle:style reuseIdentifier:reuseIdentifier];
    if (self) {
        // Initialization code
    }
    return self;
}

- (void)setSelected:(BOOL)selected animated:(BOOL)animated
{
    [super setSelected:selected animated:animated];

    // Configure the view for the selected state
}

@end

В RootViewController.h нет ничего, кроме классадекларация и стандартный импорт.Переменные или методы не определены.Это подклассы UITableViewController.

RootViewController.m:

#import "RootViewController.h"
#import "StatesAppDelegate.h"
#import "PointCoordinatesCell.h"

@implementation RootViewController

- (void)viewDidLoad {
    [super viewDidLoad];


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

- (void)didReceiveMemoryWarning {
    [super didReceiveMemoryWarning]; // Releases the view if it doesn't have a superview
    // Release anything that's not essential, such as cached data
}

#pragma mark Table view methods

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


// Customize the number of rows in the table view.
- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section {
    return 50;
}


- (CGFloat)tableView:(UITableView *)tableView heightForRowAtIndexPath:(NSIndexPath *)indexPath;
{
    return 293;
}

// Customize the appearance of table view cells.
- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {

    static NSString *CellIdentifier = @"PointCoordinatesCell";

    PointCoordinatesCell *cell = (PointCoordinatesCell *) [tableView dequeueReusableCellWithIdentifier:CellIdentifier];
    if (cell == nil) {

        NSArray *topLevelObjects = [[NSBundle mainBundle] loadNibNamed:@"PointCoordinatesCell" owner:self options:nil];

        for (id currentObject in topLevelObjects){
            if ([currentObject isKindOfClass:[UITableViewCell class]]){
                cell =  (PointCoordinatesCell *) currentObject;
                break;
            }
        }
    }

    //cell.capitalLabel.text = [capitals objectAtIndex:indexPath.row];
    //cell.stateLabel.text = [states objectAtIndex:indexPath.row];

    return cell;
}


- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath {
    // Navigation logic may go here. Create and push another view controller.
    // AnotherViewController *anotherViewController = [[AnotherViewController alloc] initWithNibName:@"AnotherView" bundle:nil];
    // [self.navigationController pushViewController:anotherViewController];
    // [anotherViewController release];
}



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


@end

Ответы [ 2 ]

0 голосов
/ 28 августа 2011

Кажется, вы выполняете сложное приведение в методе cellForRowAtIndexPath.Я не думаю, что это необходимо.Мне не кажется логичным проверять объект класса UITableViewCell и затем приводить его в пользовательскую ячейку.Ячейка в вашем перо уже должна быть пользовательской ячейкой.

В образце Apple загрузка ячейки намного более прямолинейна.Вы связываете свою пользовательскую ячейку с IBOutlet в вашем контроллере представления и затем делаете это:

CustomCell *cell = (CustomCell *) [aTableView dequeueReusableCellWithIdentifier:CellIdentifier];
if (cell == nil) {
    [[NSBundle mainBundle] loadNibNamed:@"CustomCell" owner:self options:nil];
    cell = customTableViewCell;
    self.customTableViewCell = nil;
    // etc.
}
0 голосов
/ 28 августа 2011

Нет, не стоит не выпускать этикетку.Вы объявили свойство с указанием retain.Это означает, что вам придется выпустить его, по крайней мере, в dealloc (безопасный способ сделать это: self. levelLabel = nil;).

Как вы заметили, потребление памяти увеличится при прокрутке, если вы не отпуститеобъекты (утечки памяти!).

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

...