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

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

ただいまの
回答率

89.20%

クラスのプロパティを通信処理で得た値を使って定義する方法

解決済

回答 2

投稿

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

swifty

score 36

チャートを描くためのChartViewというクラスを定義しています。
その中の変数 lastSevenRecords の中に同じクラス内で定義されたファンクション createData(completion: @escaping ([RecordData]) -> Void) 内の通信で得た結果を渡してからChartViewクラスを読み込んでもらいたいです。

その場合はどのようにすればよいでしょうか。

import Foundation
import Macaw
import Firebase
import FirebaseFirestore

class ChartView: MacawView {

    static var lastSevenRecords: [RecordData] = []
    static let maxValue = 20
    static let maxValueinHeight = 20
    static let lineWidth: Double = 385

    static let dataDivisor = Double(maxValue/maxValueinHeight)
    static let adjustedData: [Double] = lastSevenRecords.map({ Double($0.score) / dataDivisor })
    static var animations: [Animation] = []

    required init?(coder aDecoder: NSCoder) {
        super.init(node: ChartView.createChart(), coder: aDecoder)
        backgroundColor = .clear

    }

    private static func createChart() -> Group {
        var items: [Node] = addYAxisItems() + addXAxisItems()
        items.append(createBars())

        return Group(contents: items, place: .identity)
    }

    private static func addYAxisItems() -> [Node] {
        let maxLines = 10
        let lineInterval = Int(maxValue/maxLines)
        let yAxisHeight: Double = 600
        let lineSpacing: Double = 60

        var newNodes: [Node] = []

        for i in 1...maxLines {
            let y = yAxisHeight - (Double(i) * lineSpacing)
            let valueLine = Line(x1: -5, y1: y, x2: lineWidth, y2: y).stroke(fill: Color.black.with(a: 0.10))
            let valueText = Text(text: "\(i * lineInterval)", font: Font(name: "Avenir-Light", size: 14), align: .max, baseline: .mid, place: .move(dx: -10, dy: y))
            valueText.fill = Color.black


            newNodes.append(valueLine)
            newNodes.append(valueText)
        }

        let yAxis = Line(x1: 0, y1: 0, x2: 0, y2: yAxisHeight).stroke(fill: Color.black.with(a: 0.25))
        newNodes.append(yAxis)

        return newNodes
    }

    private static func addXAxisItems() -> [Node] {
        let chartBaseY: Double = 600
        var newNodes: [Node] = []

     //ここでエラーが出てしまいます
        for i in 1...adjustedData.count {
            let x = (Double(i) * 50)
            let valueText = Text(text: lastSevenRecords[i - 1].recordDate!, font: Font(name: "Avenir-Light", size: 14), align: .max, baseline: .mid, place: .move(dx: x, dy: chartBaseY + 15))
            valueText.fill = Color.black
            newNodes.append(valueText)
        }

        let xAxis = Line(x1: 0, y1: chartBaseY, x2: lineWidth, y2: chartBaseY).stroke(fill: Color.black.with(a: 0.25))
        newNodes.append(xAxis)

        return newNodes
    }

    private static func createBars() -> Group {
        let fill = LinearGradient(degree: 90, from: Color(val: 0xff4704), to: Color(val: 0xff4704).with(a: 0.33))
        let items = adjustedData.map { _ in Group() }

        animations = items.enumerated().map { (i: Int, item: Group) in
            item.contentsVar.animation(delay: Double(i) * 0.1) { t in
                let height = adjustedData[i] * t * 30
                let rect = Rect(x: Double(i) * 50 + 25, y: 600 - height, w: 30, h: height)

                return [rect.fill(with:fill)]
            }
        }

        return items.group()
    }

    static func playAnimations() {
        animations.combine().play()

    }


    static func createData(completion: @escaping ([RecordData]) -> Void) {

     //こちらで通信処理を行いDBからデータを取得する
       //↓こちらのrecordDataArrayが得られて使いたいデータ
     completion(recordDataArray)

        }
    }
}

そのために以下のようにRecordViewControllerのviewDidLoad()でCompletion Handlerで得た値を渡そうと試みました。ですがChartViewクラスはこちらのviewDidLoad()に来る前に既に読み込まれてしまっていて lastSevenRecords の配列が空のまま処理を進めてしまっているので lastSevenRecords を使って定義されたadjustedDataが使われる部分で

Thread 1: Fatal error: Can't form Range with upperBound < lowerBound

とエラーが出てしまいます。ChartViewクラスが読み込まれる段階で通信結果を反映させるにはどのようにすればよいのでしょうか?

