PHPで条件に合わせて配列の出力を分けるのに苦戦しています。

Question

###【実現したいこと】
ＰＨＰで、次の【対象の配列】を、後述する【目的の出力】にしたいです。

そのための【考えたコード】について直すべき部分を教えてもらえませんでしょうか？

かなり長いです。すみません。。

###【対象の配列】

<?php
/*
配列
 - 同じ日付で 'adtion' の値は重複しない （'2019-01-02'に'escape'が２つあるとかは無い）
 - 'time' と 'id' は同じ数（いつ誰がやったかを示す）
 - 'page' は配列３つずつ増えていく（ページ分割に使う）
*/
$array = array(
	'2019-01-02'=> array(
        array(
        'action'=>'escape',
        'dialogue'=>'覚えてやがれー！',
        'time'=>array('2019-01-02 18:00:00.000','2019-01-01 13:00:00.000'), 
        'last_update'=>'2019-01-02 18:00:00.000',        
        'id'=>array('9','1'), 
        'page'=>'1',
        ),
        array(
        'action'=>'attack',
        'target'=>'6', 
        'time'=>array('2019-01-02 15:00:00.000','2019-01-02 01:00:00.000'), 
        'last_update'=>'2019-01-02 15:00:00.000',        
        'id'=>array('14','8'),   
        'page'=>'1', 
        ), 
	),   
	'2019-01-01'=> array( 		  
        array(
        'action'=>'propose',
		'dialogue'=>'たこ焼きはいかがでしょうか？',
        'time'=>array('2019-01-01 15:00:00.000','2019-01-01 01:00:00.000'), 
        'last_update'=>'2019-01-01 15:00:00.000',        
        'id'=>array('36','5'),
        'page'=>'1',
        ),
        array(
        'action'=>'attack', 
        'target'=>'7', 
        'time'=>array('2019-01-01 16:00:00.000','2018-12-31 22:00:00.000','2018-12-25 12:00:00.000'),     
        'last_update'=>'2019-01-01 16:00:00.000',        
        'id'=>array('1','15','3'),
        'page'=>'2', // ４つ目の配列なので２になる
        ), 		
    ), 		
);
?>

###【目的の出力】
上の【対象の配列】をこのようにしたいです。

※'page'=>'2'の配列は次のページとして扱うので、上の配列は４つありますがここの出力は'page'=>'1'の３つになります。

<ul>
	<li> 
		<p>escapeした人の一覧です。</p>
		<ul><li>9番さん</li><li>1番さん</li></ul>
		<p>覚えてやがれー！</p>
		<p>2019-01-02</p>
	</li>
	<li> 
		<p>6番さんがattackされました。</p>
		<p>attackした人の一覧です。</p>
		<ul><li>14番さん</li><li>8番さん</li></ul>
		<p>2019-01-02</p>
	</li>
	<li> 
		<p><a href="36">36番</a>さんと<a href="5">5</a>番さんがproposeしています。</p>
		<p>たこ焼きはいかがでしょうか？</p>
		<p>2019-01-01</p>
	</li>
<ul>

###【考えたコード】
追記：質問時のコードが支離滅裂であったので、papinianus様から質問内容に沿った形でご修正頂きました。

<?php
/*
配列
 - 同じ日付で 'adtion' の値は重複しない （'2019-01-02'に'escape'が２つあるとかは無い）
 - 'time' と 'id' は同じ数（いつ誰がやったかを示す）
 - 'page' は配列３つずつ増えていく（ページ分割に使う）
*/
$array = array(
    '2019-01-02'=> array(
        array(
        'action'=>'escape',
        'dialogue'=>'覚えてやがれー！',
        'time'=>array('2019-01-02 18:00:00.000','2019-01-01 13:00:00.000'), 
        'last_update'=>'2019-01-02 18:00:00.000',        
        'id'=>array('9','1'), 
        'page'=>'1',
        ),
        array(
        'action'=>'attack',
        'target'=>'6', 
        'time'=>array('2019-01-02 15:00:00.000','2019-01-02 01:00:00.000'), 
        'last_update'=>'2019-01-02 15:00:00.000',        
        'id'=>array('14','8'),   
        'page'=>'1', 
        ), 
    ),   
    '2019-01-01'=> array(           
        array(
        'action'=>'propose',
        'dialogue'=>'たこ焼きはいかがでしょうか？',
        'time'=>array('2019-01-01 15:00:00.000','2019-01-01 01:00:00.000'), 
        'last_update'=>'2019-01-01 15:00:00.000',        
        'id'=>array('36','5'),
        'page'=>'1',
        ),
        array(
        'action'=>'attack', 
        'target'=>'7', 
        'time'=>array('2019-01-01 16:00:00.000','2018-12-31 22:00:00.000','2018-12-25 12:00:00.000'),     
        'last_update'=>'2019-01-01 16:00:00.000',        
        'id'=>array('1','15','3'),
        'page'=>'2', // ４つ目の配列なので２になる
        ),         
    ),         
);

