一ヶ月を分割する様な計算

Question

１月を複数分割する様な計算をしたいのですが簡潔に記載できる様な気がするのですがいかんせん思いつかずこまっています
特にif文のto_dateを分割した日として処理で使う必要があるのですが、ここが自分で書いてて気持ち悪いです
書いたコードはjavaで３分割していますがshや別言語で記述するかもしれないため日付を文字列にしています

int MAX_LIMIT = 3;
String taget_ymd="20170731";
String taget_day = taget_ymd.substring(6, 8);
int date_term =  Integer.parseInt(taget_day) / MAX_LIMIT;
String to_date="";
for(int i=0;i<= MAX_LIMIT-1;i++){
	if(i==MAX_LIMIT-1){
		to_date = taget_day;
	} else {
		to_date = String.valueOf(i * date_term + date_term);
	}
	//１回ごとの処理処理
	System.out.println(to_date);
}

Comments

[takito]

分割で出た余りの処置はどのようにする方針ですか？最後を多くするのか、なるべく等分になるよう配分したいのか

[think49]

私もそれは気になります。私の回答では、あげられている「31」の事例を元に改善し、余剰分は後方から等しく配分する方針と仮定しました。29を3等分すると「9 -> 19 -> 29」になります。「10 -> 20 -> 29」の方が見た目はいいですが、そうすると「31」が「10 -> 20 -> 31」として扱う要件と合致しなくなります。個人的には余剰分は前方に等しく配分して「11 -> 21 -> 31」「10 -> 20 -> 29」とする方が好みです。

[takito]

「１月を複数分割する様な計算をしたい」ことの目的を教えてほしいですね。

[enigumalu]

1ヶ月の処理を複数分割して分割回毎に分割した期間の日(dd)を利用して処理を行いたいのです。内容的に3回程度がちょうどいいので3回としています。 余剰分に関しては3等分されていて月末がきちんとデータに適宜できればそれでいいとしています。 綺麗な状態にするには回答にあるように年月日の区分けを持つ必要がありますがそこまでを求めていないのです。

Answer 1

自分ならこうすると思います。

int day =  Integer.parseInt(taget_day);
double date_term =  (double)day / MAX_LIMIT;

for(int i = MAX_LIMIT - 1; i >= 0; i--){
    //１回ごとの処理処理
    System.out.println(String.valueOf(day - Math.floor(date_term * i)));
}

あと、割とどうでもいいことですが、

for(int i = 0; i <= MAX_LIMIT - 1; i++){

マイナス1して<=で比べるなら、

for(int i = 0; i < MAX_LIMIT; i++){

マイナスせずに<で比べたほうが見やすいです。
doubleとの比較等では上も有用ですが、int同士なら下の方がいいと思います。
個人的な意見なので、そういうこと考える人もいるんだなーくらいに思っていただければ。

Answer 2

int date = Integer.parseInt(taget_day);
for (int i = 1; i <= MAX_LIMIT; i++) {
    to_date = String.valueOf(date * i / MAX_LIMIT);
}

これでどう？

Answer 3

JavaScript タグがあるので、JavaScript で回答します。

元のコードの問題点

質問文のコードには、29 の三等分を試みた場合に三等分にならない問題があります。
[9, 9, 11] の配分となりますが、[9, 10, 10] もしくは [10, 10, 9] になるべき状況です。

• 整数値をN等分した配列を返す - JSFiddle

'use strict';
var MAX_LIMIT = 3;
var taget_ymd = "20120229";
var taget_day = taget_ymd.substring(6, 8);
var date_term =  Math.floor(taget_day / MAX_LIMIT);
var to_date="";

for (var i = 0; i <= MAX_LIMIT - 1; i++){
  if (i === MAX_LIMIT - 1){
    to_date = taget_day;
  } else {
    to_date = i * date_term + date_term;
  }
  //１回ごとの処理処理
  console.log(to_date); // 9 -> 18 -> 29
}

修正版コード

余剰分の数値は質問文のコードに合わせて、後方に持ってくるようにしました。
すなわち、29 を与えた場合は [9, 10, 10] の配分となります。

• 整数値をN等分した配列を返す (修正版) - JSFiddle

'use strict';
function toDividedInteger (number, divisor) {
  var floor = Math.floor,
      integers = [],
      divided, remainder;

  number = floor(number);
  divisor = floor(divisor);
  remainder = number % divisor;
  divided = (number - remainder) / divisor;

  for (var i = 0, j = divisor - remainder - 1, temp = 0; i < divisor; ++i) {
    temp += divided + Number(j < i);
    integers.push(temp);
  }

  return integers;
}

var integers = toDividedInteger('20120229'.slice(-2), 3);
console.log(JSON.stringify(integers)); // [9, 19, 29]

Re: eugenio さん

Comments

[ku__ra__ge]

質問文のコードは型宣言がありSystem.out.printlnを使っていることからjavaだと思いますが……。javaの場合、int型とint型の除算の結果は端数切り捨てなので問題ないと思います。

[think49]

Javaに詳しくないので外していたら申し訳ないのですが、端数の問題ではないと考えています。

String taget_ymd="20120229"; と仮定すると、date_term は 9 になります。
すると、出力される値が「9 -> 18 -> 29」となり、index 0, index 1 の範囲が「9」であるのに対して、index 2 だけ「11」の範囲を取るように読めました。
最終インデックスだけ最終値をとる帳尻合わせをした場合、三等分にはなりません。
私のコードの読み方が間違っているのでしょうか。

[ku__ra__ge]

失礼しました。回答の意図を読み違えていました。
確かに3つには分かれても、等分にはならない問題がありますね。

Answer 4

ループの変数をわざわざ1小さい数で回して、ループ内でそれの分を足してるような感じがします。
以下のようなコードでいいのでは？

for(int i=1; i<=MAX_LIMIT; i++){
	String to_date = String.valueOf(i * date_term);
	if (i == MAX_LIMIT) { to_date = taget_day; }
	
	System.out.println(to_date);
}

Answer 5

date_termがいらないんじゃないかな

to_date = (double)target_day / MAX_LIMIT * (i+1);

となるようにすれば、if文もいらない

Answer 6

こんなんどうでしょ?

int MAX_LIMIT = 3;
String taget_ymd = "20170731";
String taget_day = taget_ymd.substring(6, 8);
int date_term = Integer.parseInt(taget_day) / MAX_LIMIT;
for (int i=1; i<=(MAX_LIMIT - 1); i++) {
    to_date = String.valueOf(i * date_term);
    System.out.println(to_date);
}
System.out.println(taget_day);

Answer 7

おっしゃりたいことがよくわからないのですが、こういう感じでしょうか？

public class MonthSeparater {
    private final int[] dayTable = { 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31 };
    private int divisionNum;

    public MonthSeparater( int divisionNum ) {
        this.divisionNum = divisionNum;
    }
    public int computePeriod( String ymd ) {
        int month = computeMonthByYmd( ymd );
        int day = computeDayByYmd( ymd );
        return day * divisionNum / ( dayTable[ month ] + 1 ) + 1;
    }

    //
    // private methods
    private int computeMonthByYmd( String ymd ) {
        return Integer.parseInt( ymd.substring( 4, 6 ) );
    }
    private int computeDayByYmd( String ymd ) {
        return Integer.parseInt( ymd.substring( 6, 8 ) );
    }
}

微妙なネーミング、漂うマジックナンバー感、改善の余地は大いにあるコードですが。