カレンダーを作成中ですが、祝日の色変更をしようとしています。

Question

import java.awt.Color;
import java.awt.Dimension;
import java.awt.GridLayout;
import java.util.Calendar;

import javax.swing.BoxLayout;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.SwingConstants;
import javax.swing.UIManager;
import javax.swing.UnsupportedLookAndFeelException;
import javax.swing.border.LineBorder;

public class debugTest {
 public static void main(String[] args) {
  JFrame frame = new JFrame("Swing Calender");
  frame.add(new SwingCalendar());
  frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
  frame.pack();
  frame.setVisible(true);
 }
}

class SwingCalendar extends JPanel {
 private JLabel  monthLabel = new JLabel();
 private JLabel  yearLabel = new JLabel();
 Calendar cal = Calendar.getInstance();
 
 //曜日、日付ラベル作成
 private JLabel[]    weekLabels = new JLabel[7];
 private JLabel[][]  dayLabels = new JLabel[6][7];

　//各パーツのColorの作成
　private static final Color WEEK_BG = new Color(102,102,102);
　//private static final LineBorder WEEK_BORDER = new LineBorder(new 　Color(160,160,230), 2, false);

　//日付パネルの色
　private static final Color DAY_BG = new Color(102,102,102);
　private static final Color DAY_FG = Color.BLACK;
　private static final LineBorder DAY_BORDER = new LineBorder(new Color(102,102,102), 2, false);

　//日付のLabel
　private static final Dimension DAY_LABEL_SIZE = new Dimension(50,50);
　private static final String[] WEEK_NAMES = new String[] {"日", "月", "火", "水", "木", "金", "土"};
　private static final String[] MONTH_NAMES = new String[] {"１月", "２月", "３月", "４月", "５月", "６月", "７月", "８月", "９月", "１０月", "１１月", "１２月"};

　public SwingCalendar() {
　try {
　UIManager.setLookAndFeel(
　"javax.swing.plaf.nimbus.NimbusLookAndFeel"
　);
　} catch (ClassNotFoundException | InstantiationException
　| IllegalAccessException | UnsupportedLookAndFeelException e) {
　e.printStackTrace();
　}

　/* MONTH Label */
　Dimension dm = monthLabel.getPreferredSize();
　dm.width += 10; //dm.height += 1;
　monthLabel.setPreferredSize(dm);
　monthLabel.setForeground(DAY_FG);

　/* YEAR Label */
　Dimension dy = yearLabel.getPreferredSize();
　dy.width += 10; //dy.height += 1;
　yearLabel.setPreferredSize(dy);
　yearLabel.setForeground(DAY_FG);

　/* DAY PANEL */
　JPanel dayPanel = new JPanel();
　GridLayout layout = new GridLayout(7, 7);
　layout.setHgap(0);
　layout.setVgap(0);
　dayPanel.setLayout(layout);
　dayPanel.setBackground(DAY_BG);

　//曜日の配列
　for( int i=0; i<7; i++) {
　　weekLabels[i] = new JLabel(WEEK_NAMES[i]);
　　weekLabels[i].setHorizontalAlignment(SwingConstants.CENTER);
　　weekLabels[i].setVerticalAlignment(SwingConstants.CENTER);
　　weekLabels[i].setPreferredSize(DAY_LABEL_SIZE);
　　weekLabels[i].setOpaque(true);
　　weekLabels[i].setBackground(WEEK_BG);
　
　　if( i == 0 ) {
　　　　weekLabels[i].setForeground(Color.RED);
　　} else if ( i == 6 ) {
　　　　weekLabels[i].setForeground(Color.BLUE);
　　}

　　System.out.println("kakunin" + i);
　　dayPanel.add(weekLabels[i]);
　}


　//日付の配列
  for( int i=0; i<6; i++) {
  　　　for( int j=0; j<7; j++) {
       　dayLabels[i][j] = new DayLabel("test");

            if(j == 0 ) {
            　　dayLabels[i][j].setForeground(Color.RED);
            } else if (j == 6 ) {
            　　dayLabels[i][j].setForeground(Color.BLUE);
            }  

　　　　　　dayPanel.add(dayLabels[i][j]);
　　　　　}
　　　　　　System.out.println("i" + i); //確認
         　System.out.println("j" + j); //確認
　　}

　　JPanel monthPanel = new JPanel();
　　monthPanel.setLayout(new GridLayout());
　　monthPanel.add(yearLabel);
　　monthPanel.add(monthLabel);
　　monthPanel.setBackground(DAY_BG);

　　setLayout(new BoxLayout(this, BoxLayout.Y_AXIS));
　　GridLayout gl = new GridLayout(1,2);
　　gl.setHgap(10); gl.setVgap(10);

　　//年月ボタン表示パネル、日にちパネルの追加
　　add(monthPanel);
　　add(dayPanel);

　}

}

