JPanelを斜めに移動させる方法

Question

前提・実現したいこと

現在、独学でシューティングゲームを作っています。その際に玉(JPanel)を斜めに動かす必要があります。もし角度が45°であれば各座標を定める変数に1ずつ足せばいい(ソースコード①のBallクラスのgoメソッド)と思うのですが、その角度がもっと斜めだった場合にどうすればいいのかわかりません。案としてソースコード②が思いついたのですが、他にいい方法はありませんか？
ソースコード②はsetLocationをする際に数値を1/10にすることで精度を保っているつもりです。
また、他にソースコードの改善点があれば教えてください。

元のソースオード①

import javax.swing.*;
import java.awt.*;
public class Main{
	public static void main(String[] args){
		new Main();
	}
	Main(){
		Game f = new Game();
		f.setVisible(true);
		while(true){
			long st = System.currentTimeMillis();
			
			f.ball.go(f.fx,f.fy);
			
			long en = System.currentTimeMillis();
			try{
				Thread.sleep(1000/f.fps-(en-st));
			}catch(Exception ex){
				System.out.println("<Error> +"+(1000/f.fps-(en-st))+"ms");
			}
		}
	}
}
class Game extends JFrame{
	int fps = 60;
	int fx=500,fy=500;	//フレームサイズ
	Ball ball;
	Game(){
		setTitle("ゲーム画面");
		setSize(fx+14,fy+37);
		setLocationRelativeTo(null);
		setLayout(null);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBackground(Color.white);
		
		ball = new Ball();
		add(ball);
	}
}
class Ball extends JPanel{
	int x=0,y=0;	//座標
	Ball(){
		setBounds(x,y,10,10);
		setBackground(Color.black);
	}
	void go(int fx,int fy){
		if(fx==x || fy==y){
			x=0;
			y=0;
		}
		x++;
		y++;
		setLocation(x,y);
	}
}

変更点②

class Ball extends JPanel{
	int x=0,y=0;	//座標
	Ball(){
		setBounds(x,y,10,10);
		setBackground(Color.black);
	}
	void go(int fx,int fy){
		if(fx==x/10 || fy==y/10){
			x=0;
			y=0;
		}
		x+=9;
		y+=3;
		setLocation(x/10,y/10);
	}
}

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

java version "16.0.1" 2021-04-20
Java(TM) SE Runtime Environment (build 16.0.1+9-24)
Java HotSpot(TM) 64-Bit Server VM (build 16.0.1+9-24, mixed mode, sharing)

Answer 1

そんなまどろっこしいことをするくらいなら、x・yをdoubleで持てばよくないですか？

class Ball extends JPanel {
    double x = 0;
    double y = 0;
    double rad = Math.toRadians(18);

    Ball() {
        setBounds((int) x, (int) y, 10, 10);
        setBackground(Color.black);
    }

    void go(int fx, int fy) {
        if (fx <= x || fy <= y) {
            x = 0;
            y = 0;
        }

        x += Math.cos(rad);
        y += Math.sin(rad);
        setLocation((int) x, (int) y);
    }
}

---

JavaでゲームならProcessingもおすすめです^^
Welcome to Processing! / Processing.org

Comments

[cretter]

たしかに！
言われるまで気づきませんでした。
ありがとうございます。

Answer 2

他にいい方法はありませんか？

基本的には 2 番の感じになると思います。

他にソースコードの改善点があれば

フレームを表示して以降は、Swing の部品を外部から操作してはいけません。(より正確には、フレームの表示自体もSwing上でやらないといけないのですが…。)
また、JFrame内をJPanelが動き回る構造よりも、JPanelをゲーム画面としてその中に各キャラクタを画像として書き込むほうが、自由度が高く、JPanelという重そうな(計ったことはありません)モノを使うより良いように思います。

ボタンを押すとボールが動き回ります。

package teratail_java.q357613;

import java.awt.Color;
import java.awt.Dimension;
import java.awt.FlowLayout;
import java.awt.Graphics;
import java.awt.Rectangle;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.concurrent.Executors;
import java.util.concurrent.ScheduledExecutorService;
import java.util.concurrent.TimeUnit;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class GameFrame extends JFrame {
  public static void main(String[] args) {
    new GameFrame().setVisible(true);
  }
  GameFrame() {
    super("ゲーム画面");
    setLocationRelativeTo(null);
    setDefaultCloseOperation(EXIT_ON_CLOSE);
    setResizable(false);
    add(new GamePanel());
    pack();
  }
  private class GamePanel extends JPanel {
    private Ball ball;
    private Rectangle frame;
    private ScheduledExecutorService executor;
    GamePanel() {
      super(new FlowLayout());
      Dimension size = new Dimension(200, 200);
      setPreferredSize(size);
      frame = new Rectangle(size);

      ball = new Ball(90, 100, 0.9f, 0.5f);

      executor = Executors.newSingleThreadScheduledExecutor();

      JButton startButton = new JButton("start");
      add(startButton);
      startButton.addActionListener(new ActionListener() {
        @Override
        public void actionPerformed(ActionEvent e) {
          startButton.setEnabled(false);
          executor.scheduleAtFixedRate(new Runnable() {
            @Override
            public void run() {
              ball.move(frame);
              repaint();
            }
          }, 10, 10, TimeUnit.MILLISECONDS);
        }
      });
    }
    @Override
    public void paintComponent(Graphics g) {
      super.paintComponent(g);
      ball.draw(g);
    }
  }
  private class Ball {
    private float x, y, dx, dy;
    private int w=20, h=20;
    Ball(int x, int y, float dx, float dy) {
      this.x = x;
      this.y = y;
      this.dx = dx;
      this.dy = dy;
    }
    void move(Rectangle frame) {
      for(boolean xover=false, yover=false; !xover || !yover; ) {
        xover = frame.contains(x+dx, y, w, h);
        if(!xover) dx *= -1;
        yover = frame.contains(x, y+dy, w, h);
        if(!yover) dy *= -1;
      }
      x += dx;
      y += dy;
    }
    void draw(Graphics g) {
      g.setColor(Color.BLUE);
      g.fillOval((int)x, (int)y, w, h);
    }
  }
}