javaswing repaintメソッドを呼び出しても画面が背景色でクリアされない

Question

実現したいこと

現在,javaswingでの描画処理の勉強をしてテストプログラムを作っています。
コントローラークラスで、描画するオブジェクトの計算をし、imageインスタンスにグラフィックの情報を代入しています。そして、frameにそれを引数としてわたし、repaintメソッドを呼び出しています。

実行すると描画自体はされているのですが、
これまでに描画していた、画像をがrepaintメソッドを呼び出しても、
描画されてしまっています。
適切に実装するにはどのようにすればよいのでしょうか。

該当のソースコード

package TestRpgWalk;

public class TestArpgWalk {
public static void main(String[] args) {
	new GameController();
}
}

package TestRpgWalk;

import java.awt.Color;
import java.awt.Graphics;
import java.awt.Image;
import java.awt.Point;

public class GameController {
	MainJFrame mainJFrame = null;
	Image image = null;
	Enemy enemy = new Enemy();
	Hero hero = new Hero();

	public GameController() {
		mainJFrame = new MainJFrame();
		image = mainJFrame.createImage(mainJFrame.getWidth(), mainJFrame.getHeight());
		Thread thread = new Thread(new GameLoop());
		thread.start();
	}

	class GameLoop implements Runnable {
		boolean isGame = true;

		@Override
		public void run() {
			init();
			while (isGame == true) {
				logic();
				draw();
				try {
					Thread.sleep(20);
				} catch (InterruptedException e) {
					// TODO 自動生成された catch ブロック
					e.printStackTrace();
				}
			}
		}

		public void init() {
			enemy.init();
			hero.init();
		}

		public void logic() {
			enemy.move();
			hero.move();
		}

		public void draw() {
			Graphics g = image.getGraphics();
			g.setColor(Color.blue);
			g.fillOval(enemy.point.x, enemy.point.y, 30, 30);
			mainJFrame.draw(image);
		}

	}
}

class Character {
	Point point = new Point();

	public void move() {
		point.x++;
		point.y++;
	}
}

class Hero extends Character {
	public void init() {
		point.setLocation(20, 20);
	}
}

class Enemy extends Character {
	public void init() {
		point.setLocation(50, 50);
	}
}

package TestRpgWalk;

import java.awt.Graphics;
import java.awt.Image;

import javax.swing.JFrame;

public class MainJFrame extends JFrame {
	Image image  =null;
	public MainJFrame() {
setDefaultCloseOperation(EXIT_ON_CLOSE);
setSize(600,600);
setLocationRelativeTo(null);
setTitle("Title");
setVisible(true);
	}
	public void paint(Graphics graphics) {
		graphics.drawImage(image, 0, 0, this);
	}
	public void draw(Image image) {
		this.image = image;
		repaint();
}
}

Answer 1

これまでに描画していた、画像をがrepaintメソッドを呼び出しても、
描画されてしまっています。

これまでに描画していたモノを消していないなら残っているのは当然ではないでしょうか。
一番簡単には、 image をクリアする処理を入れることでしょう。

GameController.GameLoopクラス

    public void draw() {
      Graphics g = image.getGraphics();
      g.setColor(new Color(0x80c080)); //追加
      g.fillRect(0, 0, mainJFrame.getWidth(), mainJFrame.getHeight()); //追加
      g.setColor(Color.blue);

---

JFrame に直接ペイントせず、 JPanel を土台に paintComponent を Override して直接(間に offscreen の image を挟まず)描画します。
描画タイミングには Swing の Timer クラスを使用します。ウインドウのアクティブ状態に合わせて再開/停止を制御します。
データの描画と更新の衝突を防ぐため、 syncronized を入れています。
キャラクタの描画はそれぞれのクラス自身が行います。Hero も赤四角として横に移動させるようにしました。

import java.awt.*;
import java.awt.event.WindowAdapter;
import java.awt.event.WindowEvent;
import java.util.ArrayList;
import java.util.List;

import javax.swing.*;

public class TestArpgWalk extends JFrame {
  public static void main(String[] args) {
    SwingUtilities.invokeLater(() -> new TestArpgWalk().setVisible(true));
  }

  TestArpgWalk() {
    super("Title");
    setDefaultCloseOperation(EXIT_ON_CLOSE);

    GamePanel gamePanel = new GamePanel();
    setContentPane(gamePanel);

    addWindowListener(new WindowAdapter() {
      @Override
      public void windowActivated(WindowEvent e) {
        //System.out.println("windowActivated");
        gamePanel.resume();
      }
      @Override
      public void windowDeactivated(WindowEvent e) {
        //System.out.println("windowDeactivated");
        gamePanel.pause();
      }
      @Override
      public void windowClosing(WindowEvent e) {
        //System.out.println("windowClosing");
        windowDeactivated(e);
      }
    });

    pack();
    setLocationRelativeTo(null);
  }
}

class GamePanel extends JPanel {
  private List<Character> charaList = new ArrayList<>();
  private Timer gameLoop;

  public GamePanel() {
    super(null);
    setSize(600, 600);
    setPreferredSize(getSize());
    setMinimumSize(getSize());

    setBackground(new Color(0x80c080));

    charaList.add(new Hero());
    charaList.add(new Enemy());
  }

  void resume() {
    if(gameLoop != null) throw new IllegalStateException("'gameLoop' NOT null");
    gameLoop = new Timer(20, e -> { tick(); repaint(); } );
    gameLoop.start();
  }

  void pause() {
    if(gameLoop != null) {
      gameLoop.stop();
      gameLoop = null;
    }
  }

  @Override
  synchronized public void paintComponent(Graphics g) {
    super.paintComponent(g);
    for(Character c : charaList) c.draw(g);
  }

  synchronized private void tick() {
    logic();
  }

  private void logic() {
    for(Character c : charaList) c.move();
  }
}

abstract class Character {
  protected Point point = new Point();

  void move() {}
  abstract void draw(Graphics g);
}

class Hero extends Character {
  Hero() {
    point.setLocation(20, 20);
  }

  @Override
  void move() {
    point.x++;
  }

  @Override
  void draw(Graphics g) {
    g.setColor(Color.RED);
    g.fillRect(point.x, point.y, 30, 30);
  }
}

class Enemy extends Character {
  Enemy() {
    point.setLocation(50, 50);
  }

  @Override
  void move() {
    point.x++;
    point.y++;
  }

  @Override
  void draw(Graphics g) {
    g.setColor(Color.BLUE);
    g.fillOval(point.x, point.y, 30, 30);
  }
}