Q&A
URLは https://teratail.com/help#about-markdown の[リンク]を使ってください。
現在下記のサイトを勉強しています。
ブログを作る
質問なのですが、テーブル名をpostの所をdai、commentの所をhenjiとしました。
そのようにしたら、サイトの下記の部分でエラーが出てしまい、どの部分を書き換えれば良いのか分からなくなり質問します。
phpmyadminからテーブル名を変えるのも考えましたが、結局意味がないと思い教えて頂きたいと思います。
blog/index.php
php
1<?php 2 $pdo = new PDO("mysql:dbname=blog", "root"); 3 $st = $pdo->query("SELECT * FROM post ORDER BY no DESC"); 4 $posts = $st->fetchAll(); 5 for ($i = 0; $i < count($posts); $i++) { 6 $st = $pdo->query("SELECT * FROM comment WHERE post_no={$posts[$i]['no']} ORDER BY no DESC"); 7 $posts[$i]['comments'] = $st->fetchAll(); 8 } 9 require 't_index.php'; 10?>
blog/t_index.php
php
1<!DOCTYPE html> 2<html> 3<head> 4<meta charset="utf-8"> 5<title>Special Blog</title> 6<link rel="stylesheet" href="blog.css"> 7</head> 8<body> 9<h1>Special Blog</h1> 10<?php foreach ($posts as $post) { ?> 11 <div class="post"> 12 <h2><?php echo $post['title'] ?></h2> 13 <p><?php echo nl2br($post['content']) ?></p> 14 <?php foreach ($post['comments'] as $comment) { ?> 15 <div class="comment"> 16 <h3><?php echo $comment['name'] ?></h3> 17 <p><?php echo nl2br($comment['content']) ?></p> 18 </div> 19 <?php } ?> 20 <p class="commment_link"> 21 投稿日：<?php echo $post['time'] ?> 22 <a href="comment.php?no=<?php echo $post['no'] ?>">コメント</a> 23 </p> 24 </div> 25<?php } ?> 26</body> 27</html>
エラーメッセージです
Fatal error: Uncaught Error: Call to a member function fetchAll() on boolean in /Applications/XAMPP/xamppfiles/htdocs/php/blog/index.php:4 Stack trace: #0 {main} thrown in /Applications/XAMPP/xamppfiles/htdocs/php/blog/index.php on line 4
回答3件
0
ベストアンサー
phpmyadminからテーブル名を変えるのも考えましたが、結局意味がないと思い教えて頂きたいと思います。
考え方が逆です。
用意したデータベース情報（テーブル・カラム含む）にPHPから接続してSQLを作ってください。プログラムが先ではありません。
エラーについて：
データが取得できる前提で組まれています。
php
1 $st = $pdo->query("SELECT * FROM post ORDER BY no DESC"); 2 $posts = $st->fetchAll();
pdo::queryは失敗したらfalseを返します。
返り値
PDO::query() は、PDOStatement オブジェクトを返します。 失敗した場合は FALSE を返します。
ということは、falseが返った場合はfetchAllどころじゃないですよね。
せめて$stのtrue/false判定を入れてください。
php
1 $st = $pdo->query("SELECT * FROM post ORDER BY no DESC"); 2 if($st){ 3 $posts = $st->fetchAll(); 4 }
もちろんこれだけでは十分ではありません。
次のcommentsのところはループの中で実行しているわけですし、紐づく情報を得ようとしているのでしたら1回失敗したらそれ以降もうまくいくとは限りません。（むしろうまくいくはずがありません）
できればtry-catchで囲み、Exceptionを捕捉するような仕組みにしてください。
というか何度もSQLを実行するのは結構無駄です。
yambejpさんのような一発で取得するSQLを実行するのが良いのではないでしょうか。
取得した後に表示部分は多少うまいことやる必要がありますが、
何度もSQLを実行してループするよりは良いと思います。
※そもそも質問者さんはプログラムからDBを利用するというところの感覚がまだ薄いように思います。基本形の解説記事を熟読されたほうが良いのではないでしょうか。
投稿2019/01/31 11:45
編集2019/01/31 21:50
総合スコア80850
とても分かりやすくありがとうございます。
どのように解決されたのでしょうか。
私はヒント回答と考え方しか指南していません。
理解が不足しているとまた同じ質問を繰り返すことになり、この質問とそれについた回答が無駄になります。
お恥ずかしいのですが、Atomでプログラムを変更しているときに、フォルダの方でプログラムを別の場所に移動する作業をしていました。その後もAtomでプログラムの変更などしていたのですが、移動した先のプログラムは変更されていないままの状態でした。
結局は、blog/index.phpにあるテーブル名を適切な名前に変更すると言う作業だけでよかったです。
事情はよくわからないですが、あくまで「設定をした情報を使う」のが原則なので、プログラムの都合で設定を振り回すのは不具合の原因となり問題の切り分けが難しくなるので、手順を守るようにしてください
気をつけます。
解決のヒントになったようで何よりです。
あなたの回答
tips
プレビュー
