ios - Swift 3 UITableViewCell indexPath.row 搞砸了

Question

我有一个 TableView 来显示一堆电影。movies 是 Movie 对象的数组。 movieIDs 是电影 ID 数组。 ID 只是字符串。

override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "movieCell", for: indexPath) as! MovieCell

        // editing the cell here.

        cell.movieNameLabel.text = movies[indexPath.row].movieName
        cell.movieYearLabel.text = movies[indexPath.row].year

        // source of all hell here.

        for id in movieIDs {

            if id == movies[indexPath.row].movieID {

                print(id + " is equal to " + movies[indexPath.row].movieID)
                cell.myButton.setImage(/*there is an image here*/), for: .normal)

            }

        }

cellForRowAt 方法中的 for 循环:

for id in movieIDs {

        if id == movies[indexPath.row].movieID {

            print(id + " is equal to " + movies[indexPath.row].movieID)
            cell.myButton.setImage(//there is an image here), for: .normal)
        }

    }

我正在将 movieIDs 中的所有 ID 与单元格中的电影 ID 进行比较，即 movies[indexPath.row].movi​​eID。如果它返回 true，我将替换单元格内按钮的图像。当我在 if 语句中打印时，它实际上并没有执行，但它仍然会替换随机单元格中的按钮图像。如果我上下滚动得太快，按钮的图像将在几乎所有单元格中被替换，而它只是为了改变 ID 匹配的单元格。

Answer 1

细胞被塞满的原因是因为它们是可重复使用的细胞。

例如，如果您为单元格 #1 设置了图像，当您向下滚动并且该单元格 #1 离开屏幕并变为单元格 #10(例如)时，它仍然显示图像。

解决方案是你必须通过检查它是否与 movieID 不匹配来删除之前设置的图像，将图像设置为 nil。

您不必在此处执行 for 循环，而是使用 contains 数组。所以替换这段代码:

for id in movieIDs {

    if id == movies[indexPath.row].movieID {

        print(id + " is equal to " + movies[indexPath.row].movieID)
        cell.myButton.setImage(//there is an image here), for: .normal)
    }

}

用这个:

if movieIDs.contains(movies[indexPath.row].movieID) {
    cell.myButton.setImage(//there is an image here), for: .normal)
}
else{
    cell.myButton.setImage(nil)
}