usatie/fractal

ctxのリファクタ

Opened this issue · 2 comments

そもそも何が問題?
書き出してみよう

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