Comments

[opyon]

https://i.gyazo.com/b20e2780d7df8ec95f99a3d3e71629c6.png

[opyon]

コードが枠内に入っていないようなので修正して頂けませんか？

[shuriboo]

@opyonさん　ご指摘ありがとうございます。修正いたしました。

[m.ts10806]

コードブロック内はコードだけを書いてください。コード以外のものが入ると再現確認の際の弊害にしかなりません。また、インデントが全くない状態なので非常に読みづらいですし、ミスが起きやすくなっています。調整してください。あと「うまくいきません」では何が起きているか伝わらないのて起きている現象を具体的に記載してください。このあたりは「質問するときのヒント」を熟読して質問内容を調整すると良いです（質問内容が充実することで的確な回答を得やすくなります） https://teratail.com/help/question-tips

[m.ts10806]

説明文全部削除したらただコード投げただけのも丸投げですよ・・・。コードも大事ですがそのコードで起きている現象やエラー、何がどう想定通りにできていないのか具体的に書くのも必要です。

Answer 1

曜日の日付部分を色替えしたいという仕様と想定して回答いたします。
一般的なDatepickerのコードからやりたい事を憶測して
コードを見た感じでは

　//日付の配列
　for( int i=0; i<6; i++) {
　　for( int j=0; j<7; j++) {
       // ↓ ※1　JLabelのテキスト文字を設定していません。
       dayLabels[i][j] = new JLabel(); //String.valueOf(i),(j)
       // 中略
　　　　if( i == 0 && j == 0 ) { // ←　※2
　　　　　　dayLabels[i][j].setForeground(Color.RED);
　　　　} else if ( i == 6 && j == 0 ) { // ← ※3
　　　　　　dayLabels[i][j].setForeground(Color.BLUE);
　　　　}

この3箇所が変です。j==0の時しか前景色が変わりません。
あとi == 6と判定していますが、変数:iがループの中で6になることはfor( int i=0; i<6; i++)ループ上ありえないかと。

というわけでこんな感じでは。

　　　　if(j == 0) {
　　　　　　dayLabels[i][j].setForeground(Color.RED);
　　　　} else if (j == 6) {
　　　　　　dayLabels[i][j].setForeground(Color.BLUE);
　　　　}

土、日の判定はこれで大丈夫ですが、
春分の日と秋分の日が年によって違うので祝日を判定するにはこれではダメです。
祝日の情報をどこかに持つ必要があります。
LocalDateを使って、LocalDate#plusDays
で日付を1づつ加算して表を作るのをお勧めいたします。

---

余談
以下のようなコードは、JLabelを継承したクラスを1個作るとコード記述が減らせます。

　　　dayLabels[i][j] = new JLabel(); //String.valueOf(i),(j)
　　　dayLabels[i][j].setHorizontalAlignment(SwingConstants.CENTER);
　　　dayLabels[i][j].setVerticalAlignment(SwingConstants.CENTER);
　　　dayLabels[i][j].setOpaque(true);
　　　dayLabels[i][j].setBackground(DAY_BG);
　　　dayLabels[i][j].setPreferredSize(DAY_LABEL_SIZE);

class DayLabel extends JLabel {
　　　public DayLabel(String s) {
　　　　　　super(s);
　　　　　　this.setHorizontalAlignment(SwingConstants.CENTER);
　　　　　　this.setVerticalAlignment(SwingConstants.CENTER);
　　　　　　this.setOpaque(true);
　　　　　　this.setBackground(DAY_BG);
　　　　　　this.setPreferredSize(DAY_LABEL_SIZE);
　　　}
}
// 使い方
dayLabels[i][j] = new DayLabel("test");

曜日とコードがほぼ同じなので、コンストラクタを改良して共通化するのも一つの手かと。

---

LocalDateを使ったサンプルコードです、あとは自力で頑張ってくださいな。

import java.time.LocalDate;
import java.time.temporal.TemporalAdjusters;
import java.util.ArrayList;
import java.util.Calendar;
import java.util.List;
import java.util.stream.Collectors;
import java.util.stream.IntStream;
import java.util.stream.Stream;

class SwingCalendar extends JPanel {
	private JLabel[] dayLabels; // 一次元配列(0オリジン)
	public SwingCalendar() {
		// 前略
		dayPanel.setBackground(DAY_BG);
		// 当月1日
		final LocalDate firstDay = LocalDate.now().with(TemporalAdjusters.firstDayOfMonth());
		// 一ヶ月間のラベルのリスト(最後に配列化)
		final List<JLabel> dayList = new ArrayList<>();

		for (LocalDate day : getDateList(firstDay)) {
			final int d = day.getDayOfMonth();
			System.out.println(d);
			JLabel label = new JLabel(String.valueOf(d));
			label.setHorizontalAlignment(SwingConstants.CENTER);
			label.setVerticalAlignment(SwingConstants.CENTER);
			label.setOpaque(true);
			label.setBackground(DAY_BG);
			label.setPreferredSize(DAY_LABEL_SIZE);
			// 土、日の判定のみなので、ここに祝日判定が必要
			switch (day.getDayOfWeek()) {
			case SUNDAY:
				label.setForeground(Color.RED);
				break;
			case SATURDAY:
				label.setForeground(Color.BLUE);
				break;
			default:
				label.setForeground(Color.WHITE);
				break;
			}
			dayList.add(label);
			dayPanel.add(label);
		}

		// GridLayoutが7*7=49で残りをFillする。
		IntStream.range(0, 49 - dayList.size()).forEach(x -> {
			// GridLayoutに項目を追加する。
			dayPanel.add(new JLabel(""));
		});
		// リストから配列に変換
		dayLabels = dayList.toArray(new JLabel[0]);
		JPanel monthPanel = new JPanel();
		//　以下省略
	}
	public static List<LocalDate> getDateList(final LocalDate start) {
		final int days = start.lengthOfMonth();
		return Stream.iterate(start, d -> d.plusDays(1)).limit(days).collect(Collectors.toList());
	}

Comments

[shuriboo]

＠umyuさん
ご丁寧な回答ありがとうございます。
// ↓ ※1　JLabelのテキスト文字を設定していません。
この部分について、配列[j]がテキストとして表示されていないということでしょうか？
度々で申し訳ありませんが、もう少し詳しくご説明いただけると助かります。

[umyu]

いえ、コード実行すると分かりますが、ラベルに日(1～31)の文字出てないですよね。
JLabel#setTextを行うか、JLabelのコンストラクタに日(1～31)の文字が必要なのではというだけなのですが。

[umyu]

個人的にはLocalDateを使ってループを回して1次元配列で管理するほうが、iとjの2重ループで2次元配列で管理するより楽なのではないかと思うのですが。

[shuriboo]

＠umyuさん
LocalDateで書き直してみます。
ありがとうございます！

[shuriboo]

＠umyuさん
重ね重ねありがとうございます。