/*
idをリストで出力するための関数
*/
function echo_list_id($array/* //here 追加*/){
    foreach ($array['id'] as $id) { //here
       echo '<li>'.$id.'番さん</li>'; //here
    }
}

/*
配列をリストで出力するための関数
*/
function echo_list_content($array/* //here $pageはやめた*/, $page = "1" /* //here 追加*/){
    // リスト出力
    echo '<ul>';
    foreach ($array as $key => $ks) {
        foreach($ks as $k){ //here
        $date = substr($k['last_update'], 0, strcspn($k['last_update'],' ')); // 日付だけ取得
        // 該当ページの配列だけ出力する
        if( $k['page'] == $page ){
            // attackの場合
            if( $k['action'] == 'attack' ){
                echo '<li>';  
                    echo '<p>'.$k['target'].'番さんがattackされました！</p>';
                    echo '<p>attackした人の一覧です。</p>';
                    echo '<ul>'.echo_list_id($k).'</ul>';
                    echo '<p>'.$date.'</p>';
                echo '</li>';    
            } 
            // escapeの場合
            if( $k['action'] == 'escape' ){
                echo '<li>';  
                    echo '<p>escapeした人の一覧です。</p>';
                    echo '<ul>'.echo_list_id($k).'</ul>';
                    echo '<p>'.$k['dialogue'].'</p>';
                    echo '<p>'.$date.'</p>';                    
                echo '</li>';    
            } 
            // proposeの場合
            if( $k['action'] == 'propose' ){
                echo '<li>';  
                    //here echo '<p><a href="'.$k['page'][0].'">'.$k['page'][0].'番</a>さんと<a href="'.$k['page'][1].'">'.$k['page'][1].'番</a>さんがproposeしています。</p>';
                    echo '<p>'.$k['dialogue'].'</p>';
                    echo '<p>'.$date.'</p>';                    
                echo '</li>';    
            }                     
        }
        } //here
    }
    echo '</ul>';
    // 次のページを更新
    $nextpage = $page++;
    //setcookie("page", $nextpage); //here 出力後にcookieかいちゃだめやで
}

/*
ページごとにリスト出力
*/
// ページを取得
$page = isset($_COOKIE["page"]) ? $_COOKIE["page"] : "";
// リスト出力
echo_list_content($array);

Comments

[plato]

具体的に躓いている点はどこになるのでしょう？
エラーや現状の出力がうまくいかない等の場合，それらを記載されたほうがいいと思います．

[m.ts10806]

この内容だとそもそも対象のfunctionに$arrayが届いていないような？

[love_kinniky]

遅くなりました。ご指摘ありがとうございます。解決いたしました。

Answer 1

エラーなく実行できるところがこれなんで、とりあえず、質問をこれに修正してもらえませんか？
//hereっていうところが添削したところです。すみませんが、どう違うかは自分でみてください。

まだ理想になっているかまで見てませんが、とにかくここが出発点です。

<?php
/*
配列
 - 同じ日付で 'adtion' の値は重複しない （'2019-01-02'に'escape'が２つあるとかは無い）
 - 'time' と 'id' は同じ数（いつ誰がやったかを示す）
 - 'page' は配列３つずつ増えていく（ページ分割に使う）
*/
$array = array(
    '2019-01-02'=> array(
        array(
        'action'=>'escape',
        'dialogue'=>'覚えてやがれー！',
        'time'=>array('2019-01-02 18:00:00.000','2019-01-01 13:00:00.000'), 
        'last_update'=>'2019-01-02 18:00:00.000',        
        'id'=>array('9','1'), 
        'page'=>'1',
        ),
        array(
        'action'=>'attack',
        'target'=>'6', 
        'time'=>array('2019-01-02 15:00:00.000','2019-01-02 01:00:00.000'), 
        'last_update'=>'2019-01-02 15:00:00.000',        
        'id'=>array('14','8'),   
        'page'=>'1', 
        ), 
    ),   
    '2019-01-01'=> array(           
        array(
        'action'=>'propose',
        'dialogue'=>'たこ焼きはいかがでしょうか？',
        'time'=>array('2019-01-01 15:00:00.000','2019-01-01 01:00:00.000'), 
        'last_update'=>'2019-01-01 15:00:00.000',        
        'id'=>array('36','5'),
        'page'=>'1',
        ),
        array(
        'action'=>'attack', 
        'target'=>'7', 
        'time'=>array('2019-01-01 16:00:00.000','2018-12-31 22:00:00.000','2018-12-25 12:00:00.000'),     
        'last_update'=>'2019-01-01 16:00:00.000',        
        'id'=>array('1','15','3'),
        'page'=>'2', // ４つ目の配列なので２になる
        ),         
    ),         
);

