Q&A
単に 3レコード抽出したいだけだったら SQL でやった方がよさそう。
プログラミング初心者です。貸し会議室の運営サイトを作成しています。
Amazonなどのサイトで見る、「この商品を購入した人はこちらも購入しています」のような一覧をphpを用いて作りたいのですが、うまく行きません。
具体的には、mySQL上のデータを用いて、スライダーの中に、商品情報を３つずつ入れたいのですが、持ってきた情報を３つずつピックアップして、ループで回したくていろいろ調べたのですが、ループに必要なコードの書き方がわかりません。
発生している問題・エラーメッセージ
### $pdo = new PDO("mysql:host=******;dbname=********", "****", "***"); ↑※実際はちゃんとした情報を入れています。 if (isset($_GET['id_produit'])) { $pdo->query("SETSET lc_time_names = 'ja_JP'"); $result = $pdo->query("SELECT date_format(product.date_arrivee,'%Y %b %d %k:%i') AS new_date_arri,date_format(produit.date_depart,'%Y %b %d %k:%i') AS new_date_depa, EXTRACT(YEAR_MONTH FROM date_arrivee) AS ym, product.*, room.* FROM product JOIN room ON product.id_room = room.id_room WHERE id_product = ".$_GET['id_product'].""); $item=$result->fetch(PDO::FETCH_ASSOC); echo '<div class="belt">'; echo "<h2>" .$item ['room_name'] . "</h2>"; echo '<div class="photobox">'; echo'<a href=""><img id="boucle" src="' . $item['photo'] . '"></a>'; echo '</div>';//photobox echo "<p> 商品詳細:" . $item['description'] . " </p>"; echo "<p> 定員:" . $item['capacity'] . " 名</p>"; echo "<p> 分類:" . $item['category'] . " </p>"; echo '<div class="clear"></div>'; echo '</div>';//belt echo "<p><b>詳細情報</b></p>"; echo "<p> 国:" . $item['country'] . " </p>"; echo "<p> 地域:" . $item['city'] . " </p>"; echo "<p>郵便番号:".$item['zc']."</p>"; echo "<p>貸し出し開始日:" . $item['new_date_arri'] . "'日</p>"; echo "<p>貸し出し終了日:" . $item['new_date_depa'] . "'日</p>"; echo "<p>金額:".$item['price']."<span id='aka'>*</span></p>"; echo "<p><span id='aka'>*</span>税別</p>"; $slide = $pdo->query("SELECT R.photo, date_format( P.date_arrivee,'%Y %b %d %k:%i') AS new_date_arri,date_format(P.date_depart,'%Y %b %d %k:%i') AS new_date_depa, R.city, P.price ,R.capacity, P.id_product FROM product AS P JOIN (SELECT EXTRACT(YEAR_MONTH FROM date_arrivee) AS ym, EXTRACT(YEAR_MONTH FROM date_depart) AS ym2, id_produit FROM product) AS ps ON P.id_produit = ps.id_product JOIN room AS R ON P.id_room = S.id_room WHERE (ps.ym = ".$item['ym']." OR ps.ym2 = ".$item['ym']." ) AND S.city = '".$item['city']."' AND P.id_product <>".$item['id_product'].""); $slider = $slide->fetchAll(); $slider_array3 = array_chunk($slider,3); // debug($slider_array3); foreach($slider_array3 as $key1 =>$value1){ foreach($value1 as $key2 =>$value2){ foreach($value2 as $key3 =>$value3){ echo debug($key3.$value3); //ここから先がわかりません。// } } } } } ```PHP 最初$sliderの部分で"(PDO::FETCH_ASSOC)"を使っていたのですが、array_chunkを使ったときに正しく区切れなかったため、arrayfetchAllを使いました。そこまでは良かったのですが、そこから先の値の受け取り方がわかりませんでした。独学で初心者のため、調べるために必要な専門用語の語彙なども不足しています。調査不足、言葉足らずな部分があったら申し訳ありません。 ### 補足情報（FW/ツールのバージョンなど） PHP 7.4.9 MySQL Version 5.7.30
ループに必要なコードというのがよく分かりませんが、そもそもDBの定義はどうなっているのでしょうか。
手法よりも設計をしっかりやっておく内容かと思います
抽出だけであれば、SQLでできています!
こちらは職業訓練学校からの課題で、SQLの定義は指定されていたので、変えられないのですが、
会議室情報(room)というテーブルと日付や値段などの商品情報
(product)という2つのテーブルから成り立っています。2つの情報をまとめたい時は、JOINで、id_roomが同じ部分同士で繋げています。テーブルの中身はこのようになっています。↓
roomテーブル
CREATE TABLE `salle` (
`id_room` int(5) NOT NULL,
`country` varchar(20) NOT NULL,
`city` varchar(20) NOT NULL,
`adress` text NOT NULL,
`zc` varchar(7) NOT NU5LL, 郵便番号
`room_name` varchar(200) NOT NULL,
`description` text NOT NULL,
`photo` varchar(200) NOT NULL,
`capacity` int(3) NOT NULL,
`category` enum('reunion') NOT NULL
) ENGINE=InnoDB DEFAULT CHARSET=utf8;
INSERT INTO `room` (`id_room`, `country`, `city`, `adress`, `zc`, `room_name`, `description`, `photo`, `capacity`, `category`) VALUES
(1, '日本', '東京都', '住所A', '0000000', '会議室A', 'description', '/PHP/LOKISALLE/photo/salle1.jpg', 50, '会議'),
(2, '日本', '東京都', '住所B', '0000000', '会議室B, 'description', '/PHP/LOKISALLE/photo/salle2.jpg', 70, '会議室'),
(3 '日本', '東京都', '住所C', '0000000', '会議室C', 'description', '/PHP/LOKISALLE/photo/salle3.jpg', 20, '会議室'),
(4, '日本', '東京都', '住所D', '0000000', '会議室D', 'description', '/PHP/LOKISALLE/photo/salle4.jpg', 80, '会議室'),
(5, '日本', '東京都', '住所E', '0000000', '会議室E', 'descriotion', '/PHP/LOKISALLE/photo/salle5.jpg', 50, '会議室'),
ALTER TABLE `salle`
ADD PRIMARY KEY (`id_room`);
ALTER TABLE `room`
MODIFY `id_room` int(5) NOT NULL AUTO_INCREMENT, AUTO_INCREMENT=22;
Productテーブル
CREATE TABLE `product` (
`id_product` int(5) NOT NULL,
`date_arrivee` datetime NOT NULL,
`date_depart` datetime NOT NULL,
`id_room` int(5) NOT NULL,
`id_discount` int(2) DEFAULT '0',
`price` int(5) NOT NULL,
`status` int(1) NOT NULL DEFAULT '0' (予約の有無)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;
INSERT INTO `produit` (`id_product`, `date_arrivee`, `date_depart`, `id_room`, `id_discount`, `price`, `status`) VALUES
(1, '2021-03-22 09:00:00', '2021-03-25 18:00:00', 1, NULL, 300, 0),
(2, '2021-04-04 09:00:00', '2021-04-08 18:00:00', 3, NULL, 350, 0),
(3, '2021-03-28 09:00:00', '2021-04-01 18:00:00', 1, NULL, 300, 1),
(4, '2021-04-04 09:00:00', '2021-04-07 18:00:00', 1, NULL, 300, 0),
(5, '2021-03-28 09:00:00', '2021-04-01 18:00:00', 2, NULL, 500, 0),
(6, '2021-03-21 19:04:33', '2021-04-01 19:04:37', 6, NULL, 666, 1),
(7, '2021-03-28 09:00:00', '2021-04-01 18:00:00', 5, NULL, 500, 0),
(8, '2021-03-28 09:00:00', '2021-04-01 18:00:00', 4, 1, 600, 0),
(10, '2021-01-14 00:00:00', '2021-01-22 00:00:00', 1, 0, 1000, 0);
ALTER TABLE `product`
ADD PRIMARY KEY (`id_product`);
質問は編集できますので。
ただ、課題であれば講師に聞かれるか、これまで学んだことを復習して対応されたほうが良いですね。
teratailの回答者の多くはそれなりに実務を経験してきている人なので、実務寄りの回答となり、課題に寄った内容にはなりにくいです。
回答1件
0
ループを3つ回すんじゃなく、3回fetchすればいいんじゃね？（小並感
ループを3つ回すって、
Aに関係あるB、Bに関係あるC、Cに関係あるDを順にたどって取得するようなイメージ。
だけど、やりたいことって、
Aに関係あるBを3つほしい、
ってことなんじゃないの？
MySQLを実務で使ったことはないけど、これはおかしいんじゃないの？ってところはツッコミ入れたい。
php
1$pdo->query("SETSET lc_time_names = 'ja_JP'");
SETSETじゃなくてSET。
MySQL :: MySQL 5.6 リファレンスマニュアル :: 10.7 MySQL Server のロケールサポート
で、いつもいつもこれを実行する必要性ってあるのかどうか。
日本向けじゃなく世界に向けてユーザーごとのロケールに合わせて表示します、
とかいうのでもない限り、データベース構築時にロケール情報を日本用にするはず。
それについては自分で調べてみてね。
php
1$pdo = new PDO("mysql:host=******;dbname=********", "****", "***");
try～catchしましょうね。
ベテランでも何度も読み返すこちらの記事を参照。
PHPでデータベースに接続するときのまとめ - Qiita
php
1$result = $pdo->query("SELECT date_format(product.date_arrivee,'%Y %b %d %k:%i') AS new_date_arri,date_format(produit.date_depart,'%Y %b %d %k:%i') AS new_date_depa, EXTRACT(YEAR_MONTH FROM date_arrivee) AS ym, product.*, room.* FROM product JOIN room ON product.id_room = room.id_room WHERE id_product = ".$_GET['id_product']."");
SQL文字列を作って実行するのは理解しているようだけど、
$_POSTや$_GETなど外部から受信するパラメータを直接連結するのは
必ずしも自分らで作ったフォームじゃなくてもデータ受信できてしまうこともあり
SQLインジェクションによる攻撃を受けやすいため、
対策が必要。
不意にSQL文を終了させようとする記号や文字が入っていないことを検証するのが必要なのと、
せっかくPDO経由の接続を使っているのだからプリペアドステートメントを使いましょうってのと。
php
1$id = filter_input(INPUT_GET, 'id_product', FILTER_VALIDATE_INT); 2$sql = <<<EOT 3SELECT 4 date_format(product.date_arrivee, '%Y %b %d %k:%i') AS new_date_arri, 5 date_format(produit.date_depart, '%Y %b %d %k:%i') AS new_date_depa, 6 EXTRACT(YEAR_MONTH FROM date_arrivee) AS ym, 7 product.*, 8 room.* 9FROM 10 product JOIN room ON product.id_room = room.id_room 11WHERE 12 id_product = :id 13EOT; 14$stmt = $pdo->prepare($sql); 15$stmt->bindValue(':id', $id, PDO_PARAM_INT); 16$result = $stmt->execute();
SQL文の中身は特に評価していないけど、PDOっぽく書き直すとこんな感じとか。見やすさ大事。
PHP: filter_input - Manual
PHP: PDO::prepare - Manual
PHP: PDOStatement::bindValue - Manual
「product., room.」みたいなのは個人的にはナシだな。
SQL文としては間違っていないけど、なんのどのデータがほしいか明示して
無駄にデータを引っ張り出すことに依る無駄なリソース消費を抑えたいかな。
投稿2021/01/21 00:45
編集2021/01/21 01:11
退会済みユーザー
総合スコア0
あなたの回答
tips
プレビュー
