#pdoの検索結果を代入した変数$resultがnullになってしまう

###pdoにて検索した値を$resultに代入しているのですが、それがhtmlで使う時にnullになってしまいます。

いつもお世話になっております。
よろしくお願い致します。

現在、phpにて記事タイトルのtext検索機能を実装しております。

##出来ないこと

splで検索し、それを$resultに代入しておりますが、そのresulutの中身がnullになってしまっています。var_dump()でpdo実行後の$resultの中身を見ると値が入っているのですが、それが使えません。
また、ファイルまたぎでのデータの受け渡しではなく、1つのファイルで利用しております。

初学者なので、コードも非常に長く汚いです。見にくくて大変申し訳ありません。
初歩的な質問で申し訳ありませんが、よろしくお願い致します。

###以下、ソースコードです。

makeNews.php

php
1if ($_SERVER['REQUEST_METHOD'] === 'GET') {
2
3    if (isSet($_GET['name'])) {
4        $name = $_GET['name'];
5        try {
6          $db = new PDO('mysql:dbname=test_db;host=localhost', 'root', '');
7         //この辺りはページャです。
8          define('MAX','10');
9  
10          $page = $_GET['page'];
11          
12          if (isset($_GET['page'])) {
13            $now = 1;
14          } else {
15            $now = $_GET['page_id'];
16          }
17          
18          if ($now > 1) {
19            $start = ($now * 10) - 10;
20          } else {
21            $start = 0;
22          }
23         //問題の部分です。
24          $result = $db->prepare("
25            SELECT *
26            FROM news
27            WHERE title = ?
28            ORDER BY id DESC
29            LIMIT {$start}, 10
30          ");
31          $result->bindValue(1, $name);
32          $result->execute();
33          $result = $result->fetchAll(PDO::FETCH_ASSOC);
34          //ここでvar_dump();を行い$resultを確認すると、検索した値が入っています。
35          
36         //以下もページャです。
37          $page_num1 = $db->prepare("
38            SELECT COUNT(*) id
39            FROM news
40            WHERE title = :name
41          ");
42          $page_num1->bindParam(':name', $name, PDO::PARAM_STR);
43          $page_num1->execute();
44          $page_num1 = $page_num1->fetchColumn();
45          
46          $pagination = $page_num1 / 10;
47          $pagination = ceil($page_num1 / 10);
48          $max_page = ceil($page_num1 / MAX);
49
50        } catch (PDOException $exception) {
51            echo $exception->getMessage();
52            exit();
53        }
54    }
55}
56

そして、上記で取得した$resultを使いたいのが

makeNews.php

php
1
2
3<?php if ($_SERVER['REQUEST_METHOD'] === 'GET') { ?>
4              //結果がなければ該当結果なしと表示されます。今回はこちらの値になってしまいます。
5              <?php if (empty($result)) {
6               //上記の$resultをvar_dumpすると配列には中身が入っていないことが確認できました。
7                ?>
8                  <strong>該当する結果はなし。</strong>
9              <?php } else { ?>
10                <table>
11                  <tr>
12                    <th>ID</th> <th>タイトル</th> <th>表示時間</th> <th>編集</th> <th>削除</th>
13                  </tr>
14                  <?php foreach ($result as $value) {?>
15                  <tr>
16                    <td><?php echo $value['id'] ?></td> <td><?php echo $value['title'] ?></td> <td><?php echo date('Y年m月d日',  strtotime($value['day'])) ?></td> <td><a href="http://iwaki.icoma.jp/edit_news.php?id=<?php echo $value['id'] ?>">編集<a></td> <td><a href="http://iwaki.icoma.jp/delete_news.php?id=<?php echo $value['id'] ?>">削除<a></td>
17                  </tr>
18                  <?php } ?>
19                </table>

どこかで$resultの値がnullに変わってしまっているのでしょうか。
どなたか、ヒントでもなんでも宜しいのでご教授いただけますと幸いです。

値を入れるフォームは

makeNews.php

php
1//日付のフォームは省略しております。
2
3            <label>
4                <input type='text' name='name'>
5            </label>
6            <input type='submit' value='検索'>
7            <input type="hidden" name="day" value="<?php echo $_GET['year1'].'-'. $_GET['month1'].'-'. $_GET['date1']; ?>">
8            <input type="hidden" name="dayAfter" value="<?php echo $_GET['year'].'-'. $_GET['month'].'-'. $_GET['date']; ?>">
9        </form>

###行ったこと

ちなみに、このpdoのコードの下に日付での検索コードがあります。
そちらのコードのせいで初期化されているのでしょうか。
一応、そちらも含めたコードを貼っておきます。

makeNews.php

php
1
2if ($_SERVER['REQUEST_METHOD'] === 'GET') {
3
4    if (isSet($_GET['name'])) {
5        $name = $_GET['name'];
6        try {
7          $db = new PDO('mysql:dbname=test_db;host=localhost', 'root', '');
8          define('MAX','10');
9  
10          $page = $_GET['page'];
11          
12          if (isset($_GET['page'])) {
13            $now = 1;
14          } else {
15            $now = $_GET['page_id'];
16          }
17          
18          if ($now > 1) {
19            $start = ($now * 10) - 10;
20          } else {
21            $start = 0;
22          }
23          $result = $db->prepare("
24            SELECT *
25            FROM news
26            WHERE title = ?
27            ORDER BY id DESC
28            LIMIT {$start}, 10
29          ");
30          $result->bindValue(1, $name);
31          $result->execute();
32          $result = $result->fetchAll(PDO::FETCH_ASSOC);
33          
34          $page_num1 = $db->prepare("
35            SELECT COUNT(*) id
36            FROM news
37            WHERE title = :name
38          ");
39          $page_num1->bindParam(':name', $name, PDO::PARAM_STR);
40          $page_num1->execute();
41          $page_num1 = $page_num1->fetchColumn();
42          
43          $pagination = $page_num1 / 10;
44          $pagination = ceil($page_num1 / 10);
45          $max_page = ceil($page_num1 / MAX);
46
47        } catch (PDOException $exception) {
48            echo $exception->getMessage();
49            exit();
50        }
51    }
52}
53
54if ($_SERVER['REQUEST_METHOD'] === 'GET') {
55
56  if (isset($_GET["year"]) && isset($_GET["month"]) && isset($_GET["date"])){
57      $day = $_GET['year1'].'-'. $_GET['month1'].'-'. $_GET['date1']; 
58      $dayAfter = $_GET['year'].'-'. $_GET['month'].'-'. $_GET['date'];
59      try {
60        $db = new PDO('mysql:dbname=test_db;host=localhost', 'root', '');
61        define('MAX','10');
62
63        $page = $_GET['page'];
64        
65        if (isset($_GET['page'])) {
66          $now = 1;
67        } else {
68          $now = $_GET['page_id'];
69        }
70        
71        if ($now > 1) {
72          $start = ($now * 10) - 10;
73        } else {
74          $start = 0;
75        }
76        $result = $db->prepare("
77          SELECT *
78          FROM news
79          WHERE day BETWEEN ? AND ?
80          ORDER BY id DESC
81          LIMIT {$start}, 10
82        ");
83        $result->bindValue(1, $day, PDO::PARAM_STR);
84        $result->bindValue(2, $dayAfter, PDO::PARAM_STR);
85        $result->execute(array($day,$dayAfter));
86        $result = $result->fetchAll(PDO::FETCH_ASSOC);
87        
88        $page_num1 = $db->prepare("
89          SELECT COUNT(*) id
90          FROM news
91          WHERE day BETWEEN ? AND ?
92        ");
93        $page_num1->bindValue(1, $day, PDO::PARAM_STR);
94        $page_num1->bindValue(2, $dayAfter, PDO::PARAM_STR);
95        $page_num1->execute();
96        $page_num1 = $page_num1->fetchColumn();
97        
98        $pagination1 = $page_num1 / 10;
99        $pagination1 = ceil($page_num1 / 10);
100        $max_page = ceil($page_num1 / MAX);
101
102          
103      } catch (PDOException $exception) {
104          echo $exception->getMessage("接続できませんでした");
105          exit();
106      }
107  }
108}
109

日付での検索は正しく表示されておりますので、両者の違いを確かめてみましたが特に分かりませんでした。

お手数ですが、エンジニアの先輩方の知識を貸していただけませんでしょうか。
何卒、よろしくお願い致します。

退会済みユーザー

2020/04/30 05:58 編集

{$start} とかこれもプリペアドステートメントにしたらいいとおもう。isSet()じゃなくisset()。

退会済みユーザー

2020/04/30 06:08

$result がファイルをまたいで使おうとしていたりしないですよね？コードが断片的にしか示されていないので、リダイレクトされた後の別phpファイルだったりしないかと心配になった。

tech_first

2020/04/30 06:12

ご質問いただきありがとうございます。別のphpファイルではなく、1つのファイル内で$resultを利用しようとしております。スコープの可能性も考えましたが、日付検索も同じ実装でコードが正常に動いているのでその可能性は低いと思っています。プリペアドステートメントの件もありがとうございました。

退会済みユーザー

2020/04/30 06:51

あと、念のため、phpのバージョン、テスト実行している環境のOSとかmysql/mariadbやwebサーバーなどの情報も。

tech_first

2020/04/30 07:12

ありがとうございます！今後は最初からバージョンなども記載します。 php 7.1 mariadb です。

行動規範の内容に同意します

回答2件

ベストアンサー

isSet()って関数はないから、そのスコープ内のクエリーとか全然実行していないのではないかと。
PHP: isset - Manual
is_set()でもなく、isset()なのです。

もしもコンソール（windowsならコマンド・プロンプト）が使えるなら、

php -l makeNews.php

などとして文法エラーが生じていないかチェックしてください。

$resultとか$page_num1とか、
場面で中身が変わってしまうような使い方は避けるべきです。
例えばSQL文を入れておくなら$sql_*、
プリペアドステートメントにしたら$stmt_*
クエリー実行結果は$result_*などとネーミングをルールを決めておかないと、
1週間後、1ヶ月後、1年後などに読み返したときに意味がわからなくなってしまいがちです。

投稿2020/04/30 06:21

編集2020/04/30 06:35

退会済みユーザー

総合スコア0

tech_first

2020/04/30 06:45

ご回答いただきましてありがとうございます。命名規則にまで頭が回っておりませんでした。ご回答いただいた内容のissetを直したのですが、検索結果は変わりませんでした。 isset($result) がexecuteの後のvar_dump()で値が入っているということはクエリは実行されていると思っておったのですが、実行されていなかったということでしょうか？

退会済みユーザー

2020/04/30 06:50

var_dump()してそれなりに意味のある値が入っているのか、エラーなりに別の値が入っているのかわからないですが。 var_dump()しているということはちゃんとそこは処理が通過しているということになりますね。 try～catch()の外側で予め初期化しておくと、どうでしょう？

tech_first

2020/04/30 14:31

なるほど、データの初期化はしておりませんでしたし、やったこともありませんでした。丁寧にご回答いただきまして、誠にありがとうございます。

行動規範の内容に同意します