Processingの円の当たり判定

Question

球の当たり判定

キャラクターをマウスで動かして、ランダムな位置に置かれた円に触れたら、また別のランダムな位置に円を置くというプログラムを作成したいです。
操作する円を bullet 、ランダムに描かれる円を target としました。

疑問点は、

・if以下に打ち込むべきコードがわからない
・sqrt()を用いると良い とヒントが与えられているのですが、調べても求める検索結果が出てこないのでわからない

です

float targetX = random(380);
float targetY = random(380);
int manX;
int manY;
int random;
float r1; //bulletの半径
float r2; //targetの半径
float d = r1 + r2; 

void man() {
  fill(0);
  rect(manX, manY, 50, 30);
  rect(manX - 10, manY + 20, 70, 10);
  fill(255);
  rect(manX, manY + 10, 50, 10);
  rect(manX + 2, manY + 30, 46, 25);
  circle(manX + 20, manY + 40, 15);
  circle(manX + 45, manY + 40, 15);
}

void bullet() {
  fill(#FF0000);
  ellipse(manX, manY, 30, 30);
}

void target() {
  fill(#DCE83D);
  circle(targetX, targetY, 50);
}

void draw() {
  background(255);

  manX = 100 - 30;
  manY = 100 - 30;
  man();
  
  manX = mouseX;
  manY = mouseY;
  bullet();
  

  bullet();
  target();

  if ((targetX - mouseX) * (targetX - mouseX) + (targetY - mouseY) * (targetY - mouseY) <= (r1 + r2) * (r1 + r2)) {
    
  }
}

void setup() {
  size(480, 480);
}

Answer 1

まずr1とr2を初期化しましょう
この場合の初期化とは
**「変数に最初に具体的な値を代入する事」**です

例えばコードの1・2行目であなたは変数の宣言と初期化を同時にやっています

float targetX = random(380);
float targetY = random(380);

しかしr1とr2に関してはこれが行われている所がありません

またr1とr2は
bullet()とtarget()の関数定義の中で使用されていないので
このままだとr1とr2はそれぞれ半径としての役割を果たせません

上の2カ所を修正しさえすれば
あなたの書いた下の条件式は
書き直さなくてもあなたの要求を一応は満たします

(targetX - mouseX) * (targetX - mouseX) + (targetY - mouseY) * (targetY - mouseY) <= (r1 + r2) * (r1 + r2)

※この式は「距離」を比較する式ではなく「距離の2乗」を比較する式になっています

---

おたずねのif文の中に何を書けばいいかですが
単純にターゲットの座標を初期化すればいいでしょう
この場合の初期化は
**「今までの流れとは無関係に変数に新たな値を代入する事」**を意味します

---

ちなみに
様々なコーディングスタイルがあると思いますが
私は下のような感じでProcessingのコードを書いています

この形のほうが変数の初期化・変化など見通しが良くなると思うのですが......

int num = 120;//プログラム実行中、変化しない値はここで宣言+初期化
float a,b,c,d;//プログラム実行中、変化する値は宣言のみ

//setup 関数
//プログラムの開始に1回実行される
void setup()
{
        size(480, 480);
        
        //プログラム実行中、変化する値はここで初期化
        a = 10.0;
        b = 30.0;
        
        //初期化用の関数を使って初期化することもある
        reset();
}

//draw 関数
//setupの実行後、自動的に繰り返し実行される
void draw()
{
        background(255);
        
        c += 3.0;
        d += 3.0;
        func0();

        if(frameCount%num==0){
                func1();
                reset();
        }
        
        if(frameCount%(num*5)==0){
                setup();//setup 関数を使用すると強制的に最初の状態にすることができる
        }
}



//↓以降その他の関数定義
void reset()
{
        c =  20.0;
        d =  40.0;
}

void func0()
{
        ellipse(c,d,a,a);
        ellipse(c+2*a,d+2*a,b,b);
}

void func1()
{
        a +=  10.0;
        b +=  30.0;
}

Comments

[thkana]

私だったら、定数にはfinalを付けて変更不可にしちゃいます。あと、定数名は大文字(これはC系の文化かな?)。
final int NUM=12; //NUMの値は変更できない

Answer 2

プログラミングってのは検索の結果を写すものじゃないんですけど...

・if以下に打ち込むべきコードがわからない

「別のランダムな位置に円を置く」でいいんでしょ? 細かく言うと「別の」がちょっと引っかかるけど(ランダムというのは今の状況と関係なく新しい場所が決まるわけだから、本当にランダムにすると今と同じ位置になる可能性はわずかながらある)、
とりあえずプログラムの最初に「ランダムな位置に円を置く」をやっているのだから、それに準じた(そのまま、じゃダメかもね)処理をすればいいんじゃないですか?

・sqrt()を用いると良い とヒントが与えられているのですが

それが絡みそうなのは(targetX - mouseX) * (targetX - mouseX) + (targetY - mouseY) * (targetY - mouseY) <= (r1 + r2) * (r1 + r2)のところぐらいですけれど、これにsqrtを絡ませたところで当たり判定の結果に影響はないので、意味がないです。むしろ計算が増えるだけ無駄と言えましょう。Processingでは2点間の距離を求めるdist()という関数があるので、これを使えというならまだ理解できます。用意されている関数をわざわざ自分で再現するのは訓練以外には全く利がありません。