コンストラクタが処理されません

Question

###実現したいこと
ボタンを押すとHello Worldが10回表示されて終了するソースコードを書きたいです。
###発生している問題
動かしてボタンを押してみると、"ボタンクリック確認"は表示されるのですがUの値やHello Worldは表示されません。
恐らく、public void paint(Graphics g)のコンストラクタが動作していないのだと思います。
何故動作しないのか・どう改善すれば良いのか教えてください。
###ソースコード

import java.awt.*;
import java.awt.event.*;
import java.awt.Graphics.*;

public class ButtonTest extends Frame {
    public static void main(String [] args) {
        new ButtonTest();
    }
    ButtonTest() {
        super("ButtonTest");
        setSize(400, 200);
        setLayout(new FlowLayout());
        Button b1 = new Button("OK");
        b1.addActionListener(new MyActionListener());
        add(b1);
        show();
    }

   class MyActionListener implements ActionListener {
	private int U = 0;
	public void actionPerformed(ActionEvent e){
System.out.println("ボタンクリック確認");
	   U = 1;
	}

	public void paint(Graphics g){
System.out.println(U);
	   if(U == 1){
              String str;
              str = "Hello World";
              int Y = 50;
	         try{
	            for(int a=1; a<=10; a++){
                       g.drawString(str, 20, Y);
	               Thread.sleep(100);
	                     Y = Y + 10;
	            }
	         }
	         catch(InterruptedException b){}
	   }
	   else{}
	         System.out.println("処理を終了します");
	         System.exit(0);
	}
   }
}

###補足情報(言語/FW/ツール等のバージョンなど)

Answer 1

こんばんわ
今まで回答に出ている通りですが処理の流れは。。。
１：main実行
２：ButtonTestコンストラクタ初期化
３：とりあえずボタン押す
４：ボタンにセットしているリスナーのactionPerformed(e)が動く
５：ボタンをクリックしました表示して終了

修正ポイントは。。。
１：HellowWorld１０回表示をactionPerformed(e)の中に移動
２：HellowWorld１０回表示をやった後repaint();で再描画
３：System.exit(0);を消す

とりあえず載せてあるそーすから抜粋して一部書き換え、手前の環境で動くものを作ってみたので載せます
参考程度にどうぞ

実行環境：Java1.6

import java.awt.;
import java.awt.event.;
import java.awt.image.BufferedImage;
import javax.swing.JApplet;

@SuppressWarnings("serial")
public class ButtonTest extends JApplet {

//描画領域
Image image;
//描画内容
Graphics g;

public static void main(String[] args) {
	new ButtonTest();
}

public ButtonTest() {
	//画面の縦横サイズをセット
	setSize(400, 400);
	//描画領域を確保
	image = new BufferedImage(getHeight(), getWidth(), BufferedImage.TYPE_INT_ARGB);
	//描画内容はとりあえず白紙にする（初期化）
	g = image.getGraphics();
	//画面自体を中央寄せにする
	setLayout(new FlowLayout());
	//ボタン初期化
	Button b1 = new Button("OK");
	//ボタンにリスナー（ボタンの振る舞い）を追加
	b1.addActionListener(new MyActionListener());
	//画面にボタンを追加
	add(b1);
}

//リスナー（ボタンの振る舞い）を定義
class MyActionListener implements ActionListener {

	//ボタンを押した時の動きを定義する
	public void actionPerformed(ActionEvent e) {
		System.out.println("ボタンクリック確認");

		//HelloWorld 10回表示！！！
		String str = "Hello World";
		int Y = 50;
		try {
			for (int a = 1; a <= 10; a++) {
				g.setColor(Color.BLACK);
				g.drawString(str, 20, Y);
				Y = Y + 10;
			}
		} catch (Exception b) {
			b.printStackTrace();
		}
		//再描画
		repaint();

		System.out.println("処理を終了します");
		//これあると画面消えちゃうよ

// System.exit(0);
}

}

//描画内容を描画領域に描く
public void paint(Graphics g) {
	g.drawImage(image, 0, 0, null);
}

}

Comments

[OMDM1991]

