回答編集履歴

簡単な例を追記

2020/09/10 10:53

投稿

TsukubaDepot

スコア5086

test CHANGED Viewed

@@ -175,3 +175,103 @@
     }
 ```
+##追記
+今回の処理を簡素化したモデルです。
+Playground でも実行できますので、よかったら試してみてください。
+```Swift
+import UIKit
+func delayFunc(completion: @escaping (String) -> ()) {
+    DispatchQueue.main.asyncAfter(deadline: .now() + 1.0) {
+        // 1秒後に実行されるコード
+        // (1) に相当
+        print("(1) 遅延処理クロージャの中の処理を開始")
+        completion("クロージャへ渡す文字列")
+    }
+    // （2）に相当
+    print("(2) クロージャ外の処理")
+}
+delayFunc(completion: { (message) in
+    print("(3) 処理完了クロージャでの処理")
+    print("渡された文字列は「(message)」")
+})
+```
+クロージャだけでなく、引数の型と個数が合えば関数も渡せます。
+```Swift
+import UIKit
+func delayFunc(completion: @escaping (String) -> ()) {
+    DispatchQueue.main.asyncAfter(deadline: .now() + 1.0) {
+        // 1秒後に実行されるコード
+        // (1) に相当
+        print("(1) 遅延処理クロージャの中の処理を開始")
+        completion("クロージャへ渡す文字列")
+    }
+    // （2）に相当
+    print("(2) クロージャ外の処理")
+}
+func completionFunction(_ msg: String) {
+    print("(4) 処理完了クロージャは関数でもいい")
+    print("渡された文字列は「(msg)」")
+}
+delayFunc(completion: completionFunction)
+```

MVC に沿って変更

2020/09/10 10:53

投稿

TsukubaDepot

スコア5086

test CHANGED Viewed

@@ -1,4 +1,4 @@
-あるメソッドに引数として取られたクロージャからは、return を使ってメソッドの外側に値を返すことができないことはおそらくご存知かと思います。
+あるメソッドに引数として取られたクロージャからは、`return` を使ってメソッドの外側に値を返すことができないことはおそらくご存知かと思います。
@@ -12,13 +12,13 @@
 ```Swift
-getElevation(_ Locations: [CLLocation], completionHandler: @escaping (Double?) -> Void, errorHandler: (Error?) -> Void )
+getElevation(_ Locations: [CLLocation], completionHandler: @escaping (String?) -> Void, errorHandler: @escaping (Error?) -> Void )
 ```
-`completionHandler: `で与えるクロージャは、引数として `Double?`型が与えられます。処理内で正しい標高が得られなかった場合には `0.0` を返すようになっていましたが、実際は標高 `0.0` m という場所もあるので、`nil` を返したほうが現実的と思います。
+`completionHandler: `で与えるクロージャは、引数として `String?`型が与えられます。処理内で正しい標高が得られなかった場合には `"0.0"` を返すようになっていましたが、実際は標高 `0.0` m という場所もあるので、`nil` を返したほうが現実的と思います。
@@ -30,59 +30,39 @@
+- Controller
 ```Swift
-    func locationManager(_ manager: CLLocationManager, didUpdateLocations locations: [CLLocation]) {
-        getElevation(locations, completionHandler: { (elevation) in
+　　　　model.getElevation(locations, completionHandler: { (elevationString) in
-            if let elevation = elevation {
-                // 標高が得られたときの処理
-                self.afterElevation = elevation
-                self.measuringElevation()
-                self.elevationString = String(format: "%.2fm", self.gotElevation)
-                self.beforeElevation = self.afterElevation
-            } else {
-                // 何らかの原因で標高が取れなかった場合
-                self.elevationString = "0.0"
-            }
             DispatchQueue.main.async {
+                // ここでは View Controller にラベルに表示しているが、他のクラスのプロパティに書き込む場合でも考え方は同じ
-                self.customView.elevation = self.elevationString
+                //self.customView.elevation = elevationString
+                self.altitudeLabel.text = elevationString ?? "取得できませんでした"
             }
         }, errorHandler:  { (error) in
-            print(error?.localizedDescription)
+            print(error?.localizedDescription ?? "エラー情報なし")
         })
-    }
+```
-    func getElevation(_ Locations: [CLLocation], completionHandler: @escaping (Double?) -> Void, errorHandler: (Error?) -> Void ) {
+- Model
-        self.locations = Locations
+```Swift
+func getElevation(_ locations: [CLLocation], completionHandler: @escaping (String?) -> Void, errorHandler: @escaping (Error?) -> Void ) {
-        guard let coordinate = Locations.first?.coordinate else {
+        guard let coordinate = locations.first?.coordinate else {
             // 終了処理のクロージャに nil を渡す
@@ -94,9 +74,7 @@
-        // userLocaton = coordinate で良いのでは?
-        userLocation = CLLocationCoordinate2DMake(coordinate.latitude, coordinate.longitude)
+        let userLocation = CLLocationCoordinate2DMake(coordinate.latitude, coordinate.longitude)
@@ -104,21 +82,45 @@
-        if let userLocation = userLocation {
+        let lonUrl = "&lon=" + userLocation.longitude.description
-            let lonUrl = "&lon=" + userLocation.longitude.description
+        let latUrl = "&lat=" + userLocation.latitude.description
-            let latUrl = "&lat=" + userLocation.latitude.description
+        let outtypeUrl = "&outtype=JSON"
-            let outtypeUrl = "&outtype=JSON"
+        let listUrl = baseUrl + lonUrl + latUrl + outtypeUrl
+        print(listUrl)
+        guard let url = URL(string: listUrl) else {
+            // 終了処理のクロージャに nil を渡す
+            completionHandler(nil)
+            return
+        }
+        let task = URLSession.shared.dataTask(with: url) { (data, response, error) in
+            if error != nil {
+                // エラー処理のクロージャに error を渡す
+                errorHandler(error)
+                return
+            }
-            let listUrl = baseUrl + lonUrl + latUrl + outtypeUrl
-            guard let url = URL(string: listUrl) else {
+            guard let data = data else {
                 // 終了処理のクロージャに nil を渡す
@@ -130,47 +132,39 @@
-            let task = URLSession.shared.dataTask(with: url) { (data, response, error) in
+            // error を返せるように、do-try-catch で処理する
-                if error != nil {
+            do {
-                    // エラー処理のクロージャに error を渡す
+                let json = try JSONDecoder().decode(ElevationData.self, from: data)
-                    errorHandler(error)
-                    return
-                    //                print(error!.localizedDescription)
+                // 以下の処理は全体の文脈がわからないので、とりあえずコメントアウト
+//                self.afterElevation = json.elevation
-                }
+//                self.measuringElevation()
+//                self.beforeElevation = self.afterElevation
-                guard let data = data else {
+                // JSON で得られた座標を直接処理
-                    // 終了処理のクロージャに nil を渡す
-                    completionHandler(nil)
+                self.gotElevation = json.elevation
-                    return
-                }
-                // error を返せるように、do-try-catch で処理する
+                let elevationString = String(format: "%.2fm", self.gotElevation)
-                do {
-                    let json = try JSONDecoder().decode(ElevationData.self, from: data)
-                    completionHandler(json.elevation)
+                completionHandler(elevationString)
-                } catch {
+            } catch {
-                    errorHandler(error)
+                errorHandler(error)
-                }
             }