在数据源中隐藏表格视图单元或过滤器更好吗？ （性能问题）

Question

我有一个 UITableViewController，其中有要隐藏的单元格。

我目前正在做的是隐藏 heightForRowAt 返回 0 的单元格和 cellForRowAt 返回 isHidden = false 的单元格。但由于我使用的是这个解决方案，我注意到当我在 tableView 中滚动时应用程序变慢了。

// Currently returning a height of 0 for hidden cells
override func tableView(_ tableView: UITableView, heightForRowAt indexPath: IndexPath) -> CGFloat {
    if let post = timeline?.postObjects?[indexPath.row], post.hidden ?? false {
        return 0.0
    }
    return UITableView.automaticDimension
}

// And a cell with cell.isHidden = false (corresponding to identifier "hiddenCell")
override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
    if let post = timeline?.postObjects?[indexPath.row] {
        if post.hidden ?? false {
            return tableView.dequeueReusableCell(withIdentifier: "hiddenCell", for: indexPath)
        } else {
            return (tableView.dequeueReusableCell(withIdentifier: "postCell", for: indexPath) as! PostTableViewCell).with(post: post, timelineController: self, darkMode: isDarkMode())
        }
    }
}

我在想为什么不对数组应用过滤器以完全删除tableView的隐藏单元格，但我不知道每次过滤它们是否对性能很好......

// Returning only the number of visible cells
override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
    return timeline?.postObjects?.filter{!($0.hidden ?? false)}.count
}

// And creating cells for only visible rows
override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
    if let post = timeline?.postObjects?.filter{!($0.hidden ?? false)}[indexPath.row] {
        return (tableView.dequeueReusableCell(withIdentifier: "postCell", for: indexPath) as! PostTableViewCell).with(post: post, timelineController: self, darkMode: isDarkMode())
    }
}

什么是最好的选择？生成单元格时隐藏单元格（第一个）还是将它们排除在列表中（第二个）？

Answer 1

我建议让表格视图数据源方法来处理timeline 的过滤 版本。但是，不要在cellForRowAt 方法中这样做，因为我们需要这样做一次，而不是每次绘制单元格。

因此，您可以做的是声明filteredTimeline 并在viewDidLoad 方法中执行一次过滤（例如）：

class TableViewController: UIViewController {
    // ...
    var filteredTimeline // as the same type of `timeline`


    override func viewDidLoad() {
        // ...

        filteredTimeline = timeline?.postObjects?.filter{!($0.hidden ?? false)}

        // ...
    }

    // Returning only the number of visible cells
    override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return filteredTimeline.count ?? 0
    }

    // And creating cells for only visible rows
    override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        if let post = filteredTimeline?.postObjects?.filter{!($0.hidden ?? false)}[indexPath.row] {
            return (tableView.dequeueReusableCell(withIdentifier: "postCell", for: indexPath) as! PostTableViewCell).with(post: post, timelineController: self, darkMode: isDarkMode())
        }
    }

    // ...
}

如果filteredTimeline = timeline?.postObjects?.filter{!($0.hidden ?? false)} 比viewDidLoad 有更好的位置，您可能需要致电tableView.reloadData()。

你可以做的另一种选择：

如果您认为不需要原始的timeline，您可以自行过滤：

timeline = timeline?.postObjects?.filter{!($0.hidden ?? false)}
tableView.reloadData()

而且您不需要额外的过滤数组。

---

额外提示：

如果在heightForRowAt 方法中为某一行返回0.0 值，cellForRowAt 甚至不会被调用；例如：

func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
    return 2
}

func tableView(_ tableView: UITableView, heightForRowAt indexPath: IndexPath) -> CGFloat {
    return indexPath.row == 0 ?? 0.0 : 100.0
}

func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
    // ...
}

此时，cellForRowAt 应该只被调用一次，因为第一行的高度是 0.0。

Answer 2

单元格大小为 0 是没有意义的。最好的办法是过滤数据源，但我的建议是同时保留两个数组。

但是在其他地方处理过滤然后在 numberOfRowsInSection 中。

var filteredObjects = []

func filterObjects() {
    filteredObjects = timeline?.postObjects?.filter{!($0.hidden ?? false)}
}


// Returning only the number of visible cells
override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
    return filteredObjects.count
}

// And creating cells for only visible rows
override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
    if let post = filteredObjects[indexPath.row] {
        return (tableView.dequeueReusableCell(withIdentifier: "postCell", for: indexPath) as! PostTableViewCell).with(post: post, timelineController: self, darkMode: isDarkMode())
    }
}

我不知道您是如何处理过滤器的，但只要您想应用过滤器，您只需

filterObjects()
tableView.reloadData()