Improve performance

[uhooi]

概要

LegoColor(r:g:b:) の処理に時間がかかるため、rgb の値をキーとしてキャッシュするようにしました。

私のアイコンを StudType: Round ・ Max Stud: 128 で試したところ、処理時間が約 1/33 になりました。
同じ色を使っている画像ほど処理が速くなります。
逆にほぼ異なる色だと、キャッシュを捜査する分だけ処理が遅くなりますが、具体的な時間は計測していません。

処理時間の計測は colorMap 生成の前後に以下を仕込んで行いました。

+       let startDate = Date()
        let colorMap = (0 ..< Int(size.width * size.height))
            .map { i -> LegoColor in
                let r = CGFloat(rgbaData[4 * i]) / 255.0
                let g = CGFloat(rgbaData[4 * i + 1]) / 255.0
                let b = CGFloat(rgbaData[4 * i + 2]) / 255.0
                return LegoColor(r: r, g: g, b: b)
            }
+       print("Time: \(Date().timeIntervalSince(startDate))")

Before

Time
2.201616048812866
2.205173969268799
2.2091729640960693
2.2038110494613647
2.204527974128723

After

time
0.06664800643920898
0.06678497791290283
0.06787693500518799
0.06665492057800293
0.06649196147918701

[uhooi]

以下の画像で試したところ、Before と After で処理時間がほぼ変わりませんでした。
そのため本 PR でパフォーマンスが向上することがあっても、極端に低下することはなさそうです。

https://imgur.com/a/O9OyA
https://i.imgur.com/wvKAQUvh.jpg

[Kyome22]

コミットありがとうございます！
さらに軽量化の最適化に挑戦してみます。

[uhooi]

ぜひぜひ！
キーとなる RGB の組み合わせを減らせば、さらに高速になります！