/*
idをリストで出力するための関数
*/
function echo_list_id($array/* //here 追加*/){
    foreach ($array['id'] as $id) { //here
       echo '<li>'.$id.'番さん</li>'; //here
    }
}

/*
配列をリストで出力するための関数
*/
function echo_list_content($array/* //here $pageはやめた*/, $page = "1" /* //here 追加*/){
    // リスト出力
    echo '<ul>';
    foreach ($array as $key => $ks) {
        foreach($ks as $k){ //here
        $date = substr($k['last_update'], 0, strcspn($k['last_update'],' ')); // 日付だけ取得
        // 該当ページの配列だけ出力する
        if( $k['page'] == $page ){
            // attackの場合
            if( $k['action'] == 'attack' ){
                echo '<li>';  
                    echo '<p>'.$k['target'].'番さんがattackされました！</p>';
                    echo '<p>attackした人の一覧です。</p>';
                    echo '<ul>'.echo_list_id($k).'</ul>';
                    echo '<p>'.$date.'</p>';
                echo '</li>';    
            } 
            // escapeの場合
            if( $k['action'] == 'escape' ){
                echo '<li>';  
                    echo '<p>escapeした人の一覧です。</p>';
                    echo '<ul>'.echo_list_id($k).'</ul>';
                    echo '<p>'.$k['dialogue'].'</p>';
                    echo '<p>'.$date.'</p>';                    
                echo '</li>';    
            } 
            // proposeの場合
            if( $k['action'] == 'propose' ){
                echo '<li>';  
                    //here echo '<p><a href="'.$k['page'][0].'">'.$k['page'][0].'番</a>さんと<a href="'.$k['page'][1].'">'.$k['page'][1].'番</a>さんがproposeしています。</p>';
                    echo '<p>'.$k['dialogue'].'</p>';
                    echo '<p>'.$date.'</p>';                    
                echo '</li>';    
            }                     
        }
        } //here
    }
    echo '</ul>';
    // 次のページを更新
    $nextpage = $page++;
    //setcookie("page", $nextpage); //here 出力後にcookieかいちゃだめやで
}

/*
ページごとにリスト出力
*/
// ページを取得
$page = isset($_COOKIE["page"]) ? $_COOKIE["page"] : "";
// リスト出力
echo_list_content($array);

Comments

[love_kinniky]

お世話になっております。質問のコードを仰るように修正致しました。proposeのとこでpageを出すとか、支離滅裂すぎましたね。ご修正ありがとうございます。後のご回答もこれからじっくり読ませて頂きたいと思います。ご連絡遅くなってしまい申し訳ございませんでした。

Answer 2

前の回答で質問が更新された「ゼンテイ」で回答します。

• 改行とかインデントはまあとりあえずやったけどこの際無視する

• 問題点

1. echo_list_idがダメ。配列の構造も引数の取りかたも何もわかってないけど、本質的にはechoしているのがだめ。呼び出し元のecho文が引数(っていうのかな)を評価したときに、この関数がはしるからここでechoするとこちらのechoが終わってから、元のechoが走る

従って、「liタグ内容は変動する」→「<ul></ul>」の順で出力される。
0. そこでsetcookieしちゃだめ
0. 自分で自分のデータ構造が分かってない、proposeのとこでpageを出そうとしていたが、id。このレベルで違うと仕様を知らない他人が読み取るのは不可能に近い

<?php
/*
配列
 - 同じ日付で 'adtion' の値は重複しない （'2019-01-02'に'escape'が２つあるとかは無い）
 - 'time' と 'id' は同じ数（いつ誰がやったかを示す）
 - 'page' は配列３つずつ増えていく（ページ分割に使う）
*/
$array = array(
    '2019-01-02'=> array(
        array(
        'action'=>'escape',
        'dialogue'=>'覚えてやがれー！',
        'time'=>array('2019-01-02 18:00:00.000','2019-01-01 13:00:00.000'), 
        'last_update'=>'2019-01-02 18:00:00.000',        
        'id'=>array('9','1'), 
        'page'=>'1',
        ),
        array(
        'action'=>'attack',
        'target'=>'6', 
        'time'=>array('2019-01-02 15:00:00.000','2019-01-02 01:00:00.000'), 
        'last_update'=>'2019-01-02 15:00:00.000',        
        'id'=>array('14','8'),   
        'page'=>'1', 
        ), 
    ),   
    '2019-01-01'=> array(           
        array(
        'action'=>'propose',
        'dialogue'=>'たこ焼きはいかがでしょうか？',
        'time'=>array('2019-01-01 15:00:00.000','2019-01-01 01:00:00.000'), 
        'last_update'=>'2019-01-01 15:00:00.000',        
        'id'=>array('36','5'),
        'page'=>'1',
        ),
        array(
        'action'=>'attack', 
        'target'=>'7', 
        'time'=>array('2019-01-01 16:00:00.000','2018-12-31 22:00:00.000','2018-12-25 12:00:00.000'),     
        'last_update'=>'2019-01-01 16:00:00.000',        
        'id'=>array('1','15','3'),
        'page'=>'2', // ４つ目の配列なので２になる
        ),         
    ),         
);

