jframeの画面上で動かしている時刻をJSliderを使って自由に任意の時刻へ飛べるようにしたい

Question

前提・実現したいこと

jframeの画面上で動かしている時刻(自分で作成したものでリアルの時刻ではない）を同じく画面上にあるスライダーを使って自由に任意の時刻へ飛べるようにしたいです。
プログラミング初心者でいろいろ調べながらここまで作成しましたがこれ以上進まず行き詰っています。
どなたかアドバイスしてくださるとうれしいです。

発生している問題・エラーメッセージ

今現在、時間の作成、スライダーの作成、画面上への表示はできているのですが、時間とスライダーの対応付けができておらず、また、スライダーを左右に動かそうとしても動かないという状況です。

該当のソースコード

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Font;
import java.awt.Graphics;
import java.util.Hashtable;

import javax.swing.JComponent;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JSlider;
public class Windows6 extends JFrame {
	private static final long serialVersionUID = 1L;
		//画面の作成
	public Windows6() {
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		Hashtable<Integer, JComponent> table = new Hashtable<Integer, JComponent>();

	    table.put(new Integer(0), new JLabel("8")); //8時
	    table.put(new Integer(10), new JLabel("9"));
	    table.put(new Integer(20), new JLabel("10"));
	    table.put(new Integer(30), new JLabel("11"));
	    table.put(new Integer(40), new JLabel("12"));
	    table.put(new Integer(50), new JLabel("13"));
	    table.put(new Integer(60), new JLabel("14"));
	    table.put(new Integer(70), new JLabel("15"));
	    table.put(new Integer(80), new JLabel("16"));
	    table.put(new Integer(90), new JLabel("17"));
	    table.put(new Integer(100), new JLabel("18"));

	    JSlider slider = new JSlider();
	    slider.setValue(0);
	    slider.setLabelTable(table);
	    slider.setPaintLabels(true);

	    JPanel p = new JPanel();
	    p.add(slider);

	    getContentPane().add(p, BorderLayout.CENTER);
		// フレームを表示
		setVisible(true);

	}
	//画面表示
	public void paint(Graphics g){
		super.paint(g);

		int  sec=20,min=31, hour=8;

			for(int t=0; t<10000; t++) {

				try{

					Thread.sleep(100);



				}catch(InterruptedException e){
				}


				if(sec==60) {
					min=min+1;
					sec=0;
				}
				if(min==60) {
					hour=hour+1;
					min=0;
				}
				if(hour==24){

					hour=0;
					break;
				}


				String time=hour+":"+min+":"+sec;

				System.out.println(hour+":"+min+":"+sec);
				g.setColor(Color.white);
				g.fillRect(50, 100, 100, 100);
				g.setColor(Color.black);
				Font font1 = new Font("ＭＳ Ｐゴシック",Font.PLAIN,30);
				g.setFont(font1);
				g.drawString(time,50,150);



				sec++;
	

	}
			


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

		JFrame f = new Windows6();
		f.setTitle("スライダー");
		 f.setBounds(10, 10, 300, 300);
		
		f.setVisible(true);

	}
}

Comments

[jimbe]

何よりもまず、 paint 系のメソッドで sleep 等は行ってはいけません。
逆に極力処理を早く終えなければなりません。

そして、swing では paint ではなく paintComponent を上書きしてください。

Answer 1

スライダーを左右に動かそうとしても動かない

Swing はシングルスレッドで動作します。
ボタンを押された時にリスナを呼び出したり、画面サイズが変わったら再描画したり等は全てその唯一のスレッドが行っています。
paint もそのスレッドが呼び出しています。
しかし、ご提示のコードでは paint は for 文が終わるまで終わらず、その for 文内では sleep までしています。
ですので、 Swing 唯一のスレッドは paint が終わるまで何も出来ず、結果アプリは反応しなくなります。

GUI プログラムは、単に自分の都合でだけ動作すれば良いわけではありません。
全体としてどう協調しなければならないかも知っている必要があります。

以下は Timer を使って時間をカウントするようにしています。

SlideTimer.java

import java.awt.*;
import java.awt.geom.Rectangle2D;
import java.util.Hashtable;

import javax.swing.*;

public class SlideTimer extends JFrame {
  public static void main(String[] args) {
    new SlideTimer().setVisible(true);
  }

  public SlideTimer() {
    super("スライダー");
    setBounds(10, 10, 300, 300);
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

    TimePanel timePanel = new TimePanel();
    add(timePanel, BorderLayout.CENTER);

    JSlider slider = new JSlider(8, 18, 8);
    slider.setBorder(BorderFactory.createEmptyBorder(10,10,10,10));
    slider.addChangeListener(e -> timePanel.start(slider.getValue(), 0, 0));
    slider.setLabelTable(createLabelTable(8, 18));
    slider.setPaintLabels(true);
    add(slider, BorderLayout.SOUTH);

    timePanel.start(8, 31, 20);
  }

  private Hashtable<Integer,JComponent> createLabelTable(int from, int to) {
    Hashtable<Integer,JComponent> table = new Hashtable<>();
    for(int i=from; i<=to; i++) table.put(i, new JLabel(""+i));
    return table;
  }
}

class TimePanel extends JPanel {
  private static final Font FONT = new Font("ＭＳ Ｐゴシック", Font.PLAIN, 30);

  private final FontMetrics fontMetrics = getFontMetrics(FONT);

  private class TimeCounter {
    private int hour, min, sec;
    TimeCounter(int hour, int min, int sec) {
      this.sec = sec;
      this.min = min;
      this.hour = hour;
      normalize();
    }
    void countUp() {
      sec ++;
      normalize();
    }
    private void normalize() {
      while(sec < 0) { sec+=60; min--; }
      while(sec >= 60) { sec-=60; min++; }
      while(min < 0) { min+=60; hour--; }
      while(min >= 60) { min-=60; hour++; }
      while(hour < 0) { hour+=24; }
      while(hour >= 24) { hour-=24; }
    }
    @Override
    public String toString() {
      return String.format("%02d:%02d:%02d", hour, min, sec);
    }
  }
  private TimeCounter time;
  private Object[] counterLock = new Object[0];
  private Timer timer;

  TimePanel() {
    super(null);
    setBackground(Color.WHITE);
  }

  void start(int hour, int min, int sec) {
    stop();

    synchronized(counterLock) {
      time = new TimeCounter(hour, min ,sec);
    }

    if(timer == null) {
      timer = new Timer(1000, e -> {
        synchronized(counterLock) {
          time.countUp();
          //System.out.println(time);
        }
        repaint();
      });
      timer.start();
    } else {
      timer.restart();
    }
    repaint();
  }

  void stop() {
    if(timer != null && timer.isRunning()) {
      timer.stop();
    }
  }

  @Override
  public void paintComponent(Graphics g) {
    super.paintComponent(g);

    String str;
    synchronized(counterLock) {
      str = time.toString();
    }

    g.setColor(Color.black);
    g.setFont(FONT);
    Rectangle2D rect = fontMetrics.getStringBounds(str, g);
    int x = (getWidth() -(int)rect.getWidth() )/2;
    int y = (getHeight()-(int)rect.getHeight())/2;
    g.drawString(str, x, y+fontMetrics.getMaxAscent()); //ベースライン

    //Graphics2D g2 = (Graphics2D)g;
    //g2.drawRect(x, y, (int)rect.getWidth(), (int)rect.getHeight());
  }
}

Comments

[asagao0]

わざわざコードまで書いていただき本当にありがとうございます！！
また、2回もコメントを下さり本当にありがとうございました＞＜
実際に動いてるのを見て感激しました！！
paintのなかでfor文を回していたことが動かない原因だったのですね、、、
paintの中になんでも詰め込もうとした考えがだめでした。。
sleepではなくtimerを使うということが全く考えられていなかったのでコードを読み込んで理解していきたいと思います。
本当にありがとうございました！！

[jimbe]

「一秒毎に『足して表示』を繰り返せばいいんだから…」というのは間違っていないんですが、そこから「表示するのは paint だったな」と詰め込んだのが始まりだったようですね。

paint はあくまで表示だけです。その呼び出しは Swing のスレッドが必要に応じて行いますので、いつ呼ばれるかは分かりません。
このプログラムのウインドウに別のプログラム(Java/Swing とは限りません)のウインドウが重なって、それが退いた時等にも呼ばれます。
稀に、その場合には Swing の方で何が表示されていたか覚えていて勝手に表示を戻してくれると考えてしまっている人がいますが、実際はその都度 paint を呼び出されて再表示させられます。
ですので「ある瞬間に表示していて欲しい状態」に関する処理だけを書くようにしなければならないのです。

そして、残りの「一秒毎に『足して…』を繰り返せば」を例えば Timer で行って、『表示』は repaint でお願いする、ということになるわけです。

ちなみに「一秒毎に repaint すれば一秒毎に paint が呼ばれる形になるんだから、 paint で『足して表示』すればいっか」とすると、先のようにウインドウを重ねたり退けたりを繰り返すと必要以上に秒が進んでしまうことになります(^_^;

Answer 2

GUIプログラムではThread.sleepは使わないでください。
UIスレッドがとまると、画面の応答ができなくなってしまいます。
こういった場合は、タイマーを使ってください。
Timer (Java Platform SE 8 )

jframeの画面上で動かしている時刻(自分で作成したものでリアルの時刻ではない）を同じく画面上にあるスライダーを使って自由に任意の時刻へ飛べるようにしたいです。

８時から１８時の範囲が、０から１００ではおおざっぱすぎますよね？（JSliderはintしか扱えず使い勝手が悪いですね^^;

秒まで表示のようですから、トータルの秒で管理してはどうでしょうか。
自分で計算してもいいですが、面倒なのでLocalTimeに計算してもらいました。
LocalTime (Java Platform SE 8 )

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Font;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.time.LocalTime;
import java.time.format.DateTimeFormatter;
import java.util.Hashtable;
import javax.swing.JComponent;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JSlider;
import javax.swing.Timer;
import javax.swing.event.ChangeEvent;
import javax.swing.event.ChangeListener;


public class Windows6 extends JFrame implements ChangeListener, ActionListener {
    public static void main(String[] args) {
        new Windows6().setVisible(true);
    }


    private final JSlider slider = new JSlider();
    private final JLabel label = new JLabel();
    private final DateTimeFormatter format = DateTimeFormatter.ofPattern("HH:mm:ss");

    private LocalTime time = LocalTime.of(8, 0);


    public Windows6() {
        setTitle("スライダー");
        setSize(300, 300);
        setLocationRelativeTo(null);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

        Hashtable<Integer, JComponent> table = new Hashtable<>();
        for (int i = 8; i <= 18; i++) {
            table.put(LocalTime.of(i, 0).toSecondOfDay(), new JLabel(String.valueOf(i)));
        }
        slider.setLabelTable(table);
        slider.setMinimum(LocalTime.of(8, 0).toSecondOfDay()); // 28800
        slider.setMaximum(LocalTime.of(18, 0).toSecondOfDay()); // 64800
        slider.setValue(time.toSecondOfDay());
        slider.setPaintLabels(true);
        slider.addChangeListener(this);
        JPanel p = new JPanel();
        p.add(slider);
        add(p, BorderLayout.NORTH);

        label.setBounds(50, 50, 120, 100);
        label.setFont(new Font("ＭＳ Ｐゴシック", Font.PLAIN, 30));
        label.setBackground(Color.WHITE);
        label.setOpaque(true);
        label.setVerticalAlignment(JLabel.CENTER);
        label.setHorizontalAlignment(JLabel.CENTER);
        JPanel p2 = new JPanel(null);
        p2.add(label);
        add(p2, BorderLayout.CENTER);

        // 100ms間隔（きっちり100msというわけではない。普通だんだん遅延していく）
        Timer timer = new Timer(100, this);
        timer.start();
    }

    // タイマーイベント
    @Override public void actionPerformed(ActionEvent e) {
        time = time.plusSeconds(1);
        label.setText(time.format(format));
        slider.setValue(time.toSecondOfDay());
    }

    // スライダーイベント
    @Override public void stateChanged(ChangeEvent e) {
        time = LocalTime.ofSecondOfDay(slider.getValue());
    }
}

Comments

[asagao0]

たくさんのアドバイス本当にありがとうございます！！！
実際にいただいたコードで動かしてみると想像通りの動きをしていてびっくりしました。。。
sleepにはそんなデメリットがあるのですね、、
時間の計算も自分で計算するものだと思っていますたがlocaltimeを使うとすごく便利ですね
本当に勉強になります。
もっとじっくり読み込み理解したいと思います！！
ありがとうございました！！