それほど高負荷でしょうか？？？

Question

月初から今日の二日前までの日付を配列で取得しようとしています。

$config['now'] = new DateTimeImmutable;

function get_day_list($config){
	$day_list = array();
	$ed_date = $config['now']->modify('-2 day')->format('Y-m-d');//2日前
	$st_date = $config['now']->modify('first day of this month')->format('Y-m-d');
	$ed_ob = new DateTimeImmutable($ed_date);
	$st_ob = new DateTime($st_date);

	$stop = 0;
	if($st_ob<$ed_ob):
		while($stop<1):
			$date = $st_ob->format('Y-m-d');
			$day_list[] = $date;
			if($date==$ed_ob):
				++$stop;
			else:
				$st_ob = $st_ob->modify('+1 day');
			endif;
		endwhile;
	elseif($st_ob==$ed_ob):
		$day_list[]=$st_date;//1日だけ
	else://($st_ob>$ed_ob)
		return;//何もせずに関数を終える。返り値なし。
	endif;

	return $day_list;
}

起きている現象

やけに時間がかかり、

Fatal error: Allowed memory size of 1342177280 bytes exhausted (tried to allocate 4096 bytes) in* * * * * function.php on line 48

となる。
該当する箇所がwhileのところなので無限ループを疑い、$stop=0;の直前でこの二つのDateTimeオブジェクトをチェックしてみたが特に異常はない。

//$st_ob
DateTime Object
(
    [date] => 2019-06-01 00:00:00.000000
    [timezone_type] => 3
    [timezone] => Asia/Tokyo
)

//$ed_ob
DateTimeImmutable Object
(
    [date] => 2019-06-26 00:00:00.000000
    [timezone_type] => 3
    [timezone] => Asia/Tokyo
)

このエラーはどう回避すればよいのでしょうか・・・。

Comments

[scsi]

While分の中の変数を標準出力に出してみたらわかるのでは？

Answer 1

       if($date==$ed_ob):

文字列とオブジェクトの比較になっているので無限ループしていると思われます。
こうかな。

       if($date==$ed_date):

Answer 2

$date==$ed_ob がずっと false
$date,$ed_obの推移を確認するといいです。

Comments

[退会済みユーザー]

実際には推移以前の問題ですけど。。。

[Take_it]

while($stop<1):
$date = $st_ob->format('Y-m-d');
$day_list[] = $date;
if($st_ob==$ed_ob):
++$stop;
else:
$st_ob = $st_ob->modify('+1 day');
endif;
endwhile;

とすべきでした。。

Answer 3

バッチなどで利用していると時々出ますね。
Fatal error: Allowed memory size of

メモリーの使用量を計測する事と、下記にて使用量を変更してみては如何でしょうか？

ini_set("memory_limit", "200M");

Answer 4

こんな感じでよいのでは？

$offset=2;
$d1=range(strtotime(date("Y-m-1")),strtotime(date("Y-m-d")),60*60*24);
$d1=array_splice($d1,0,count($d1)>$offset?count($d1)-$offset:0);
$d2=array_map(function($x){return date("Y-m-d",$x);},$d1);
print_r($d2);

Comments

[yambejp]

今日が1日や2日だったときに2日前をどうするか次第ですね
わざわざ日付の処理をwhileでループする必要もないでしょう。
m日からn日までをrangeでとってo日前ということなら後ろからo個
データを削ればよいので

Answer 5

メモリの増減を追ってみては。

• php メモリ使用量測定メモ
• 【PHP】メモリ使用量の確認方法

もし大きなデータを保持している変数があるのでしたら用が済んだらunset()するとか、
必要な情報のみを取得するように配慮した方がいいかもしれません。