/*
idをリストで出力するための関数
*/
function echo_list_id($array/* //here 追加*/){
    $str = "";
    foreach ($array['id'] as $id) { //here
       $str .= '<li>'.$id.'番さん</li>'; //here
    }
    return $str;
}

/*
配列をリストで出力するための関数
*/
function echo_list_content($array/* //here $pageはやめた*/, $page = "1" /* //here 追加*/){
    // リスト出力
    echon('<ul>');
    foreach ($array as $key => $ks) {
        foreach($ks as $k){ //here
        $date = substr($k['last_update'], 0, strcspn($k['last_update'],' ')); // 日付だけ取得
        // 該当ページの配列だけ出力する
        if( $k['page'] == $page ){
            // attackの場合
            if( $k['action'] == 'attack' ){
                echon('    <li>');  
                    echon('        <p>'.$k['target'].'番さんがattackされました！</p>');
                    echon('        <p>attackした人の一覧です。</p>');
                    echon('        <ul>'.echo_list_id($k).'</ul>');
                    echon('        <p>'.$date.'</p>');
                echon('    </li>');    
            } 
            // escapeの場合
            if( $k['action'] == 'escape' ){
                echon('    <li>');  
                    echon('        <p>escapeした人の一覧です。</p>');
                    echon('        <ul>'.echo_list_id($k).'</ul>');
                    echon('        <p>'.$k['dialogue'].'</p>');
                    echon('        <p>'.$date.'</p>');                    
                echon('    </li>');    
            } 
            // proposeの場合
            if( $k['action'] == 'propose' ){
                echon('    <li>');  
                    echon('        <p><a href="'.$k['id'][0].'">'.$k['id'][0].'番</a>さんと<a href="'.$k['id'][1].'">'.$k['id'][1].'番</a>さんがproposeしています。</p>');//here 
                    echon('        <p>'.$k['dialogue'].'</p>');
                    echon('        <p>'.$date.'</p>');                    
                echon('    </li>');    
            }                     
        }
        } //here
    }
    echon('</ul>');
    // 次のページを更新
    $nextpage = $page++;
    //setcookie("page", $nextpage);
}
function echon($str) {
    echo $str.PHP_EOL;
}

/*
ページごとにリスト出力
*/
// ページを取得
$page = isset($_COOKIE["page"]) ? $_COOKIE["page"] : "";
// リスト出力
echo_list_content($array);

Comments

[love_kinniky]

こんばんは。夜分遅くに失礼致します。
こちらのコードでイメージしていたようにできました。いつも本当にありがとうございます。
以下の２についてだけ、もしよろしければ教えて下さい。

-----
１について

echo_list_id($array)の$arrayは、echo_list_contentの中でecho_list_id($k)と使うことで取得できるわけですね。勉強になりました。

-----
２について

これだけ教えて頂きたいのですが、せっかく「$nextpage = $page++;」と増やしているのに、それをcookieに入れてあげる機会がなさそうに見えます。

いま考えているのは、「<button type="button" data-pagenumber="'.$page.'">もっと見る</button>」のようなボタンを作り、このボタンの$pageにはcookieの値を入力。そしてAjaxでその$pageにふさわしい配列を読んでいく。という機能です。
ですが$nextpageをcookieに入れてあげる機会がないと、このようなページ送りができないのではないかと思っています。

それともページ送りについては、何かもっと良い方法があったりしますでしょうか？

-----
３について

先ほども申し上げましたが、本当に解読して頂きありがとうございました。

[papinianus]

ページ送りをどうするのがベストかはともかく、echoしたあとにsetcookieしちゃだめなだけで、一回ボタンを押したときには、必ずページは1しか繰り上がらないわけですから
// ページを取得
$page = isset($_COOKIE["page"]) ? $_COOKIE["page"] : "";
の直後に
setcookieすればいいだけです。

もしくは面倒にはなりますが、echo_list_contentをechoではなく、文字列をreturnする関数に全面書き変えすれば、元の位置でsetcookieしても問題はなくなります。