Индекс вне диапазона только для последних 2 значений? - PullRequest
0 голосов
/ 15 октября 2018

Я сейчас получаю индекс вне диапазона, но это происходит только после того, как первые 3 элемента были успешно отображены.Я просмотрел код и до сих пор не знаю, где я ошибаюсь.Индекс вне диапазона происходит только при попытке загрузить текст в ячейки.

Журнал, когда метод вызывается из viewdidload:

Описание indexPath: 0 Описание indexPath: 1 ОписаниеindexPath: 2

Журнал при нажатии кнопки для загрузки остальных изображений:

Описание indexPath: 0 Описание indexPath: 1 Описание indexPath: 2 Описаниеof indexPath: 3

Метод источника данных, где indexpath находится вне диапазона:

    //MARK: - tableview datasource methods

override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {

    let cell = tableView.dequeueReusableCell(withIdentifier: "Cell", for: indexPath)

    if arrayOfImgObj.count > 0 {
        cell.imageView?.image = arrayOfImgObj[indexPath.row].imageTempt
        print("Description of indexPath: ",indexPath.row)
        cell.textLabel?.text = arrayOfUIDs[indexPath.row] // index out of range
    } else {
        print("\(arrayOfImgObj.count) not over 0 yet")
    }
    return cell
}

Метод, который вызывается при viewDidload:

    //testing array
var arrayOfUIDs = User().users

func fetchAllUserFristImage() {
    Database.database().reference().child("Posts").observe(.childAdded, with: {(snapshot) in

        if (snapshot.value as? [String: AnyObject]) != nil {
            let user = User()

            user.id = snapshot.key

            //testing array
            self.arrayOfUIDs.append(snapshot.key)

            print("\(String(describing: user.id)) <-- SHUD BE THE USERUID")

            self.databaseRef = Database.database().reference()

            let usersPostRef2 = self.databaseRef.child("Posts").child(user.id!)

            usersPostRef2.observe(.value, with: {(postXSnapshots) in

                if let postDictionary2 = postXSnapshots.value as? NSDictionary {

                    for (p) in postDictionary2 {

                        let posts = p.value as! NSDictionary

                        //to get back to where i was delete the below for i
                        for (i) in posts {

                            let imageUrlString = i.value as! NSDictionary

                            guard let postUrl = imageUrlString.value(forKey: "image1") else {return}

                            //below is ting from below
                            if postUrl != nil {

                                self.feedArray.append(Post(fetchedImageURL: postUrl as! String))

                                let imageUrlString = "\(postUrl)"

                                let imageUrl = URL(string: imageUrlString)!
                                print("\(imageUrl) this shud be img url's of posts")

                                let imageDataL = try! Data(contentsOf: imageUrl)
                                self.imageObject.img2 = UIImage(data: imageDataL)
                                let image1ToDisplay: UIImage = self.imageObject.img2!
                                self.arrayOfImgObj.append(forTable(imageTempt: image1ToDisplay))

                            } else {print("this user had no posts, was nil")}
                        }
                    }
                }
            })
        }
        //below shud stay same
        DispatchQueue.main.async {
            self.tableView.reloadData()
        }
    })
}

Редактировать:

Я изменил код, чтобы использовать только один массив объектов:

    override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {

    let cell = tableView.dequeueReusableCell(withIdentifier: "Cell", for: indexPath)

    if arrayOfImgObj.count > 0 {
        cell.imageView?.image = arrayOfImgObj[indexPath.row].imageTempt
        print("Description of indexPath: ",indexPath.row)
        cell.textLabel?.text = arrayOfImgObj[indexPath.row].users[indexPath.row] // index out of range
    } else {
        print("\(arrayOfImgObj.count) not over 0 yet")
    }

    return cell
}

, изменив одну строку в функции следующим образом:

self.arrayOfImgObj.append(forTable(imageTempt: image1ToDisplay, users: snapshot.key))

Заранее спасибо!

1 Ответ

0 голосов
/ 15 октября 2018

Ваша проблема в том, что у вас есть два массива:

  • arrayOfUIDs, который имеет одну запись для каждого пользователя
  • arrayOfImgObj, который имеет 0-n записей для каждого пользователя (Таким образом, вполне вероятно, что количество объектов в этом массиве будет больше, чем количество объектов в arrayOfUIDs.

Вы рассчитываете количество строк на arrayOfImgObj, но затем используетеномер строки для индексации в arrayOfUIDs, что приводит к исключению границ массива.

Может быть более элегантно иметь секцию табличного представления для каждого идентификатора пользователя, но если вы хотите поместить все строки водин раздел Я бы посоветовал вам использовать один массив структур в качестве модели данных.

Использовать что-то вроде:

struct UserImage {
    var userID: String
    var image: UIImage
}


var tableData = [UserImage]()

Затем, выбирая каждое изображение, создайте новую структуру и поместите еев вашем массиве:

func fetchAllUserFristImage() {
    Database.database().reference().child("Posts").observe(.childAdded, with: {(snapshot) in

        if snapshot.value as? [String: AnyObject] != nil {
            let user = snapshot.key

            self.databaseRef = Database.database().reference()

            let usersPostRef2 = self.databaseRef.child("Posts").child(user)

            usersPostRef2.observe(.value, with: {(postXSnapshots) in

                if let postDictionary2 = postXSnapshots.value as? [String:AnyObject] {

                    for (p) in postDictionary2 {

                        if let posts = p.value as? [String:AnyObject] {

                            //to get back to where i was delete the below for i
                            for (i) in posts {

                                if let imageUrlString = i.value as? [String:AnyObject], let postUrl = imageUrlString.["image1"] as? String {

                                    self.feedArray.append(Post(fetchedImageURL: postUrl))
                                    if let imageUrl = URL(string: postUrl), let imageDataL = try? Data(contentsOf: imageUrl), let image = UIImage(data: imageDataL) {

                                        self.tableData.append(UserImage(userID: user, image: image))
                                    } else {print("this user had no posts, was nil")}
                                }
                            }
                        }
                    }
                }
            })
                //below shud stay same
                DispatchQueue.main.async {
                    self.tableView.reloadData()
            }
        }
    })

}

Теперь ваш cellForRow может быть:

override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {

    let cell = tableView.dequeueReusableCell(withIdentifier: "Cell", for: indexPath)

    cell.imageView?.image = tableData[indexPath.row].image
    cell.textLabel?.text = tableData[indexPath.row].userID

    return cell
}
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...