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

Question

チャートを描くための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()
            
        }
    }
}

Comments

[fuzzball]

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

[swifty]

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

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

今の状態だと、RecordViewController側の

override func viewDidLoad() {
super.viewDidLoad()

直後でprint("test")

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

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

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

[fuzzball]

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

[swifty]

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

Answer 1

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

Comments

[swifty]

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

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

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

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

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

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

Answer 2

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

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

その上で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をコントロールしていくことができそうです。

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