別々のテーブルに一回でアクセスしたい

実現したい事

xamppを使用してログイン機能を作っている所です。
テーブル：company_dataと、テーブル：home_dataに、同一のカラム名のidとpwがそれぞれあります。

①ログイン画面から$_POSTで送られてきたidとpwが、
②テーブルAとテーブルBのどちらかのテーブルのIDとPWと合致する場合のみ、
③$resultをtrueにしてlogin()関数の呼び出し元へ返す、という処理を実装したいです。

困っている事

テーブル：company_dataと、テーブル：home_dataへの効率的なアクセス方法が分かりません。
$pdo->prepare()からfetchAllまで一緒なので使い回しをしてなるべく少ない記述で終わらせたいです。

PHP
1
2<?php
3require_once('../common/config.php');
4//ログイン処理
5function login($id, $pw){
6  $pdo = new pdo(DSN,DB_USER,DB_PASS);
7  $stt = $pdo->prepare("SELECT id,pw FROM company_data WHERE id = :id AND pw = :pw");
8  $stt->bindParam(':id', $id);
9  $stt->bindParam(':pw', $pw);
10  $stt->execute();
11  $result = $stt->fetchAll(PDO::FETCH_ASSOC);
12  //どちらかのテーブルの中にidとpwがあれば$result=trueになる。
13  if($result){
14    foreach($result as $r){
15      $result['id'] = $r['id'];
16      $result['pw'] = $r['pw'];
17    }
18  }else{
19    $stt = $pdo->prepare("SELECT id,pw FROM home_data WHERE id = :id AND pw = :pw");
20    $stt->bindParam(':id', $id);
21    $stt->bindParam(':pw', $pw);
22    $stt->execute();
23    $result = $stt->fetchAll(PDO::FETCH_ASSOC);
24    foreach($result as $r){
25      $result['id'] = $r['id'];
26      $result['pw'] = $r['pw'];
27    }
28  }
29  if(isset($result)){
30    return $result;
31  }
32}
33
34
35?>
36

試した事

INNER JOINを使用してテーブルの結合を試みましたが、コードの書き方がおかしく上手くいきませんでした。

PHP
1  $stt = $pdo->prepare("SELECT c.id,c.pw,h.id,h.pw FROM company_data AS c INNER JOIN home_data AS h ON c.id = :c.id AND c.pw = :c.pw OR h.id = :h.id AND h.pw = :h.pw");

ご回答頂けますと幸いです。
よろしくお願いいたします。

sazi

2021/05/18 09:32

どちらかという状況は、どちらか一方に存在するのでしょうか、それとも、両方に存在する事もある、のでしょうか？

takasima20

2021/05/18 11:21

自分も union に一票かな。関係ないけど、なんで id と pw 返してんの?

hoshi-takanori

2021/05/18 16:10

もしかしてパスワードを平文で保存してるってことですか？

Keichi_Negishi

2021/05/19 00:01

お返事が遅くなりました。＞どちらかという状況は、どちらか一方に存在するのでしょうか、それとも、両方に存在する事もある、のでしょうか？ →両方に存在しています。適合したものが返されます。自分も union に一票かな。関係ないけど、なんで id と pw 返してんの? →セッションで持たせるためです。もしかしてパスワードを平文で保存してるってことですか？ →平文で保存していますので、暗号化したものに変更したいと思っています。

hoshi-takanori

2021/05/19 00:11

> →セッションで持たせるためです。パスワードを (たとえハッシュ化したものでも) セッションに持たせるのは良くないと思います。 > →平文で保存していますので、暗号化したものに変更したいと思っています。暗号化ではなく password_hash を使ってハッシュ化することになると思いますが、乱数のソルトが加わるので、同じパスワードでも結果は毎回異なる値になります。このため、SELECT 文で pw = :pw という条件は使えず、取得した pw を password_verify することになると思います。

Keichi_Negishi

2021/05/19 00:22

根本的にこのコードの書き方ではマズイのですね。 https://tadworks.jp/archives/1147 コードはここのサイトのものを参考にして書きました。ログイン機能に関して参考になるページがあればよろしければ教えていただけますでしょうか。

Keichi_Negishi

2021/05/19 00:45

この辺の記事がよさそうなので参考にしてみます。 https://qiita.com/rana_kualu/items/3ef57485be1103362f56 https://qiita.com/ryo-futebol/items/5fb635199acc2fcbd3ff