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

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

ただいまの
回答率

88.23%

複数行をSELECT文で取得しようとしたのですが1行しか取得できません

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 2,318

TSKT

score 16

前提・実現したいこと

PHPでmysqlを使用するためにsql文とsql文に代入するデータを引数として渡すと処理して成功したかどうかとsqlの結果を返すデータベース関数を作成しました。
これを使用していたんですが1行のみの結果を取得する時は正常に動いていたと思うのですが、複数行を所得使用とした際に1行のみしか取得できないという状態になりました。
試しにmysql側で直接sql文を実行した結果、正しく複数行取得されていたのでsql文に間違えがあるとかではないと思います。

発生している問題・エラーメッセージ

エラーメッセージは発生していません
データベースから複数行の取得を行いたいのですが1行しか所得できません

該当のソースコード

$sql = "SELECT id FROM tbl WHERE name_0=? AND name_1=? AND name_2=? AND name_3=? AND name_4=?";
$data = [$name[0],$name[1],$name[2],$name[3],$name[4]];
$wk = db($sql,$data);
$result = $wk[0];
$data = $wk[1];

function db($sql,$data){
    $dsn = 'mysql:dbname=testdb;host=localhost;charset=utf8mb4;';
    $user = 'testuser';
    $password = 'pass';
    try{
        $dbh = new PDO($dsn, $user, $password);

        $stmt = $dbh->prepare($sql);
        $result = $stmt->execute($data);
        $row = [];
        if(isset($stmt)){
            foreach($stmt as $row){}
        }
        return [$result,$row];
    }catch (PDOException $e){
        print('Error:'.$e->getMessage());
        die();
    }
    $dbh = null;
}
PHP,MYSQL

試したこと

returnされた後の$resultと$rowをvar_dump()した結果、$result=true、$rowは1行の取得が行われていました。

補足情報(FW/ツールのバージョンなど)

mysql:5.7.x
php:7.2.17

データベース構造です

CREATE TABLE `tbl` (
 `id` INT(9) UNSIGNED ZEROFILL NOT NULL AUTO_INCREMENT ,
 `name_0` VARCHAR(20) NOT NULL ,
 `name_1` VARCHAR(20) NOT NULL ,
 `name_2` VARCHAR(20) NOT NULL ,
 `name_3` VARCHAR(20) NOT NULL ,
 `name_4` VARCHAR(20) NOT NULL ,
 `name_x_0` VARCHAR(20) NOT NULL ,
 `name_x_1` VARCHAR(20) NOT NULL ,
 `name_x_2` VARCHAR(20) NOT NULL ,
 `name_x_3` VARCHAR(20) NOT NULL ,
 `name_x_4` VARCHAR(20) NOT NULL ,
 `num` INT(10) NOT NULL ,
 `num_e` INT(10) NOT NULL ,
 `user_id` INT(9) UNSIGNED ZEROFILL NOT NULL ,
 `date` TIMESTAMP NOT NULL ,
 PRIMARY KEY (`id`)
) ENGINE = InnoDB;

複数行を取得しようとしているsql文です
SELECT id FROM tbl WHERE name_0=? AND name_1=? AND name_2=? AND name_3=? AND name_4=?

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • m.ts10806

    2020/01/12 21:36

    >foreach($stmt as $row){}

    これだと回してるだけで何もセットしてないので $rowは [] が返るのでは。

    キャンセル

  • TSKT

    2020/01/12 21:48

    一応、$rowは値が入っているみたいです

    キャンセル

  • m.ts10806

    2020/01/12 21:49

    あぁ、理解。

    キャンセル

回答 3

checkベストアンサー

+2

foreachの特徴をおさえておくべきです。

foreach($stmt as $row){}

同じ状況が再現できるコードを下記に示します。

<?php
$data = [1,2,3,4,5,6];
foreach($data as $r){}

echo $r;
//出力結果
//6

これがなぜ6になるのか分かったらとるべき手法も決まります。

ただ、これはこれでforeachの特性をおさえておくべきですけど、
そもそも取得のためだけに回すのってコストの無駄遣いなので、
その場で処理が発生しないのでしたらfetchAll()で一気に全部取った方が良いと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/01/12 22:44

    foreachを誤解していました。
    foreachで何も処理をしてなかったので最後の行のみが返されていたんですね。
    解決しました。
    ありがとうございました。

    キャンセル

  • 2020/01/12 22:51

    解決されたようで何よりです。

    >foreachで何も処理をしてなかったので最後の行のみが返されていた
    そうですね。おおよそその理解であっています。
    あくまで「配列のコピーから値を1つずつ取り出している」だけなので、「取り出した」結果をasに入れていきます。
    変数は何もしなければ上書きされ続けるので最後の情報しか残らないということになります。

    キャンセル

0

テーブル定義はCREATE TABLE文で載せましょう。

複数行を所得しようとしているsql文です
SELECT id FROM tbl WHERE name_0=? AND name_1=? AND name_2=? AND name_3=? AND name_4=?

が質問のコードの中にないからでは?
コードは実行できる最小限の内容を載せた方が適切なコメントが付き易いです。

PHPでデータベースに接続するときのまとめ PDO も参考に。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/01/12 21:26

    create table文を追加しました。
    関数の呼び出し元のコードを追加しました。
    助言ありがとうございます

    キャンセル

0

function db($sql,$data){
    $dsn = 'mysql:dbname=;host=;charset=utf8mb4;';
    $user = '';
    $password = '';
    try{
        $dbh = new PDO($dsn, $user, $password);

        $stmt = $dbh->prepare($sql);
        $result = $stmt->execute($data);
        $row = $stmt->fetchAll();
        return [$result,$row];
    }catch (PDOException $e){
        print('Error:'.$e->getMessage());
        die();
    }
    $dbh = null;
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

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

  • ただいまの回答率 88.23%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る