首页 > 解决方案 > Swift - 关闭 ViewController 后停止 TableView 重新加载

问题描述

我有一个在获取请求完成后填充的表格视图。每行包含一个在 SFSafariViewController 中打开的链接。一旦用户在 SF Safari VC 中点击 Done 按钮,tableview 就会重新加载并再次调用 fetch 请求。我不希望这种情况发生,因为获取请求有时可能需要 15 秒以上的时间,而且我担心它会使我的 API 调用次数增加一倍。

下面的代码 - 一旦 Safari VC 被关闭,我能做些什么来防止 tableview 重新加载?我尝试使用布尔值来跟踪它,但是当 viewDidLoad /appear 被调用时它会再次发生变化。我应该以某种方式在 viewdidload/appear 之外调用我的获取请求吗?

import SafariServices
import UIKit

class ArticlesTVC: UITableViewController, SFSafariViewControllerDelegate {

var articles = [Article]()

let cellId = "articleCell"

var refresh: Bool = false

override func viewDidLoad() {
    super.viewDidLoad()
    registerCell()
}

override func viewWillAppear(_ animated: Bool) {
    super.viewWillAppear(true)
    
    refresh = true
    
    if refresh == true {
        DispatchQueue.main.async {
            self.fetchArticles()              
            self.refresh = false
        }
    }
   
}

func registerCell() { tableView.register(UITableViewCell.self, forCellReuseIdentifier: cellId) }

override func numberOfSections(in tableView: UITableView) -> Int { return 1 }

override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {  return articles.count }

override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
    let cell = tableView.dequeueReusableCell(withIdentifier: cellId, for: indexPath)
    
    let article = articles[indexPath.row]
    cell.textLabel?.text = article.title
    cell.detailTextLabel?.text = article.description
    return cell
}

override func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
    let article = articles[indexPath.row]
    loadArticle(articleURL: article.url!)
    
}

func loadArticle(articleURL: String) {
    if let url = URL(string: articleURL) {
       
        let vc = SFSafariViewController(url: url)
        
        vc.delegate = self
        present(vc, animated: true)
    }
}

func safariViewControllerDidFinish(_ controller: SFSafariViewController) {
    dismiss(animated: true)
}

func fetchArticles() {
    
    let baseURL = "url removed for privacy"
    
    guard let url = URL(string: "\(baseURL)") else {
        print("url failed")
        return
    }
    
    let session = URLSession(configuration: .default)
    
    let task = session.dataTask(with: url) { data, response, error in
        
        if error != nil {
            print(error)
            return
        }
        
        if let safeData = data {
            self.parseJSON(data: safeData)
        }
        
    }.resume()
}

func parseJSON(data: Data) {
    let decoder = JSONDecoder()
    
    do {
        let decodedData = try decoder.decode(ArticleEnvelope.self, from: data)
        let newsArticles = decodedData.news
        for item in newsArticles {
            let title = item.title
            let description = item.description
            let url = item.url
            let image = item.image
            let published = item.published
            
            let article = Article(title: title, description: description, url: url, image: image, published: published)
            
            DispatchQueue.main.async {
                self.articles.append(article)
                self.tableView.reloadData()
            }
            
        }
        print("articles loaded successfully")
    } catch {
        print("Error decoding: \(error)")
    }
}

标签: iosswiftuitableviewuitabbarcontroller

解决方案


当然。你已经完成了 90% 的路程。您的viewWillAppear(_:)方法检查一个布尔标志refresh,并且仅在以下情况下获取数据并重新加载表refresh == true。但是,它明确设置refresh = true. viewWillAppear(_:)每次您关闭其他视图控制器并重新显示表格视图控制器时,都会调用您的函数。

将线refresh = true移入viewDidLoad(). 问题解决了。(viewDidLoad()仅在首次创建视图控制器时调用一次,而不是每次通过关闭/弹出覆盖它的视图控制器来发现它时。)


编辑:

请注意,在此代码中:

override func viewWillAppear(_ animated: Bool) {
    super.viewWillAppear(true)
    
    refresh = true
    
    if refresh == true {
        DispatchQueue.main.async { //This bit is not needed
            self.fetchArticles()              
            self.refresh = false
        }
    }
}

不需要调用 to DispatchQueue.main.async,这会使获取数据的速度稍慢一些。viewWillAppear(_:)已经总是在主线程上调用。

像这样重写它:

override func viewDidLoad() {
    super.viewDidLoad()
    registerCell()
    refresh = true //Moved from viewWillAppear
}


override func viewWillAppear(_ animated: Bool) {
    super.viewWillAppear(true)
    
    //Remove refresh = true from this function.

    if refresh == true {
        //No need to use DispatchQueue.main.async here
        self.fetchArticles()              
        self.refresh = false
    }
}

推荐阅读