PHP,SQL,オブジェクト指向プログラミング

質問

こんにちは。僕は、現在、PHPとSQLとオブジェクト指向を用いて、掲示板を作成しています。
今ログインフォームまで完成しましたが、その中の認証処理でつまづいています。
ソースを載せますので、アドバイス等お願いします。

session_start();
include 'class.php';
include 'useclass.php';
$its=new keijibandatabase();
$its->k_conect();
$its->k_usertable();
$its2=new userdata;
$its2->k_login();

function k_usertable() { //テーブルuserの呼び出し
$sql = 'SELECT * FROM user';
$stmt=$this->pdo->query($sql);
while($row = $stmt -> fetch(PDO::FETCH_ASSOC)){
}　//データベースクラス

function k_login() { //ユーザ認証
$row;
$user=$_POST['user'];
$pass=$_POST['pass'];

if($user===$row['uid'] && $pass===$row['pass']){
echo "成功";
}else{
echo "失敗";
}

}

以上です。
これらのコードは別々のファイルに記述されています。
回答よろしくお願いします

回答No.4

LancerVII · ベストアンサー

こんにちは。

＞($row = $stmt -> fetch(PDO::FETCH_ASSOC)){」
＞で$rowに各ユーザのレコードを格納したわけなんですが、この「$row」を

全てのレコードを利用するなら１レコード毎に$rowを他の変数に入れてあげる必要があります。
var $rows = array();
while($row = $stmt -> fetch(PDO::FETCH_ASSOC)){
$rows[] = $row;
}　//データベースクラス

とすると$rowsという配列に全てのレコードが入ります。

それを他のクラスに渡すならクラスの作りによりますがメソッドで受け取れば良いのでは？
$user->setRecord ( $rows );

// どっかのクラスのメソッド
function setRecord ( $rows ) {
}

ただ質問者様がやろうとしていることは一般的ではありません。
質問内容やソースの書き方からすると
クラスの作り方、変数の渡し方等々再度学ぶ必要があるレベルだと思います。

複数のレコードを取得しているのに
if($user===$row['uid'] && $pass===$row['pass']) {
}
では誰と一致させようとしているか？

お礼

わかりやすい解説ありがとうございました。
なんとか解決できるように頑張ります

回答No.3

LancerVII

こんにちは。

まず認証の仕方の構想がずれているようです。

通常、送信されたユーザIDとパスワードを条件としてSQLを発行してレコードが存在するかどうかで認証します。
例：select * from user where userid='userid' and password='password'

今の動きですと全てのデータを取得して一致させようとしていますが、そんな無駄なことはしません。

どちらにせよ今のソースですと$rowは別クラスで取得しているのでそのまま利用できるはずがありません。
もう少しクラスの作り方とかを学ぶ必要があると思います。

例えばユーザIDとパスワードをクラスに設定するなら

class User {
private $userId;
private $password;
public function __construct ( $userId, $password ) {
$this->userId = $userId;
$this->password = $password;
}
}

こんな感じにして
$user = new User ( 'ユーザID', 'パスワード' );

として呼び出します。

補足

回答ありがとうございます。
また、別の質問なんですが、
質問１の補足から、
これはデータベースクラスなのですが、「function k_usertabale() {　～　「while($row = $stmt -> fetch(PDO::FETCH_ASSOC)){」
で$rowに各ユーザのレコードを格納したわけなんですが、この「$row」を

ユーザクラスである、
「function k_login() {
$user=$_POST['user']; //このPOSTは入力フォームのところで設定してあります
$pass=$_POST['pass'];

if($user===$row['uid'] && $pass===$row['pass']) {
echo "成功";
}else{
echo "失敗";
}
}
に「$row」に格納されている配列を使いたいときはどうすればいいのでしょうか。
よろしくお願いします

回答No.2

Taiyonoshizuku

これじゃクラスの関連もわからねーよ。

$user === $row['uid']とかやってるけど$rowっていつ中身が入るんだ？
第一$rowがローカル変数なのかグローバル変数なのかわからないし、グローバル変数なんてクラス使ったら基本的に使わないだろ。
クラス設計から見直したほうがいいよ。

第一何に対してのアドバイスが欲しいのかぜんぜんわからん。

お前さん、自分でこれに回答しようとしたら回答できると思う？

回答No.1

LancerVII

こんにちは。

＞ソースを載せますので、アドバイス等お願いします。
何に対するアドバイスでしょうか。
ソースレビュー？

それとも
＞その中の認証処理でつまづいています。
ってところでしょうか？
それだとしたら「どうするとどうなる」という情報が足りなすぎます。

＞これらのコードは別々のファイルに記述されています。
どこからどこまでが別のファイルなんだかわかりません。

認証もどのようにして考えているか読み取れません。

補足下さい。

補足

すみません。
かなりわかりにくかったようですね。
捕捉させていただきますと、

一番最初の質問に対してですが、
ソースレビューということで構いません

まず、
session_start();
から
$its2->k_login();
までで、一つ

次に
function k_usertable() { //テーブルuserの呼び出し
から
while($row = $stmt -> fetch(PDO::FETCH_ASSOC)){
}　//データベースクラス
までで、一つ

最後に
function k_login() { //ユーザ認証
から
}else{
echo "失敗";
}

}
までで、全部３つのファイルになっています。

この処理だと
ユーザＩＤとパスワードを入力すると今の場合だと認証に成功した場合、「成功」、失敗した場合だと、「失敗」と出力できるようにしたいのですが、これだと、失敗とししか出力されませんでした。

＞認証もどのようにして考えているか読み取れません。
ということに関してですが、

まず、ログインフォームみたいのがあります。
ユーザＩＤとパスワード入力することによって、
ＳＱＬ文で、「user」というテーブルを呼び出し、uid（ユーザＩＤ)とpass(パスワード）、ユーザネームを$rowという変数に格納ということになっています。ちなみに、試作なので、「user」というテーブルにはあらかじめ、４人分のアカウントが入っています。

一度while句のところでvar_dump($row);してみたのですが、配列にちゃんとなっているようなのですが、その配列したデータが「function k_login() { //ユーザ認証」のところに行かないようです。

こんな感じの補足でよろしかったでしょうか？
よろしくお願いします