ios - 滚动时uitableview单元格更改值

Question

下面的代码创建了对多个字符串的搜索。最初有 5 行，当你到达第五行时，它会添加另一行。我没有直接编辑行，而是加载了一个过滤器 Controller (另一个 View Controller ，当您键入它时会为您完成单词)。当用户找到一个单词后，他点击它并返回到这个 View Controller 。现在我想用过滤器中的文本填充最初点击的单元格。

我之前试着问过，但没有得到任何具体的答案。

我遇到了一个问题，当我滚动时(添加新行后)，它开始用表格中已有的信息填充那些行，(而不是保持空白)

请帮我看看我哪里出错了

//global indexpath to remember which cell tapped
NSIndexPath *globalPath;
@interface SearchViewController ()

@end

@implementation SearchViewController

//Load implementation once per launch
- (void)viewDidLoad
{
    [super viewDidLoad];
    [self linkInputTableToDelegate];
    _temporaryResultsArray =[[NSMutableArray alloc]init];
    _flurryArray=[[NSMutableArray alloc]init];
    _numberOfSections=6;
}

-(void)viewWillAppear:(BOOL)animated{
    [super viewWillAppear:NO];
    [InputTable reloadData];

    textFromUserDefaults=[[[HelperMethods alloc]init]getObjectUserDefault:@"textFiltered"];
    [self addTextToFlurryArrayForFlurryAndSavedLists:_textFromUserDefaults];
}

-(void)viewDidDisappear:(BOOL)animated{

}

- (IBAction)searchButtonPressed:(UIButton *)sender {
     self.tabBarController.selectedIndex = 1;
}

//Makes the input table respond to delegate table view methods
-(void)linkInputTableToDelegate{
    _inputTable.dataSource=self;
    _inputTable.delegate=self;
}

-(void)performSearch:(NSString*)text{
    //do search
}

#pragma mark - Table view data source

- (NSInteger)numberOfSectionsInTableView:(UITableView *)tableView
{
    int numberOfRows=_numberOfSections;

    //Rows for iPhone 4
    if ([[UIScreen mainScreen]bounds].size.height==480) {
        numberOfRows=numberOfRows;
        //Rows for iPhone 5
    }else if ([[UIScreen mainScreen]bounds].size.height==568){
        numberOfRows=numberOfRows+1;
    }
    return numberOfRows;
}

- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section
{
    //In reality groups are created with 1 row inside, this is to allow spacing between the rows
    return 1;

}

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

    UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:kCellID];

    if (!cell) {
        cell = [[UITableViewCell alloc]initWithStyle:UITableViewCellStyleDefault reuseIdentifier:kCellID];
    }

    //Is the cell the same as the one clicked when going to ingredient filter
    BOOL cellIndexPathSameAsSelected=[self isCellIndexSameAsPreviousClicked:indexPath];

    cell.textLabel.textColor=[UIColor blackColor];
    if (cellIndexPathSameAsSelected && _textFromUserDefaults!=nil) {

        if (![cell.textLabel.text isEqualToString:_textFromUserDefaults]) {

            cell.textLabel.text=_textFromUserDefaults;
            [self performTextSearch:_textFromUserDefaults];
        }

    }
    return cell;
}

//Compares the previous clicked cell with the cell now selected
-(BOOL)isCellIndexSameAsPreviousClicked: (NSIndexPath*)cellPath{

    if (cellPath.row == globalPath.row && globalPath.section==cellPath.section) {
        return YES;
    }
    else{
        return NO;
    }
}

- (void)updateTableViewWithExtraRow :(NSIndexPath*)rowSelected{
    NSLog(@"number of sections =%i",_numberOfSections);
    if (rowSelected.section == _numberOfSections) {
        _numberOfSections ++;
    }
}

#pragma mark - Table view delegate

- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath
{
    NSString *cellText = [tableView cellForRowAtIndexPath:indexPath].textLabel.text;
    [[[HelperMethods alloc]init]saveObjectToUserDefaults:cellText :@"textFiltered"];
    globalPath = indexPath;
    [self updateTableViewWithExtraRow:indexPath];
}

-(void)addTextToFlurryArrayForFlurryAndSavedLists:(NSString*)text{
    if ([_flurryArray count]==0 &&[text length]>0) {
        [_flurryArray addObject:text];
    }
    for (int i=0;i<[_flurryArray count];i++) {
        NSString *textInArray=[_flurryArray objectAtIndex:i];
        if (![textInArray isEqualToString:text]) {
            [_flurryArray addObject:text];
        }

    }
    NSLog(@"Total number of saved items = %i",[_flurryArray count]);
}

// Dispose of any resources that can be recreated.
- (void)didReceiveMemoryWarning
{
    [super didReceiveMemoryWarning];
}

Answer 1

我在查看代码时有几个 react :

1. 关于正确使用 UITableViewDataSource 方法的一些观察，特别是 numberOfRowsInSection、numberOfSectionsInTableView 和 cellForRowAtIndexPath :

   • 这些真的应该由一些模型数据结构(例如 NSMutableArray)驱动，而不是别的；

   • 这些方法应该是无状态的。他们不应该依赖于一些 NSString 实例变量的值，比如 _textFromUserDefaults)，而是总是在 NSMutableArray 模型结构中查找值indexPath 参数值的基础。您根本无法对何时调用 cellForRowAtIndexPath 做出任何假设。这很可能会解释您的重复值。

   • 除了响应 UITableView 查询之外，这些都不应该做任何事情。例如，您的 cellForRowAtIndexPath 正在调用 performTextSearch。除了返回单元格之外，它真的不应该做任何事情。

   • 您的 cellForRowAtIndexPath 当前具有条件逻辑，并且仅在满足特定条件时才更新单元格。因为单元格被重用，所以您真的要确保无论如何都初始化单元格。您不能保证获取单元格时它是空白的，也不能保证之前的内容就是该 indexPath 的以前的值。因为单元格被重复使用，所以它可能用于完全不同的行。这也可能是您重复输入的原因。

2. 关于主视图 Controller 和详细 View Controller 的交互，有比通过 NSUserDefaults 来回传递数据更优雅的方法。例如，当你启动细节 View Controller 时，你可以只传递它需要的信息。完成后，它应该调用主视图 Controller 中的方法来更新主视图中的数据。为此，主视图 Controller 应符合您自己创建的某些协议(protocol)。如果您看到我通过聊天分享的示例，您可以看到它可能是什么样子。无论如何，通过在主视图 Controller 中使用一些委托(delegate)方法，细节 View Controller 在完成时调用它，这消除了使用 viewDidAppear 控制主 TableView 更新的相当脆弱的技术。

3. 您可能想考虑使用“编辑”(它允许您删除，也可能编辑特定行)和“添加”按钮，例如 Xcode 提供的标准“主从”模板。这里有许多标准约定可能比您可以点击的空白单元格数组更好。显然，您的用户体验完全取决于您，但您始终可以考虑是否存在您可以采用的现有的、熟悉的约定。