エラー解消相談： ExamGMousePressAndReleaseはabstractでなく、MouseMotionListener内のabstractメソッドmouseMoved(MouseEven

Question

前提・実現したいこと

javaのappletviewerでマウスドラッグの最初の位置から現在の位置を対角線とする矩形を書こうとしています。
コンパイル時にエラーが出ますが理解できず解消ができません。

MousePressedでP1、public void MouseDraggedでP2にポイントの座標を取り込み、paintの内部で処理を行い矩形を描くつもりです。

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

.\ExamGMousePressAndRelease.java:6: エラー: ExamGMousePressAndReleaseはabstractでなく、MouseMotionListener内のabstractメソッドmouseMoved(MouseEvent)をオーバーライドしません
public class ExamGMousePressAndRelease extends Applet implements MouseMotionListener {

該当のソースコード

import java.awt.*;
import java.awt.geom.*;			
import java.awt.event.*;	
import java.applet.*;

public class ExamGMousePressAndRelease extends Applet implements MouseMotionListener {
Point p1;//マウスドラッグ１
Point p2;//マウスドラッグ２

	public void init(){
		addMouseMotionListener(this);
	}		
		public void paint(Graphics g){
			super.paint(g);
			int x;
			int y;
			int l;
			int w;

			if(p1.x>=p2.x){
				x=p1.x;
			}
			if(p1.x<p2.x){
				x=p2.x;
			}
			if(p1.y>=p2.y){
				y=p1.y;
				}
			if(p1.y<p2.y){
				y=p2.y;
			}
			w=Math.abs(p1.x-p2.x);
			l=Math.abs(p1.y-p2.y);

			g.drawRect(x,y,l,w);
			}

	public void MousePressed(MouseEvent e){
		p1 = e.getPoint();
		repaint();
	}
	public void MouseDragged(MouseEvent e) {
		p2 = e.getPoint();
		repaint();
	}

}

Answer 1

MouseMotionListenerはInterfaceであり、そのInterface内で定義されたメソッドがオーバーライドされていないとコンパイルエラーが起こります。

MouseMotionListenerには

void mouseMoved(MouseEvent event);
void mouseDragged(MouseEvent event);

この2つのメソッドが定義されています。

しかし、ExamGMousePressAndRelease内ではどちらもオーバーライドされていません。（両方の頭文字が大文字になっている）
これらのメソッドの頭文字を小文字にすれはエラーは無くなりますが、まずはメソッドのオーバーライド時に@Overrideアノテーションを付けるようにしましょう。
ミスも減りますし、間違いにも気付きやすくなります。

Comments

[umintyu00]

回答ありがとうございます。オーバーライドというのは'ExamGMousePressAndRelease'内で'MouseMotionListener'Interfaceを適用させるという解釈であっているでしょうか。
またおっしゃる通りに編集したところ'''.\ExamGMousePick.java:7: エラー: ExamGMousePickはabstractでなく、MouseListener内のabstractメソッドmouseExited(MouseEvent)をオーバーライドしません'''
とエラーになってしまいました。
javaの知識が足りておらず回答を理解できずすいません。

[退会済みユーザー]

「適応」という曖昧な表現では、あなたがオーバーライドのことを理解されているかどうかは判断できません。
理解できていないことを前提に説明すると、「本来のメソッドの役割を上書きすること」です。
さらにInterfaceのメソッドには中身が元々無いので、実質的に動作の定義をしているのと同じです。
今回の場合は、「マウスが動いたらどうするか」を定義しています。

エラーに関してですが、
「編集したら別のエラーが出た」のではなく
「元々ダメだったがコンパイラは見てもいなかった」ということです。
コンパイラがExamGMousePressAndReleaseのエラーを見つけた時点で、それ以上コードを見る意味がなくなっていたのです。

そしてExamGMousePickのエラーですが、質問にあるエラーと全く同じです。この場合はMouseMotionListenerではなくMouseListenerなので、
void mouseClicked(MouseEvent event)
void mousePressed(MouseEvent event)
void mouseReleased(MouseEvent event)
void mouseEntered(MouseEvent event)
void mouseExited(MouseEvent event)
の5つの定義があります。
エラーメッセージではmouseExitedがオーバーライドできていないと言われていますが、先述のようにコンパイラはエラーを見つけた時点で終了します。たまたま最初にmouseExitedが無いことに気づいただけの可能性もあるので、ご自分でソースコードを確認してください。実際のコードがないためこれ以上わかることはありません。

[umintyu00]

わかりやすく説明してくれてありがとうございます。Interface内のメソッドをExamGMousePick内ですべて動作の定義をしオーバーライドしなければならないのですね。

「○○はオーバーライドしません」というエラーが「○○オーバーライドしてないからしろ」というエラーメッセージだったということの理解であっていますか？

Interfaceを複数継承することも初めてで継承の部分にも間違いを見つけました。また、MouseListener内のすべてのメソッドをオーバーライドし、ほかにもいくつか修正したところ、無事完成させることができましたため、解決方法にソースを張らせていただきます。

大変ありがとうございました。

[退会済みユーザー]

エラーメッセージの解釈は合っています。おそらくEclipseを使っているのだと思いますが、日本語で使っているとエラーメッセージも変な日本語で出てしまいます。プログラミング全般に言えることですが、IDE等のエディタ類は日本語で使用しないことを推奨します。（あくまでも私の主張です）

Answer 2

・Interfaceを複数用いるときはメソッドの定義が抜けないようにすべて行うこと
・InterfaceをAとBを用いるときimplements A,Bで継承
osaru2002729さんありがとうございました。

以下ソースコードとなります。

// <APPLET  CODE = ExamGMousePick.class WIDTH=300 HEIGHT=300>
import java.awt.*;
import java.awt.geom.*;
import java.awt.event.*;
import java.applet.*;

public class ExamGMousePick extends Applet implements MouseMotionListener,MouseListener{　//MouseMotionListener,MouseListenerの二つを継承するとき ,(コンマ)　で並べる
	Point p,p1,p2;
	int r,flag;
	int x1,x2,xx,y1,y2,yy;
	int xa,ya;
	public void init() {
		addMouseMotionListener(this);
		addMouseListener(this);
		
		
		r = 50; //円半径
		xx = 150; //初期中心
		yy = 150; //初期中心
		

	}
	public void paint(Graphics g){
		g.drawString("programmed by kaito abe",10,10);
		g.setColor(Color.black);

		g.fillOval(xx-r, yy-r, 2*r, 2*r); //初期円描画
		
	
		if(flag==1){
			xx=x2;　//円の中心座標更新
			yy=y2;
			flag=1;
			}
	

		g.drawString("flag="+flag,10,20);
		}
		
					
			//g.fillOval(xx-r, yy-r, 2*r, 2*r);
	@Override
	public void mousePressed(MouseEvent e){
		p1 = e.getPoint();
		x1 = p1.x;
		y1 = p1.y;
			if(((x1-xx)*(x1-xx))+(((y1-yy))*(y1-yy)) <r*r){
				flag=1; //円の外に押されると円の座標を更新するようにする
			}
			else
				flag=0;
		repaint();

	}
	@Override
	public void mouseDragged(MouseEvent e){
		p2 = e.getPoint();
		x2 = p2.x;
		y2 = p2.y;
		repaint();
	}

	@Override
	public void mouseMoved(MouseEvent e){
	}
	@Override
	public void mouseExited(MouseEvent e){}
	@Override
	public void mouseEntered(MouseEvent e){}
	@Override
	public void mouseReleased(MouseEvent e){}
	@Override
	public void mouseClicked(MouseEvent e){}



}