PHP： データベースの行を指定して表示させたいです。

Question

PHP：　データベースの行を指定して表示させたいです。
動くプログラムは書いたのですが、データベースの一行目しか表示されません。
プログラムを修正して、指定された行のデータを表示させたいです。

試したことは
最後の方の
print_r($result);　　　の部分を

print_r （$result[1]）;
と書いてみて動かした。

ソース

<?php $dsn = 'mysql:dbname=chatlog;host=localhost'; $user = 'testuser'; $password = 'yasushi'; try { $dbh = new PDO($dsn, $user, $password); echo "接続成功\n"; } catch (PDOException $e) { echo "接続失敗: " . $e->getMessage() . "\n"; exit(); } $sql = "SELECT * from `chatlog`"; $stmt = $dbh->query($sql); $result = $stmt->fetch(PDO::FETCH_ASSOC); print_r($result); ?>

＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿

このソースならちゃんと動いてくれます。
結果は　　接続成功 Array ( [id] => 1 [name] => ヤスシ [log] => テスト
というように表示されます。

これの最後の方の部分を　　 print_r （$result[1]）;　　と書き直すと

Parse error: syntax error, unexpected variable "$result" in C:\xampp\htdocs\php\db.php on line 28

というエラーが出ます。

よろしくお願いします。

Comments

[退会済みユーザー]

markdown を適切に使用してください。

[y-sasaki]

読みにくくてすいません。markdown のこと、よくわかってませんでした。

[退会済みユーザー]

質問は編集できます。あとから見る人のためにも修正をしておいてください。

Answer 1

エラーが出るのは
print_r （$result[1]）;　
の括弧が全角だからじゃないかと思います。
エラーが出ないコードに[1]だけを追加すれば、parseErrorからnoticeに変わると思います。
後述の理由で欲しいデータは入っていませんが。

文法エラーはIDEやエディタにチェックさせるのが楽なので、PHP IDEやPHP 開発環境などで検索して効率的な開発環境を構築されることをお勧めします。
（質問中に記述されているのがコピペミスであれば、実際にエラーが出ているコードを質問に追記して下さい）

また、質問中のfetch()では一行しか取り出せないので、
seastar3さんの回答の様にループで全件取得するか、

PDOStatement::fetchAll
を使って全件取得する必要があります。（上記サンプルコードを読んでみるとやりたいことに近いと思います）

---

事前にID等が分かり、IDを変数で指定したい場合は
PDO::prepareを使ってプレースホルダを使う事で安全に条件指定をすることが出来ます。
条件となる数値や文字列を文字列連結によってSQLに組み込み、PDO::query()で実行することはSQLインジェクションの温床になる可能性が高いうえ、メリットは特に無いので避けるべきです。

Comments

[y-sasaki]

ありがとうございます。:fetchAllにしてみました。

修正したソースは
try {
$dbh = new PDO($dsn, $user, $password);
echo "接続成功\n";
}

catch (PDOException $e) {
echo "接続失敗: " . $e->getMessage() . "\n";
exit();
}

$sql = "SELECT * from chatlog";
$stmt = $dbh->query($sql);

$result = $stmt->fetchAll();
print_r($result);

ですが、結果が繰り返しで表示されます。
Array ( [0] => Array ( [id] => 1 [0] => 1 [name] => ヤスシ [1] => ヤスシ [log] => テスト [2] => テスト )
のように、これは１行だけですが、すべての行について繰り返し表示されてしまいます。
　繰り返さないようにしたいのですが、どうすればいいですか

[tanat]

全行取得している配列を丸ごとprint_r()すれば全行表示されるのは当然です。
(提示したURLのサンプルコードをしっかりと読んで理解するように努めてください。)

質問にあるように$result[1]を直接扱うか、where句でidを指定してfetch()してください。

[tanat]

もし絞り込みをするべきSQL自体がよく分からないという状態であれば
`SQL 入門 WHERE`等で調べて基本を学習した上で、PHPMyAdminやMySQL Workbench等のツールを使ってSQLを実際に実行できる環境を用意する事を強くお勧めします。

[y-sasaki]

私の質問の仕方が悪かったです。
全行が表示されるのは、分かっていたのですが、　　なぜか、各行で2回ずつ表示されてしまいます。日本語が変でしたら、すいません。

[tanat]

そういう風にデータが格納されてしまってるのではと推測します。
推測を重ねても仕方無いのでまずは前述の通り、
MySQLのデータをPHP以外の方法で確認できる環境(phpMyAdminやMySQL Workbench等)を用意して、データを直接確認してください

[y-sasaki]

表示結果は　
　接続成功 Array ( [0] => Array ( [id] => 1 [0] => 1 [name] => ヤスシ [1] => ヤスシ [log] => テスト [2] => テスト ) [1] => Array ( [id] => 2 [0] => 2 [name] => ひろし [1] => ひろし [log] => あ [2] => あ ) [2] => Array ( [id] => 3 [0] => 3 [name] => つよし [1] => つよし [log] => い [2] => い ) )

phpMyadmin に入れたデータは
1
ヤスシ
テスト

2
ひろし
あ

3
つよし
い

みたいな感じです。

[tanat]

ああ、なるほど。
であれば
回答中の
https://www.php.net/manual/ja/pdostatement.fetchall.php
の
例2 結果セットから単一カラムの全ての値を取得する
のあたりのサンプルを熟読して、fetchAll()のパラメータを指定してください。
`PHP fetchモード`等で検索して調べるのもありです。

実は大体のことはPHPマニュアル(回答中のリンクは全部PHPマニュアルのページです)のサンプルをよく読めば解決するので、とっつきにくいとは思いますがしっかりと読むようにしてください。

[y-sasaki]

マニュアルみたら、、なんとなく　分かりました
[name] => ヤスシ [1] => ヤスシ 　のように　実際は1回だけの表示なんですが、
表示形式？のせいで　2回繰り返してるように見えただけかもです。
何度もありがとうございます。

Answer 2

print_r()は配列情報を吐き出すデバッグ用の関数なので、

print($result[1]);

を

print($result['id'].' '.$result['name'].' '.$result['log'].'<br>');

のようにして表示します。

また指定した行を表示するには

$sql = "SELECT * from chatlog";

を

$siteiid = 1010;

$sql = 'SELECT * FROM chatlog WHERE id = :id';
$stmt = $dbh->prepare($sql);
$stmt->bindValue(':id', $siteiid, PDO::PARAM_INT);
$stmt->execute();
while( $result = $sth->fetch(PDO::FETCH_ASSOC) ){
  printf("%d|%d|%d\n", $result['id'], $result['name'], $result['log']);
}

のようなSQLでの指定で取り出します。

（tanatさんの御指摘の通り、単純なエスケープではSQL対策として不備があるので、PDOを尊重してプレースホルダを用いたコードに改善しました。）

Comments

[y-sasaki]

丁寧なご回答、　ありがとうございます。
$siteiid = 1010;
$sql = "SELECT * from chatlog where id = '".$siteiid."'";　の部分の　".$siteiid."'　は　".$siteid."'でよろしいでしょうか

$siteidが 2 の時のデータを取り出したい場合。この部分を
$siteid = 2;
$sql = "SELECT * from `chatlog where id =".$siteid."`";
$stmt = $dbh->query($sql);
と書いてみたのですが、エラーが出てしまいました。
エラーメッセージは
致命的エラー：キャッチされないPDOException：SQLSTATE [42S02]：ベーステーブルまたはビューが見つかりません：1146テーブル 'chatlog.chatlog where id = 2'がC：\ xampp \ htdocs \ php \ db.php：25に存在しませんスタックトレース：＃0 C：\ xampp \ htdocs \ php \ db.php（25）：PDO-> query（ 'SELECT * from `...'）＃1 {main}がC：\ xampp \ htdocs \にスローされました25行目のphp \ db.php
です。
ちなみに２５行目は
$stmt = $dbh->query($sql);　　　の部分です。

たびたびすいません。

[tanat]

SQLを構築する際にhtmlspecialchars()を使用するのは危険です。
参考: https://blog.tokumaru.org/2013/02/security-measures-of-own-way-are-unsafe.html

元の回答だと直前で変数指定をしているので、ほぼ固定値として扱っていることが分かるのですが、
- htmlspecialchars()は誤ったSQLインジェクション対策として用いられた歴史的経緯がある（割と新しめの記事でもダメな記述が書かれてたりする）
- 固定値として扱うのであれば追記する必要がなく、（誤った）SQLインジェクション対策として追記されたように見える
ため、低評価といたしました。

[tanat]

修正と追記ありがとうございました。
低評価を取り消しました。

[seastar3]

tanatさん、御助言どうもありがとうございました。
現在のSQLインジェクション対策の常識がよく分かりました。
常に自己研鑽を重ねなければ、おいて行かれることを痛感しましたので、今後とも精進して参ります。

Answer 3

SQL で抽出するのがよさそうだと思うけど、質問の箇所は array_filter() で対応かなぁ。。。

<?php
$arr = [
    [
        'id' => 1,
        'name' => 'ヤスシ',
        'log' => 'テスト',
    ],
    [
        'id' => 2,
        'name' => 'ヤスシ2',
        'log' => 'テスト2',
    ],
 ];
 
var_dump($arr);
var_dump($arr[0]);
var_dump(array_filter($arr, function($x){
    return $x['id'] === 2;
}));

//var_dump($arr);
array(2) {
  [0]=>
  array(3) {
    ["id"]=>
    int(1)
    ["name"]=>
    string(9) "ヤスシ"
    ["log"]=>
    string(9) "テスト"
  }
  [1]=>
  array(3) {
    ["id"]=>
    int(2)
    ["name"]=>
    string(10) "ヤスシ2"
    ["log"]=>
    string(10) "テスト2"
  }
}

//var_dump($arr[0]);
array(3) {
  ["id"]=>
  int(1)
  ["name"]=>
  string(9) "ヤスシ"
  ["log"]=>
  string(9) "テスト"
}

//array_filter
array(1) {
  [1]=>
  array(3) {
    ["id"]=>
    int(2)
    ["name"]=>
    string(10) "ヤスシ2"
    ["log"]=>
    string(10) "テスト2"
  }
}

~~SQL の文字リテラルを文字列連結で構成している回答がありますが、~~SQL インジェクションの原因となります。
普段から プリペアードステートメントを使用することをおすすめします。

Comments

[退会済みユーザー]

DB への接続に関しては、初期設定すべき parameter が複数あるので、以下のテンプレートを参考にすると良いです。
https://qiita.com/te2ji/items/56c194b6cb9898d10f7f

あと、php のマニュアルでは足りない部分があるので以下も
https://qiita.com/mpyw/items/b00b72c5c95aac573b71
上記は若干内容が古くなりつつありますが、今でもよく参考にしています。