ctxのリファクタ
Opened this issue · 2 comments
usatie commented
そもそも何が問題?
書き出してみよう
usatie commented
-
put_ctx_to_window
は他と揃えるならput_config_to_window
かな - メソッド名のブレ
ctx_xxx
vsxxx_ctx
vsxxx_ctx_xxx
- メンバ変数が多すぎるので、さらに意味単位での構造体にまとめたい
- config
- doubleの値を持っているのは精度的には微妙なので、intの値から都度計算するようにしたい(intの値をキーにしてcacheするのはアリ)
-
win_mouse_pnt
,mouse_pnt
,o
,step
などは、表したい情報に重複がある -
step
とo
があれば事足りるはずだが、実際にはzoomをするためにはマウスがどこにあるかの情報も必要。しかし、マウスがどこにあるかの情報からmouse_pntは計算可能なため、持っておく必要はないかも。 - colorとjuliaの回転に関しては、normalとpsychedelicの2つしかないならboolでもいい
- mlx_ptr, win_ptr, fractal_img, config_clear_imgは一つにまとめられるかも
usatie commented
- on_ctx_updateは本来setterの最後に書けたらいいんだけど(SwiftでいうdidSet)、特にそういう機構はないのでやめた方がいいかも。使うたびに呼び出した方が良さそう。