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

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

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

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

SQL

SQL(Structured Query Language)は、リレーショナルデータベース管理システム (RDBMS)のデータベース言語です。大きく分けて、データ定義言語(DDL)、データ操作言語(DML)、データ制御言語(DCL)の3つで構成されており、プログラム上でSQL文を生成して、RDBMSに命令を出し、RDBに必要なデータを格納できます。また、格納したデータを引き出すことも可能です。

PHP

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

Q&A

解決済

1回答

823閲覧

渡されたIDを元にデータベースからデータを持ってくる

ryohasegawa

総合スコア437

MySQL

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

SQL

SQL(Structured Query Language)は、リレーショナルデータベース管理システム (RDBMS)のデータベース言語です。大きく分けて、データ定義言語(DDL)、データ操作言語(DML)、データ制御言語(DCL)の3つで構成されており、プログラム上でSQL文を生成して、RDBMSに命令を出し、RDBに必要なデータを格納できます。また、格納したデータを引き出すことも可能です。

PHP

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

0グッド

0クリップ

投稿2017/06/01 12:40

編集2017/06/01 14:05

このコードは、ある講座で使用したログイン時のユーザデータ取得のコードです。
なお下2つのコードは、呼び出しと、SQL実行の関数は別ファイルにあり、SQL実行の関数は2つとも同一ファイルにあります。
ですが、ファイルは呼び出せています。

php

1try { 2 $userModel = new \MyApp\Model\User(); 3 $user = $userModel->login([ 4 'email' => $_POST['email'], 5 'password' => $_POST['password'] 6 ]); 7} catch (\MyApp\Exception\UnmatchEmailOrPassword $e) { 8 $this->setErrors('login', $e->getMessage()); 9 return; 10} 11 12// ログイン 13 // 渡されたデータでSQLを組み立てる 14 public function login($values) { 15 // ユーザーの取得 16 $stmt = $this->db->prepare("select * from users where email = :email"); 17 $stmt->execute([ 18 ':email' => $values['email'] 19 ]); 20 // データはオブジェクトモードで取得 21 $stmt->setFetchMode(\PDO::FETCH_CLASS, 'stdClass'); 22 $user = $stmt->fetch(); 23 24 // ユーザーが存在しない場合 25 if (empty($user)) { 26 throw new \MyApp\Exception\UnmatchEmailOrPassword(); 27 } 28 // パスワードがマッチしなかった場合 29 if (!password_verify($values['password'], $user->password)) { 30 throw new \MyApp\Exception\UnmatchEmailOrPassword(); 31 } 32 return $user; 33 }

このコードは、呼び出し元から引数で値を渡し、その結果をうまく返しているんですが、これを少しいじって自分で作ってみると上手く動かすエラーが出ます。

php

1$Get_Id = substr($_SERVER['QUERY_STRING'], 3); 2 3// IDでユーザの情報を探す 4try { 5 $userModel = new \MyApp\Model\User(); 6 $user = $userModel->findUser([ 7 'number' => $Get_Id 8 ]); 9 10} catch (\MyApp\Exception\UnmatchUserData $e) { 11 $this->setErrors('Get_User', $e->getMessage()); 12 return; 13} 14 15 16// 引数で指定されたユーザのデータを返す 17 public function findUser($values) { 18 $stmt = this->db->prepare("select * from users where anumber = :number"); 19 $stmt->execute([ 20 ':number' => $values['number'] 21 ]); 22 $stmt->setFetchMode(\PDO::FETCH_CLASS, 'stdClass'); 23 return $stmt->findUser(); 24 } 25 26

下のエラーが出ます。

text

1Fatal error: Uncaught Error: Call to undefined method PDOStatement::findUser() in /Users/hasegawaryou/Dropbox/alpha/lib/Model/User.php:89 Stack trace: #0 /Users/hasegawaryou/Dropbox/alpha/public/Admin/User_Data/index.php(59): MyApp\Model\User->findUser(Array) #1 {main} thrown in /Users/hasegawaryou/Dropbox/alpha/lib/Model/User.php on line 89

###追記です。現状報告

php

