質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.35%
Xcode

Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。

Swift

Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

API

APIはApplication Programming Interfaceの略です。APIはプログラムにリクエストされるサービスがどのように動作するかを、デベロッパーが定めたものです。

Q&A

解決済

1回答

987閲覧

MVCでAPI通信が完了してから変数に値を入れたいのですが

Raleigh

総合スコア8

Xcode

Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。

Swift

Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

API

APIはApplication Programming Interfaceの略です。APIはプログラムにリクエストされるサービスがどのように動作するかを、デベロッパーが定めたものです。

0グッド

0クリップ

投稿2020/09/10 06:25

編集2020/09/10 07:58

前提・実現したいこと

現在位置情報から国土地理院のAPIを使って標高を取得するアプリを作成しています。
標高を取得する操作をControllerとModelに分けて実装しようとしたところうまくいきませんでした。

発生している問題・エラーメッセージ

標高の取得が完了する前にUIの更新をしてしまって、画面に標高の表示がうまく反映されない。

該当のソースコード

Controllerで位置情報を取得したら、ModelのgetElevationで標高を取得しています。
このAPIを使って取得した標高をViewにわたすことができずにいます。
常に0.0mが表示されてしまう状況です。

Controller

1func locationManager(_ manager: CLLocationManager, didUpdateLocations locations: [CLLocation]) { 2 self.customView.elevation = customModel.getElevation(locations) 3}

Model

