mouseClickedイベントが発生しない（拾えない）場合が度々ある

Question

実現したいこと

添付JavaプログラムでmouseClickedイベントを ”常に” 拾えるようにしたい。

前提

画面に表示された５ｘ５のJLabelを次々にクリックし、mouseClickedイベントを拾ってJLabellの色を変更（白<->水色）したいのですが、イベントを拾えない場合がしばしば発生します。同じJLabelを繰り消しクリックすると拾えることが多いです。

該当のソースコード

import java.awt.Color;
import java.awt.Container;
import java.awt.Dimension;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;

import javax.swing.BorderFactory;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JRootPane;
import javax.swing.border.LineBorder;

public class TEST extends JFrame {
	//----------------------------------------------------------
	static public void main(String args[]) { new TEST(); }
	//----------------------------------------------------------
	public TEST() {
		
		JRootPane root = getRootPane();
		root.setBorder(BorderFactory.createMatteBorder(1,1,1,1,Color.RED));
		setLocationRelativeTo(null);
		setResizable(false);

		JPanel p = new JPanel();
		p.setPreferredSize(new Dimension( 50*5, 50*5 ));
		p.setLayout(null);
		Container cp = getContentPane();
		cp.add(p);
		
		for (int row=0; row<5; row++) for (int col=0; col<5; col++)
			p.add(new Box(row, col));
		
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		pack();
		setVisible(true);
	}
}
//--------------------------------------------------------------
class Box extends JPanel {
	int r, c;
	Box(int row, int col) {
		r = row;
		c = col;
		setBounds(50*c,50*r, 50,50);
		setBorder(new LineBorder(Color.BLACK, 1, true));
		setBackground(Color.WHITE);
		setLayout(null);
		addMouseListener(new MyMouseAdapter());
	}
	private class MyMouseAdapter extends MouseAdapter {
		@Override
		public void mouseClicked(MouseEvent e) {
			System.out.println("mouseClicked "+(r+1)+"-"+(c+1));
			Color c = getBackground();
			setBackground(c==Color.WHITE ? Color.CYAN: Color.WHITE);
		}
	}
}

試したこと

MouthAdapterではなくMouseListenerでも同じ現象でした。

補足情報（FW/ツールのバージョンなど）

jdk21 windows-x64_bin.exeをインストール
Eclipse 2025上で開発

Comments

[jimbe]

コードは「コードのマークダウン」を用いてご提示ください。でないと、コードの一部が文字修飾等のマークダウンとされて消えてしまう場合があります。
具体的には、コードを 「```java」の行と「```」 の行で挟んでください。

[jimbe]

「イベントを拾えない」とはどのように判断されたのでしょうか。

[t0.me_ji]

MyMouseAdapterの mouseClicked()が呼ばれれば、Eclipseのコンソールに
System.out.println("mouseClicked "+(r+1)+"-"+(c+1));
の結果が表示されるのですが、されないことで確認しました。
rとc は5行5列のBoxのどれがクリックされたかを表しています。また、イベントが拾えれば、画面上のBoxの色が変化しますが、変化しません。
イベントを拾える・拾えないは、画面上の5x5のBoxを連続的にクリックすると、両方の場合が混在して現れます。

Answer 1

画面に表示された５ｘ５のJLabelを次々にクリックし、mouseClickedイベントを拾ってJLabellの色を変更（白<->水色）したいのですが、イベントを拾えない場合がしばしば発生します。

こちらの手元（Windows11）では急いでクリックしようとして、mousePressed・mouseReleasedの間にマウスが動くとmouseClicked判定にはなりませんでした。
[JDK-4218549] an allowable "smudge factor" for all mouse clicks - Java Bug System

---

マウスが動いても良いようにするなら、JToggleButtonベースにしたらいかがでしょうか。

import javax.swing.*;
import java.awt.Color;
import java.awt.Dimension;

public class TEST extends JFrame {
    static void main(String[] args) {
        new TEST().setVisible(true);
    }

    TEST() {
        getRootPane().setBorder(BorderFactory.createLineBorder(Color.RED));

        var p = new JPanel(null);
        p.setPreferredSize(new Dimension(50 * 5, 50 * 5));
        add(p);

        for (var row = 0; row < 5; row++)
            for (var col = 0; col < 5; col++)
                p.add(new Cell(row, col));

        pack();

        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setLocationRelativeTo(null);
        setResizable(false);
    }
}

class Cell extends JToggleButton {
    Cell(int row, int col) {
        setBounds(50 * col, 50 * row, 50, 50);
        setBorder(BorderFactory.createLineBorder(Color.BLACK, 1, true));
        setBackground(Color.WHITE);
        addActionListener(e -> {
            System.out.println("Clicked " + (row + 1) + "-" + (col + 1));
            setBackground(getBackground() == Color.WHITE ? Color.CYAN : Color.WHITE);
        });

        // [java - Changing the background color of a selected JToggleButton - Stack Overflow](https://stackoverflow.com/questions/5808022)
        setContentAreaFilled(false);
        setOpaque(true);
    }
}

---

Boxクラスはすでにあるので他の名前がいいでしょう。
Box (Java SE 21 & JDK 21)

Comments

[TN8001]

解決されたようですが^^;

[t0.me_ji]

