1

Суть такая есть менеджер для парса данных

    import Foundation


protocol NetworkDataManagerDelegate: AnyObject {
    func updateInterface( _: NetworkDataManager, with currenData: CurrentData)
}

class NetworkDataManager {
    weak var delegate: NetworkDataManagerDelegate?
    func fetchCurrentData() {
        let urlString = "https://run.mocky.io/v3/1d1cb4ec-73db-4762-8c4b-0b8aa3cecd4c"
        guard let url = URL(string: urlString) else {return}
        let urlSession = URLSession(configuration: .default)
        let task = urlSession.dataTask(with: url) { data, responce, error in
            if let data = data {
                if let currentData = self.parseJSON(withData: data){
                    self.delegate?.updateInterface(self, with: currentData)
                }
            }
        }
        
        task.resume()
    }
    
    func parseJSON(withData data: Data) -> CurrentData?{
        
        let decoder = JSONDecoder()
        do {
            let currentDataData = try decoder.decode(CurrentDataData.self, from: data)
            guard let currentData = CurrentData(currentDataData: currentDataData) else {return nil}
            return currentData
        } catch let error as NSError {
            print(error)
        }
        return nil
    }
    
}

На tableViewController предаю через протокол в котором есть класс updateInterface() вопрос такой как передать данные в массив cell что бы подтягивалось нужно кол-во ячеек? У ячеек есть отдельный viewController на который они собственно и подключены.

view для ячеек

    import UIKit

class CustomCell: UITableViewCell {

    @IBOutlet weak var skilsLable: UILabel!
    @IBOutlet weak var numbeLable: UILabel!
    @IBOutlet weak var nameLable: UILabel!
    
    override func awakeFromNib() {
        super.awakeFromNib()
        // Initialization code
    }

}

Вот и сам tableViewController

    class MyTableViewController: UITableViewController {

    var cell: [String] = []
    let network = NetworkDataManager()
    
    override func viewDidLoad() {
        super.viewDidLoad()
        network.fetchCurrentData()
        network.delegate = self

        tableView.rowHeight = 150
        self.title = "Team of Developer"
    }

    // MARK: - Table view data source

    override func numberOfSections(in tableView: UITableView) -> Int {
        // #warning Incomplete implementation, return the number of sections
        return 1
    }

    override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        // #warning Incomplete implementation, return the number of rows
        return cell.count
    }

    override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "Cell", for: indexPath) as! CustomCell
        return cell
    }
}


extension MyTableViewController: NetworkDataManagerDelegate{
    func updateInterface(_: NetworkDataManager, with currenData: CurrentData) {
        
    }
    
    
}

введите сюда описание изображения

2
  • Не совсем понятно в чем трудность - в пребразовании CurrentData в массив [String]?
    – schmidt9
    5 сен 2021 в 7:49
  • суть такая, вот отсюда run.mocky.io/v3/1d1cb4ec-73db-4762-8c4b-0b8aa3cecd4c нужно получить список сотрудников из массива employees и передать на tableView в numberOfRowsInSection для правильного формирования кол-во ячеек а так же с распарсенные даные предать в ячейки, в func updateInterface я не могу достучаться до ячеек с типом IULable, так как они находятся в другом классе class CustomCell: UITableViewCell и файле, вот в этом то как раз таки и вопрос.
    – Роман
    5 сен 2021 в 10:31

1 ответ 1

3

Вы не привели в вопросе модель данных, предположим она такая, хотя конечно CurrentDataData лучше бы переименовать или вообще обойтись без него, использовав сразу CurrentData

class CurrentData {
    var currentDataData: CurrentDataData?
    
    init?(currentDataData: CurrentDataData?) {
        self.currentDataData = currentDataData
    }
}

struct Employee : Decodable {
    var name: String
    var phone_number: String
    var skills: [String]
}

struct Company : Decodable {
    var name: String
    var employees: [Employee]
}

struct CurrentDataData : Decodable {
    var company: Company
}

