java カレンダー作成

Question

追記：JDKのバージョンは1.6.0_19です。
そして指示によりimportは使用できないようになっています。

---

コマンドライン引数で年月の値を引き渡し(201604の様に6桁で)、それでカレンダーを表示させるプログラムを考えています。

一応理想の形で表示させることが出来るようになったのですが、もっと『ここはこうするべき』『こうした方がスマートで分かりやすい』『最初から大幅にやり直すべき』などご指摘がございましたら是非お願いいたします！

---

public class CalShow {

public static void main(String[] args) {

	java.util.Calendar cal = java.util.Calendar.getInstance();

	int year = Integer.parseInt(args[0].substring(0, 4));
	int month = Integer.parseInt(args[0].substring(4));

	System.out.println( year + "年" + month + "月");
	System.out.println(" 日 月 火 水 木 金 土");

	//year,month,1で年月日設定
	cal.set(year, month - 1, 1);

	//月初日が何曜日なのか設定
	int startDay = cal.get(java.util.Calendar.DAY_OF_WEEK);

	int lastDay = cal.getActualMaximum(java.util.Calendar.DATE);

	if(startDay != 7){
		for(int i = 1 ; i < startDay; i++ ){
			System.out.print("   ");
		}
	}

	int day = 1;
	for(int i = startDay; i <= 7; i++) {
		if(day < 10){
			System.out.print(" ");
		}
		System.out.print(" "+ day);
		day ++;
	}

	System.out.println("");

	for(int j = 1; j < 7; j++){
		for(int k = 0; k < 7; k++){
			if(day <= lastDay){
				if(day < 10){
					System.out.print(" ");
				}
				System.out.print(" "+ day);
				day ++;
			}
		}
		System.out.println("");
	}
}

}

Comments

[KiyoshiMotoki]

お使いのJDKのバージョンは何ですか？それによって、回答が変わってくる可能性がありますので。

[退会済みユーザー]

JDKは1.6.0_19です。あと申し訳ありません、記載するのを忘れていたのですが、importは使用せずに作成したいと思っております。

[KiyoshiMotoki]

返信ありがとうございます。お手数ですが、「JDKは1.6.0_19です。」は質問欄に追記願います。誰もがこの"情報の追加・修正の依頼をする"欄を読むとは限りませんので。あと、「importは使用せずに作成したいと思っております。」は、なぜですか？理由によっては、その方針自体、考え直すべきかもしれません。

Answer 1

for(int j = 1; j < 7; j++){
    for(int k = 0; k < 7; k++){
        if(day <= lastDay){
            if(day < 10){
                System.out.print(" ");
            }
            System.out.print(" "+ day);
            day ++;
        }
    }
    System.out.println("");
}

とりあえず、ここのfor文のネストは可読性を落とすだけなのでやめたほうがいいでしょう。
そのうえでjava.util.Calendarを有効活用するならこんなコードもありですね。

java.util.Calendar cal = java.util.Calendar.getInstance();
cal.set(year, month - 1, 1);

int monthStartDayOfWeek = cal.get(java.util.Calendar.DAY_OF_WEEK);

int dateConst = java.util.Calendar.DATE;

System.out.println( year + "/" + (month--) + "");
System.out.print(" sun mon tue wed thu fri sat");

for(int i = 1; ; i++) {
    if(i % 7 == 1){ System.out.println(); }
    if (i < monthStartDayOfWeek) {
        System.out.print("    ");
        continue;
    }
    System.out.print(String.format("% 4d", cal.get(dateConst)));
    cal.add(dateConst, 1);
    if (cal.get(java.util.Calendar.MONTH) != month) {
        System.out.println();
        break;
    }
}

Comments

[退会済みユーザー]

可読性についてのご指摘ありがとうございました。
なるべくfor文がネストしないよう作っていきたいと思います。
まだまだ分からないことばかりで、有効活用の記載分についてはまだ理解出来かねる部分が多いのですが、理解できるよう頑張っていきます。
またよろしくお願いいたします。

Answer 2

入力値をチェックしていない

入力値が渡されなかった場合、ArrayIndexOutOfBoundsExceptionが発生してプログラムが異常終了します。

また、"201613"や"123456789"など、年月として無効な値が渡された場合でも、そのまま実行されてしまいます。
これに関しては、cal.setを実行する前にcal.setLenient(false)を実行しておくことで、おかしな日付を黙って計算してしまう事態を防ぐことができます。
https://docs.oracle.com/javase/jp/6/api/java/util/Calendar.html#setLenient(boolean)

文字数の調整方法

if (day < 10) {
    System.out.print(" ");
}
System.out.print(" " + day);

部分で一桁の日付の表示幅を調整しながら出力していますが、String.formatメソッドを使用すれば、以下のように簡潔に書けます。

System.out.print(String.format("% 3d", day));

定数の値に依存した実装

for (int i = startDay; i <= 7; i++) {

などの部分は、Calendar.SUNDAY = 1であることなどに依存した実装になっています。
しかし、万が一、後のバージョンアップで曜日も（月のように）0から始まる連番に変更された場合、このプログラムは正しく動かなくなります。
定数の値が何なのかを気にしなくても良いように実装すべきです。

例えば、あまり良いコードではありませんが、以下のような感じです。

// year,month,1で年月日設定
cal.set(year, month - 1, 1);

// 理由は不明だが、ここでこれを呼び出しておかないと1月を指定したときにおかしな動きをする。
// おそらく年・月・日以外のフィールドの値が変更されていないためと思われ。
cal.getTime();

// さらに、その週の日曜日に日付を設定
cal.set(Calendar.DAY_OF_WEEK, Calendar.SUNDAY);

while (("" + cal.get(Calendar.YEAR) + (cal.get(Calendar.MONTH) + 1)).compareTo("" + year + month) <= 0) {
    if (cal.get(Calendar.MONTH) + 1 != month) {
        // 前月の間は空欄表示
        System.out.print("   ");
    } else {
        System.out.print(String.format("% 3d", cal.get(Calendar.DAY_OF_MONTH)));
    }

    // 土曜日なら折り返し
    if (cal.get(Calendar.DAY_OF_WEEK) == Calendar.SATURDAY) {
        System.out.println();
    }
    cal.add(Calendar.DAY_OF_MONTH, 1);
}

そして指示によりimportは使用できないようになっています。

これを指示した方に、その理由を質問することを強くおすすめします。

何のための制限なのか、正直 理解できません。

あるいは、
「java.langパッケージ以外は使わないこと」
という意味なのかもしれません。

Comments

[退会済みユーザー]

定数に依存しない実装を教えていただきありがとうございます。
表示幅の簡潔な表し方についても大変勉強になりました！

ちなみにimportを使うな、という指示は私がフルネームでAPIメソッドを使用出来るかどうかをチェックしていただけだそうでした；

[KiyoshiMotoki]

お役に立てたなら、幸いです。

> 私がフルネームでAPIメソッドを使用出来るかどうかをチェックしていただけだそうでした；

それができたところで何がうれしいのか分かりかねますが(笑)、了解しました。
丁寧なご報告、ありがとうございます。

Answer 3

その月の最終日はgetActualMaximumというメソッドにCalendar.DATEを渡せば取得できます。
また、もっとCalendarクラスを活用すれば表示の部分も簡略化できそうです。

Comments

[退会済みユーザー]

最終日取得についてのご指摘ありがとうございます。
簡略化は是非とも行いたいところですので、『もっとCalendarクラスを活用』についてももう少し詳しくご教授願えませんでしょうか？