質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

Q&A

解決済

2回答

639閲覧

PDOでsqlを複数実行

tege

総合スコア29

MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

0グッド

0クリップ

投稿2019/10/15 13:21

編集2019/10/15 14:30

商品の検索とページングを行うために下記のコードのようにPDOで複数のsqlを実行させようとしているのですが、このやり方はただしいですか?可読性などに問題はありませんか?
他に短縮できたり可読性をあげる書き方があれば教えてほしいです。

<?php require_once($_SERVER["DOCUMENT_ROOT"] . "/ph/classes/conf.php"); require_once($_SERVER["DOCUMENT_ROOT"] . "/ph/classes//Book.php"); define('PER_PAGE',10); // formからnameを受け取る if (isset($_GET['name'])) { $name = $_GET['name']; } else { exit; } $where = []; $data = []; foreach (explode(' ', $name) as $val) { if (trim($val) === '') continue; $where[] = '(name like ?)'; $data[] = '%' . preg_replace('/(?=[!_%])/', '!', $val) . '%'; } //現在ページ取得 if (isset($_GET['page'])) { $page = (int)$_GET['page']; } else { $page = 1; } $bookList = []; try { $db = new PDO(Conf::DB_DNS, Conf::DB_USERNAME, Conf::DB_PASSWORD); $db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); $db->setAttribute(PDO::ATTR_EMULATE_PREPARES, false); //全ページカウントで値を取得し÷10でページ数を取得 $page_num = $db->prepare("SELECT COUNT(*) book_id FROM product"); $page_num ->execute(); $page_num = $page_num->fetchColumn(); $pagenation = ceil($page_num / PER_PAGE); $offset = PER_PAGE * ($page - 1); $sql = 'SELECT book_id,book_name,isbn_id,price,book_text,bookimg FROM product'; if (count($where) > 0) $sql .= ' where ' . implode('and', $where); $sql .= ' LIMIT ' . ($offset > 0 ? $offset . ',' : '') . PER_PAGE; $stmt = $db->prepare($sql); $stmt->execute($data); while ($row = $stmt->fetch(PDO::FETCH_ASSOC)) { $dp = []; foreach ($row as $col => $val) { $dp[$col] = $val; } $book = new Book(); $book->setId($dp['book_id']); $book->setText($dp['book_text']); $book->setBookimg($dp['book_img']); $book->setBoName($dp['book_name']); $book->setIsbn($dp['isbn_id']); $book->setPrice($dp['price']); $bookList[$dp['book_id']] = $book; } } catch (PDOException $ex) { $_SESSION["errorMsg"] = "DB接続に失敗しました。"; } finally { $db = null; } ?> <!DOCTYPE html> <html lang="ja"> <head> <meta charset="utf-8"> <title>商品リスト</title> <link rel="stylesheet" href="/ph34/EP32/css/desing.css" type="text/css"> </head> <body> <section> <form method="get"> <div class="form-group"> <input name="name" value="<?php echo isset($_GET['name']) ? htmlspecialchars($_GET['name']) : '' ?>"> <button type="submit" name="serch">検索</button> </div> </form> <tbody> <?php if (empty($bookList)) { ?> <tr> <td colspan="5">該当商品は存在しません。</td> </tr> <?php } else { foreach ($bookList as $book) { ?> <ul id="bookimg"> <li id="box2"><?= $book->getBoName() ?></li> <li>¥<?= $book->getPrice() ?>+税</li> </ul> <?php } } ?> <br> <?php for($i=1; $i <= $pagenation; $i++) :?> <a href="?page=<?php echo $i ?>"><?php echo $i; ?></a> <?php endfor; ?> </tbody> </section> </body> </html>

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2019/10/15 13:32

どこを問題視してます?
退会済みユーザー

退会済みユーザー

2019/10/15 14:01

単にレビューして欲しいと言うことでも、どこまで指摘すべきか非常に困ります。 プロとして金を稼げるレベル? 趣味としてやるくらいのレベル? どの程度を求めているんでしょうか?
m.ts10806

2019/10/15 14:35

想定の結果が出ているなら「正しい」ですよ。やろうと思ったら幾らでもできるものを聞くのは質問ではなく作業依頼です(大喜利目的ならそう書いた方が良いですが質問者には到底理解が及ばない宇宙コードか実務で使っちゃいけないお遊びコードのどちらかになります)
guest

回答2

0

ベストアンサー

nameをスペース区切りで入力してget処理しているんですよね
先頭や末尾のスペースやスペースの連続が入ると

name like '%%'

となるのでトリム処理が必要です

$data[] = '%' . preg_replace('/(?=[!_%])/', '!', $val) . '%';

ワイルドカードの処理も微妙です
_は_、%は%に変更しないと、クリティカルにヒットさせられないでしょう
なお前方後方一致はインデックスが利用できないので
パフォーマンスは期待できません

投稿2019/10/16 01:05

yambejp

総合スコア114784

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

where句が指定される条件の時に、ページングが正しく機能しないので間違っています。

可読性などに問題はありませんか?

せめてインデントはつけましょう。

投稿2019/10/15 14:43

編集2019/10/15 14:45
退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問