java 日付の取得と変更について

Question

javaの日付の取得と変更に関するコードについて質問です

参考書のとおりに、下記のコードを書いて実行してみたのですが
実行してみても、現在の日付から5日後の日付が出力されず、現在の日付が出力されてしまいます。
どこを直せばよいのでしょうか。

import java.text.SimpleDateFormat;
import java.util.Calendar;
import java.util.Date;

public class Test{
public static void main(String[] args){
Date now = new Date();
Calendar c = Calendar.getInstance();
c.setTime(now);
int day = c.get(Calendar.DAY_OF_MONTH);
day += 5;
Date future = c.getTime();
c.set(Calendar.DAY_OF_MONTH,day);
SimpleDateFormat f = new SimpleDateFormat("西暦yyyy年MM月dd日");
System.out.println(f.format(future));
}
}

Answer 1

java での日付けの扱いは大きく変化しました。

java7 までの方法と、java8 の新しいクラスを利用した方法の両方を示します。

import java.text.SimpleDateFormat;
import java.time.LocalDateTime;
import java.time.format.DateTimeFormatter;
import java.util.Calendar;

public class DateSample{
    public static void main(String[] args) {
    	sub0();
    	System.out.println();
    	sub1();
    }

    // java7 までの方法
    static void sub0() {
        SimpleDateFormat sdf = new SimpleDateFormat("西暦yyyy年MM月dd日");

        Calendar cal = Calendar.getInstance(); // 今日
        System.out.println(sdf.format(cal.getTime()));

    	cal.add(Calendar.DAY_OF_MONTH, 5);     // 5 日後
        System.out.println(sdf.format(cal.getTime()));
    }

    // java8 の新しい方法
    static void sub1() {
    	LocalDateTime d = LocalDateTime.now(); // 今日
    	DateTimeFormatter f = DateTimeFormatter.ofPattern("西暦yyyy年MM月dd日");
    	System.out.println(d.format(f));

    	LocalDateTime d5 = d.plusDays(5);     // 5 日後
    	System.out.println(d5.format(f));
    }
}

実行例：

西暦2015年09月23日
西暦2015年09月28日

西暦2015年09月23日
西暦2015年09月28日

参考情報；

• ImmutableでスレッドセーフになったJavaの新しい日時APIの基礎知識 http://www.atmarkit.co.jp/ait/articles/1412/16/news041.html
• Java 8日時APIの主なメソッドとフォーマット用のパターン文字の使い方 http://www.atmarkit.co.jp/ait/articles/1501/29/news016.html
• Java8の日時APIはとりあえずこれだけ覚えとけ http://qiita.com/tag1216/items/91a471b33f383981bfaa

Answer 2

デバッグの仕方を覚えると、どこで間違っていたか発見しやすくなります。
一番のおすすめは、デバッグログを仕込んで、期待値通りの動きをしているかチェック。
下記は独自のデバッグログを入れてますが、log4jを使用した方がいいです。

private static final boolean DEBUG_LOG_MODE = true;

public static void main(String[] args) {
    Date now = new Date();
    debugLog("1.now",now); // ←1行1行デバックログを出力させる
    Calendar c = Calendar.getInstance();
    debugLog("2.c", c);
    c.setTime(now);
    debugLog("3.c", c);
    int day = c.get(Calendar.DAY_OF_MONTH);
    debugLog("4.day", day);
    day += 5;
    debugLog("5.day", day);
    Date future = c.getTime();
    debugLog("6.future",future);
    c.set(Calendar.DAY_OF_MONTH,day);
    debugLog("7.c", c);
    SimpleDateFormat f = new SimpleDateFormat("西暦yyyy年MM月dd日");
    debugLog("8.f", f);
    System.out.println(f.format(future));
}

/**
 *  デバッグ用
 *  objのクラス名とtoStringの内容を表示
 *  DEBUG_LOG_MODE=falseで非出力
 *
 * @param id どのログがどの位置で出力されたか識別できる文字
 * @param obj 出力させるオブジェクト
 */
private static void debugLog(String id,Object obj) {
	if(DEBUG_LOG_MODE == false) return;

	System.out.println(
			String.format("[DEBUG]%s Class名:%s.toString()=%s",
					id, obj.getClass(),obj.toString()
			)
	);
}

↓
出力結果