Также я подкорректировал вызов делегата, который лучше сделать сразу в основном потоке

    func fetchCurrentData() {
        let urlString = "https://run.mocky.io/v3/1d1cb4ec-73db-4762-8c4b-0b8aa3cecd4c"
        guard let url = URL(string: urlString) else {return}
        let urlSession = URLSession(configuration: .default)
        let task = urlSession.dataTask(with: url) { data, responce, error in
            if let data = data {
                if let currentData = self.parseJSON(withData: data){
                    DispatchQueue.main.async {
                        self.delegate?.updateInterface(self, with: currentData)
                    }
                }
            }
        }
        
        task.resume()
    }

Ну и соответствующие правки в контроллере

class MyTableViewController: UITableViewController {

    var data: CurrentDataData?
    let network = NetworkDataManager()
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        tableView.register(UINib(nibName: "CustomCell", bundle: nil), forCellReuseIdentifier: "Cell")
        tableView.rowHeight = 150
        
        network.delegate = self
        network.fetchCurrentData()

        self.title = "Team of Developer"
    }

    // MARK: - Table view data source

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

    override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return data?.company.employees.count ?? 0
    }

    override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "Cell", for: indexPath) as! CustomCell
        
        guard let employees = data?.company.employees else {
            return cell
        }
        
        let employee = employees[indexPath.row]
        
        cell.nameLable.text = employee.name
        cell.numbeLable.text = employee.phone_number
        cell.skilsLable.text = employee.skills.joined(separator: ",")
        
        return cell
    }
}

extension MyTableViewController: NetworkDataManagerDelegate{
    func updateInterface(_: NetworkDataManager, with currenData: CurrentData) {
        data = currenData.currentDataData
        tableView.reloadData()
    }
}

При этом и тут можно предложить ряд улучшений:

  • вместо делегирования использовать замыкание, что упростит код
    func fetchCurrentData(_ completion: @escaping (CurrentData) -> Void) {
        let urlString = "https://run.mocky.io/v3/1d1cb4ec-73db-4762-8c4b-0b8aa3cecd4c"
        guard let url = URL(string: urlString) else {return}
        let urlSession = URLSession(configuration: .default)
        let task = urlSession.dataTask(with: url) { data, responce, error in
            if let data = data {
                if let currentData = self.parseJSON(withData: data){
                    DispatchQueue.main.async {
                        completion(currentData)
                    }
                }
            }
        }
        
        task.resume()
    }
    network.fetchCurrentData {
        self.data = $0.currentDataData
        self.tableView.reloadData()
    }
  • вместо прямого доступа к аутлетам в ячейке увеличить инкапсуляцию и передавать туда данные, а сами аутлеты сделать приватными
class CustomCell: UITableViewCell {
    
    @IBOutlet private weak var skilsLable: UILabel!
    @IBOutlet private weak var numbeLable: UILabel!
    @IBOutlet private weak var nameLable: UILabel!
    
    var employee: Employee? {
        didSet {
            skilsLable.text = employee?.name
            numbeLable.text = employee?.phone_number
            nameLable.text = employee?.skills.joined(separator: ",")
        }
    }

}
    override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "Cell", for: indexPath) as! CustomCell
        
        guard let employees = data?.company.employees else {
            return cell
        }
        
        cell.employee = employees[indexPath.row]
        
        return cell
    }
6
  • вот так выглядит модель ` import Foundation struct CurrentDataData: Codable { let company: Company } struct Company: Codable { let name: String let employees: [Employees] } struct Employees: Codable { let name: String let phoneNumber: String let skills: [String] enum CodingKeys: String, CodingKey { case name case phoneNumber = "phone_number" case skills } } `
    – Роман
    5 сен 2021 в 19:38
  • а по поводу delegate или Closure читал что лучше делегат использовать, Спасибо в мне очень помогли
    – Роман
    5 сен 2021 в 19:40
  • Может, что то порекомендуете почитать что бы подобные задачи не вызывали трудностей
    – Роман
    5 сен 2021 в 23:21
  • @Роман что-то конкретное к сожалению порекомендовать не могу
    – schmidt9
    6 сен 2021 в 10:15
  • ну ладно. Спасибо большое, что помогли с проблемой. А менторством занимаетесь ?
    – Роман
    6 сен 2021 в 10:18

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.