APIレスポンスをUITableViewCellに各々表示するにあたり不明な点

Question

前回の質問からの進展

前回思うように結果を受け取れたので
1.queryを入れ複数の情報に対応させました。
2.TextFieldを追加してTableViewで検索機能を追加。
3.TableViewCellを作成しnibに表示したい項目を追加。
4.新たにidをAPiから取得してdidselectを追加。

ソースコード

import UIKit
import SafariServices

class ViewController: UIViewController,UITextFieldDelegate {

    @IBOutlet weak var movieTableView: UITableView!
    @IBOutlet weak var field: UITextField!
    
    var movies = [Movie]()

    override func viewDidLoad() {
        super.viewDidLoad()
        movieTableView.register(MovieTableViewCell.nib(), forCellReuseIdentifier: MovieTableViewCell.identifier)
        
        movieTableView.delegate = self
        movieTableView.dataSource = self
        field.delegate = self
        
        fechData()
    }
    
    func textFieldShouldReturn(_ textField: UITextField) -> Bool {
        fechData()
        return true
    }
    
    func fechData(){

        field.resignFirstResponder()
        
        guard let text = field.text, !text.isEmpty else {
            return
        }
        
        let query = text.replacingOccurrences(of: " ", with: "%20")
        let url = text.addingPercentEncoding(withAllowedCharacters: .urlQueryAllowed)
        
        movies.removeAll()
       
        URLSession.shared.dataTask(with: URL(string: "https://api.themoviedb.org/3/search/movie?api_key=自分のAPI&language=ja&query=(query)&page=1")!,
                                   completionHandler: {data, response, error in
                                    
                                    guard let data = data else {
                                        print(error?.localizedDescription ?? "Unknown error")
                                        return
                                    }
                                    
                                    var result: MovieStruct?
                                    do {
                                        result = try JSONDecoder().decode(MovieStruct.self, from: data)
                                    }
                                    catch{
                                        print("JSON perse error")
                                    }
                                    
                                    guard let finalResult = result else{
                                        return
                                    }
                                    
                                    let newMovies = finalResult.results
                                    self.movies.append(contentsOf: newMovies)
                                    
                                    DispatchQueue.main.async {
                                        self.movieTableView.reloadData()
                                    }
            }).resume()
    }
}

extension ViewController:UITableViewDelegate,UITableViewDataSource{
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return movies.count
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = movieTableView.dequeueReusableCell(withIdentifier: MovieTableViewCell.identifier, for: indexPath) as! MovieTableViewCell
        cell.configure(with: movies[indexPath.row])
        return cell
    }
    
    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        tableView.deselectRow(at: indexPath, animated: true)
        let url = "https://www.themoviedb.org/movie/(movies[indexPath.row].id)?language=ja"
        let vc = SFSafariViewController(url: URL(string: url)!)
        present(vc, animated: true)
    }
}

struct MovieStruct:Codable {
    let results: [Movie]
}

struct Movie: Codable {
    let title: String
    let release_date:String?
    let poster_path:String?
    let id:Int?
}

TableViewCell

import UIKit

class MovieTableViewCell: UITableViewCell {

    @IBOutlet var movieTitleLabel: UILabel!
    @IBOutlet var movieReleaseDateLabel: UILabel!
    @IBOutlet var moviePosterImageView: UIImageView!
    
    override func awakeFromNib() {
        super.awakeFromNib()
        // Initialization code
    }

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

    }
    
    static let identifier = "MovieTableViewCell"
    
    static func nib() -> UINib{
        return UINib(nibName: "MovieTableViewCell", bundle: nil)
    }
    
    func configure(with model:Movie){
        self.movieTitleLabel.text = model.title
        self.movieReleaseDateLabel.text = model.release_date
        let url = model.poster_path
        if let data = try? Data(contentsOf: URL(string: url!)!){
            self.moviePosterImageView.image = UIImage(data: data)
        }
    }
}

不明点&エラー

1.検索Hitにばらつきがあり、urlにnillが入ってしまう。

例えばstarと入力するとSTARWARSの情報が出るのですが
titanicと入力するとconfigureのところで

Fatal error: Unexpectedly found nil while unwrapping an Optional value

となりurlにnillが入っています。

