php POSTされたデータを用いてDBにSELECT

Question

###前提・実現したいこと
Unityでシステムをつくりながら、プログラムを勉強しています。
今回実現したいことは、php、MySQLを用いてUnityでログイン機能を作成することです。

現在、Login.C#でUnityからInpurFiledに入力されたnameとpasswordをphpにPOSTし、POSTされたデータをlogin.phpで取得し、そのデータを用いてMySQLにSELECT文を発行し、nameとpasswordが一致しているときのidを取得し、一致していた場合1を、していなければ0が返されるプログラムを作りましたが、nameとpasswordが同じでも、0が返ってしまいます。

###発生している問題・エラーメッセージ
UnityでInputFieldにnameとpasswordを入力後、ログインボタンを押したときのログには、

Invalid argument supplied for foreach() in ～～

のようなエラーが表示されますが、返り値もかえって来ますが、値は0の状態です。
DBに登録されているnameとpasswordをしっかり入力しているのですが…
このエラーも何か関係しているのでしょうか。

###該当のソースコード

using UnityEngine;
using System.Collections;
using UnityEngine.UI;

public class login : MonoBehaviour
{   
    public InputField inputName;
    public InputField inputPass;

    public void OnClickButton()
    {
        StartCoroutine(Connect());
    }

    IEnumerator Connect()
    {
        string name = inputName.text;
        string pass = inputPass.text;

        string url = ("http://～～～.php");

        WWWForm wwwForm = new WWWForm();

        wwwForm.AddField("name", name);
        wwwForm.AddField("pass", pass);

        WWW form = new WWW(url, wwwForm);

        yield return form;
        Debug.Log(form.text);

    }
}

<?php
	
	require("./executeQuery.php");

    //初期化
	$name = '';
	$pass = '';
	$ref = 0;

    //受け取り
	$name = $_POST['name'];
	$pass = $_POST['pass'];
	
	
	//確認
    echo "name: $name ";
    echo "pass: $pass ";

	$sql = "SELECT * FROM users WHERE name=".$name." AND password=".$pass."";
	
	$result = executeQuery($sql);
	
	foreach ($result as $row) {
		$ref = $row["id"];
	}
	
	
	if($ref>0)
		echo "1";
	else
		echo "0";
?>

<?php
function executeQuery($sql){
	$dsn = 'mysql:dbname=～～～;host=～～～';		
	$user = '～～～';											
	$password = '～～～';								

	// クエリ文実行結果受け取り用変数
	$result = "";
	
	try{
		$dbh = new PDO($dsn, $user, $password);

		$dbh->query('SET NAMES sjis');

	
		$result = $dbh->query($sql);

	}catch (PDOException $e){
		print('Error:'.$e->getMessage());		
		die();
	}

	$dbh = null;

	// 戻り値
	return($result);
}
?>

###試したこと
phpで、 確認にecho "name: $name "; echo "pass: $pass ";を追加し、POSTられた値がしっかり送れているのかの確認のために表示しましたが、しっかり送れているようです。

なぜDBに登録されているnameとpasswordと同じ文字列をPOSTしているのに返り値が0になってしまうのでしょうか。

まだプログラミングを始めたばかりで、至らない点等多くあるかと思いますが、何卒ご教授お願いできますでしょうか。
よろしくお願い致します。

Answer 1

executeQuery() の内容が分からないので断定的なことはいえませんが、
とりあえず、SQLエラーになってないスか?
文字列を指定する場合はシングルクオートで囲む必要があると思います。

// セキュリティ的な話もありますが、とりあえず。

Comments

[Hutax]

takasima20様
ご回答ありがとうございます。
executeQuery() は、PDOでDBにアクセスするコードを書いています。
Unity側で出ているエラーは上記に示したエラーのみです。

[takasima20]

とりあえず、var_dump($result); で確認するとか
先頭に ini_set( 'display_errors', 1 ); を追加してみるとか。

[Hutax]

var_dump($result);を挿入し、デバッグを行ったところ、
Unity側のログでは、
object(PDOStatement)#2 (1) {
["queryString"]=>
string(88) "SELECT * FROM teachers WHERE name='user' AND password='hoge'"
}
と出力されました。

Answer 2

まずこちらのエラーですが配列ではないものをforeachで回そうとする際に出るエラーです。
おそらく$resultの中身がnullか文字列なのでないかと思います。

Invalid argument supplied for foreach() in ～～

"SELECT * FROM users WHERE name=".$name." AND password=".$pass.""

こちらのSQLですが、下記のように書き換えたらどうでしょうか？

"SELECT * FROM users WHERE name='".$name."' AND password='".$pass."'"

SQLもまた文字列はクォーテーションで囲む必要があります。

また、細かい話になりますが、SQLインジェクションを考えてSQLを発行する際はプリペアードステートメントを使った方が良いです。

Comments

[Hutax]

tokino0219様
ご回答ありがとうございます。
提案いただいたソースで書き換えたところ、
Parse error: syntax error, unexpected '$result' (T_VARIABLE) in ～～
とエラーが表示されてしまい、返り値も帰らなくなってしまいました。

[tokino0219]

文法エラーになってしまっているようですね。;が削れたりはしていませんか？

[Hutax]

失礼致しました。
こちらのミスで；を抜かしていました。
；を加えて実行したところ、初めに提示したforeachのエラーはなくなりましたが、返り値は0のままで、解決には至りませんでした。

[tokino0219]

なるほど。。。
となりますと、takasima20さんがおっしゃっている通り、executeQuery()の内容が分からないのでこれ以上の推測が立てれませんね。。。
executeQueryのコードを提示していただければ他の方もより詳細な回答ができるかと思います。

[Hutax]

お手数をおかけいたします。
executeQueryのコードを提示させていただきますので、ご教授よろしくお願い致します。

[tokino0219]

パッと見る限りではこれといっておかしい点は見受けられないですね。。。
下記のメソッドを使ってSQLのエラーを表示したら原因がわかるかもしれません。
http://php.net/manual/ja/pdo.errorinfo.php

[Hutax]

考察ありがとうございます。
PDO::errorInfoは、
新たに作成したphpに、参考サイトに書かれているようなコードを書き、
上記のlogin.phpのSELECT文を$stmt = $dbh->prepare('');の中に書くという形でよろしいのでしょうか。

[tokino0219]

$result = $dbh->query($sql);

$dbh-> errorInfo();// 追記

一例ですが、私はこうしますね。

[Hutax]

お手数をおかけしております。
今朝方、一度Unityを再起動してみたところ、ご教授していただいた方法で実装できました。
昨日はなぜ起動しなかったのか原因は不明ですが、Untiy側に何かしら問題があったようです。
たくさんのご助言感謝いたします。ありがとうございました。