親切な解答ありがとうございます。
"実現したいこと"に不備がありました。ボタンを押すとHello Worldが0.1秒おきに10回表示されて終了するソースコードを書きたいのですが、書いてくださったソースコードの
for (int a = 1; a <= 10; a++) {
g.setColor(Color.BLACK);
g.drawString(str, 20, Y);
Y = Y + 10;
}
を
for (int a = 1; a <= 10; a++) {
g.setColor(Color.BLACK);
g.drawString(str, 20, Y);
repaint();
Y = Y + 10;
Thread.sleep(100);
}
と書き直して実行しても、1秒経過してからHello Worldが10個一気に表示されるのですが、どう書き直せばいいでしょうか？

[退会済みユーザー]

こんばんわ
普通に動きそうですが、いざやってみて同じ結果になりましたorz
ネットで参考になりそうなページを見つけたので読んでみてはどうでしょうか
http://stackoverflow.com/questions/16498174/thread-sleep-in-actionperformed

要するに使えませんよ残念。。。ってところですか

[退会済みユーザー]

ということで代わりに使えと書いてあるTimerを使ってサンプルを作ってみました
おそらくですが実現したいことに近い動きだと思います

実行環境：Java1.6

import java.awt.*;
import java.awt.event.*;
import java.awt.image.BufferedImage;
import java.util.TimerTask;
import java.util.Timer;

import javax.swing.JApplet;

@SuppressWarnings("serial")
public class ButtonTest extends JApplet {

//描画領域
Image image;
//描画内容
Graphics2D g;

//タイマー
Timer timer;
//HelloWorld位置
int w = 20;
int h = 50;

public static void main(String[] args) {
new ButtonTest();
}

public ButtonTest() {
//画面の縦横サイズをセット
setSize(400, 400);
//描画領域を確保
image = new BufferedImage(getHeight(), getWidth(), BufferedImage.TYPE_INT_ARGB);
//描画内容はとりあえず白紙にする（初期化）
g = (Graphics2D) image.getGraphics();
//画面自体を中央寄せにする
setLayout(new FlowLayout());
//ボタン初期化
Button b1 = new Button("OK");
//ボタンにリスナー（ボタンの振る舞い）を追加
b1.addActionListener(new MyActionListener());
//画面にボタンを追加
add(b1);

}

//リスナー（ボタンの振る舞い）を定義
class MyActionListener implements ActionListener {

//ボタンを押した時の動きを定義する
public void actionPerformed(ActionEvent e) {
System.out.println("ボタンクリック確認");

//HelloWorld 10回表示！！！
//を１秒間隔で動かすタイマーを定義
timer = new Timer();
PrintHello ph = new PrintHello();
timer.schedule(ph, 1000, 1000);

System.out.println("処理を終了します");
//これあると画面消えちゃうよ
// System.exit(0);
}

}

//描画内容を描画領域に描く
public void paint(Graphics g) {
g.drawImage(image, 0, 0, null);
}

//HelloWorldを指定位置に表示する
//（TimerTaskを継承して一定時間ごとに繰り返す処理に変身！！）
public class PrintHello extends TimerTask{
@Override
public void run(){
String str = "Hello World";
try {
//１０回（条件はイマイチですが。。。）繰り返して〜
//タイマー実行タスクを終わる
if(h == 185){
timer.cancel();
setEnabled(false);
}else{
//これが無いと同じ行に複数回出力して文字が太くなる。。。
g.setBackground(Color.WHITE);
g.clearRect(w, h-15, getWidth(), getHeight());
}
g.setColor(Color.BLACK);
g.drawString(str, w, h);
repaint();
h += 15;
} catch (Exception b) {
b.printStackTrace();
}
}
}

}

Answer 2

paintメソッドは、コンポーネントのpaintメソッドをオーバーライドしないといけません。この場合だと、Frameを継承したButtonTestでオーバーライドするのが簡単でしょう。
そして、ボタンが押された時に（actionPerformの中で）repaintを呼んであげれば、paintが実行されます。

それと、System.exit(0)がif文の外にあるので、おそらく再描画されるとボタンが押されていなくても終了してしまいます。

最後に、これは余計なお世話だと思いますが、今から勉強するならAWTではなくてSwingの方を選んだ方が良いと思います。

Answer 3

class MyActionListener implements ActionListener {
public void paint(Graphics g){
}
}

paintを呼ぶ記述がないからじゃないですか？

再描画要求をだして、MyActionListener.paint() しないと呼ばれないと思います。

そもそも MyActionListener にペイントを置くのではなくButtonTest におく方が良いと思いましたけど・・・。

Answer 4

実際に描写処理を行うpaintメソッドをどこからも呼んでいないためです．
actionPerformedのU=1のあとでpaintメソッドを起動して下さい．