java swing マウスをドラッグして画像を移動したい

Question

前提・実現したいこと

マウスをドラッグすることで画像を移動したいです。
java初心者でしっかり構造を理解できていないのが原因だと思います。
どこに問題があるか教えていただきたいです。
よろしくお願いいたします。

発生している問題

ドラッグしても画像が表示されません。

該当のソースコード

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;

class mouseExec extends JFrame implements MouseMotionListener {
	int x=0;
	int y=0;
	ImageIcon icon = new ImageIcon("画像.png");
	JLabel label = new JLabel(icon);

	public static void main(String[] args) {
             new mouseExec();
	}

	mouseExec(){
             setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
             getContentPane().setLayout(null);
             addMouseMotionListener(this);
             setSize(700, 550);
             setVisible(true);

	public void mouseDragged(MouseEvent e) {
             x = e.getX();
             y = e.getY();
             repaint();
    }
    public void mouseMoved(MouseEvent e) {}

    public void paint(Graphics g){
            label.setBounds(x,y,50,50);
    }
}

Comments

[m.ts10806]

インデントがおかしくなっている部分があるので調整してください

[entd270]

インデントを修正しました。

Answer 1

複数の原因があります。

◇原因
a, mouseExecクラスのコンストラクタでgetContentPane().setLayout(null);とレイアウトマネージャを無効化しているが、labelの座標(label.setBounds(0, 0,50,50);)を初期配置座標を指定していない。
◇参考情報
座標指定で配置する

b, ContentPaneに変数:labelを追加していない。
◇参考情報
コンテナとコンポーネント

c, JFrameのpaintメソッドをオーバーライド(上書きしている)
最初の内はSwingでpaintメソッドはオーバーライドしてはダメ、独自の描画処理をしたいときはpaintComponentをオーバーライドすると覚えれば良いです。
◇参考情報
翻訳　AwtとSwingにおける描画処理

他にもいろいろ直しましたが修正版のソースコードです。ご参考まで。

import javax.swing.*;
import java.awt.event.*;

public class mouseExec extends JFrame implements MouseMotionListener {
	ImageIcon icon = new ImageIcon("画像.png");
	// JLabel label = new JLabel(icon);
	JLabel label = new JLabel("こんにちわ");

	public static void main(String[] args) {
		// 1, SwingUtilities.invokeLaterを使う。
		// 2, mouseExecのコンストラクタ内ではsetVisible(true);を行わない。
		SwingUtilities.invokeLater(() -> {
			new mouseExec().setVisible(true);
		});
	}

	mouseExec() {
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		// 3, 画面のウィンドウサイズ指定はこの位置で。
		setSize(700, 550);
		getContentPane().setLayout(null);
		// 4, 回答の原因:aの対策
		label.setBounds(0, 0, 50, 50);
		// 5, 回答の原因:bの対策
		getContentPane().add(label);
		addMouseMotionListener(this);
	}

	// 6, @Overrideアノテーションを付ける。
	@Override
	public void mouseDragged(MouseEvent e) {
		// 7, サイズが変わらないのであれば、setBoundsではなくsetLocationを呼び出す。
		label.setLocation(e.getPoint());
		// 8, repaintは呼び出さない。
	}

	@Override
	public void mouseMoved(MouseEvent e) {
	}
}

Comments

[entd270]

とても分かりやすい説明で勉強になりました。
本当にありがとうございます。