search

phinajs / phina.js

phina.js is game library
http://phinajs.com
MIT License
308 stars 51 forks source link

fix shape render timing #211

Open axion014 opened 7 years ago

axion014 commented 7 years ago

fix for #170 superMethodで実装しています。このやり方に問題があるようなら、もう少し議論しましょう。

simiraaaa commented 7 years ago

superMethodはあまり速くないのであまり良い気はしませんね 自分の場合ですが、仮にやるならコードが汚くなっても、同じコードをそのまま持ってきます。

axion014 commented 7 years ago

速度… しかし、継承する意味が… PlainElementのdraw自体、何か別のメソッドにすべき？

simiraaaa commented 7 years ago

PlainElementのdraw内に_draw等で抽象メソッド用意しておいて、そこで描画させるようにするとかが現状での最適実装かと思います

simiraaaa commented 6 years ago

今実装よりはかなりスッキリしたと思います