Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix shape render timing #211

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

axion014
Copy link
Contributor

@axion014 axion014 commented Sep 9, 2017

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

@simiraaaa
Copy link
Contributor

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

@axion014
Copy link
Contributor Author

axion014 commented Sep 9, 2017

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

@simiraaaa
Copy link
Contributor

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

@axion014 axion014 force-pushed the axion014/rendertiming branch from 9f8d3c5 to dcfcf5e Compare September 19, 2017 14:28
@simiraaaa
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants