カレンダーの日付の色を変えたい

Question

$hori=[1,2,3,4,5]
for( $day=1; $day <= $countdate; $day++ ){
/////
}

上記のfor文で表示したカレンダーに
$horiに該当する日付を色を変えたいです。

array_searchを試しましたがうまくいきません。
よろしくお願いいたします。

for( $day=1; $day <= $countdate; $day++ ){
$w = date("w", mktime( 0, 0, 0, $next_month, $day, $now_year ) );
								
//スタイルシートの値設定ここから-----------------------------------
								  
if ( in_array( $day, (array)$hori, true ) ) {
								  	
$style = "color:#C30;";
								  
}
								
switch( $w ){
								    
case 0: //日曜日の文字色
								        
$style = "color:#C30;";
								       
 break;
								    
case 6: //土曜日の文字色
								        
$style = "color:#03C;";
								        
break;
								    
default: //月～金曜日の文字色
								        
$style = "color:#333;";
								}
								 
								
//スタイルシートの値設定ここまで-----------------------------------
								 
								
$line = $day."日（".$weekday[$w]."）"; //１行の定義：日付（曜日）
								 
								
/スタイルシートを挿入・1行ごとに改行して出力
								
echo '<tr><td style="'.$style.'">.$line."</td>"
								
}

Answer 1

$hori=[1,2,3,4,5]
for ( $day=1; $day <= $countdate; $day++ ) {
    if ( in_array( $day, $hori, true ) ) {}
} 

【PHP: in_array - Manual】
https://www.php.net/manual/ja/function.in-array.php

【PHPのin_arrayは罠が多いので注意喚起が必要 - Qiita】
https://qiita.com/tadsan/items/2a4c3e6b0b74a408c038#in_arrayに第三引数を渡す

Comments

[kazu130]

if ( in_array( $day, (array)$hori, true ) ) {
$style = "color:#C30;";
}
でやってみましたが色がかわらなかったのでもう少し試してみます。

[kei344]

とりあえず判定できているかをまず確認されては？

[kazu130]

判定結果がboolでした。まずは判定されるように考えてみます。

[kei344]

> 判定結果がbool
true/falseが採れませんか？

[kazu130]

falseでした！!

[kei344]

追記されたコードの問題は、ifで分岐したとしても必ずその後で$styleが上書きされていることが問題です。

[kazu130]

曜日の定義をしているのでうまくいかないのですね。他の方法で試してみます。

Answer 2

if ( in_array( $day, $hori, true ) ) {
   $style = "color:#C30;";
}
else { // ←これがないから
   switch( $w ){
...
}

Comments

[kazu130]

いけました！！ありがとうございます！！

Answer 3

「対象の配列に該当の情報が含まれるか」ならin_array()です。

• in_arrayを使うときは黙って第三引数を付けること

$horiに該当する日付を色を変えたいです。

あとは該当する日付に対してCSSで色指定するだけです。

---

ただ、もし「$hori」に該当するものが「日本の祝日」なのであれば
japanese-holidayというプラグインが有用です。
※composer必須、あと未来の年の祝日データ更新が年末ですのでcomposer updateの必要あり

会社所定の休日などであればこの限りではありませんが。

追記：
インデントがかなりおかしいのでそこをきちんとキレイにそろえる必要があるのと、提示コードのそのままだと構文エラーもあります。
エラー表示ONにしてますか？（構文チェックを自動で行ってくれるIDE導入を強くすすめます）

構文エラー箇所（あくまで一例）：
echo '<tr><td style="'.$style.'">.$line."</td>"
最後に;がないですしクォーテーションの組が抜けてたり合ってなかったり。

というのを一通り回避して提示されてない変数の未定義回避補完もした

<?php
//未定義回避のため勝手に補完したコードstart
$next_month = date('m',strtotime('+1 month'));
$now_year = date('Y');
$countdate = date('t',strtotime('+1 month'));
$weekday = [0=>'日',1=>'月',2=>'火',3=>'水',4=>'木',5=>'金',6=>'土'];
//未定義回避のため勝手に補完したコードend

$hori = [1, 2, 3, 4, 5];
for ($day = 1; $day <= $countdate; $day++)
{
    $w = date("w", mktime(0, 0, 0, $next_month, $day, $now_year));
    
    // スタイルシートの値設定ここから-----------------------------------
    
    if (in_array($day, (array)$hori, true))
    {
        $style = "color:#C30;";
    }
    
    switch ($w)
    {
        case 0: //日曜日の文字色
            $style = "color:#C30;";
            break;
            
        case 6: //土曜日の文字色
            $style = "color:#03C;";
            break;
            
        default: //月～金曜日の文字色
            $style = "color:#333;";
    }

    // スタイルシートの値設定ここまで-----------------------------------
    
    $line = $day."日（" . $weekday[$w] . "）";
    echo '<td style="'. $style .'">'.$line.'</td>'.PHP_EOL;
}

Comments

[kazu130]

in_array()を使用してやってみます。ありがとうございます。

[m.ts10806]

html出力のところまで全てのコードを提示された方が良さそうですね。
質問を編集してコード追記してください(そのときは<code>ご利用ください)

[kazu130]

返信ありがとうございます。
長い構文でしたので；漏れがあり申しわございません。
$next_month = date('m',strtotime('+1 month'));
$now_year = date('Y');
$countdate = date('t',strtotime('+1 month'));
$weekday = [0=>'日',1=>'月',2=>'火',3=>'水',4=>'木',5=>'金',6=>'土'];
は定義していました。
漏れていてすみません。

[m.ts10806]

変数提示の漏れや構文エラーよりもインデントのほうが気になります（エンジニアの職業病ですね・・）

[kazu130]

インデント直します・・・ありがとうございます！

[m.ts10806]

質問を提示するコードを直すのではなく必ずご自身の手元のコードを治してご提示くださいね。
普段のコーディングから意識するのはもちろんですが構文チェック＋フォーマット機能のあるエディタ利用するのが確実です。

Answer 4

話の腰を容赦なくバッサリと折らせてもらいます。

phpにおいて日付・時刻を扱う術を身に着けたほうがよろしいかと思います、
その理由は、日付情報をひとまとめにして必要なときに年や月は日や曜日などを抽出したほうが、
何日後や何日前、何週間前、何ヶ月前などの計算も自己流でやることによるミスを防げるからです。

PHP: Date/Time - Manual
DateTimeクラスやDateTimeImmutableクラスを今後もジャンジャン使うことになるかと思いますので
早めに慣れてください。

そして、日本固有の祝日を考慮した休みの日を判定するのに、
japanese-holiday/japanese-holiday - Packagist
【PHP】日本の祝日を扱うときは「Yasumi」を使ってみてはどうですか - Qiita
GitHub - holiday-jp/holiday_jp-php: Japanese holiday.
などいくつかあるようです。お好きなのをどうぞ。

Comments

[kazu130]

ありがとうございます

[tacsheaven]

https://www8.cao.go.jp/chosei/shukujitsu/gaiyou.html
ここから辿れる csv ファイル　https://www8.cao.go.jp/chosei/shukujitsu/syukujitsu.csv　（Shift-JIS 形式）
を使えば、当年と翌年の祝日設定が自動で可能だったりします。
（振替休日も休日として登録されています）

[kazu130]

上記を配列に格納して祝日判定を行おうとしていました。ありがとうございます

Answer 5

PHPはサーバー側の言語なので「色」という認識はありません
HTML上で色を変えるのであればタグ毎にスタイルシートを設定するか
クラスなど適当な属性をつけてCSSで色をつけてください

sample

$hori=[1,2,3,4,5];
$countdate=10;
for( $day=1; $day <= $countdate; $day++ ){
  $style=in_array($day,$hori)?" style=\"color:red\"":"";
  print "<span{$style}>{$day}</span>";
}

Comments

[kazu130]

質問の際にcssとつけずに申し訳ありませんでした。