SwiftでJSONをURLから取得しLabelに流し込みたい

Question

試行錯誤の上、たどり着いたのは以下のコードです。

        struct Record:Codable {
                    let shindo : String
                    let equrl : String
                }
        let url = URL(string: "https://dokasen.com/eqimg/new.json")!  //URLを生成
        let request = URLRequest(url: url)               //Requestを生成
        let task = URLSession.shared.dataTask(with: request) { (data, response, error) in  //非同期で通信を行う
            guard let data = data else { return }
            do {
        
                let object = try JSONSerialization.jsonObject(with: data, options: [])  // DataをJsonに変換
                print(object)
                
                guard let jsonData = try? JSONDecoder().decode(Record.self, from: object as! Data) else {
                                print("json変換に失敗しました")
                                return
                            }
                            print(jsonData.shindo)
                            self.label.text = jsonData.shindo
                
            } catch let error {
                print(error)
            }
        }
        task.resume()

このコードでビルドできるのですが、エミュレータがクラッシュして動作しません。
直すべき箇所や、他にいい方法があればご教示ください。
参考になるURLでも何でもいいです。
よろしくおねがいします。

Answer 1

ご提示のコードだと確かにコンパイルは通るかもしれませんが、強制的なキャストなどを行ったり、try? にしてしまうことでエラーハンドリングが適切に行われないため、容易に発見できるミスも発見できなくなってしまっています。

ミスですが、

1. 構造体 Record のプロパティ名が JSON のキー名と一致していない
2. Data 型で渡されてきたデータを不必要に JSON オブジェクトにしている。
3. また、JSON オブジェクトになった型（__NSDictionary）を Data に強制キャストしているため、実行時エラーで落ちている
4. UILabel の書き換えはメインスレッドで行わなければならないが、離脱するクロージャ内で行っているため、ここも実行時エラーで落ちる要因となっている。

といった具合です。

大きな修正は必要ないですが、ご提示いただいている部分だけで修正したものは次の通りとなります。

struct Record:Codable {
    let shindo : String
    // MARK: JSON のキー名と一致していないのでエラーが出る
    //let equrl : String
    let urleq : String
}

let url = URL(string: "https://dokasen.com/eqimg/new.json")!  //URLを生成
let request = URLRequest(url: url)               //Requestを生成
let task = URLSession.shared.dataTask(with: request) { (data, response, error) in  //非同期で通信を行う
    guard let data = data else { return }
    do {
        // MARK: URLSession に渡すクロージャのうち、data は既に Data 型になっているので下記の作業は不要
        //        let object = try JSONSerialization.jsonObject(with: data, options: [])  // DataをJsonに変換
        //        print(object)
        //
        
        //　MARK: from の部分には直接 data を渡す。また、try? にするとエラーハンドリングできなくなるので、try で実行する。
        //        guard let jsonData = try? JSONDecoder().decode(Record.self, from: object as! Data) else {
        
        let jsonData = try JSONDecoder().decode(Record.self, from: data)
        
        print(jsonData.shindo)
        
        // MARK: UIパーツに値を渡すときにはメインスレッドで実行させる
        DispatchQueue.main.async {
            self.label.text = jsonData.shindo
        }
        
    } catch let error {
        print(error)
    }
}
task.resume()

Comments

[mitdok]

修正頂いたもので試したところ、無事動きました。
ミスの部分まで丁寧に解説して頂いて、勉強になりました。
確かにJSONで届いてるはずのデータを謎にJSONにしてる箇所あったり・・・。
本当に、本当にありがとうございました！