回答編集履歴

あ

2022/09/18 18:19

投稿

TN8001

スコア9855

test CHANGED Viewed

@@ -1,201 +1,101 @@
 課題ということなのでヒントにとどめます。
 ヒントが足りなければ追記しますので、コメントしてください。
 > pngのRGBはイラレでしっかり指定して作ってあります。
-添付画像が`pic.png`でればどう見ても、`color(255,0,0)`や`color(0,0,255)`はありません。
+添付画像が`pic.png`であれば、どう見ても`color(255,0,0)`や`color(0,0,255)`はありません。
 ```Processing
 println(hex(get(mouseX, mouseY), 6));
 ```
 とすると16進でff0000のように表示して確認できます。
 `color(255,0,0)`の代わりに`#ff0000`のような表記もできるので、if文を画像の色に合わせましょう。
 [hex() \ Language (API) \ Processing 3+](https://processing.org/reference/hex_.html)
 [color \ Language (API) \ Processing 3+](https://processing.org/reference/color_datatype.html)
 > 下手に書き換えられず宣言している変数で使っていない変数もあります。
 （イライラ棒の跡形もないので）もうすでにかなり書き換えてると思うのですが、このコード量なら変数を使っているかどうかは一目では？
 例えば`final int STANDBY = 0;`を削除します。
 もしどこかで使われている変数なら、実行ボタンを押したときにエラーになります。
 エラーになったら元に戻せばいいだけです。
 > こうした方がスッキリするなどのご意見も欲しいです。
 それ以前にやりたいことにコードがなっていません。
 ```Processing
 startTime = millis();
 finishTime= millis()-startTime;
 ```
 これはどういうつもりで書いたのでしょう？
 スタートするのは赤ボタンを押したときですよね？
 `setup()`（プログラム開始）時の値をとっても仕方ありません。
 `finishTime`も`millis()-millis()`になるわけですから結局０です。
 ---
 ```Processing
 text((millis()-startTime)*0.001,256,260);
 ```
 ここの意図もわかりません。赤を押したときに何か表示するんですか？
 ---
 ```Processing
 text(finishTime*0.001,256,260);
 ```
 ここの気持ちはわかります。
 しかし`finishTime`は０です。青ボタンを押したとき（if文の中）に、`finishTime`を計算する必要があります。
 あと2点
 文字色は今のままでは白です。
 [text() \ Language (API) \ Processing 3+](https://processing.org/reference/text_.html)
 [fill() \ Language (API) \ Processing 3+](https://processing.org/reference/fill_.html)
 `mousePressed()`で文字を書きますが、すぐ`draw()`に入ってしまい画像で上書きされて消えてしまいます。
 簡単にやるならループを止めることです。
 [noLoop() \ Language (API) \ Processing 3+](https://processing.org/reference/noLoop_.html)
 ---
 全体コードを追記
 `_`になっている部分を埋めてください。
 ```Processing
 int startTime;
 PImage pic;
 void setup() {
   size(512, 512);
   pic = loadImage("pic.png");
   textSize(50);
   fill(0); // 塗りつぶし色（文字色）黒
 }
 void draw() {
   image(pic, 0, 0);
 }
 void mousePressed() {
   println(hex(get(mouseX, mouseY), 6)); // 717272 や FFFFFF 等
   if (get(mouseX, mouseY) == #______) { // 赤ボタンを押したとき #FF0000 等確認した色を入れる
     startTime = millis(); // startTimeを記録
     loop(); // 再チャレンジ時のためにループを再開
   }
   if (get(mouseX, mouseY) == #______) { // 青ボタンを押したとき #0000FF 等確認した色を入れる
     // 先に赤ボタンを押してなければ青ボタンは押せない（赤を押したときに変えた変数は？）
     if(_________ == 0) return;
     // finishTimeの計算（finishTimeはここでしか使わないのでローカル変数でよい）
     int finishTime = millis() - startTime;
     text(finishTime * 0.001, 256, 260); // 位置が全然合ってない
     noLoop(); // ループを止める
   }
 }
 ```

インデント・スペース

2021/02/02 16:05

投稿

TN8001

スコア9855

test CHANGED Viewed

@@ -168,12 +168,14 @@
   if (get(mouseX, mouseY) == #______) { // 赤ボタンを押したとき #FF0000 等確認した色を入れる
-  startTime = millis(); // startTimeを記録
+    startTime = millis(); // startTimeを記録
     loop(); // 再チャレンジ時のためにループを再開
   }
   if (get(mouseX, mouseY) == #______) { // 青ボタンを押したとき #0000FF 等確認した色を入れる
@@ -182,11 +184,11 @@
     if(_________ == 0) return;
     // finishTimeの計算（finishTimeはここでしか使わないのでローカル変数でよい）
-    int finishTime = millis() - startTime;
+    int finishTime = millis() - startTime;
     text(finishTime * 0.001, 256, 260); // 位置が全然合ってない

全体コードを追記

2021/02/02 16:05

投稿

TN8001

スコア9855

test CHANGED Viewed

@@ -117,3 +117,83 @@
 簡単にやるならループを止めることです。
 [noLoop() \ Language (API) \ Processing 3+](https://processing.org/reference/noLoop_.html)
+---
+全体コードを追記
+`_`になっている部分を埋めてください。
+```Processing
+int startTime;
+PImage pic;
+void setup() {
+  size(512, 512);
+  pic = loadImage("pic.png");
+  textSize(50);
+  fill(0); // 塗りつぶし色（文字色）黒
+}
+void draw() {
+  image(pic, 0, 0);
+}
+void mousePressed() {
+  println(hex(get(mouseX, mouseY), 6)); // 717272 や FFFFFF 等
+  if (get(mouseX, mouseY) == #______) { // 赤ボタンを押したとき #FF0000 等確認した色を入れる
+  startTime = millis(); // startTimeを記録
+    loop(); // 再チャレンジ時のためにループを再開
+  }
+  if (get(mouseX, mouseY) == #______) { // 青ボタンを押したとき #0000FF 等確認した色を入れる
+    // 先に赤ボタンを押してなければ青ボタンは押せない（赤を押したときに変えた変数は？）
+    if(_________ == 0) return;
+    // finishTimeの計算（finishTimeはここでしか使わないのでローカル変数でよい）
+    int finishTime = millis() - startTime;
+    text(finishTime * 0.001, 256, 260); // 位置が全然合ってない
+    noLoop(); // ループを止める
+  }
+}
+```