年月のリスト化

Question

いつもおせわになっております。
動的に取得できる年月があって
例2015/01～2016/01とあって
2015/01
2015/02
2015/03
.
.
.
2016/01と
出力したいのですが
カレンダークラスを使用してListを使う方法はわかりますが
どうしても処理が難読化してしまうためスマートな方法はありますでしょうか？

import java.util.ArrayList;
import java.util.Calendar;

public class Test3 {

	public static void main(String[] args) {
		// TODO 自動生成されたメソッド・スタブ

		String fromym = "201501";
		String toym = "201601";
		ArrayList<String> yms = new ArrayList<String>(10);
		Calendar calendar = Calendar.getInstance();
		calendar.set(Integer.valueOf(fromym.substring(0, 4)), Integer.valueOf(fromym.substring(4)) - 1, 1);

		// 最初の年月の確認
		System.out.println(String.valueOf(calendar.get(Calendar.YEAR))
				+ String.format("%02d", (calendar.get(Calendar.MONTH) + 1)));

		while (!toym.equals(String.valueOf(calendar.get(Calendar.YEAR))
				+ String.format("%02d", (calendar.get(Calendar.MONTH) + 1)))) {
			// 一月増やす
			calendar.add(Calendar.MONTH, 1);
			yms.add(String.valueOf(calendar.get(Calendar.YEAR))
					+ String.format("%02d", (calendar.get(Calendar.MONTH) + 1)));
		}
		// 出力して確認
		for (String ym : yms) {
			System.out.println(ym);
		}
	}

}

このようになり、かなり分かりづらくなってしまうのでもう少しわかりやすくしたいのです。

Comments

[yona]

スマートかどうかは比較しないと判断できません。あなたの書いたコードを追記してください。

[Zuishin]

複雑になりますか？
どの程度をもって複雑と思われてます？

与えられた範囲の最初の 1 日を取得して、それを 1 月ずつ増やせばいいだけのように思いますが、どのような形(文字列なのか Date なのか)で入力があってどのような形で出力するのでしょう？

Answer 1

簡潔さにこだわるならintでなんとかするのが良さそうな気もしますが、いろいろやり方はあるからなおさらどういう風に決めるのかでけっこう悩みそうですね。
その方法での回答はきりがないと思うのでやめておきます。

Java8以降でしか使えませんが、やはりDateTimeAPIとStreamで書くのが簡潔だと思います。
後出し気味で申し訳ないですが、同様の回答が無さそうなので書いておきます。

// import java.time.YearMonth;
// import java.time.format.DateTimeFormatter;
// import java.util.List;
// import java.util.stream.*;

YearMonth ym = YearMonth.of(2015, 1);
DateTimeFormatter dtf = DateTimeFormatter.ofPattern("yyyy/MM");
List<String> monthList = IntStream.range(0, 13)
                                  .mapToObj(x -> ym.plusMonths(x).format(dtf))
                                  .collect(Collectors.toList());
// => [2015/01, 2015/02, 2015/03, 2015/04, 2015/05, 2015/06, 2015/07, 2015/08, 2015/09, 2015/10, 2015/11, 2015/12, 2016/01]

IntStream.rangeで生成した数列は加算する月数で、それぞれの数値を基準となる年月に足したものに変換しつつフォーマットしてできあがりです。

---

（追記）

無限ストリームを使った方がよりスッキリする気がします。
（この辺、久しぶりに使ったのですぐに思いつきませんでした。）

// import java.time.YearMonth;
// import java.time.format.DateTimeFormatter;
// import java.util.List;
// import java.util.stream.*;

DateTimeFormatter dtf = DateTimeFormatter.ofPattern("yyyy/MM");
List<String> monthList = Stream.iterate(YearMonth.of(2015, 1), x -> x.plusMonths(1))
                               .limit(13)
                               .map(dtf::format)
                               .collect(Collectors.toList());
// => [2015/01, 2015/02, 2015/03, 2015/04, 2015/05, 2015/06, 2015/07, 2015/08, 2015/09, 2015/10, 2015/11, 2015/12, 2016/01]

2015/01から1月ごと加算する年月の無限ストリームを生成して、13個に制限し、フォーマットで文字列に変換しています。

---

（追記2）

上記の13（月数）を年月から算出するには下記のようにします。
（sipadan2003さんにご指摘いただきました。）
こうすると、YearMonth.of(2015, 1)が複数登場してしまうので、変数startYMなどに代入して重複を解消してください。

// import java.time.YearMonth;
// import java.time.temporal.ChronoUnit;

long monthCount = YearMonth.of(2015, 1).until(YearMonth.of(2016, 1), ChronoUnit.MONTHS) + 1;
// => 13

// これもほぼ同じ
// long monthCount = ChronoUnit.MONTHS.between(YearMonth.of(2015, 1), YearMonth.of(2016, 1)) + 1;

---

Java7以前なら、素直にforループで書いた方が無難です。

Comments

[sobue]

加算する月数はベタ打ちしないとだめなのですか？

[argius]

ベタ打ちというのはどれのことでしょうか？

[退会済みユーザー]

limit(13)のことでしょう。
通常、「13ヶ月分出力する」というより、「いつからいつまでを出力する」が多いでしょう。

