UITableView“隐藏”部分导致每次拉动刷新时分配更多内存
Posted
技术标签:
【中文标题】UITableView“隐藏”部分导致每次拉动刷新时分配更多内存【英文标题】：UITableView 'hidden' section causing more memory allocations every time on pull to refresh 【发布时间】：2014-02-21 18:40:37 【问题描述】：我在 Code Review 上发布了这篇文章，但被告知作为 Stack Overflow 问题可能会更好，因此发布了这篇文章。
我有一个 UITableView，当用户点击该部分的标题时，我正在“隐藏”该部分。我认为这是我想出的一个非常酷的实现，但事实证明，当这部分被隐藏并且用户拉动刷新时，每次拉动刷新时（刷新完成后）都会将大约 1-2MB 的内存添加到内存中。如果 UITableView 的部分没有隐藏并且它们拉动刷新，则不会分配额外的内存（正确的方式/行为）。我知道 1-2 MB 并不多，但都加起来了。我在 Profiler 中找不到任何漏洞，所以这就是我来这里的原因。
我可能忽略了某些事情，或者我的逻辑有缺陷，如果提供任何帮助，我将不胜感激。对不起，下面的冗长代码：
更新：已解决的代码：
- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section
switch (section)
case 0:
return [self friendRequests];
case 1:
return [self friends];
default:
if (self.showingBlockedUsers == YES)
return [self blockedUsers];
else
return 0;
原始问题的旧代码：
- (void)refreshing:(UIRefreshControl *)refreshControl
[refreshControl beginRefreshing];
[UIApplication sharedApplication].networkActivityIndicatorVisible = YES;
NSMutableURLRequest *request = [NSMutableURLRequest requestWithURL:[NSURL URLWithString:@"http://foobar.com/test.plist"]];
request.cachePolicy = NSURLRequestReloadIgnoringLocalAndRemoteCacheData;
request.timeoutInterval = 5.0;
[NSURLConnection sendAsynchronousRequest:request
queue:[NSOperationQueue mainQueue]
completionHandler:
^(NSURLResponse *response, NSData *data, NSError *connectionError)
if (data)
NSPropertyListFormat format;
NSMutableDictionary *dictionary = [NSPropertyListSerialization propertyListWithData:data options:NSPropertyListImmutable format:&format error:nil];
self.tableDataSource = dictionary;
[self.tableView reloadData];
[refreshControl endRefreshing];
[UIApplication sharedApplication].networkActivityIndicatorVisible = NO;
- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath
UITableViewCell *cell;
NSString *identifier = @"cellIdentifier";
UILabel *userNameLabel = [self userNameLabel];
UIView *requestsView = [self requestsView];
UIButton *approveButton = [self approveButton];
UIButton *denyButton = [self denyButton];
UIImageView *profileImageView = [self profileImageView];
if (cell == nil)
cell = [[UITableViewCell alloc]initWithStyle:UITableViewCellStyleDefault reuseIdentifier:identifier];
cell.selectionStyle = UITableViewCellSelectionStyleDefault;
[requestsView addSubview:approveButton];
[requestsView addSubview:denyButton];
[cell.contentView addSubview:requestsView];
[cell.contentView addSubview:profileImageView];
[cell.contentView addSubview:userNameLabel];
if (indexPath.section == 0)
cell.selectionStyle = UITableViewCellSelectionStyleNone;
userNameLabel.frame = CGRectMake(82, 0, 167, 68);
approveButton.tag = indexPath.row;
denyButton.tag = indexPath.row;
NSArray *keys = [self.tableDataSource objectForKey:@"Requests"];
id aKey = [keys objectAtIndex:indexPath.row];
userNameLabel.text = [aKey objectForKey:@"User"];
NSMutableURLRequest *request = [NSMutableURLRequest requestWithURL:[NSURL URLWithString:[aKey objectForKey:@"URL"]]];
request.cachePolicy = NSURLRequestReloadIgnoringLocalAndRemoteCacheData;
request.timeoutInterval = 5.0;
if ([imageCache objectForKey:aKey])
profileImageView.image = [imageCache objectForKey:aKey];
else
[NSURLConnection sendAsynchronousRequest:request
queue:[NSOperationQueue mainQueue]
completionHandler:
^(NSURLResponse *response, NSData *data, NSError *error)
if (data)
dispatch_async(dispatch_get_main_queue(), ^
profileImageView.image = [UIImage imageWithData:data];
[imageCache setObject:[UIImage imageWithData:data] forKey:aKey];
);
else
profileImageView.image = [self profileImage];
];
else if (indexPath.section == 1)
requestsView.hidden = YES;
NSArray *keys = [self.tableDataSource objectForKey:@"Friends"];
id aKey = [keys objectAtIndex:indexPath.row];
userNameLabel.text = [aKey objectForKey:@"User"];
NSMutableURLRequest *request = [NSMutableURLRequest requestWithURL:[NSURL URLWithString:[aKey objectForKey:@"URL"]]];
request.cachePolicy = NSURLRequestReloadIgnoringLocalAndRemoteCacheData;
request.timeoutInterval = 5.0;
if ([imageCache objectForKey:aKey])
profileImageView.image = [imageCache objectForKey:aKey];
else
[NSURLConnection sendAsynchronousRequest:request
queue:[NSOperationQueue mainQueue]
completionHandler:
^(NSURLResponse *response, NSData *data, NSError *error)
if (data)
dispatch_async(dispatch_get_main_queue(), ^
profileImageView.image = [UIImage imageWithData:data];
[imageCache setObject:[UIImage imageWithData:data] forKey:aKey];
);
else
profileImageView.image = [self profileImage];
];
else if (indexPath.section == 2)
requestsView.hidden = YES;
NSArray *keys = [self.tableDataSource objectForKey:@"Blocked"];
id aKey = [keys objectAtIndex:indexPath.row];
userNameLabel.text = [aKey objectForKey:@"User"];
return cell;
- (void)displayBlockedUsers
if (self.showingBlockedUsers == YES)
self.showingBlockedUsers = NO;
[self.tableView reloadSections:[NSIndexSet indexSetWithIndex:2] withRowAnimation:UITableViewRowAnimationNone];
else
self.showingBlockedUsers = YES;
[self.tableView reloadSections:[NSIndexSet indexSetWithIndex:2] withRowAnimation:UITableViewRowAnimationNone];
- (NSInteger)friendRequests
return [[self.tableDataSource objectForKey:@"Requests"]count];
- (NSInteger)friends
return [[self.tableDataSource objectForKey:@"Friends"]count];
- (NSInteger)blockedUsers
return [[self.tableDataSource objectForKey:@"Blocked"]count];
- (CGFloat)tableView:(UITableView *)tableView heightForRowAtIndexPath:(NSIndexPath *)indexPath
if (indexPath.section == 2 && self.showingBlockedUsers == NO)
return 0;
else
return 68;
- (CGFloat)tableView:(UITableView *)tableView heightForHeaderInSection:(NSInteger)section
if (section == 0)
if ([self friendRequests] < 1)
return 0;
else
return 22;
else if (section == 1)
if ([self friends] < 1)
return 0;
else
return 22;
else
if ([self blockedUsers] < 1)
return 0;
else
return 22;
- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section
switch (section)
case 0:
return [self friendRequests];
case 1:
return [self friends];
default:
return [self blockedUsers];
- (UIView *)tableView:(UITableView *)tableView viewForHeaderInSection:(NSInteger)section
UIView *headerView = [[UIView alloc]init];
headerView.backgroundColor = [UIColor colorWithRed:248.0/255.0 green:248.0/255.0 blue:248.0/255.0 alpha:1.0];
UILabel *headerLabel = [[UILabel alloc]initWithFrame:CGRectMake(12, 0, 320, 22)];
headerLabel.font = [UIFont systemFontOfSize:13.5];
[headerView addSubview:headerLabel];
if (section == 0)
NSInteger friendRequests = [self friendRequests];
if (friendRequests < 2)
headerLabel.text = [NSString stringWithFormat:@"%d Request",friendRequests];
else
headerLabel.text = [NSString stringWithFormat:@"%d Requests",friendRequests];
else if (section == 1)
NSInteger friends = [self friends];
if (friends < 2)
headerLabel.text = [NSString stringWithFormat:@"%d Friend",friends];
else
headerLabel.text = [NSString stringWithFormat:@"%d Friends",friends];
else
UIButton *theButton = [UIButton buttonWithType:UIButtonTypeCustom];
[theButton addTarget:self action:@selector(displayBlockedUsers) forControlEvents:UIControlEventTouchUpInside];
theButton.frame = CGRectMake(0, 0, 320, 22);
[headerView addSubview:theButton];
NSInteger blocked = [self blockedUsers];
if (self.showingBlockedUsers == YES)
headerLabel.text = [NSString stringWithFormat:@"%d Blocked - Tap to Hide",blocked];
else
headerLabel.text = [NSString stringWithFormat:@"%d Blocked - Tap to Show",blocked];
return headerView;
- (CGFloat)tableView:(UITableView *)tableView heightForFooterInSection:(NSInteger)section
return 0;
- (NSInteger)numberOfSectionsInTableView:(UITableView *)tableView
return 3;
【问题讨论】：
您可以为您的拉动刷新处理程序和您的 cellForRowAtIndexPath 方法编写代码吗？ @JasonCoco 我刚刚用代码块顶部的代码更新了问题。 【参考方案1】：您似乎通过在 heightForRowAtIndexPath: 中返回 0 来隐藏该部分。这种方法的问题在于表格视图仍会创建这些单元格 - 因此，除了为实际显示的所有单元格创建单元格之外，表格视图还会为所有被阻止的用户创建其他单元格。
要隐藏一个部分，只需在该部分“关闭”时从tableView:numberOfRowsInSection: 返回0。这样，表格视图就不会调用heightForRowAtIndexPath:、cellForRowAtIndexPath: 或其他任何东西，因为您告诉它那里没有单元格。
【讨论】：
完美！现在不再分配内存。请在原始问题顶部查看我的新代码，了解我是如何解决的。 @troop231 很高兴它成功了！顺便说一句，您的异步图像下载代码看起来可能是错误的 -profileImageView 可能会在用户快速滚动时被重用；这可能会导致显示错误的个人资料图片。
有什么改进的建议吗？
@troop231 在这里查看我的答案 - ***.com/a/21896596/1445366 - 基本上是完全相同的问题。如果您在那之后需要更多帮助，请发布另一个问题，我会看看。【参考方案2】：
好吧，快速查看它，它正在使用内存，因为 cellForRowAtIndexPath: 仍然被调用并且您仍然返回单元格，就好像它是可见的一样。这听起来真的很简单，但是您是否尝试过检查该部分是否隐藏然后返回一个空单元格？
更新：问题是您实际上并没有告诉表格视图您隐藏了单元格（只是降低了高度）。在 MVC 中，您的模型知道但控制器不知道。在您使用模型中的布尔值检查您是否隐藏了单元格后，只需在 - (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section 中返回 0。
【讨论】：
我也是这么想的，我会尽快实施并回复您。 不要这样做。你不能从那个方法返回nil！它会破坏一切！
我可以确认返回 nil 会导致我的应用崩溃。
哦，对不起.. 我怎么会错过呢？但是，当我谈到这个话题时，你究竟是如何“隐藏”你的部分/单元格的。因为如果它真的被隐藏了，那么 cellForRowAtIndexPath 应该被调用！
我更新了我的答案以返回一个空单元格（框架为 0 的单元格）以上是关于UITableView“隐藏”部分导致每次拉动刷新时分配更多内存的主要内容，如果未能解决你的问题，请参考以下文章
UITableView上方的自定义UIView（用户拉动刷新时）