アップレットのアクションリスナーが動作しません

質問

• アップレットにアクションリスナーをつけて、アップレットとキャンバスにアクションが来るようにコードを書いたが、動作しない。
• mouseClickedで四角を表示するが、最初に表示される。mouseEnteredでも四角を表示しない。
• "cs.paint(g);"の書き方が悪いのか、"gを解決できません"と表示される。

回答No.7

ngsvx · ベストアンサー

＞実装してやると「インターフェイスのメソッドが重複しています」とのエラーが出ます。

＃5のお礼に記述されたコードに実装してもそのようなメッセージは出ないと思います。
すでにこのコードとはかなり違うものになっていませんか？
違うコードになっていると、こちらからは適切なアドバイスはできません。

＃2のお礼にあるコードに＃3の回答を修正したものが、＃4でうまくいっているようなので、
そこから、やり直したほうが早いかもしれません。

＞#　このような書き方は、具体的にはどのように書くのでしょうか。
　また、プログラムの書き方としては良くないというのはどのような理由なのでしょうか。

これはケースバイケースなのですが、今回はやめたほうがいいと思います。
この方法では、Canvasで発生したイベントを、Appletで処理することになります。
すると、CanvasとAppletの関係が密となり、切り離せない関係となります。
そうすると、プログラムの変更/修正などがしにくいプログラムとなってしまうのが理由です。
クラスはなるべく関係性を疎にしておいたほうが応用が利きやすいのです。

最初の方法、つまりCanvasで発生したイベントをCanvasで処理するということは、
処理がCanvasで完結し、Canvasの独立性が保たれます。

さて、具体的な方法ですが、

イベントのリスナーに定義されるメソッドの引数は、EventObjectを継承したものですので、getSource()というメソッドで、イベントを発生させたオブジェクトを取得できます。

これを使って、処理を振り分けます。
public void mouseClicked(MouseEvent e)

{
　Object sourceObject = e.getSource();
　if( sourceObject == canvasObject ){
　　//Canvas用の処理

　}
　if( sourceObject == appletObject ){
　　//Applet用の処理

　}

}

なお、「canvasObject 」「appletObject 」は、インスタンス変数としておくことが必要です。

お礼

回答有難う御座います。
　何とかうまく動くようになりました。

　インスタンス変数を作ってイベントの振分けの方法も勉強しているのですが、その方法が分かりません。

　下のように書いてやりましたが、うまく実行できません。
　宜しくお願いします。

===================================================================

public class ActionTest6 extends Applet implements MouseListener
{
Canvas cs ;
boolean flag = false ;
Graphics g ;
myCanvas mc ;
boolean flag2 = false ;

public void init()
{
this.setBackground(Color.pink);
this.addMouseListener( this ) ;

myCanvas mc = new myCanvas( this , flag2) ;
this.addMouseListener( mc ) ;
this.add( mc ) ;
}

public void paint( Graphics g )
{
if( flag == true )
{
g.fillRect( 50 , 50 , 10 , 10 ) ;
}
}

public void mouseClicked( MouseEvent e )
{
Graphics g = getGraphics() ;
setBackground( Color.cyan ) ;
flag = true ;
repaint();
}

public void mouseEntered( MouseEvent e )
{
Object sourceObject = e.getSource();
if( sourceObject == mc )
{
flag2 = true ;
}
}

public void mouseExited(MouseEvent arg0) { }
public void mousePressed(MouseEvent arg0) { }
public void mouseReleased(MouseEvent arg0) { }

class myCanvas extends Canvas implements MouseListener
{
ActionTest6 a ;
boolean fc = false ;

public myCanvas( ActionTest6 a , boolean f)
{
this.a = a ;
this.fc = f ;
setBackground( Color.green ) ;
setSize( 100 , 100 ) ;
addMouseListener( this ) ;
}

public void paint( Graphics g )
{
g.setColor( Color.black ) ;
g.drawOval( 10 , 10 , 20 , 20 ) ;
if(fc == true)
{
mc.setBackground( Color.black ) ;
repaint() ;
}
}

public void mouseEntered(MouseEvent e) {}
public void mouseClicked(MouseEvent e) { }
public void mouseExited(MouseEvent arg0) { }
public void mousePressed(MouseEvent arg0) { }
public void mouseReleased(MouseEvent arg0) { }
}

}