質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

Swing

SwingはJavaに標準で付属するグラフィック関連のクラスライブラリを指します。

Q&A

解決済

2回答

6040閲覧

[Java][Swing]ストップウォッチを作りたい

sirokuman

総合スコア45

Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

Swing

SwingはJavaに標準で付属するグラフィック関連のクラスライブラリを指します。

0グッド

0クリップ

投稿2017/03/04 09:28

編集2017/03/04 11:17

###前提・実現したいこと
JavaでSwingを用いて、windows上で動作するストップウォッチ(スタート、ポーズ、ストップが出来る一番単純なモノ)を作っています。
以下に記載するWEBサイトを参考にして自分なりに作ってみたのですが、うまいこと動作してくれません。
どうかアドバイスをお願いします。

http://java-study.blog.jp/archives/1005563757.html
https://codezine.jp/article/detail/2597

###[追記]発生している障害
TimeTable.javaの11行目(型変換のところ)で、NPEが発生していました。(KSwordOfHasteさん、ありがとうございました!)
該当箇所をif文で処理(下のソースにあります)したところ、NPEはでないでLabelに時間が表示されるようになったのですが、ミリ秒のままで。HH:mm:ssの形に直ってくれません。また、上記のWEBサイトでは初期化について触れていなかったため、出来ればそれも含めてアドバイスのほうをお願いします。

###該当のソースコード(関係無いと思われる処理や記述は省略してあります)
MainPage.java(ストップウォッチを表示するメインページ)

Java

1public class MainPage extends JPanel{ 2static JLabel TimeLabel; 3 MainPage(){ 4 TimeLabel = new TimeLabel(); 5 TimeLabel.setBounds(30,140,200,200); 6 add(TimeLabel); 7 } 8}

MyActionListener.java(アクションリスナー)

Java

1public class MyActionListener implements ActionListener{ 2 static StopWatch stopWatch; 3 if(StartButtonが押された){ 4 stopWatch = new StopWatch(); 5 stopWatch.start(); 6 }else if(PauseButtonが押された){ 7 stopWatch.suspend(); 8 }else if(StopButtonが押された){ 9 stopWatch.stop(); 10 } 11}

TimeLabel.java

Java

1class TimeLabel extends JLabel{ 2 private DateFormat format; 3 public TimeLabel(){ 4 setFont(new Font("Dialog",Font.BOLD,40)); 5 format = new SimpleDateFormat("HH:mm:ss"); 6 Timer t = new Timer(); 7 t.schedule(new TimerLabelTask(), 0,1000);//一秒毎に更新 8 } 9 class TimerLabelTask extends TimerTask { 10 public void run(){ 11 if(MyActionListener.stopWatch != null){ 12 String Time = String.valueOf(MyActionListener.stopWatch.getTime());//long to String 13 setText(Time); 14 }else{ 15 System.out.println("nullnullだ!"); 16 } 17 } 18 } 19}

###補足情報(言語/FW/ツール等のバージョンなど)
Java 8 , Apache Commons lang3を使用しております。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

ミリ秒のままで。HH:mm:ssの形に直ってくれません。

https://codezine.jp/article/detail/2597 の例では、SimpleDateFormatを使っています。format.format()でHH:mm:ss形式にします。これはうまくゆきません。JSTの時差が9時間出ます。

Java

1String Time = format.format(MyActionListener.stopWatch.getTime());

StopWatch.toString()を使えば、hours:minutes:seconds.milliseconds 形式の表示になります。SimpleDateFormatは捨ててください。

Java

1String Time = MyActionListener.stopWatch.toString();

上記のWEBサイトでは初期化について触れていなかったため、出来ればそれも含めてアドバイスのほうをお願いします。

MyActionListenerのインスタンスを作った時にStopWatchのインスタンスもつくるのはどうでしょうか。

Java

1public class MyActionListener implements ActionListener{ 2public final StopWatch stopWatch = new StopWatch();

この場合、MyActionListenerは一度しかnewできません。

Java

1 MyActionListener myListerner = new MyActionListener(); 2 JButton btn1 = new JButton("StartButton"); 3 btn1.addActionListener(myListerner); 4 JButton btn2 = new JButton("PauseButton"); 5 btn2.addActionListener(myListerner); 6 JButton btn3 = new JButton("StopButton"); 7 btn3.addActionListener(myListerner);

イベント処理です。stop()は一度しか呼べないので使いません。MyActionListenerで以下のメソッドがペアで呼ばれるようにします。
resume() は suspend() が呼ばれた後でなければ呼べない。
start() は reset() が呼ばれた後でなければ2回呼べない。

初期状態では、PauseButton/StopButtonは押せない。StartButtonだけ押せる。

Java

1 if(StartButtonが押された){ 2 stopWatch.start(); 3 // StartButtonを押せなくする。 4 // PauseButton/StopButtonを押せるようにする。 5 }else if(PauseButtonが押された){ 6 // suspend()とresume()が交互に呼ばれるようにする。 7 }else if(StopButtonが押された){ 8 stopWatch.reset(); 9 // PauseButton/StopButtonを押せなくする。 10 // StartButtonを押せるようにする。 11 }

MVCパターンで考えましょう

アドバイスが中途半端になってしまいました。MVCパターンを導入しましょう。初期化やボタン制御はMVCパターンで考えてください。

投稿2017/03/04 13:02

編集2017/03/04 14:08
退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

関係無いと思われる処理や記述は省略してあります

適度な省略は問題点に焦点を合わせやすいので良い考えと思いますが、本件については少々省略しすぎのところがあります。例えばstopWatchをいつどこでどのように初期化しているかがわからないので、ひょっとして初期化を忘れていてNullPointerExceptionが発生しているのかも知れないし、さりとてNPEが発生しているとは書いてないので別のことが起こっているかもしれないし・・・という具合に、問題の可能性が色々考えられるので原因がしぼりにくくなります。

質問の際にどこが肝心な点かわからない場合は元のプログラムを極力短くはするがそのままで動くはずの完全なプログラムコードを載せるとよいと思います。

うまいこと動作してくれません。

残念ながら「うまく動かない」では「何が期待通りでないのか」ほぼわかりません。「期待と何が違うかを具体的に」書きましょう。例えば・・・

以下のコードでNullPointerExceptionが発生する

java

1public void run(){ 2 String Time = String.valueOf(MyActionListener.stopWatch.getTime());//<=ここで例外 3 setText(Time); 4}

とか、「何秒経っても同じ時刻が表示されたまま」
とか、「時間がたつにつれて時刻がずれる」等々・・・

投稿2017/03/04 10:19

KSwordOfHaste

総合スコア18394

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問