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

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

ただいまの
回答率

88.31%

SwiftでJSONのデータを構造体に入れる際のエラー

受付中

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 821

pompom3

score 13

前提・実現したいこと

Swift初心者です.プログラミング経験はC言語のif文あたりまでの知識しかないのですが,急遽アプリを作らないといけなくなったので作成しています.Swiftの文法をあまり理解せずにプログラムを書いています.
サイトに書かれているプログラムをコピペしてなんとかやっている状態ですのでしょうもないミスをしているかもしれませんが,よろしくお願い致します.

ユーザの位置情報を取得し,その場所から最も近い場所にある駅に近づいた際に通知を送るアプリを作成しています.
ユーザの緯度と経度を取得し,APIにその情報を送るとレスポンスとしてその場所からの最寄駅の情報が返ってきます.
その情報の中に「distance=32m」というように現在地と最寄駅の直線距離の値が含まれています.
この値を取り出して使いたいのですが,その工程でエラーが出ています.

プログラム内の構造体JsonFeedItem内にある「name,prefecture」などがAPIから返ってくる最寄り駅の情報です.
この構造体からdistanceを取り出したいです.

let feed = try? decoder.decode(JsonFeed.self, from: Data)
の行の「decoder」の部分に「Cannot use instance member 'decoder' within property initializer; property initializers run before 'self' is available」のエラーが出ています.

エラー内容を検索しているのですがさっぱりです.
どうかよろしくお願いします.

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

Cannot use instance member 'decoder' within property initializer; property initializers run before 'self' is available

該当のソースコード

import UIKit
import CoreLocation

class LocationViewController: UIViewController, CLLocationManagerDelegate {

    struct JsonFeed: Codable {
        var response: JsonFeedItems
    }
    struct JsonFeedItems: Codable {
        var station: [JsonFeedItem]
    }
    struct JsonFeedItem: Codable {
        var name: String
        var prefecture: String
        var line: String
        var x: Float
        var y: Float
        var postal: String
        var distance: String
        var prev: String
        var next: String
    }

    //distanceを取り出す
    let decoder = JSONDecoder()
    let feed = try? decoder.decode(JsonFeed.self, from: data)
    let distance = feed.JsonFeedItems[0].JsonFeedItem.distance

    //distanceの単位を削除し数値のみにする
    var distance : String = ""
    var endPoint = distance.count(distance) - 1
    var kyori = (distance as NSString).substringToIndex(endPoint)


    @IBOutlet weak var latTextField: UITextField!
    @IBOutlet weak var lngTextField: UITextField!

    var locationManager: CLLocationManager!

    override func viewDidLoad() {
        super.viewDidLoad()

        if CLLocationManager.locationServicesEnabled() {
            locationManager = CLLocationManager()
            locationManager.delegate = self
            locationManager.startUpdatingLocation()
        }
    }

    override func viewDidDisappear(_ animated: Bool) {
        super.viewDidDisappear(animated)

        if CLLocationManager.locationServicesEnabled() {
            locationManager.stopUpdatingLocation()
        }
    }

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        // Dispose of any resources that can be recreated.
    }

    // MARK: - CLLocationManager delegate
    func locationManager(_ manager: CLLocationManager, didChangeAuthorization status: CLAuthorizationStatus) {
        switch status {
        case .notDetermined:
            locationManager.requestWhenInUseAuthorization()
        case .restricted, .denied:
            break
        case .authorizedAlways, .authorizedWhenInUse:
            break
        }
    }

    func locationManager(_ manager: CLLocationManager, didUpdateLocations locations: [CLLocation]) {
        guard let newLocation = locations.last,
            CLLocationCoordinate2DIsValid(newLocation.coordinate) else {
                self.latTextField.text = "Error"
                self.lngTextField.text = "Error"
                return
        }
        self.latTextField.text = "".appendingFormat("%.4f", newLocation.coordinate.latitude)
        self.lngTextField.text = "".appendingFormat("%.4f", newLocation.coordinate.longitude)

        let url = URL(string: String(format: "http://express.heartrails.com/api/json?method=getStations&x=%.4f&y=%.4f",newLocation.coordinate.longitude,newLocation.coordinate.latitude))
        var request = URLRequest(url: url!)
        // POSTを指定
        request.httpMethod = "POST"
        // POSTするデータをBodyとして設定
        request.httpBody = "os=iOS&version=11&language=日本語".data(using: .utf8)
        let session = URLSession.shared
        session.dataTask(with: request) { (data, response, error) in
            if error == nil, let data = data, let response = response as? HTTPURLResponse {
                // HTTPヘッダの取得
                print("Content-Type: \(response.allHeaderFields["Content-Type"] ?? "")")
                // HTTPステータスコード
                print("statusCode: \(response.statusCode)")
                print(String(data: data, encoding: .utf8) ?? "")

            }
            }.resume()

    }


}
    /*
    // MARK: - Navigation

    // In a storyboard-based application, you will often want to do a little preparation before navigation
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        // Get the new view controller using segue.destination.
        // Pass the selected object to the new view controller.
    }
    */

補足情報(FW/ツールのバージョンなど)

使用している環境
macOS Mojave 10.14.2
Xcode 10.1
Swift 4.2.1

利用しているAPIのサイト
http://express.heartrails.com/api.html#nearest
このサイトの最寄駅情報取得 APIを利用しています.

JSONのデータを構造体に入れるために参考にしたサイト
http://gtech.hatenablog.com/entry/2017/07/07/175238
このサイトの「JSONのデータを構造体に入れる」のところを参考にしています.

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

回答 1

0

//distanceを取り出す
let decoder = JSONDecoder()
let feed = try? decoder.decode(JsonFeed.self, from: data)
let distance = feed.JsonFeedItems[0].JsonFeedItem.distance

//distanceの単位を削除し数値のみにする
var distance : String = ""
var endPoint = distance.count(distance) - 1
var kyori = (distance as NSString).substringToIndex(endPoint)

などですが、

Swiftではクラスのメンバ変数の定義時に、selfを(暗黙的に)使った演算結果を初期値として格納することはできません。
(できる方法もありますが、一旦忘れてください。)

演算等はイニシャライザや各種関数内で適切なタイミングで行い、必要最低限の情報をメンバ変数へ格納するようにしてください。
また、メンバーでdistance変数を二回定義しようとしていますが、それもNGです。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

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

  • ただいまの回答率 88.31%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る