編集履歴

回答編集履歴

改行削除

2024/03/04 09:30

投稿

スコア10213

answer CHANGED Viewed

@@ -1,73 +1,72 @@
-大変素晴らしいコードなのですが、クラスが多くProcessingは内部クラスになる仕様上アクセス修飾子が意味がなくなるので、ちょっとコードが追いにくいですね。
-しかし順番に落ち着いて読んでいくと、案外すぐ目的のものに行き当たりました。
-まず要件をまとめると、
-1. どこでクリックしても拡散する
-2. クリックしたら描画されている円を全て拡散する
-3. クリックをしなくてもランダムな色の円がランダムな場所に描画され続ける
-ですね。
----
-**[注意]**
-これ以降同名のメソッドが多いので、本体？PApplet？直に置かれてるsetup()やdraw()をApp.setup()と書き、class Noteのdraw()をNote.draw() と書くこととします。
----
-1. 2.はクリックに関連するでしょうから、`App.mouseClicked()`を見ます。
-`NoteController.mouseClicked()`に任せています。
-ちゃんと読んでもいいんですが、きっと`find()`でnoteのindexを探し、なければ何もせずあれば`Note.toggle()`ってことでしょう。
-まずここで全部拡散なので、findする必要はなくなります。
-```Processing
-void mouseClicked() {
-  for (Note note : notes) {
-    note.toggle();
-  }
-}
-```
-実行します。
-はい、もう1.2.は終わってしまいました。
-実は問題は3.です。
-拡散した後一瞬でパララと10個円が出たと思いますが、追加のペースが遅いと全部消えた後寂しいですし、早すぎるとあっという間に画面中覆いつくされそうです。
-どういうイメージなのかがわからないので、調整ポイントを押さえて、
-細かい調整はmswさんに任せます。
-では追加部分を見てみます。
-起点はApp.draw()内の`controller.add(playerIn);`です。
-`NoteController.add()`を見ます。
-`if (notes.size() > 10) return;` 円の最大個数ですね。
-`if (playerIn < 20) return;` 音量が小さすぎる場合は無視するようです。
-`notes.add(new Note(x, y, playerIn));` 円を追加してるんでしょう。
-なるほど。ここは毎フレーム呼ばれるので拡散した後、一気に増えたわけですね。
-一気に増えないように、
-```Processing
-void add(final float playerIn) {
-  if (notes.size() > 100) return;//check
-  if (playerIn < 20) return;
-  if (frameCount % 30 != 0) return;
-  final float x = random(xmin, xmax);
-  final float y = random(ymin, ymax);
-  notes.add(new Note(x, y, playerIn));
-}
-```
-としてみましたが規則的過ぎていけませんね。
-`if (frameCount  % (int)random(10, 30) != 0) return;`私はこんなもんかな?と思いましたが、今ある円の個数によって変えるとか方法はいろいろありそうです。お好きなように調整してください。
-あとやってるうちに気が付いたんですが、一回拡散した後、円が水色系統だけになりませんか?
-そういう仕様ならいいのですがなんか色が減って寂しいので、どうにかならないかと試行錯誤したところ、`Note.draw()`
-```Processing
-void draw() {
-  noStroke();
-  colorMode(HSB, 360, 100, 100, 100);
-  fill(col);
-  ellipse(x, y, w, h);
-}
-```
+大変素晴らしいコードなのですが、クラスが多くProcessingは内部クラスになる仕様上アクセス修飾子が意味がなくなるので、ちょっとコードが追いにくいですね。
+しかし順番に落ち着いて読んでいくと、案外すぐ目的のものに行き当たりました。
+まず要件をまとめると、
+1. どこでクリックしても拡散する
+2. クリックしたら描画されている円を全て拡散する
+3. クリックをしなくてもランダムな色の円がランダムな場所に描画され続ける
+ですね。
+---
+**[注意]**
+これ以降同名のメソッドが多いので、本体？PApplet？直に置かれてるsetup()やdraw()をApp.setup()と書き、class Noteのdraw()をNote.draw() と書くこととします。
+---
+1. 2.はクリックに関連するでしょうから、`App.mouseClicked()`を見ます。
+`NoteController.mouseClicked()`に任せています。
+ちゃんと読んでもいいんですが、きっと`find()`でnoteのindexを探し、なければ何もせずあれば`Note.toggle()`ってことでしょう。
+まずここで全部拡散なので、findする必要はなくなります。
+```Processing
+void mouseClicked() {
+  for (Note note : notes) {
+    note.toggle();
+  }
+}
+```
+実行します。
+はい、もう1.2.は終わってしまいました。
+実は問題は3.です。
+拡散した後一瞬でパララと10個円が出たと思いますが、追加のペースが遅いと全部消えた後寂しいですし、早すぎるとあっという間に画面中覆いつくされそうです。
+どういうイメージなのかがわからないので、調整ポイントを押さえて細かい調整はmswさんに任せます。
+では追加部分を見てみます。
+起点はApp.draw()内の`controller.add(playerIn);`です。
+`NoteController.add()`を見ます。
+`if (notes.size() > 10) return;` 円の最大個数ですね。
+`if (playerIn < 20) return;` 音量が小さすぎる場合は無視するようです。
+`notes.add(new Note(x, y, playerIn));` 円を追加してるんでしょう。
+なるほど。ここは毎フレーム呼ばれるので拡散した後、一気に増えたわけですね。
+一気に増えないように、
+```Processing
+void add(final float playerIn) {
+  if (notes.size() > 100) return;//check
+  if (playerIn < 20) return;
+  if (frameCount % 30 != 0) return;
+  final float x = random(xmin, xmax);
+  final float y = random(ymin, ymax);
+  notes.add(new Note(x, y, playerIn));
+}
+```
+としてみましたが規則的過ぎていけませんね。
+`if (frameCount  % (int)random(10, 30) != 0) return;`私はこんなもんかな?と思いましたが、今ある円の個数によって変えるとか方法はいろいろありそうです。お好きなように調整してください。
+あとやってるうちに気が付いたんですが、一回拡散した後、円が水色系統だけになりませんか？
+そういう仕様ならいいのですがなんか色が減って寂しいので、どうにかならないかと試行錯誤したところ、`Note.draw()`
+```Processing
+void draw() {
+  noStroke();
+  colorMode(HSB, 360, 100, 100, 100);
+  fill(col);
+  ellipse(x, y, w, h);
+}
+```
 で色が戻りました。ここはそれでよいのか自信ありません。