PHPでMysqlから値を取り出すときにエラーが出る。

###前提・実現したいこと
PHPでMySQLから値を取り出す部分を作っています。
値を機能を実装中に以下のエラーメッセージが発生しました。

PHP
1Notice: Undefined index: active in C:\xampp\htdocs\v3\function.php on line 20

###該当のソースコード

PHP
1<?php
2//Mysqlへ接続
3$db = new mysqli('localhost', 'root', 'husetemasu', 'husetemasu');
4$db->set_charset('utf8');
5
6/* 接続状況をチェックします */
7if (mysqli_connect_errno()) {
8    printf("Connect failed: %s\n", mysqli_connect_error());
9    exit();
10}
11
12
13function alertmessage(){
14  $sql = 'SELECT * FROM news WHERE active="1"';
15  $db = new mysqli('localhost', 'root', 'husetemasu', 'husetemasu');
16  $db->set_charset('utf8');
17  try {
18  $rows = $db->query($sql)->fetch_all(MYSQLI_ASSOC);
19  echo $rows['active'];
20  } catch (mysqli_sql_exception $e) {
21    echo $e->getMessage();
22  }
23}
24
25 ?>

HTML
1require_once("function.php");
2php alertmessage();
3

###補足情報(言語/FW/ツール等のバージョンなど)
XAMPPを使用しています。

こういった方法で書くのは初めてで何が初歩的なミスがあるかもしれません....
よろしくお願いします。

s8_chu

2017/06/04 12:08

データベースはどのようになっているのか追記していただけませんか？また、質問文のHTMLのコードはできるだけ省略せずに記述していただけると嬉しいです。

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

回答2件

ベストアンサー

s8_chuさんの回答をベースに細かいところを直しました

2回接続してるのが無駄
mysqli_report()でエラーモードを設定しないとmysql_sql_exceptionは投げられないのでtry~catchが無意味

sql
1CREATE TABLE `news`(
2    `id` int(11) UNSIGNED NOT NULL AUTO_INCREMENT PRIMARY KEY,
3    `active` tinyint(1) NOT NULL DEFAULT 1,
4    `text` varchar(255) NOT NULL DEFAULT ''
5) ENGINE = InnoDB DEFAULT CHARSET = utf8mb4 COLLATE = utf8mb4_unicode_ci;
6
7INSERT INTO `news`(`text`) VALUES('sample_text1'),('sample_text2');

php
1<?php
2
3// エラーを例外としてスローさせる
4mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
5
6/**
7 * MySQLに接続
8 *
9 * @return mysqli
10 */
11function connectDb()
12{
13    $db = new mysqli('localhost', 'root', 'password', 'sample01');
14    $db->set_charset('utf8');
15    return $db;
16}
17
18/**
19 * アクティブなニュースを取得
20 *
21 * @param mysqli MySQL接続
22 * @return array 2次元配列
23 */
24function fetchActiveNews(mysqli $db)
25{
26    $sql = 'SELECT * FROM news WHERE active="1"';
27    return $db->query($sql)->fetch_all(MYSQLI_ASSOC);
28}
29
30/**
31 * 変数出力時に <?=h()?> のイディオムで使う
32 *
33 * @param string $s 生の文字列
34 * @return string HTMLエスケープ済みの文字列
35 */
36function h($s)
37{
38    return htmlspecialchars($s, ENT_QUOTES, 'UTF-8');
39}
40
41// MySQLに関わる処理全体をtry~catchで括る
42// （原則的には個々の関数の中にtry~catchを書いてはいけない）
43try {
44
45    $db = connectDb();
46    $news = fetchActiveNews($db); 
47
48} catch (mysql_sql_exception $e) {
49
50    // 接続に失敗したときやSQL実行に失敗したときはここに飛ぶ
51    // 500エラーでテキストを吐いておく
52    header('Content-Type: text/plain; charset=UTF-8', true, 500);
53    exit($e->getMessage());
54
55}
56
57
58?>
59<!DOCTYPE html>
60
61<meta charset="UTF-8">
62
63<ul>
64<?php foreach ($news as $new): ?> 
65    <li><?=h($new['text'])?></li>
66<?php endforeach; ?>
67</ul>

備考: PHPでデータベースに接続するときのまとめ - PDOとmysqliの比較

個人的にはPDOをおすすめしておきます。mysqliはあまり使いやすいとは言えない・・

投稿2017/06/04 12:53

編集2017/06/04 13:00

mpyw

総合スコア5223

退会済みユーザー

2017/06/04 13:10

エスケープまでありがとうございます。いらないコードを自分は書いてたみたいですね（´・ω・｀） POD、勉強してきます

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

データベースを以下のように定義して

SQL
1CREATE TABLE `sample_database`(
2    `id` int(11) UNSIGNED NOT NULL AUTO_INCREMENT PRIMARY KEY,
3    `active` tinyint(1) NOT NULL DEFAULT 1,
4    `news` varchar(255) NOT NULL DEFAULT ''
5)ENGINE = InnoDB DEFAULT CHARSET = utf8mb4 COLLATE = utf8mb4_unicode_ci;

以下のようにデータを入れました。

SQL
1INSERT INTO `sample_database` (`id`, `active`, `news`) 
2VALUES 
3  (NULL, '1', 'sample_text1'), 
4  (NULL, '1', 'sample_text2')

そのとき、質問者さんが行いたいことは以下のように行えると思いますが、いかがでしょうか？

PHP
1<?php
2//Mysqlへ接続
3$db = new mysqli("localhost", "root", "password", "sample01");
4$db->set_charset("utf8");
5
6/* 接続状況をチェックします */
7if (mysqli_connect_errno()) {
8    printf("Connect failed: %s\n", mysqli_connect_error());
9    exit();
10}
11
12
13function alertmessage()
14{
15    $sql = "SELECT * FROM sample_database WHERE active=1";
16    $db = new mysqli("localhost", "root", "password", "sample01");
17    $db->set_charset("utf8");
18    try {
19        $rows = $db->query($sql)->fetch_all(MYSQLI_ASSOC);
20        foreach ($rows as $value) { // foreachを使う。
21            echo $value["active"];
22        }
23    } catch (mysqli_sql_exception $e) {
24        echo $e->getMessage();
25    }
26}
27
28alertmessage();// 今回は11が出力される。
29
30?>