商品の検索とページングを行うために下記のコードのように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>
回答2件
あなたの回答
tips
プレビュー