2.posterが表示されない
上記の検索Hitした、していないにかかわらず
Hitした場合MovieTableViewCellで設定したmoviePosterImageViewが何も表示されずに真っ白です。
titleとrelease_dateは表示されるのでposter_pathのみ受け取れていないのかどこかでnilを設定してしまわれているのだと思います。

3.日本語対応ができない
以前APIの日本語を受け取るためには

let url = text.addingPercentEncoding(withAllowedCharacters: .urlQueryAllowed)

で日本語対応できると習ったのですがURLSessionのところで

 Fatal error: Unexpectedly found nil while unwrapping an Optional value

と出てしまいます。

4.didselectで思うようなサイトに飛べない。

let url = "https://www.themoviedb.org/movie/(movies[indexPath.row].id)?language=ja"
let vc = SFSafariViewController(url: URL(string: url)!)
present(vc, animated: true)

このようにIMDBの映画情報詳細ページに飛びたいのですが
IMDBのHPサイトには飛ぶのですが探しているページは見つかりませんと出ます。

STARWARS 最後のジェダイの詳細URL
https://www.themoviedb.org/movie/181808-star-wars-the-last-jedi?language=ja
ドラえもん　宝島
//https://www.themoviedb.org/movie/495925?language=ja

上記のように検索URLにばらつきがあるので

let url = "https://www.themoviedb.org/movie/(movies[indexPath.row].id)?language=ja"

上記だとidのみを設定しているのでTitileも考慮して設定しなければなりませんか？
質問、コード等が多くなってしまい大変申し訳ありません。

Answer 1

1.検索Hitにばらつきがあり、urlにnillが入ってしまう。
2.posterが表示されない

前回指摘しましたが、model.poster_pathは nil の場合があります。
映画によってはポスター情報（画像URL）が存在しない映画があるようで、そのような場合には nil が戻ってきます。

「titanic」で検索した場合、多くの映画はポスターがあるのですが、「titanic 2000」はポスターがないため、セルを作成している途中でエラーとなってしまいます。

また、model.poster_pathにはベース URL（という表現が正しいかわかりませんが）がなく、画像名しか入っていないため、ベース URL を追加する必要があります。

• [https://developers.themoviedb.org/3/getting-started/images

](https://developers.themoviedb.org/3/getting-started/images)

これらをまとめて修正すると

    func configure(with model:Movie){
        self.movieTitleLabel.text = model.title
        self.movieReleaseDateLabel.text = model.release_date
        // poster_path に URL を追加する。
        // ちなみに、posther_path が nil の場合には無視
        if let poster_path = model.poster_path {
            let url = "https://image.tmdb.org/t/p/w500/" + poster_path
            if let data = try? Data(contentsOf: URL(string: url)!){
                self.moviePosterImageView.image = UIImage(data: data)
            }
        } else {
            // 該当するポスターがない場合
            // titanic 2000 など
            moviePosterImageView.image = nil
        }
    }

のような感じになります。

Cell は使い回しされるため、もし画像データがない場合には、moviePosterImageView.image に nil を入れたり、あるいは「ポスターがない」に相当する画像を表示するようにした方がよいかもしれません。

3.日本語対応ができない

let url = text.addingPercentEncoding(withAllowedCharacters: .urlQueryAllowed)

addingPercentEncoding(withAllowedCharaters:)の戻り値はオプショナル型なので、そのことを念頭に置く必要があります。

• addingPercentEncoding(withAllowedCharacters:)

Return Value

Returns the encoded string, or nil if the transformation is not possible.

簡単に修正すると

        //let query = text.replacingOccurrences(of: " ", with: "%20")
        //let url = text.addingPercentEncoding(withAllowedCharacters: .urlQueryAllowed)
        
        guard let query = text.addingPercentEncoding(withAllowedCharacters: .urlQueryAllowed) else {
            // 変換できなかった時の処理
            print("パーセントエンコーディング失敗")
            return
        }

このような感じで一応日本語でも検索できるようになります。

4.didselectで思うようなサイトに飛べない。

これはご自身で API リファレンスを読み込んでいただき、詳細情報の URL を取得できるような方法を調べていただくしかないかと思います。

少なくとも今回検索している JSON にはそのような情報は入っていないように思えます。

Comments

[Ytan]

返信が遅くなってしまい申し訳ありません。
APIのリファレンスにそれぞれ使い方があるのですね、二つとも問題なく稼働して思うようなアプリにようやく仕上がりました。誠にありがとうございます。