Проблема итерации по массиву, который был сгруппирован - PullRequest
0 голосов
/ 30 октября 2018

У меня есть tableViewCell, который содержит collectionView, который будет отображать некоторые ячейки. Я связал данные с firebase, которые будут показаны в этом tableView.

В настоящее время я сталкиваюсь с проблемой, когда я сгруппировал массив определенной структуры, используя новую функцию группировки словаря (это я сделал в моем TableViewCell).

После того, как это было сделано, я пытаюсь выполнить итерацию в ячейке collectionView, в cellforItemAtIndexPath.

Что я заметил, так это то, что когда я запускаю код, мои ячейки отображают последнюю часть итерации, поэтому последний ее сегмент.

Пожалуйста, найдите ниже мой код для tableView:

struct WebsiteStruct{
var title: String
var description: String
var banner: String
var link: String
}

var siteInfos: [WebsiteStruct] = []
var siteSections: [String] = []

class Websites: UIViewController, UITableViewDelegate, UITableViewDataSource{

@IBOutlet weak var tableView: UITableView!

override func viewDidLoad() {
    super.viewDidLoad()
    self.navigationController?.navigationBar.setBackgroundImage(UIImage(), for: UIBarMetrics.default)
    self.navigationController?.navigationBar.shadowImage = UIImage()

    let websitesRef = Database.database().reference().child("Schools").child(Variables.schoolName).child("Websites")

    websitesRef.observe(.value, with: { snapshot in
        for site in snapshot.children {
            if let siteSnapshot = site as? DataSnapshot{
                let siteInfo = siteSnapshot.value as? NSDictionary
                siteInfos.append(WebsiteStruct(title: siteInfo?.value(forKey: "title") as! String, description: siteInfo?.value(forKey: "description") as! String, banner: siteInfo?.value(forKey: "banner") as! String, link: siteInfo?.value(forKey: "link") as! String))
            }
        }
        self.tableView.reloadData()
    })
}

func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
    for website in siteInfos{
        let section = website.description
        if !siteSections.contains(section){
            siteSections.append(section)
        }
    }
    return siteSections.count
}

func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
    let cell = tableView.dequeueReusableCell(withIdentifier: "websiteSectionCell") as! WebsiteSectionCell
    cell.displayContent(sectionTitle: siteSections[indexPath.row], indexPath: indexPath.row)
    return cell
}
}

Вот код для моей коллекции ViewCell:

class WebsiteSectionCell: UITableViewCell, UICollectionViewDelegate, UICollectionViewDataSource{

@IBOutlet weak var websiteCornerView: UIView!
@IBOutlet weak var Title: UILabel!
@IBOutlet weak var websiteCollectionView: UICollectionView!

var currentIndexPath: Int = 0
let groupedSites = Dictionary(grouping: siteInfos, by: { $0.description })

func displayContent(sectionTitle: String, indexPath: Int){
    websiteCornerView.layer.cornerRadius = Variables.cornerRadius
    websiteCornerView.layer.masksToBounds = true

    Title.text = sectionTitle
    currentIndexPath = indexPath
}

override func awakeFromNib() {
    super.awakeFromNib()
    websiteCollectionView.dataSource = self as UICollectionViewDataSource
    websiteCollectionView.delegate = self as UICollectionViewDelegate
}

override func setSelected(_ selected: Bool, animated: Bool) {
    super.setSelected(selected, animated: animated)
    //Set selected
}

func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
    for site in groupedSites[siteSections[currentIndexPath]]!{
        print(site.title)
    }
    print(groupedSites[siteSections[currentIndexPath]]!)
    return groupedSites[siteSections[currentIndexPath]]!.count
}

func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
    let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "cell", for: indexPath) as! WebsiteCell
    for site in groupedSites[siteSections[currentIndexPath]]!{
        cell.displayContent(title: site.title, banner: Variables.schoolBanner!)
        print(indexPath)
    }
    return cell
}
}

Вот скриншот того, как это выглядит.

Normally, the

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

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

[EEB3App.WebsiteStruct(title: "Site 1", description: "School", banner: "/", link: "/"), EEB3App.WebsiteStruct(title: "Site 2", description: "School", banner: "/", link: "/")]

[EEB3App.WebsiteStruct(title: "Site 3", description: "Utility", banner: "/", link: "/"), EEB3App.WebsiteStruct(title: "Site 4", description: "Utility", banner: "/", link: "/")]

1 Ответ

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

Прежде всего, ваш код очень трудно читать. Я предлагаю, по крайней мере, реорганизовать структуру в другой файл.

Другое дело, что вам нужно проверить эту конкретную строку кода. Это не упорядоченный массив, и он может вернуть вам то же значение.

let groupedSites = Dictionary(grouping: siteInfos, by: { $0.description })

Это должно было быть объявлено и использовано в WebsitesViewController вместо логики в ячейке. Вы можете заменить массив siteSections и массив siteInfos на этот.

...