1var elevationString: String = "0.0m" 2private var afterElevation:Double = 0.0 3private var beforeElevation:Double = pow(10, 5) 4 5func getElevation(_ Locations: [CLLocation]) -> String{ 6 7 self.locations = Locations 8 9 guard let coordinate = Locations.first?.coordinate else {return elevationString} 10 11 userLocation = CLLocationCoordinate2DMake(coordinate.latitude, coordinate.longitude) 12 13 // 国土地理院のURL 14 let baseUrl = "https://cyberjapandata2.gsi.go.jp/general/dem/scripts/getelevation.php?" 15 // ロングタップ地点でクエリを設定する 16 if let userLocation = userLocation { 17 let lonUrl = "&lon=" + userLocation.longitude.description 18 let latUrl = "&lat=" + userLocation.latitude.description 19 // アウトプット形式をJSONに設定する 20 let outtypeUrl = "&outtype=JSON" 21 // URLとクエリを連結 22 let listUrl = baseUrl + lonUrl + latUrl + outtypeUrl 23 // URLを生成する 24 guard let url = URL(string: listUrl) else { return elevationString} 25 26 let task = URLSession.shared.dataTask(with: url) { (data, response, error) in 27 if error != nil { 28 print(error!.localizedDescription) 29 } 30 guard let data = data else { return } 31 // JSONを取得する 32 if let json = try? JSONDecoder().decode(ElevationData.self, from: data) { 33 DispatchQueue.main.async { 34 // JSONから標高を取得する 35 self.afterElevation = json.elevation 36 self.measuringElevation() 37 self.elevationString = String(format: "%.2fm", self.gotElevation) 38 self.beforeElevation = self.afterElevation 39 print(#function, self.elevationString) 40 } 41 } else { 42 self.afterElevation = 0.0 43 44 } 45 } 46 task.resume() 47 } 48 return elevationString 49} 50 51func measuringElevation() { 52 if afterElevation > beforeElevation { 53 gotElevation += afterElevation - beforeElevation 54 } 55} 56

ElevationData

1import Foundation 2 3struct ElevationData: Codable{ 4 let elevation: Double 5}

試したこと

非同期通信などを調べてみてものの、解決に至らなかったのでどうかよろしくおねがいします

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

TsukubaDepot

2020/09/10 06:42

if let json = try? JSONDecoder().decode(ElevationData.self, from: data) { 以降、json に代入された値を直接使っている形跡がないのですが、高度はどのようにして計算しているのでしょうか。 self.measuringElevation() からは json は見えないので、計算できていない気もしています。
TsukubaDepot

2020/09/10 06:44

if let json = try? JSONDecoder().decode(ElevationData.self, from: data) { この行以降、json という変数は直接扱われていませんが、得られた JSON はどのように加工しているのでしょうか。 self.measuringElevation()を読んでいるようですが、このメソッドから上記の json は見えないのではないかと思います。
Raleigh

2020/09/10 06:55

すいませんご指摘ありがとうございます。 jsonの記述の self.afterElevation = json.elevation この部分が抜けいていました。
guest

回答1

0

ベストアンサー

あるメソッドに引数として取られたクロージャからは、return を使ってメソッドの外側に値を返すことができないことはおそらくご存知かと思います。

一方、非同期処理を行うクロージャで得られたデータを処理させるためには、直接クロージャ外の変数を処理するよりも、データ取得完了、あるいはエラー発生時の処理を行うクロージャを渡せるようにして、そこで処理をさせるほうが確実かとおもます。

ということで、ご提示いただいたコードのgetElevation(_:)を次のようにしてみました。

Swift

1getElevation(_ Locations: [CLLocation], completionHandler: @escaping (String?) -> Void, errorHandler: @escaping (Error?) -> Void )

completionHandler: で与えるクロージャは、引数として String?型が与えられます。処理内で正しい標高が得られなかった場合には "0.0" を返すようになっていましたが、実際は標高 0.0 m という場所もあるので、nil を返したほうが現実的と思います。

errorHandler:は、Error? 型が与えられます。こちらはエラーが起きたときの処理を記述します。

いただいた範囲で想像で記述し、コンパイルしていないためもしかしたらエラーが出る可能性もありますが、概ねこんな感じで使えるのではないでしょうか。

  • Controller

Swift

1    model.getElevation(locations, completionHandler: { (elevationString) in 2 DispatchQueue.main.async { 3 // ここでは View Controller にラベルに表示しているが、他のクラスのプロパティに書き込む場合でも考え方は同じ 4 //self.customView.elevation = elevationString 5 self.altitudeLabel.text = elevationString ?? "取得できませんでした" 6 } 7 }, errorHandler: { (error) in 8 print(error?.localizedDescription ?? "エラー情報なし") 9 })
  • Model

Swift

1func getElevation(_ locations: [CLLocation], completionHandler: @escaping (String?) -> Void, errorHandler: @escaping (Error?) -> Void ) { 2 guard let coordinate = locations.first?.coordinate else { 3 // 終了処理のクロージャに nil を渡す 4 completionHandler(nil) 5 return 6 } 7 8 let userLocation = CLLocationCoordinate2DMake(coordinate.latitude, coordinate.longitude) 9 10 let baseUrl = "https://cyberjapandata2.gsi.go.jp/general/dem/scripts/getelevation.php?" 11 12 let lonUrl = "&lon=" + userLocation.longitude.description 13 let latUrl = "&lat=" + userLocation.latitude.description 14 let outtypeUrl = "&outtype=JSON" 15 16 let listUrl = baseUrl + lonUrl + latUrl + outtypeUrl 17 print(listUrl) 18 guard let url = URL(string: listUrl) else { 19 // 終了処理のクロージャに nil を渡す 20 completionHandler(nil) 21 return 22 } 23 24 let task = URLSession.shared.dataTask(with: url) { (data, response, error) in 25 if error != nil { 26 // エラー処理のクロージャに error を渡す 27 errorHandler(error) 28 return 29 } 30 31 guard let data = data else { 32 // 終了処理のクロージャに nil を渡す 33 completionHandler(nil) 34 return 35 } 36 37 // error を返せるように、do-try-catch で処理する 38 do { 39 let json = try JSONDecoder().decode(ElevationData.self, from: data) 40 41 // 以下の処理は全体の文脈がわからないので、とりあえずコメントアウト 42// self.afterElevation = json.elevation 43// self.measuringElevation() 44// self.beforeElevation = self.afterElevation 45 46 // JSON で得られた座標を直接処理 47 self.gotElevation = json.elevation 48 49 let elevationString = String(format: "%.2fm", self.gotElevation) 50 51 completionHandler(elevationString) 52 } catch { 53 errorHandler(error) 54 } 55 } 56 task.resume() 57 }

##追記
今回の処理を簡素化したモデルです。
Playground でも実行できますので、よかったら試してみてください。

Swift

1import UIKit 2 3func delayFunc(completion: @escaping (String) -> ()) { 4 DispatchQueue.main.asyncAfter(deadline: .now() + 1.0) { 5 // 1秒後に実行されるコード 6 7 // (1) に相当 8 print("(1) 遅延処理クロージャの中の処理を開始") 9 completion("クロージャへ渡す文字列") 10 } 11 // (2)に相当 12 print("(2) クロージャ外の処理") 13} 14 15delayFunc(completion: { (message) in 16 print("(3) 処理完了クロージャでの処理") 17 print("渡された文字列は「(message)」") 18})

クロージャだけでなく、引数の型と個数が合えば関数も渡せます。

Swift

1import UIKit 2 3func delayFunc(completion: @escaping (String) -> ()) { 4 DispatchQueue.main.asyncAfter(deadline: .now() + 1.0) { 5 // 1秒後に実行されるコード 6 7 // (1) に相当 8 print("(1) 遅延処理クロージャの中の処理を開始") 9 completion("クロージャへ渡す文字列") 10 } 11 // (2)に相当 12 print("(2) クロージャ外の処理") 13} 14 15func completionFunction(_ msg: String) { 16 print("(4) 処理完了クロージャは関数でもいい") 17 print("渡された文字列は「(msg)」") 18} 19 20delayFunc(completion: completionFunction)

投稿2020/09/10 07:32

編集2020/09/10 10:53
TsukubaDepot

総合スコア5086

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Raleigh

2020/09/10 07:40

MVCでコードを記述したかったので、getElevationとmeasuringElevationはModelのファイルにあるのですがどうしたらいいでしょうか?
TsukubaDepot

2020/09/10 07:48

そういう意味だったんですね。失礼しました。 そうなると、Double? で json.elevation の結果を返すのではなく、ある程度処理したあと、最終結果だけ String? で返して、getElevation(locations, completionHandler:errorHandler:)では表示に専念させるほうがいいのかもしれません。
TsukubaDepot

2020/09/10 07:48

ちょっときちんと試してみたいので、Struct ElevationData の定義もご質問に追記していただけないでしょうか(これがあれば多分こちらでも動くアプリが作れると思います)。
Raleigh

2020/09/10 07:59

ややこしくてすいません。 追加しました。よろしくおねがいします。
TsukubaDepot

2020/09/10 08:58

Model の中でどの程度隠蔽を行いたいのかわからなかった部分もありますが、こちらで試した感じだとこのような感じになるかと思います(要は、細かい計算まで済ませて、文字列だけ返すようにしただけ)。 回答内のコードは全般的に書き換えてあります。 こちらのテストでは、GPSの座標は使わずに、いくつか座標を適当に入れてみて、それを公表されている結果と合わせてみましたが、概ね妥当な値がでてきるようです。
Raleigh

2020/09/10 09:29

できました!!!ありがとうございます。 自分自身なぜうまくいったのか理解が追いついていないので、これからもっと勉強しようと思います! コードの添削のようなこともしていただいて、頭が上がりません。 ちなみに以下のようなアプリを作っていて、MVCで書くことに苦戦しているところです。 https://twitter.com/QINIx9fgWSEuyF4/status/1300783075383672838?s=20
Raleigh

2020/09/10 10:07

最後になぜ自分のコードはelevationStringが思ったように更新されなかったか、理由を教えていただけると嬉しいです。
TsukubaDepot

2020/09/10 10:44

すごく抽象的な書き方をしますが、 let task = URLSession.shared.dataTask(with: url) { (data, response, error) in // (1) } // (2) における、(1)(2)の両者において、この場合は(1)->(2)の順番に処理が行われるとは限らないためです。 ネットワークの通信状態によっては、(2)->(1)のこともあり得ますし、どんな低遅延であっても、処理量を考えれば(1)->(2)となることはないかと思います。 すると、(1)の部分である変数にデータを書き込んだとしても、(2)の部分(たとえば return )は通り過ぎており、意図した通り実行することができないわけです。 なので、(1)の部分で行わせたい処理を「クロージャ」の形で外部から渡し、任意のタイミングで処理するようにしています。
TsukubaDepot

2020/09/10 10:55

簡単な例を追記しましたので、参考にしていただければとおもます。 アプリですが、面白そうですね。 とりあえず作ってみるのか、モデル化を突き詰めるのか、UIを極めるのか、選択も難しいと思いますが、目標があることはいいことだと思うので、成功するのを楽しみにしています。
Raleigh

2020/09/10 12:29

少しわかったような気がします! 今までクロージャの存在だけは知っていたものの、今回始めてその使い所が理解できました。 とても親切に回答してくださってありがこうございました。すごく助かりましたm(_ _)m アプリ開発大変ですががんばります。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問