うまく動かない原因を教えてください

Question

以下のコードで実行したら
初期の座標で止まったまま落下しません
原因を教えてください

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

public class action extends JFrame implements ActionListener{
	
	JPanel Panel;
	
	JLabel Player;
	JLabel Block;
	
	ImageIcon PlayerIcon;
	ImageIcon BlockIcon;
	
	Timer timer;
	
	int PlayerX = 20;//プレイヤーのX座標
	int PlayerY = 40;//プレイヤーのY座標
	
	static boolean Jflag = false;//ジャンプフラグ
	
	public static void main(String args[]){
		
		action Frame = new action();
		
		Frame.setSize(900, 500);//ウィンドウのサイズ設定(X, Y)
		Frame.setTitle("アクションゲーム");//ウィンドウのタイトル
		Frame.setLocationRelativeTo(null);//ウィンドウを画面の中央に表示する
		Frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);//ｘで閉じるようにする
		Frame.setVisible(true);//ウィンドウを可視化
		
	}
	
	action(){
		
		timer = new Timer(10, this);
		
		Panel = new JPanel();
		Panel.setLayout(null);//レイアウトマネージャー無効化
		
		PlayerIcon = new ImageIcon("./PlayerIcon.png");//プレイヤーの画像指定
		BlockIcon = new ImageIcon("./BlockIcon.png");//ブロックの画像指定
		
		Player = new JLabel(PlayerIcon);//プレイヤー画像設定
		Player.setBounds(PlayerX, PlayerY, 20, 40);//プレイヤーの座標とサイズ設定
		
		Block = new JLabel(BlockIcon);//ブロック画像設定
		Block.setBounds(0, 442, 20, 20);//ブロックの座標とサイズ設定
		
		Panel.add(Player);//プレイヤー画像追加
		Panel.add(Block);//ブロック画像追加
		
		Container contentPane = getContentPane();
		contentPane.add(Panel, BorderLayout.CENTER);
		
		getContentPane().add(Panel, BorderLayout.CENTER);;
		
		timer.start();
		
	}
	
	public void actionPerformed(ActionEvent e){
		
		if(PlayerY <= 462){
			
			Jflag = false;
			timer.stop();
			
		}
		
		if(Jflag == true){//もしジャンプフラグがtrueなら
			
			PlayerY += 5;//プレイヤーのY座標を-5
			Player.setBounds(PlayerX, PlayerY, 20, 40);//プレイヤーの座標再設定
			Panel.repaint();//パネルを更新
			
		}else{
			
			timer.stop();
			
		}
		
	}
	
}

Comments

[karamarimo]

私はSwingというのは全く知らないんですが、なぜ依頼が来たんでしょうか...。

[Alpa]

すいません 間違えました

Answer 1

すいません
自分のミスで簡単な修正で理想通り動きました！
いろいろご指摘いただいたのですがすいませんでした

Comments

[KSwordOfHaste]

自己解決は決してわるくはないですがご自分の回答を「この質問を見ていた同様の疑問を持つ閲覧者へどのような情報を与えるか」を考えてみてください。「具体的にこうなおした」といった情報がないですね？だとすれば具体的な情報を提供している回答者さんの方へＢＡがついてしかるべきと考える閲覧者が多いと思います。そんな観点で考えてみてくださいね。

[KSwordOfHaste]

どうでもよいことですが、マイナスを付けたのは自分ではありません。しかし何も書いてないと質問者さんがわからないかもと思いお節介で上のコメントを付けました。

Answer 2

if(PlayerX <= 442){

    Jflag = false;
    timer.stop();

}

開始位置では「PlayerX == 20」なので、開始直後にtimerが止まりますね。

※以下追記

PlayerXの初期値が違ったとしても、Jflagをtrueにしている記述がどこにもないので、

if(Jflag == true){//もしジャンプフラグがtrueなら

    PlayerY += 5;//プレイヤーのY座標を-5
    Player.setBounds(PlayerX, PlayerY, 20, 40);//プレイヤーの座標再設定
    Panel.repaint();//パネルを更新

}else{

    timer.stop();

}

このブロックでelseの方が動いて、やはり処理が止まります。

Comments

[Alpa]

すいません
その部分は修正していました

[tkturbo]

次のツッコミを入れました。