javaで時刻を表示したい

Question

前提・実現したいこと

画像のような左側に地名、右側にその場所の時刻を表示させたい。

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

正直何が何だかわからない状況です。
下のコードは大きくは変えないで（上記を実現するための最小限の変更で）上記のことを実現するにはどこをどう直せば良いでしょうか？

よろしくお願いいたします。

該当のソースコード

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

class TimeObservable extends Observable implements ActionListener {
  private javax.swing.Timer timer;
  private int hour, min, sec; // 現在時間を保持するインスタンス変数
  private Calendar cal;
  public TimeObservable() {
      timer = new javax.swing.Timer(1000, this); // 1秒毎にactionPerformedを呼び出し
      timer.start();
  }
  public String getValue(int diff) { //時差補正後の時間を返す
      String current_time = String.format("%02d:%02d:%02d", (hour+diff+24)%24, min, sec);
      return current_time;
  }
  public void setValue() {
    cal = Calendar.getInstance();
    hour = cal.get(Calendar.HOUR_OF_DAY);
    min = cal.get(Calendar.MINUTE);
    sec = cal.get(Calendar.SECOND);
  }
  public void actionPerformed(ActionEvent e) {
      this.setValue();
      setChanged();
      notifyObservers();
  }
}

class ClockPanel extends JPanel implements Observer {
    private TimeObservable timeObservable;
    private String place_name;
    private int diff; //時差
    private JLabel label_place, label_time;
    public ClockPanel(TimeObservable to, String s, int d) {
        timeObservable = to;
        place_name = s; diff = d;
        timeObservable.addObserver(this);
        this.setLayout(new GridLayout(1, 2));
        Font font = new Font(Font.SANS_SERIF, Font.BOLD, 32);
        label_place = new JLabel(place_name, JLabel.CENTER);
        label_place.setFont(font);
        this.add(label_place);
        this.add(label_time);
    }
    public void update(Observable o, Object arg) {
        label_time.setText(timeObservable.getValue(diff));
    }
}

class ClockFrame extends JFrame {
    public ClockFrame(){
      this.setTitle("Clock Frame");
      this.setLayout(new GridLayout(2,1)); //これ必要ですか？
      TimeObservable t = new TimeObservable();
      this.add(new ClockPanel(t, "London", -9)); //とりあえず2つだけ
      this.add(new ClockPanel(t, "New York", -14));
      this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      this.pack();
      this.setVisible(true);
    }
    public static void main(String argv[]) {
      new ClockFrame();
   }
}

Comments

[cx20]

とりあえず label_time が null な為、時間が表示されていないように思います。
this.add(label_time);

Answer 1

label_time = new JLabel("", JLabel.CENTER);

を追加したら表示されるようになりました。

■ 実行結果

■ ソース

class ClockPanel extends JPanel implements Observer {
    private TimeObservable timeObservable;
    private String place_name;
    private int diff; //時差
    private JLabel label_place, label_time;
    public ClockPanel(TimeObservable to, String s, int d) {
        timeObservable = to;
        place_name = s;
        diff = d;
        timeObservable.addObserver(this);
        this.setLayout(new GridLayout(1, 2));
        Font font = new Font(Font.SANS_SERIF, Font.BOLD, 32);
        label_place = new JLabel(place_name, JLabel.CENTER);
        label_place.setFont(font);

        label_time = new JLabel("", JLabel.CENTER);
        label_time.setFont(font);
        this.add(label_place);
        this.add(label_time);
    }
    public void update(Observable o, Object arg) {
        label_time.setText(timeObservable.getValue(diff));
    }
}

Comments

[yomoanyan]

回答ありがとうございます。
自分の質問の仕方が悪かったです。よくよく見たら講義内で与えられていた部分のコードを打ち間違えており、自分で追加した部分のミスが見当たらなかったので丸投げ的な仕方になっていました。cx20さんの回答を見て打ち間違いに気づけました。以後気をつけます。

[cx20]

恐らく java.lang.NullPointerException が出ていたのではないかと思います。
スタックトレースにエラーが発生している行番号も出ていたかと思いますので、このエラーが出た場合は、そのソース箇所を確認してみて下さい。そうすれば打ち間違いに気が付いていたのではないかと思います。

■ java.lang.NullPointerExceptionの意味と主な原因を現役エンジニアが解説【初心者向け】
https://techacademy.jp/magazine/22803