sakatore / graghview Goto Github PK

// GraghViewCellのレイアウトを決定するためのデータ
    struct GraghViewCellLayoutOptions {
        // MARK: Shared
        
        // cellAreaHeight / frame.height
        var barAreaHeightRate: CGFloat = 0.8
        // maxGraghValueRate / maxBarAreaHeight
        var maxGraghValueRate: CGFloat = 0.8
        // cell width
        var cellAreaWidth: CGFloat = 50
        // if over label is hidden
        var valueLabelIsHidden: Bool = false
        
        // MARK: Only Bar
        
        // bar.width / rect.width
        var barWidthRate: CGFloat = 0.5
        // Bar Color
        var barColor = UIColor.init(red: 1.0, green: 0.7, blue: 0.7, alpha: 1.0)
        // Label backgroundColor
        var labelBackgroundColor = UIColor.init(white: 0.95, alpha: 1.0)
        // Gragh backgroundColor
        var GraghBackgroundColor = UIColor.init(white: 0.9, alpha: 1.0)
        
        // MARK: Only Round
        
        // round size / cellAreaWidth
        var roundSizeRate: CGFloat = 0.1
        // round color
        var roundColor = UIColor.init(red: 0.7, green: 0.7, blue: 1.0, alpha: 1.0)
        // if round is hidden
        var onlyPathLine: Bool = false
        
        // MARK: Only Jaggy
        
        // jaggy color
        var jaggyColor = UIColor.init(red: 1.0, green: 1.0, blue: 0.6, alpha: 1.0)
        
        
    }
    
    // GraghViewCellsに付加するViewsのレイアウトを決定するためのデータ
    struct GraghLayoutOptions {
        // MARK: Comparison Value
        
        var comparisonLabelBackgroundColor = UIColor.lightGray.withAlphaComponent(0.7)
        var comparisonLineColor = UIColor.red
        var comparisonLineWidth: CGFloat = 1
        
        // MARK: Round Path
        
        var roundPathWidth: CGFloat = 2
        
        // MARK: Average Value
        var avarageLabelBackgroundColor = UIColor.init(red: 0.8, green: 0.7, blue: 1, alpha: 0.7)
        var averageLineColor = UIColor.init(red: 0.7, green: 0.6, blue: 0.9, alpha: 1)
        var averageLineWidth: CGFloat = 1
        
        
    }

これらの構造体はGraghViewの中に定義されてるから、少し抽象的な命名でいい気がする。
CellLayoutOptionsと LayoutOptionsでいい気がするな.cellって名前も...だけどそれは別issueで

UIColorの管理

参照Koyomi/UIColorExtension.swift

UIColorはhexで生成できるようなextensionを作成して定数として管理しておいたほうが、管理しやすい

sample

class Parent {
    var child = Child()
    
    func registerClosure() {
        child.callback = { number in
            print("Number \(number)")
        }
    }
}

class Child {
    var callback: ((Int) -> ())?
    
    var frame: CGRect = .zero
    
    func call() {
        callback?(3)
    }
}

let p = Parent()
p.registerClosure()
p.child.call()

マイナーversionリリース

今日中にひとまずリリースしたいです。

変数？定数

// GraghViewCellsに付加するViewsのレイアウトを決定するためのデータ
    struct GraghLayoutOptions {
        // MARK: Comparison Value
        
        var comparisonLabelBackgroundColor = UIColor.lightGray.withAlphaComponent(0.7)
        var comparisonLineColor = UIColor.red
        var comparisonLineWidth: CGFloat = 1
        
        // MARK: Round Path
        
        var roundPathWidth: CGFloat = 2
        
        // MARK: Average Value
        var avarageLabelBackgroundColor = UIColor.init(red: 0.8, green: 0.7, blue: 1, alpha: 0.7)
        var averageLineColor = UIColor.init(red: 0.7, green: 0.6, blue: 0.9, alpha: 1)
        var averageLineWidth: CGFloat = 1
        
        
    }

質問だけどここら辺は変数で可読性とかまとまりを持たせるために、stractでまとめてる感じ？

親子関係

いまの #22 だとcallbackパターンで、親のプロパティにアクセスしているけどちょっと微妙だから設計考えよう

親子関係の考え方

子への通知はchild.param = ..が一番簡潔。これを基本とする
親が持つ状態を使って子を扱う時は、

初期化時に渡せるpramは渡す
親のpramが更新されたタイミングで子のpramを更新するdidSetとかを使う。
関数と一緒に渡す

これを意識して設計を考えようか
#22 がマージされた後で

関数ジャンプ

// MARK: Only Bar

これ
//MARK: -
じゃないとジャンプできなくない？

refactoring

GraghView.swift
ここはまとめていい気がする

enumのcase

enum GraghViewContetOffset {
    case atMinimumDate, atMaximizeDate
}

でいい気がする

enum GraghViewContetOffset {
    case minimumDate, maximizeDate
}

framework

ライブラリ化しちゃえb

sakatore / graghview Goto Github PK

graghview's People

Contributors

Watchers

graghview's Issues

Recommend Projects

Recommend Topics

Recommend Org