データベースに数値の0だけが登録できないのは....

Question

###前提・実現したいこと
こんな初歩的なことですいません。0も登録できるようにしたいです。

###発生している問題・エラーメッセージ
データベースに他の数字は登録できるのに0だけ登録できない

Warning: mysqli_num_rows() expects parameter 1 to be mysqli_result, boolean given in /Applications/MAMP/htdocs/graduate_thesis/register.php on line 21

このwarningも関係しているのでしょうか？

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

if(array_key_exists('number',$_POST) OR array_key_exists('password',$_POST)){
    
    if($_POST['number'] == ""){
        echo "〇〇を入力してください";
    } elseif($_POST['password'] == ""){
        echo "パスワードを入力してください";
    } else {
        
        $query="SELECT 'id' FROM 'users' WHERE number='".mysqli_real_escape_string($link,$_POST['number'])."' ";
        
        $result=mysqli_query($link,$query);
        if(mysqli_num_rows($result) > 0){
            echo "すでにそのnumberは使用されています。";
        } else {
            $query = "INSERT INTO `users` (`number`,`password`) VALUES ('". mysqli_real_escape_string($link,$_POST['number']). "','" . mysqli_real_escape_string($link,$_POST['password'])."')";
            
            if(mysqli_query($link,$query)){
//                header("Location: http://localhost/graduate_thesis/index.php" );
//                
                echo "登録に成功しました。";
            } else {
                echo "登録に失敗しました。";
            }
            
        }
    }
    
}

###試したこと
課題に対してアプローチしたことを記載してください

###補足情報(言語/FW/ツール等のバージョンなど)
PHP7 phpmyadmin

Comments

[shi_ue]

コードはコードブロックで囲んでください。右側にプレビューが出ていますので、それを見ながら編集してください。

[Tomoya]

ご指摘ありがとうございました。

Answer 1

最初のmysqli_queryが失敗しています。

$query="SELECT 'id' FROM 'users' WHERE number='".mysqli_real_escape_string($link,$_POST['number'])."' ";
```フィールド名やテーブル名をシングルクォーテーション`'`で囲っていますが、正しくはバッククォート（`Shift+@`）です。

Comments

[Tomoya]

回答ありがとうございました。

Answer 2

インデントがないので、コードの詳細は確認していないですが、ざっと見て脆弱性だらけです。。。
何かを参考にコードを記述しているのであれば、すぐに変更したほうが良いです。

ログイン機能を実装するなら、このあたりを参考にすると良いです
PHPでログイン機能を実装するチュートリアル #1

追記
ちゃんと見直したら、そこまでひどくはありませんでした＾＾；
が、パスワードのハッシュ化ぐらいはしておいたほうが良いと思います。

Comments

[Tomoya]

迅速な回答ありがとうございました。

Answer 3

warningの方はご指摘のとおり,フィールド名やテーブル名をシングルクォーテーションで挟んでいたのでバッククォートに直したらwarningがとれました。

$query="SELECT `id` FROM `users` WHERE number='".mysqli_real_escape_string($link,$_POST['number'])."' ";

データベースの方は数値データを格納する際に先頭に0（ゼロ）が付いていると勝手に消されてしまうらしいのでカラムの属性にUNSIGNED ZEROFILLを追加したら解決しました。