数组索引超出范围 - 从 Parse 下载图像

Question

我正在尝试从解析中查询图像，当我打开我的应用程序时一切正常，但如果我尝试刷新，我会得到如下所示的崩溃...

不太清楚是什么原因造成的...解释一下我是如何设置的：

我有一个带有 1 个单元格的 tableView，在那个单元格中有三个 imageView 连接到一个集合 Outlet。然后我从解析中获取我的图像并将它们放在我的 imageViews 中，然后在 numberOfRowsInSection 中我将它除以 3，这样它就不会重复图像 3 次......！

下面是我的代码：

    var countries = [PFObject]()



    override func viewDidLoad() {
        super.viewDidLoad()

       loadPosts()


        // Do any additional setup after loading the view.
    }

 func loadPosts() {
        PFUser.query()
        // Build a parse query object
        let query = PFQuery(className:"Post")
        query.whereKey("user", equalTo: PFUser.currentUser()!)
        query.orderByDescending("createdAt")


        // Fetch data from the parse platform
        query.findObjectsInBackgroundWithBlock {
            (objects: [PFObject]?, error: NSError?) -> Void in

            // The find succeeded now rocess the found objects into the countries array
            if error == nil {

                self.countries.removeAll(keepCapacity: true)

                if let objects = objects {
                    self.countries = Array(objects.generate())
                }

                // reload our data into the collection view
               self.tableView.reloadData()

            } else {
                // Log details of the failure
                print("Error: \(error!) \(error!.userInfo)")
            }
        }


    }

    override func tableView(tableView: UITableView, numberOfRowsInSection section: Int) -> Int
    {

       return countries.count / 3

    }

    var counter = 0
     override func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell
     {

      let cell = tableView.dequeueReusableCellWithIdentifier("cell2") as! bdbTableViewCell


        for imageView in cell.threeImages {
            let placeHolder = UIImage(named: "camera")
             imageView.image = placeHolder

            let finalImage = countries[counter++]["imageFile"]
            finalImage!.getDataInBackgroundWithBlock {
                (imageData: NSData?, error: NSError?) -> Void in
                if error == nil {
                    if let imageData = imageData {
                        imageView.image = UIImage(data:imageData)
                    }
                }
            }}
            return cell
    }

Answer 1

如果我了解您的数据，则需要移动到数组中包含您所在国家/地区的第一张图片的位置，然后遍历接下来的 3 张图片。

基本上，您应该将计数器声明移动到函数体内，并使用第一张图像的位置对其进行初始化。有关完整实现，请参阅下面的方法定义。

您的代码第一次运行是靠运气。基本上，iOS 将请求第 1 行的单元格，并且您的计数器将通过获取索引 0、1 和 2 处的图像来增加自身。然后 iOS 将请求第 2 行的单元格，它将拉取图像 3、4、5。计数器将不断增加。然后，当您刷新表格时，计数器仍设置为高于图像数组大小的数字，因此您会崩溃。如果 iOS 出于某种原因以不同的顺序要求单元格，您也会错误地显示图像。您需要使用 indexPath.row 参数来获取正确的图像。

您的代码也不会考虑 3 的倍数结束后的其他图像。因此，如果有 17 个图像，则不会在 tableview 中为图像 16 1nd 17 提供一行。要更正此问题，请添加以下函数：

func roundUp(value: Double) -> Int {
    if value == Double(Int(value)) {
        return Int(value)
    } else if value < 0 {
        return Int(value)
    } else {
        return Int(value) + 1
    }
}

然后将tableView的numberOfRowsInSection方法改成如下：

override func tableView(tableView: UITableView, numberOfRowsInSection section: Int) -> Int
{
    return roundUp(Double(countries.count / 3.0))
}

最后，在您的 cellForRowAtIndexPath 函数中，您将不得不处理潜在的尝试查找超过数组大小的图像：

func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell
{
    var imageIndex = (indexPath.row * 3)

    let cell = tableView.dequeueReusableCellWithIdentifier("cell2") as! bdbTableViewCell

    for imageView in cell.threeImages {
        let placeHolder = UIImage(named: "camera")
        imageView.image = placeHolder

        if imageIndex < countries.count
        {
            let finalImage = countries[imageIndex++]["imageFile"]
            finalImage!.getDataInBackgroundWithBlock {
                (imageData: NSData?, error: NSError?) -> Void in
                if error == nil {
                    if let imageData = imageData {
                        imageView.image = UIImage(data:imageData)
                    }
                }
            }
        }
    return cell
}

我相信这将解决您的所有问题。我确实建议您学习有关 UITableView 如何工作以及如何逐步执行您自己的代码进行故障排除的教程。这样你就可以理解为什么你最初的尝试行不通了。

Answer 2

var counter = 0 

应该在函数之前重置

cellForRowAtIndexPath

Answer 3

问题

您将 counter 变量声明为静态变量。这就是为什么它不会在 cellForRowAtIndexPath 调用之间被重置，而是在每次调用该方法时递增。

解决办法：

将counter 声明移动到函数体。如下：

override func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
     
    var counter = 0

    let cell = tableView.dequeueReusableCellWithIdentifier("cell2") as! bdbTableViewCell

        for imageView in cell.threeImages {
            let placeHolder = UIImage(named: "camera")
             imageView.image = placeHolder

            let finalImage = countries[counter++]["imageFile"]
            finalImage!.getDataInBackgroundWithBlock {
                (imageData: NSData?, error: NSError?) -> Void in
                if error == nil {
                    if let imageData = imageData {
                        imageView.image = UIImage(data:imageData)
                    }
                }
            }}
            return cell
    }