回答、ありがとうございます。
ご指摘のように「mousePressed・mouseReleasedの間にマウスが動くとmouseClicked判定にはなりません」というのが問題の根本原因だったようです。マウスイベントは「そういう仕組みである」というのは、JDKの説明書のどこかに記載されてるのでしょうか？それとも経験則で判明した事実なのでしょうか？
私としては、当面の問題が解決できればいいので、jimbeさんのご指摘に従ってmouseClickedからmousePressedに処理を移して問題は解決しました。
なお、JPanelではなくJToggleButtonにする案は参考になります。ただ、今回の質問は、より大がかりなアプリの一部を切り出して質問しているのですが、アプリの構造上、JPanelを使わざるを得ない事情があって採用できません。
また、Boxというクラス名は既存クラスとバッティングするので別名にすべしとのご指摘は今後気をつけます。現状はBoxという名称をアプリのいたる所で使っていて改名箇所が大量になるので、このままの名称で通します。

[TN8001]

> マウスイベントは「そういう仕組みである」というのは、JDKの説明書のどこかに記載されてるのでしょうか？それとも経験則で判明した事実なのでしょうか？

わたしは動画のようにやってみて気が付きました。
ただ一般的にはドラッグ閾値未満の移動は、無視すべきと思いますね。

どうやらこれはバグのようです。
[[JDK-4218549] an allowable "smudge factor" for all mouse clicks - Java Bug System](https://bugs.openjdk.org/browse/JDK-4218549)

> mouseClickedからmousePressedに処理を移して問題は解決しました。

mousePressedで早すぎないならそれで十分でしょう^^
mouseReleasedのタイミングのほうが好ましいなら、ひと手間いりますね^^;

[t0.me_ji]

TN8001さん、情報、ありがとうございます。
pressedとreleasedの間に1ピクセルでも動くsmudgeがあるとclickedとは見做さないという厳格さが妥当かどうかが1999年から問題になってたんですね。
最新のJDKでも「妥当」とされてるのか、あるいは、「つまらないことでツベコベ言うな」として無視されてきたのかは知る由もありませんが、現状が「仕様」なら、そういうふうになっていて、smudgeは1ピクセルたりとも許容しない、ということを mouseClicked()メソッドの説明に書いて欲しいものです。現状は「コンポーネント上でマウス・ボタンをクリック(押してから離す)したときに呼び出されます」としか書いていません。
個人的意見ですが、
案1: mouseClicked()は紛らわしいので廃止
案2: mouseReleased()の直後に、無条件にmouseClicked()を呼び出す
　　　→ released座標＝clicked座標
案3: smudgeがあるとmouseClicked()は呼ばれない旨、目につく所に明記
のどれかが良さそうに思います。smudgeを許容する案は賛成できません。

[t0.me_ji]

長文コメントを投稿したらエラーになって、変な切り貼りされてしまいました。
5行目以降は、個人的には、次の３案のどれかにしてもらえるといいと思うと書きました。
案1: mouseClickedは紛らわしいので廃止
案2: mouseReleaseed直後にmouseClickedを発生（relese座標＝click座標）
案3: smudgeは許容しないことをmouseClickedの説明に明記

[t0.me_ji]

投稿システムがおかしい。文章の文字数制限がある？？？

[TN8001]

> 最新のJDKでも「妥当」とされてるのか、あるいは、「つまらないことでツベコベ言うな」として無視されてきたのかは知る由もありませんが、

クローズ理由は私には読み取れませんが、何にしろ今更手が入るとも思えませんね...
ドキュメントの修正ぐらいならあり得るかな？（その辺の手続きどうするのか知りませんが^^;

> 長文コメントを投稿したらエラーになって、変な切り貼りされてしまいました。

コメントも編集できます。コメントにマウスを乗せると右上に「…」が出るので、クリックして編集を選んでください^^

> 投稿システムがおかしい。文章の文字数制限がある？？？

質問や回答は１万文字制限だと思います。
コメントの文字数はどうなんだろう？少なくとも数千文字書いてるコメントも見たことあります。

編集してみて同様だったら、メンテナンスで制限が付いた可能性はあります（あるいはバグか^^;

[t0.me_ji]

返信ありがとうございます。
いちアジア人が「こうあるべき」と言ったところで、仕様が変るとは思えないので、言うだけ無駄でした。とにかく、やりたいことがやれるようになったので、これでオシマイにします。お付き合い、ありがとうございました。

Answer 2

基本的には、"本当に" イベントが拾えないのなら処理速度のような要因でしょうからそれは仕方ないことと思います。イベントが来たら処理するというしかないでしょう。(ちなみにコンソールへの表示は案外重いです。)

が、イベントが拾えてない "ように見える" ということはありそうです。
setBackground の実行は、必ずしも再表示はしてくれません。 setBackground したらそのパネルの repaint を呼んでみては如何でしょうか。

他に、クリックと判断するには時間が必要な為、クリックでは無くプレスで処理(色変え)するというテもあるかもしれません。

ついでに、 setLocationRelativeTo を使って真ん中に表示するなら、 pack の後にした方が良いです。
setLocationRelativeTo はその時点でのサイズで真ん中を決めますので、後から pack してもウインドウの大きさで中央が合いません。

Comments

[t0.me_ji]

回答ありがとうございます。
mousePressed()に変更したところ。毎回拾えるようになりました。
setBackground()直後のrepaint()は特に効果がありませんでした。
setLocationRelativeToをpack()後にするご提案は別途、試してみます。
今のところ、そうしなくても特に問題は無いので。

[t0.me_ji]

回答ありがとうございます。
試したところ問題が解決しました！
ベストアンサーに選ばせていただきました。