[argius]

sipadan2003さん

ご指摘ありがとうございます。
既に月数を計算済みと思い込んでしまいました。
ハードコーディングという連想もできず。ボケてました。

[sobue]

今回、ベストアンサーは非常に悩みました。最も簡潔だと思ったコードにベストアンサーとさせていただきました。今回はjava8ではないので皆さんにはjava8での回答をしていただきましたので、申し訳ありませんでした。きちんとバージョンも伝えるようにします。答えていただいたjava8に関しては次回使用することがあれば使用させていただきます。ありがとうございます。

Answer 2

何をするプログラムかいまいちわからないんですが、結局この処理の後に残るのは文字列だけですよね？
1 年は 12 カ月と決まってるんだから、カレンダーへの変換は要らないんじゃないですか？

Comments

[sobue]

最終的にはエクセルに出力して表にしたかったのです。
１年はもちろん１２ヶ月と決まっていますがそれをリスト化して出力したかったのです

[Zuishin]

わざわざカレンダーに変換しなくても、数字を 1 ずつ増やしながら順に出力して 13 になったら年を 1 増やして月を 1 にするだけでいいんじゃないでしょうか？

[Zuishin]

可読性を落としている原因は主に二つあると思います。

一つは文字列とカレンダーと数字の三種類のデータの変換が入り混じっていること。swordone さんの示されたように、全部 int にまとめるのは大変効果的です。

もう一つは main 文だけで全部やっていること。
機能を関数にまとめることでかなり見通しが良くなるはずです。

Answer 3

難読化している最大の要因は、カレンダーから文字列への変換ですね。
SimpleDateFormatを使えば、このインスタンスを1つ用意しておくだけで、文字列変換が1文で書けます。

DateFormat format = new SimpleDateFormat("yyyyMM");
String now = format.format(calendar.getTime());

なお、他の方も回答していますが、年月を管理するのにわざわざCalendarを使う必要は無いのでは？
この操作に限定するなら、intで年と月をそれぞれ管理すればいいのでは？

int fromym = 201501;
int toym = 201601

int year = fromym / 100;
int month = fromym % 100;
int cal = fromym;
while(cal <= toym){
    System.out.println(cal);
    month++;
    if(month > 12){
        year += month / 12;
        month %= 12;
    }
    cal = year * 100 + month;
}

また、Calendarはバグがあるなど潜在的な問題があるので、Java8のAPIを使うことをおすすめします。

DateTimeFormatter formatter = DateTimeFormatter.ofPattern("uuuuMM");

//ベースが文字列なら、上のformatterを使ってparseメソッドを使うがここでは省略
YearMonth from = YearMonth.of(2015, 1);
YearMonth to = YearMonth.of(2016, 1);
ArrayList<YearMonth> yms = new ArrayList<>();

YearMonth now = from;

while(!to.isBefore(now)){
    yms.add(now);
    now = now.plus(1, ChronoUnit.MONTHS);
}

for(YearMonth ym : yms){
    System.out.println(formatter.format(ym));
}

Comments

[sobue]

いつも、詳しい回答ありがとうございます。int型も考えてはいたのですがなにをしているかがわかりにくいかな？と瞬間的に選択肢から消してました。全ての可能性を捨てずに考えたいと思います。

Answer 4

年月表示を月数に変換して処理する例を示します。
(ruby でかいてますが、処理方法は伝わると思います)
時分表記を分単位に置き換えて処理するのと同様です。(
１時間２分を 62分と置き換える、 逆に 72 分は 1時間12分と置き換える)
a.rb

fromym = 201501
toym = 201601

from_m = (fromym / 100) * 12 + (fromym % 100 - 1)
to_m   = (toym / 100) * 12 + (toym % 100 - 1)

(from_m..to_m).each do |m|
  puts((m / 12) * 100 + (m % 12 + 1))
end

実行結果

$ ruby a.rb
201501
201502
201503
201504
201505
201506
201507
201508
201509
201510
201511
201512
201601

Answer 5

###コード例 (Java 8)

性能は悪いですが、コードの可読性は良いと思います。

public static void main(String[] args) {
	final SimpleDateFormat format = new SimpleDateFormat("yyyy/MM");
	final Calendar to = new GregorianCalendar(2016, 1, 1);
	final Calendar cal = new GregorianCalendar(2015, 0, 1);
	final List<String> yms = new ArrayList<>();
	do {
		yms.add(format.format(cal.getTime()));
		cal.add(Calendar.MONTH, 1);
	}while(cal.before(to));
	yms.forEach(System.out::println);
}

###実行結果

2015/01
2015/02
 :
2015/12
2016/01

Comments

[swordone]

なぜわざわざJava8を持ちだしたのにCalendarクラスなのか

[退会済みユーザー]

質問者が「カレンダークラス」と要件を出しているからです。
DateTime APIを使うことには賛成ですが、質問者は可読性にもこだわっているようですから、それは議論点でしょう。ともあれ、サンプルを出してみたわけです。

Answer 6

年月を受け取って、次の年月を返す関数作ったら?
で、返ってきたのが最終年月になるまでループ
みたいな感じで。