回答編集履歴

編集修正

2023/10/04 07:41

投稿

jimbe

スコア12721

test CHANGED Viewed

@@ -1,14 +1,23 @@
-JFrame に直接ペイントせず、 JPanel を土台に paintComponent を Override して直接(間に offscreen の image を挟まず)描画して下さい。
-Swingはデフォルトでダブルバッファリングを行います。
 >これまでに描画していた、画像をがrepaintメソッドを呼び出しても、
 >描画されてしまっています。
 これまでに描画していたモノを消していないなら残っているのは当然ではないでしょうか。
+一番簡単には、 image をクリアする処理を入れることでしょう。
+GameController.GameLoopクラス
+```java
+    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 で Timer を使って gameTick() メソッドを呼び出し、 Hiro と Enemy を動かします。
+JFrame に直接ペイントせず、 JPanel を土台に paintComponent を Override して直接(間に offscreen の image を挟まず)描画します。
+描画タイミングには Swing の Timer クラスを使用します。ウインドウのアクティブ状態に合わせて再開/停止を制御します。
-Character はそれぞれが draw() メソッドで自身を描画します。
+データの描画と更新の衝突を防ぐため、 syncronized を入れています。
+キャラクタの描画はそれぞれのクラス自身が行います。Hero も赤四角として横に移動させるようにしました。
 ```java
 import java.awt.*;
 import java.awt.event.WindowAdapter;
@@ -26,8 +35,6 @@
   TestArpgWalk() {
     super("Title");
     setDefaultCloseOperation(EXIT_ON_CLOSE);
-    setSize(600, 600);
-    setLocationRelativeTo(null);
     GamePanel gamePanel = new GamePanel();
     setContentPane(gamePanel);
@@ -49,6 +56,9 @@
         windowDeactivated(e);
       }
     });
+    pack();
+    setLocationRelativeTo(null);
   }
 }
@@ -58,13 +68,19 @@
   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 -> { gameTick(); repaint(); } );
+    gameLoop = new Timer(20, e -> { tick(); repaint(); } );
     gameLoop.start();
   }
@@ -76,12 +92,12 @@
   }
   @Override
-  public void paintComponent(Graphics g) {
+  synchronized public void paintComponent(Graphics g) {
     super.paintComponent(g);
     for(Character c : charaList) c.draw(g);
   }
-  private void gameTick() {
+  synchronized private void tick() {
     logic();
   }

修正

2023/10/04 06:51

投稿

jimbe

スコア12721

test CHANGED Viewed

@@ -1,4 +1,5 @@
-Swing では paint ではなく paintComponent をOverride して下さい。
+JFrame に直接ペイントせず、 JPanel を土台に paintComponent を Override して直接(間に offscreen の image を挟まず)描画して下さい。
+Swingはデフォルトでダブルバッファリングを行います。
 >これまでに描画していた、画像をがrepaintメソッドを呼び出しても、
 >描画されてしまっています。

コード追加

2023/10/04 04:42

投稿

jimbe

スコア12721

test CHANGED Viewed

@@ -5,5 +5,129 @@
 これまでに描画していたモノを消していないなら残っているのは当然ではないでしょうか。
+---
+JFrame を直接使うのではなく、 JPanel で Timer を使って gameTick() メソッドを呼び出し、 Hiro と Enemy を動かします。
+Character はそれぞれが draw() メソッドで自身を描画します。
+```java
+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);
+    setSize(600, 600);
+    setLocationRelativeTo(null);
+    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);
+      }
+    });
+  }
+}
+class GamePanel extends JPanel {
+  private List<Character> charaList = new ArrayList<>();
+  private Timer gameLoop;
+  public GamePanel() {
+    super(null);
+    charaList.add(new Hero());
+    charaList.add(new Enemy());
+  }
+  void resume() {
+    gameLoop = new Timer(20, e -> { gameTick(); repaint(); } );
+    gameLoop.start();
+  }
+  void pause() {
+    if(gameLoop != null) {
+      gameLoop.stop();
+      gameLoop = null;
+    }
+  }
+  @Override
+  public void paintComponent(Graphics g) {
+    super.paintComponent(g);
+    for(Character c : charaList) c.draw(g);
+  }
+  private void gameTick() {
+    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);
+  }
+}
+```

間違いじゃなかった

2023/10/04 04:29

投稿

jimbe

スコア12721

test CHANGED Viewed

@@ -1,4 +1,4 @@
-Swing では paint ではなく paintComponents をOverride して下さい。
+Swing では paint ではなく paintComponent をOverride して下さい。
 >これまでに描画していた、画像をがrepaintメソッドを呼び出しても、
 >描画されてしまっています。

メソッド名間違い

2023/10/04 03:02

投稿

jimbe

スコア12721

test CHANGED Viewed

@@ -1,4 +1,4 @@
-Swing では paint ではなく paintComponent をOverride して下さい。
+Swing では paint ではなく paintComponents をOverride して下さい。
 >これまでに描画していた、画像をがrepaintメソッドを呼び出しても、
 >描画されてしまっています。

追記

2023/10/04 02:07

投稿

jimbe

スコア12721

test CHANGED Viewed

@@ -4,3 +4,6 @@
 >描画されてしまっています。
 これまでに描画していたモノを消していないなら残っているのは当然ではないでしょうか。
+ついでに、各クラスのカプセル化が全く出来ていません。
+各クラスの役割をはっきりさせ、クラス間のインターフェースを最小にするようにしたほうが良いと思います。