1try { 2 $userModel = new \MyApp\Model\User(); 3 $user = $userModel->findUser([ 4 'number' => $Get_Id 5 ]); 6 7} catch (\MyApp\Exception\UnmatchUserData $e) { 8 $this->setErrors('Get_User', $e->getMessage()); 9 return; 10} 11 12 13 // 引数で指定されたユーザのデータを返す 14 public function findUser($values) { 15 $stmt = $this->db->prepare("select * from users where anumber = ?"); 16 $stmt->execute([$values['number']]); 17 $row=$stmt->fetch(PDO::FETCH_ASSOC); 18 return $row; 19 }

text

1Fatal error: Uncaught Error: Class 'MyApp\Model\PDO' not found in /Users/hasegawaryou/Dropbox/alpha/lib/Model/User.php:86 Stack trace: #0 /Users/hasegawaryou/Dropbox/alpha/public/Admin/User_Data/index.php(59): MyApp\Model\User->findUser(Array) #1 {main} thrown in /Users/hasegawaryou/Dropbox/alpha/lib/Model/User.php on line 86

yambejp さんの返答を元に書き直したコードです。

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

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

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

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

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

guest

回答1

0

ベストアンサー

$this->db->query()
しているようですが

$this->db->prepare()
しなくても大丈夫なのでしょうか?

投稿2017/06/01 12:46

yambejp

総合スコア114572

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

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

ryohasegawa

2017/06/01 12:49 編集

this->db->prepare("select * from users where anumber = :number"); ですね
yambejp

2017/06/01 12:52

:numberについては、bindValueしたほうがいいかもしれません $stmt->bindValue(':number',$values['number'], PDO::PARAM_INT);
ryohasegawa

2017/06/01 12:56

$stmt = $this->db->prepare("select * from users where anumber = :number"); $stmt->bindValue(':number',$values['number'], PDO::PARAM_INT); $stmt->setFetchMode(\PDO::FETCH_CLASS, 'stdClass'); こうしたのですが、結果は変わりませんでした。
yambejp

2017/06/01 12:58

$stmt->bindValue(':number',$values['number'], PDO::PARAM_INT); の直後に $stmt->execute(); してますか?
ryohasegawa

2017/06/01 13:17

してないです。 $stmt->execute() で値を挿入していた所を $stmt->bindValue(':number',$values['number'], PDO::PARAM_INT); に置き換えた感じです。
yambejp

2017/06/01 13:26

プレースホルダーの流れとしては ・prepareでSQL文を投入 ・bindValueでパラメータを渡す ・executeで実行 がセットです。 バインドする値の型指定をしない簡易的な処理では $stmt = $this->db->prepare("select * from users where anumber = ?"); $stmt->execute([$values['number']]); と省略して書くこともあるとおもいます
ryohasegawa

2017/06/01 13:44

public function findUser($values) { $stmt = $this->db->prepare("select * from users where anumber = ?"); $stmt->execute([$values['number']]); $stmt->setFetchMode(\PDO::FETCH_CLASS, 'stdClass'); return $stmt->findUser(); } 書き換えてみて実行してみたんですが、上手く行きませんでした。 これであってますか?
yambejp

2017/06/01 13:50

$stmt->execute([$values['number']]); の後にデータ抽出するのは $row=$stmt->fetch(PDO::FETCH_ASSOC); でよいかと またreturnするのは return $row["なんかid"]; になるのでは? ※戻したい値はなんなのかちょっと読み取れないので
ryohasegawa

2017/06/01 13:58 編集

えーと、呼びだし時に引数でIDが渡って来る([$values['number'])ので、そのIDを元にデータベースからそのIDのデータ(全件)をreturnで返すので、IDを返す必要はないです。 返すのは、データベースから受け取ったデータだけを返したいんです。
yambejp

2017/06/01 14:00

そうなると return $row; とすればいいのでは? $rowにはfetchしたテーブルの全カラムがはいっているはずなので
ryohasegawa

2017/06/01 14:02

はい。そうしてみました。 一度、本文に追記で今の現状コードとエラーを追記するので見てもらっていいですか?
ryohasegawa

2017/06/01 14:19

追記しました。
yambejp

2017/06/02 00:32

MyApp\Model\PDOなんてクラスないよ、といわれていますが 当該のUser.phpはDropbox側のものですかね? なにか参照の仕方が悪いのだとおもいますが?
ryohasegawa

2017/06/02 04:02

一度このsql文を変えて、実際入るであろう値をそのまま書き込んで、実際に欲しい値が帰ってくるかを先に調べた方が良さそうですかね?
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問