[DEBUG]1.now Class名:class java.util.Date.toString()=Wed Sep 23 15:19:49 JST 2015
[DEBUG]2.c Class名:class java.util.GregorianCalendar.toString()=java.util.GregorianCalendar[time=1442989189381,areFieldsSet=true,areAllFieldsSet=true,lenient=true,zone=sun.util.calendar.ZoneInfo[id="Asia/Tokyo",offset=32400000,dstSavings=0,useDaylight=false,transitions=10,lastRule=null],firstDayOfWeek=1,minimalDaysInFirstWeek=1,ERA=1,YEAR=2015,MONTH=8,WEEK_OF_YEAR=39,WEEK_OF_MONTH=4,DAY_OF_MONTH=23,DAY_OF_YEAR=266,DAY_OF_WEEK=4,DAY_OF_WEEK_IN_MONTH=4,AM_PM=1,HOUR=3,HOUR_OF_DAY=15,MINUTE=19,SECOND=49,MILLISECOND=381,ZONE_OFFSET=32400000,DST_OFFSET=0]
[DEBUG]3.c Class名:class java.util.GregorianCalendar.toString()=java.util.GregorianCalendar[time=1442989189338,areFieldsSet=true,areAllFieldsSet=true,lenient=true,zone=sun.util.calendar.ZoneInfo[id="Asia/Tokyo",offset=32400000,dstSavings=0,useDaylight=false,transitions=10,lastRule=null],firstDayOfWeek=1,minimalDaysInFirstWeek=1,ERA=1,YEAR=2015,MONTH=8,WEEK_OF_YEAR=39,WEEK_OF_MONTH=4,DAY_OF_MONTH=23,DAY_OF_YEAR=266,DAY_OF_WEEK=4,DAY_OF_WEEK_IN_MONTH=4,AM_PM=1,HOUR=3,HOUR_OF_DAY=15,MINUTE=19,SECOND=49,MILLISECOND=338,ZONE_OFFSET=32400000,DST_OFFSET=0]
[DEBUG]4.day Class名:class java.lang.Integer.toString()=23
[DEBUG]5.day Class名:class java.lang.Integer.toString()=28
[DEBUG]6.future Class名:class java.util.Date.toString()=Wed Sep 23 15:19:49 JST 2015
[DEBUG]7.c Class名:class java.util.GregorianCalendar.toString()=java.util.GregorianCalendar[time=?,areFieldsSet=false,areAllFieldsSet=true,lenient=true,zone=sun.util.calendar.ZoneInfo[id="Asia/Tokyo",offset=32400000,dstSavings=0,useDaylight=false,transitions=10,lastRule=null],firstDayOfWeek=1,minimalDaysInFirstWeek=1,ERA=1,YEAR=2015,MONTH=8,WEEK_OF_YEAR=39,WEEK_OF_MONTH=4,DAY_OF_MONTH=28,DAY_OF_YEAR=266,DAY_OF_WEEK=4,DAY_OF_WEEK_IN_MONTH=4,AM_PM=1,HOUR=3,HOUR_OF_DAY=15,MINUTE=19,SECOND=49,MILLISECOND=338,ZONE_OFFSET=32400000,DST_OFFSET=0]
[DEBUG]8.f Class名:class java.text.SimpleDateFormat.toString()=java.text.SimpleDateFormat@73b4b0ca
西暦2015年09月23日

※Eclipseのブレークポイント張ってデバッグ実行するやり方は、リモート環境でやりづらくなるのでオススメできません。

Answer 3

どのような参考書を、参考にされたかわかりませんが。

・上の方法だよ8/34等のありえない日付ができる
・Calendar c = Calendar.getInstance();は現在時刻のインスタンスを取得します。
・日付の操作はCalendarクラスだけで可能

import java.text.SimpleDateFormat;
import java.util.Calendar;
import java.util.Date;

public class Test{
    public static void main(String[] args){
        Calendar c = Calendar.getInstance();
        c.add(Calendar.DAY_OF_MONTH,5);
        SimpleDateFormat f = new SimpleDateFormat("西暦yyyy年MM月dd日");
        System.out.println(f.format(c.getTime()));
    }
}

Comments

[birdeater52]

回答ありがとうございます。参考書では、DateとCalendarを両方つかって、Calendar型にDate型で取得した日付を代入しながらやるやりかただったので、こういうコードになりました。
参考書のコードをよくみてみたら
c.set (Calendar.DAY_OF_MONTH,day); と
Date future = c.getTime(); を自分で上下逆に書いてたのに気づきました。

回答のとおりのaddメソッドを使ったやりかたも、これから自分でも書いてみて、
参考にさせていただきます
ありがとうございました

[yona]

すみません