ネストを減らすと言う意味では、guard文の採用も検討された方が良いでしょう。
task = URLSession.shared.dataTask(with: URL(string: url)!) { (data, res, err) in
guard let obj = try! JSONSerialization.jsonObject(with: data!) as? [String: Any],
let items = obj["items"] as? [[String: Any]]
else {
print("bad data")
return
}
self.repo = items
DispatchQueue.main.async {
self.tableView.reloadData()
}
}
なお、コード例の中でtry!
を使用していますが、JSONSerialization.jsonObject(with:)
は、入力となるデータに1バイトでもおかしな点があれば簡単に失敗するので、try!
を使用するとそこでアプリがクラッシュしてしまいます。「強制tryは確実に安全と分かっている場合以外は使ってはいけない」と覚えておいた方が良いと思います。
別質問とも関わりますが、その他の危険な強制アンラップも全部避け、エラーチェックもきちんとやるとこんな感じになります。「ネストを減らす」と言うご質問の趣旨からは外れることになりますが、URLSession
は様々な理由でエラー終了しますので、(今は練習用のコードなのかもしれませんが、例えそうだとしても)必要なエラー処理を省略すべきではありません。
guard let theUrl = URL(string: url) else {
print("bad URL string")
return
}
task = URLSession.shared.dataTask(with: theUrl) { (data, res, err) in
do {
if let error = err {
throw error
}
guard let data = data else {
print("data is nil")
throw MyError.dataNil
}
guard let obj = try JSONSerialization.jsonObject(with: data) as? [String: Any],
let items = obj["items"] as? [[String: Any]]
else {
throw MyError.badData
}
self.repo = items
DispatchQueue.main.async {
self.tableView.reloadData()
}
} catch {
print(error)
//必要であればエラー時の処理を追加
}
}
(どこかにこんなものを定義してある前提のコードです。)
enum MyError: Error {
case dataNil
case badData
}