円が重なっているかの判定がよくわかりません

Question

###前提・実現したいこと
円が衝突したら円の色が変わるプログラムの実行がうまくできません。どうしたらいいですか。

###発生している問題・エラーメッセージ

エラーメッセージ

###該当のソースコード
package l;

public class MyOval {
private int xPosition; //円の始点のx座標を保持するフィールド変数
private int yPosition; //円の始点のy座標を保持するフィールド変数
private int radius; //円の半径を保持するフィールド変数
private int colorRed; //円の色（赤の要素）を保持するフィールド変数
private int colorGreen; //円の色（緑の要素）を保持するフィールド変数
private int colorBlue; //円の色（青の要素）を保持するフィールド変数

//コンストラクタ
public MyOval(int x,int y,int radius1,int red,int green,int blue){

	xPosition=x;
	yPosition=y;
    radius=radius1;
	colorRed=red;
    colorGreen=green;
	colorBlue=blue;
}

//コンストラクタ
public MyOval(int x2,int y2,int radius2){
	xPosition=x2;
	yPosition=y2;
    radius=radius2;
    colorRed=0;
    colorGreen=0;
	colorBlue=255;
}

//円の始点のX座標を返す
public int getX(){
	return xPosition;
}

//円の始点のY座標を返す
public int getY(){
	return yPosition;
}

//円の半径を返す
public int getR(){
	return radius;
}

//赤色の要素を返す
public int getRed(){
	return colorRed;
}

//緑色の要素を返す
public int getGreen(){
	return colorGreen;
}

//青色の要素を返す
public int getBlue(){
	return colorBlue;
}

//赤色の要素を設定する
public void setRed(int red){
	colorRed=red;
}

//緑色の要素を設定する
public void setGreen(int green){
	colorGreen=green;
}

//青色の要素を設定する
public void setBlue(int blue){
	colorBlue=blue;
}

//円が重なっているかの判定
public boolean isCollision(MyOval MyOval){
	
	int x1=MyOval.getX();
	int y1=MyOval.getY();
	int r1=MyOval.getR();
	int x2=MyOval.getX();
	int y2=MyOval.getY();
	int r2=MyOval.getR();


	if((x1-x2)*2+(y1-y2)*2<=(r1+r2)*2){
		return true;
	}
		return false;
}

}

package p3;

import java.applet.Applet;
import java.awt.Color;
import java.awt.Graphics;

import l.MyOval;
public class Applett extends Applet{
public void paint(Graphics g){

	MyOval o2= new MyOval(30,20,80);//一つ目の円の設定
	
	//色が青なら
	if(o2.getBlue()==255){
		g.setColor(Color.blue);//青色を設定
	}
	
	g.fillOval(o2.getX(), o2.getY(), o2.getR(), o2.getR());//一つ目の円を描画
	
	
	MyOval o1= new MyOval(100,10,80,50,50,50);//二つ目の円の設定
	
	MyOval k = o2;
	
	//円が重なっている場合
	if(o1.isCollision(k)==true){
		g.setColor(Color.red);//赤色を設定
		g.fillOval(o1.getX(), o1.getY(), o1.getR(), o1.getR());//二つ目の円を描画
	}
	
	//円が重なっていない場合
	if(o1.isCollision(k)==false){
		g.setColor(Color.green);//緑色を設定
		g.fillOval(o1.getX(), o1.getY(), o1.getR(), o1.getR());//二つ目の円を描画
	}
}

}

###試したこと
課題に対してアプローチしたことを記載してください

###補足情報(言語/FW/ツール等のバージョンなど)
より詳細な情報

Comments

[kei344]

コードはコードブロックで囲んでいただけませんか？ ```（バッククオート3つ）で囲み、前後に改行をいれるか、コードを選択して「</>」ボタンを押すとコードブロックになります。

Answer 1

単純な高校数学ですね。
「2つの円が重なる」条件は、2つの円の中心同士の距離が、2つの円の半径の和よりも小さく、差よりも大きいことです。

「一方が他方に完全に取り込まれている」状態も考えるのであれば後半の「差より大きい」は不要です。

Answer 2

うまくいかないとはどううまくいかないのでしょう？
コードをちゃんと見たわけではありませんが、常に赤になってます？

int x1=MyOval.getX(); 
int y1=MyOval.getY(); 
int r1=MyOval.getR(); 
int x2=MyOval.getX(); 
int y2=MyOval.getY(); 
int r2=MyOval.getR();

チェック関数内で、同じオブジェクトからデータを取っているからじゃないでしょうか？
これだと x1=x2, y1=y2, r1=r2 ではないかと思います。
そうすると常にtureが返ると思います。