質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

Swing

SwingはJavaに標準で付属するグラフィック関連のクラスライブラリを指します。

Q&A

解決済

3回答

3096閲覧

java swing ActionListenerとMouseListenerを同時に利用できない。

entd270

総合スコア19

Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

Swing

SwingはJavaに標準で付属するグラフィック関連のクラスライブラリを指します。

0グッド

0クリップ

投稿2018/06/30 16:31

編集2018/06/30 17:57

前提・実現したいこと

ラジオボタン1が選択されているとき、クリックした場所に四角形を出現させたい。

選択されているラジオボタンに応じて、クリックした場所に出力する図形を変化させたいです。今は、ラジオボタン1(rb1)が選択されているとき、クリックした場所に四角形を出力させようとしているのですが、うまくいきません。なぜでしょうか。
よろしくお願いいたします。

該当のソースコード

<メインクラスの問題があると思われる部分のソースコード> public void actionPerformed(ActionEvent e){ if(rb1.isSelected()) { DrawRect draw= new DrawRect(); addMouseListener(draw); } } <画面のクリックした場所に四角形を出力するためのクラス> class DrawRect extends JFrame implements MouseListener{ public void mouseClicked(MouseEvent e) {   int x,y; x = e.getX(); y = e.getY(); Graphics g= getGraphics(); g.setColor(Color.black); g.drawRect(x, y, 50, 50); } public void mousePressed(MouseEvent e) {} public void mouseReleased(MouseEvent e) {} public void mouseEntered(MouseEvent e) {} public void mouseExited(MouseEvent e) {} }

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答3

0

ベストアンサー

なんか変な構造になっているのでは?

「マウスイベントの中で、ラジオボタンの状態をチェックし、
条件に合致していれば、該当する処理を実行する」

という構造に直してみて下さい。

投稿2018/07/01 09:43

ngsvx

総合スコア287

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

entd270

2018/07/02 15:49

構造を見直した結果正常に動作しました。 ありがとうございます。
guest

0

すでに挙げられている問題点の他に、もう一点描画の方法で気になる箇所があります。

java

1class DrawRect extends JFrame implements MouseListener{ 2 // ... 3 public void mouseClicked(MouseEvent e) { 4 Graphics g = getGraphics();

該当のソースコードのように、JFrame#getGraphics()を使用してもおそらく意図した結果は得られないと思います。

コンポーネントのgetGraphics()メソッドで取得できるGraphicsは一時的なもので、これに対して描画しても大抵別のメソッドで上書き再描画されてしまいます。

代わりに、JPanel#paintComponent(Graphics)メソッドなどをオーバーライドして、引数のGraphicsオブジェクトを使用して描画する方法が一般的です。

公式チュートリアルのLesson: Performing Custom Painting (The Java™ Tutorials > Creating a GUI With JFC/Swing)でコンポーネントのpaintComponent(...)をオーバーライドして独自描画を行う方法が解説されており、Step3にはMouseListenerで四角形を描画するサンプルもあるので、参考にしてみてください。

投稿2018/07/02 09:52

aterai

総合スコア158

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ngsvxさん回答で充分とも思いますが、ちょっとだけ補足してみます。

現状のコードでは「RadioButtonが選択されたらマウスリスナーを登録」という具合になってますが、「RadioButtonが選択解除された際にマウスリスナーの登録を解除」しないてないためおかしな動きになると思います。それをngsvxさんは「なんか変な構造」と表現したのだろうと思います。

常に正しいというわけでもないのですが「何かのイベントが発生したとき別のイベントリスナーを登録する」なんていう方法よりは「初期化の際に必要なイベントリスナーは固定的に設定」し各イベントが発生した際に、「さて、今は四角形を描画すべきなのかな?それとも描画すべきでないのかな?」という判定の上で必要に応じてイベントハンドリング処理内容を分岐させる方がプログラムの構造が分かり易くなると思います。

投稿2018/07/02 03:19

KSwordOfHaste

総合スコア18392

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問