import UIKit

class RecordViewController: UIViewController {

    @IBOutlet private var chartView: ChartView!

    override func viewDidLoad() {
        super.viewDidLoad()

        chartView.contentMode = .scaleAspectFit
        ChartView.createData { recordDataArray in

            //Completion Handlerで得られた値(recordDataArray)をChartView内のlastSevenRecordsへ
            ChartView.lastSevenRecords = recordDataArray
            ChartView.playAnimations()

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • fuzzball

    2019/07/01 09:57

    通信処理が終わってからcreateChart()を実行すればいいような。
    y_waiwaiさんの回答と意味合いは同じです。
    処理順を変えればいい話に見えます。

    キャンセル

  • swifty

    2019/07/01 18:16

    ご回答ありがとうございます。

    おっしゃる通りでそれが実現したいです。

    今の状態だと、RecordViewController側の

    override func viewDidLoad() {
    super.viewDidLoad()

    直後でprint("test")

    としてもそこまでいかないChartViewの段階で上記のようにエラーが出てしまうのでChartView側の動きの時系列をコントロールできずどうすればいいのだろうかと困ってしまっております。

    通信処理が終わってからcreateChart()を実行するには具体的にどのように書けば可能なのでしょうか?

    質問ばかりで大変恐縮ですが引き続き何卒よろしくお願いいたします。

    キャンセル

  • fuzzball

    2019/07/01 18:37

    createChart() の呼び出しタイミング(initの中)を変えられないのであれば、y_waiwaiさんの回答のように生成タイミングを変えるしかなさそうな。

    キャンセル

  • swifty

    2019/07/01 21:09

    お世話になっております。クラスの生成タイミングを変えたいと思うのですがどのようにしてかえればよいのでしょうか?やはりRecordViewControllerの記述をするということではないのでしょうか?いろいろ試してみておりますが思った挙動を実現できません。

    キャンセル

回答 2

checkベストアンサー

+1

通信結果を取得してからそのクラスを生成してはどうでしょう

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/07/01 18:07

    ご回答ありがとうございます。teratailでの初めての回答なので涙が出そうになるほど嬉しいです。

    「通信結果を取得してからそのクラスを生成する」というのはRecordViewControllerのcreateDataのクロージャー内でChartViewクラスのインスタンスを生成するということでしょうか?

    そうだとすると私なりに同じことを考えて、RecordViewControllerのcreateDataのクロージャー内でChartView.lastSevenRecordsにrecordDataArrayを渡してみました。でもこういった方法ではないということでしょうか?

    普通クラスのインスタンスを生成する際は

    let chartView = ChartView()
    としてから
    chartView.〇〇
    でクラス内にアクセスしますが今回はChartView内の変数やファンクションがstaticで定義されているためその過程を踏まずに作られていることが何か鍵になっているのでしょうか?

    引き続き何卒よろしくお願いいたします。

    キャンセル

0

みなさまご回答ありがとうございました。

只今無事みなさまのご回答を参考にした上で自己解決することができました。

その上でy_waiwaiさんが提案してくださった

「通信結果を取得してからそのクラスを生成してはどうでしょう」

ということについて徹底的に考えた結果答えが出せましたのでベストアンサーにさせていただきました。

ポイントになったのはStoryboardを使わずコードでオリジナルViewのサブクラスを生成し、その後そのViewを編集し直して表示するということでした。

Firestoreからデータを取得してCompletion Handlerを使って渡すという私自身の考え方とやりかたが間違っていたわけではなく、今回の問題はそちらではありませんでした。

とにかくその前段階としてオリジナルのchartView(取得したデータを使ったchartView)を生成する前にStoryboardで作っていたchartViewによりChartを表示したいページに行くと自動でそのChartViewクラスのイニシャライザが作動してしまい、Chart描画のための準備に入ってしまっており、その段階では取得したデータはまだ存在していないのでエラーになってしまっていました。

そのためChartViewクラスの初期化設定を最初にCGRect情報さえいれれば作れる以下のように修正し、データ取得後、得た情報を使って初期化時仮に作っておいたchartViewを書き換えるという作業をRecordViewController側ですることで解決しました。

    override init(frame: CGRect) {
        super.init(frame: frame)
        backgroundColor = .clear

    }

    @objc required convenience init?(coder aDecoder: NSCoder) {
        fatalError("init(coder:) has not been implemented")   
    }

今回の問題でUIViewのサブクラスの生成について深く学ぶことができました。
これで今まで以上にオリジナルのUIViewをコントロールしていくことができそうです。

みなさまのご協力ありがとうございました。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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