runメソッド内のrepaintが実行されない

Question

前提・実現したいこと

java初心者です。
runメソッド内でwhile文を用いて、等速に図形が落下するプログラムを考えています。

発生している問題

run内部で変化していくｙ座標の計算を行い、計算終了ごとにrepaintを実行するようにしたいのですがpaintComponentが呼ばれていません。(paintComponent内のプリント出力がされていないことから判断)

該当のソースコード

import javax.swing.JFrame;
import javax.swing.JPanel;
import java.awt.;
public class Fall{
public static void main(String[] args) {
JFrame fr = new JFrame("TypeGame");
fr.setSize(630,750);
fr.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
fr.getContentPane().setBackground(Color.black);
TileBasedGamePanel panel = new TileBasedGamePanel();
panel.setOpaque(false);
fr.add(panel);
fr.setVisible(true);
}
}
class TileBasedGamePanel extends JPanel implements Runnable {
private int paintX,paintY=0;
private final double speed=30;
private long deltTime=(long)0;//ms
private Thread th;
TileBasedGamePanel() {
setFocusable(true);
}
public void paintComponent(Graphics g){
g.setColor(Color.white);
g.drawOval(this.paintX,this.paintY,50,50);
System.out.println("paintComponentを呼びました");
TileBasedGamePanel tbgp = new TileBasedGamePanel();
this.th = new Thread(tbgp);
th.start();
}
private boolean isExisted(){//paintYがフレーム上にあるか判定
if(this.paintY<750)return true;
return false;
}
public void run(){
while(isExisted()){
double secTime=this.deltTime/1000.;
this.paintY=(int)(this.speed(secTime));//m/s*s=m
long sleep=(long)50;//ms スレッドの停止時間
try{
this.th.sleep(sleep);
this.deltTime+=sleep;//経過時間を計算
this.repaint();
System.out.println("paintY:"+this.paintY);
System.out.println("deltTime:"+this.deltTime);
}catch(InterruptedException e){}
}
System.out.println("runおわり");
}
}

Java

repaintが実行されない原因、解決策をご教授お願い致します。

Answer 1

TileBasedGamePanel tbgp = new TileBasedGamePanel();
this.th = new Thread(tbgp);

新たにJPanelを作ってしまっては、呼ぶrepaintは表示されていない新しいJPanelのものです。

paintComponent内でThreadを作るのも意図しているように思えません。
repaintのたびにThreadが増えてしまいます。

import java.awt.Color;
import java.awt.Graphics;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class Fall {
    public static void main(String[] args) {
        JFrame fr = new JFrame("TypeGame");
        fr.setSize(630, 750);
        fr.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        fr.getContentPane().setBackground(Color.black);
        TileBasedGamePanel panel = new TileBasedGamePanel();
        panel.setOpaque(false);
        fr.add(panel);
        fr.setVisible(true);
    }
}

class TileBasedGamePanel extends JPanel implements Runnable {
    private int paintX = 0;
    private int paintY = 0;
    private final double speed = 30;
    private long deltTime = 0;
    private Thread th;

    TileBasedGamePanel() {
        setFocusable(true);
        this.th = new Thread(this);
        th.start();
    }

    @Override
    protected void paintComponent(Graphics g) {
//        super.paintComponent(g);
        g.setColor(Color.white);
        g.drawOval(this.paintX, this.paintY, 50, 50);
        System.out.println("paintComponentを呼びました");
    }

    private boolean isExisted() {
        return this.paintY < 750;
    }

    public void run() {
        while (isExisted()) {
            double secTime = this.deltTime / 1000.0;
            this.paintY = (int) (this.speed * secTime);
            long sleep = 50;
            try {
                Thread.sleep(sleep);
                this.deltTime += sleep;
                this.repaint();
                System.out.println("paintY:" + this.paintY);
                System.out.println("deltTime:" + this.deltTime);
            } catch (InterruptedException e) {}
        }
        System.out.println("runおわり");
    }
}

Comments

[railsganbaruman]

ベストアンサーにさせていただきます。
多くの点で理解不足でした。
丁寧な解説ありがとうございます。