Swift - сохранение массива объектов в основных данных, получение только одного объекта при извлечении - PullRequest
1 голос
/ 02 мая 2019

Я извлекаю данные json из API и заполняю UITableview из них - до этого все работает нормально.

Проблема заключается в сохранении данных в основные данные . Насколько я знаю - я правильно сохраняю данные, и я отладил, чтобы увидеть, что на самом деле 15 объектов были сохранены в основных данных. Моя проблема заключается в попытке извлечь данные из основных данных - я получаю только один объект из всего списка json, обычно это первый, но иногда это какой-то другой индекс.

вот мой ВК:


import UIKit
import CoreData
import Kingfisher

class MoviesViewController: UITableViewController {

    let base_url = "https://api.androidhive.info/json/movies.json"
    let context = PersistanceService.context
    var moviesArray = [Movie]()

    override func viewDidLoad() {
        super.viewDidLoad()
        if loadData(){
            print("loading items from local presistancy")
        } else {
            print("fetching new data from api")
            dataFetchFromMoviesApi()
        }
        //dataFetchFromMoviesApi()
        tableView.estimatedRowHeight = 180
        tableView.register(UINib(nibName: "TableViewCell", bundle: nil), forCellReuseIdentifier: "MovieCell")
    }



    //MARK: - TableView Delegate Method
    override func numberOfSections(in tableView: UITableView) -> Int {
        return 1
    }

    override func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        tableView.deselectRow(at: indexPath, animated: true)
        performSegue(withIdentifier: "showMovieDetails", sender: moviesArray[indexPath.row])
    }

    override func tableView(_ tableView: UITableView, heightForRowAt indexPath: IndexPath) -> CGFloat {
        return UITableView.automaticDimension
    }


    //MARK: - TableView Datasource Methods

    override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let movie = moviesArray[indexPath.row]
        let cell = tableView.dequeueReusableCell(withIdentifier: "MovieCell", for: indexPath) as! TableViewCell
        cell.title.text = "Title: \(movie.title)"
        cell.rating.text = "Rating: \(String(movie.rating))"
        cell.releaseYear.text = "Release Year: \(String(movie.releaseYear))"
        cell.genre.text = "Genre: \(String(movie.genre[0]))"
        cell.movieImage.downloaded(from: movie.image)

        return cell

    }

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

    //MARK: - Segue Method
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if segue.identifier == "showMovieDetails" {
            let movieVC = segue.destination as! MovieDetailsViewController
            movieVC.selectedMovie = sender as? Movie
        }
    }



    //MARK: - Data Fetching Method

    func dataFetchFromMoviesApi(){
        httpRequest(urlForRequest: base_url) { [weak self] (data: Data?, error: Error?) in
            if error == nil { //no error - continue
                if let moviesData = data { //un-wrapping the data object
                    let decoder = JSONDecoder()
                    do {
                        let responseArray = try decoder.decode([Movie].self, from: moviesData)
                        //print(responseArray)
                        DispatchQueue.main.async {
                            self?.moviesArray = responseArray
                            self?.moviesArray.sort(by: {$0.releaseYear > $1.releaseYear})

                            for movie in self!.moviesArray {
                                let movieModel = MovieModel(context: self!.context)
                                movieModel.title = movie.title
                                movieModel.releaseYear = String(movie.releaseYear)
                                movieModel.rating = String(movie.rating)
                                movieModel.genre = movie.genre[0]
                                movieModel.image = movie.image
                            }
                            PersistanceService.saveContext()
                            self?.tableView.reloadData()
                        }
                    } catch {
                        print(error)
                    }
                }
            } else {
                //error fetching data
                print(error?.localizedDescription ?? "error with no description")
            }
        }
    }

    //MARK: - Http Request

    private func httpRequest(urlForRequest: String, completion: @escaping (Data?, Error?) -> Void){
        guard let url = URL(string: urlForRequest) else {return}
        let task = URLSession.shared.dataTask(with: url) { (data: Data?, urlResponse: URLResponse?, error: Error?) in
            if error == nil {
                //task was succfull
                completion(data, nil)
            } else {
                //task was unsuccessfull
                completion(nil, error)
            }
        }
        task.resume()
    }

    func saveData(){
        do {
            try context.save()
        } catch {
            print("error saving context, \(error.localizedDescription)")
        }
    }

    func loadData() ->Bool{
        let fetchRequest: NSFetchRequest<MovieModel> = MovieModel.fetchRequest()
        do {
            let movies = try PersistanceService.context.fetch(fetchRequest)
            print(movies.count)
            for movie in movies {
                let title = movie.title!
                let rating = Float(movie.rating!)!
                let image = movie.image!
                let genre = [movie.genre!]
                let releaseYear = Int(movie.releaseYear!)!
                let newMovie = Movie(title: title, image: image, rating: rating, releaseYear: releaseYear, genre: genre)
                self.moviesArray.append(newMovie)
                print("movies array count: \(moviesArray.count)")
                print("successfully loaded items")
                return true
            }
        } catch {
            print("error fetching from core data, \(error)")
            return false
        }
        return false
    }
}

Я работаю с двумя параллельными типами данных: один - объект фильма для базовых данных, другой - структура фильма, которая соответствует кодируемому протоколу, а другой - для декодирования json. поэтому, когда я возвращаю запрос на выборку, я назначаю его объекту movies, и, насколько я понимаю, этот объект movie становится массивом моей сущности movie. Я прав?

Надеюсь, кто-то может указать мне на ошибку, которую я делаю.

1 Ответ

1 голос
/ 02 мая 2019

Вы поместили свое возвращение не в то место в loadData, переместите его из цикла for

 for movie in movies {
     let title = movie.title!
     let rating = Float(movie.rating!)!
     let image = movie.image!
     let genre = [movie.genre!]
     let releaseYear = Int(movie.releaseYear!)!
     let newMovie = Movie(title: title, image: image, rating: rating, releaseYear: releaseYear, genre: genre)
     self.moviesArray.append(newMovie)
     print("movies array count: \(moviesArray.count)")
     print("successfully loaded items")
     // return true <-- This was wrong
 }
 return true //move it here
...