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

[変更予定?] 塗り面積の集計方法の変更 #7

Open
kotmw0701 opened this issue Oct 24, 2017 · 2 comments
Open

[変更予定?] 塗り面積の集計方法の変更 #7

kotmw0701 opened this issue Oct 24, 2017 · 2 comments

Comments

@kotmw0701
Copy link
Owner

kotmw0701 commented Oct 24, 2017

旧集計方法(/////で囲まれた範囲)
旧集計方法だと、戦闘終了時に一斉にステージ全体をfor分で取得して集計を行っていたため、戦闘中の負荷は発生しないが、終了時に負荷が掛かる場合がある(総面積によって差がある)

新集計方法
このメソッドをPaint.java#PaintWool(PlayerData data, Block block)メソッドから呼び出して実行、終了時の負荷はそこまで発生しないが、戦闘時の負荷が半端ないと思われる。

ちょっと誰かに聞いて意見を求めようかな(´-ω-`)

@kotmw0701
Copy link
Owner Author

新集計方法で自分が負荷等を危惧している理由として挙げられるのは

  • 呼び出されるたび(塗られるたび)にMapから元値を取り出し+書き込み
  • カウントの加算減算のタイミングでの競合(?)

なにせ1ブロック辺り毎回これを行う訳だから心配点が多い・・・

@kotmw0701
Copy link
Owner Author

後者の方で塗られるたび更新というのに実装(変更)理由として、どっちが優勢かをUIで表示出来るようにしたかったってのもあるから、塗られるたび更新のやり方で今のMapを使う方法以外に何かあるかなってのも思ってること

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

No branches or pull requests

1 participant