PHPで配列の中身をページネーションを使って表示させたいです

Question

前提・実現したいこと

配列$testに入っているデータを最大3件表示させ、ページネーションで3件目以降を表示するようにさせたいです。
$testには９件のデータが入ってます。

参考にさせて頂いた記事
https://www.sejuku.net/blog/70234

発生している問題・エラーメッセージ

現在のページを取得し、if文でリンクをつけるのかを判断しているのですが、変数への代入が上手く行われていないのか、if文が機能していません。
               

該当のソースコード

<?php if(have_posts()): the_post(); ?>
        
        <?php
            define('MAX','3');
            $test = scf::get('test');
            $test_num = count($test);
            $max_page = ceil($test_num / MAX);
            define ($URL, (empty($_SERVER['HTTPS']) ? 'http://' : 'https://').$_SERVER['HTTP_HOST'].$_SERVER['SCRIPT_NAME']);

            if(!isset($_GET['page_id'])){
                $now = 1;
            }else{
                $now = $_GET['page_id'];
            }

            $start_no = ($now - 1) * MAX;

            $disp_data = array_slice($test, $start_no, MAX, true);

            foreach ($disp_data as $fields ) {
            echo  '<p id="comment">' .$fields['comment']. '</p>';
        } 

        echo '<ul class="pagination">';

             if($now > 1){ // リンクをつけるかの判定
                    echo '<a href=' .$URL.'?page_id="'.($now - 1).'")><li id="prev"></li></a>'. '　';
                } else {
                    echo '<li id="prev"></li>'. '　';
                }

                for($i = 1; $i <= $max_page; $i++){
                    if ($i == $now) {
                        echo '<li class="page">' .$now. '</li>'. '　'; 
                    } else if ($i == 1){
                        echo '<a href=' .$URL.')><li>'. $i. '</li></a>'. '　';
                    } else{
                        echo '<a href=' .$URL.'?page_id='. $i. ')><li>'. $i. '</li></a>'. '　';
                    }
                }
                if($now < $max_page){ // リンクをつけるかの判定
                    echo '<a href=' .$URL.'?page_id="'.($now + 1).'")><li id="next"></li></a>'. '　';
                } else if ($now == $max_page){
                    echo '<li id="next"></li>';
                }
            
                echo '</ul>';
        ?>

        <?php endif; ?>
        <?php if( function_exists("the_pagination") ) the_pagination(); ?>

試したこと

echo '<a href=' .$URL.'?page_id='.($now + 1).')><li id="next"></li></a>'. '　';

「page_id='.($now + 1).'」の部分を二重引用符で囲む（page_id="'.($now + 1).'"）と２ページ目は表示されるのですが、４件目〜６件目ではなく７件目〜９件目が表示されます。
二重引用符で囲まず数字として代入した場合2ページ目は表示されず、WordPressテーマのtopページに飛ばされます。URLはwordpress/sample-page__trashed/です

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

• ulはli以外を子要素にできない
• aはブロック要素を囲めない
• 英数字及び-_.:以外の文字を含む属性値は""で囲まなくてはならない

<a href=http://example.com?page_id=2><li id="next"></li></a>
↓
<li id="next"><a href="http://example.com?page_id=2"></a></li>

cssもこれに合わせて書き直し

---

ここもおかしいですね

 define ($URL, (empty($_SERVER['HTTPS']) ? 'http://' : 'https://').$_SERVER['HTTP_HOST'].$_SERVER['SCRIPT_NAME']);

defineの使い方を確認してください
また、URLの値の方も多分意図通りのものではないでしょう、確認してみてください

現在のページのURLにパラメータをつけたいなら
せっかくWordPressがadd_query_argという便利な関数を用意してくれてるので
こっちを使ってやる方がいいでしょう

例えば表示数や表示モード切り替えの機能を追加したときに
そのURLパラメータを引き継ぎつつページ切り替えをしたい時とかには
add_query_argを使っていた方が対応が楽です

Comments

[unionbiz]

ご回答有難うございます。
```
echo '<ul class="pagination">';

if($now > 1){ // リンクをつけるかの判定
echo '<li id="prev"><a href="' .$URL.'?page_id='.($now - 1).')"></a></li>'. '　';
} else {
echo '<li id="prev"></li>'. '　';
}

for($i = 1; $i <= $max_page; $i++){
if ($i == $now) {
echo '<li class="page">' .$now. '</li>'. '　';
} else if ($i == 1){
echo '<li><a href=' .$URL.')>'. $i. '</a></li>'. '　';
} else{
echo '<li><a href="' .$URL.'?page_id='. $i. ')">'. $i. '</a></li>'. '　';
}
}
if($now < $max_page){ // リンクをつけるかの判定
echo '<li id="next"><a href="' .$URL.'?page_id='.($now + 1).')"></a></li>'. '　';
} else if ($now == $max_page){
echo '<li id="next"></li>';
}

echo '</ul>';
```
aタグをliで囲み、hrefの中身をダブルクウォーテーションで囲んでみたところURLはpage_id=2、page_id=3と表示されるのですがリンクに飛びません。page_id=2はワードプレスのテーマトップ(URL /sample-page__trashed/)に、page_id=3以降はページも移動せず、URLも変わりません。