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

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

ただいまの
回答率

90.52%

  • PHP

    20323questions

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

エラーの検知がうまくできない

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 495

ryohasegawa

score 116

<form action="" method="post" id="signup2">
                  <p>
                    メールアドレスを入力してください。
                    <input type="text" name="email" placeholder="メールアドレス" value="<?= isset($app->getValues()->email) ? h($app->getValues()->email) : ''; ?>">
                  </p><br>
                  <p class="err"><?= h($app->getErrors('email')); ?></p>

                  <p>
                    ユーザーの識別IDです。<br>半角で入力してください。
                    <input type="text" name="id" placeholder="アカウントID" value="<?= isset($app->getValues()->id) ? h($app->getValues()->id) : ''; ?>">
                  </p><br>
                  <p class="err"><?= h($app->getErrors('id')); ?></p>

                    パスワードを入力してください。
                    <input type="password" name="password" placeholder="パスワード" value="<?= isset($app->getValues()->password) ? h($app->getValues()->password) : ''; ?>">
                  </p><br>
                  <p class="err"><?= h($app->getErrors('password')); ?></p>
                  <div class="btn" onclick="document.getElementById('signup2').submit();">新規登録</div>
                  <input type="hidden" name="token" value="<?= h($_SESSION['token']); ?>">
                  <input type="hidden" name="posttype" value="signup">
                </form>


ユーザ認証の勉強でフォームを作りました。そこでうまくIDを英数で入力しないとエラーがでるようにしたく下のコードを書きました。

        try {
            $this->signup_validate();
        } catch (\MyApp\Exception\InvalidEmail $e) {
            $this->setErrors('email', $e->getMessage());
        } catch (\MyApp\Exception\InvalidPassword $e) {
            $this->setErrors('password', $e->getMessage());
        }catch (\MyApp\Exception\InvalidId $e){
            $this->setErrors('id', $e->getMessage());
        }


    private function signup_validate()
    {
        //  送られてきたトークンが本物かどうか調べる
        if (!isset($_POST['token']) || $_POST['token'] !== $_SESSION['token']) {
            echo "送信中エラー エラー番号T01";
            exit;
        }

        //  入力したデータ(メール)が形式じゃなかったらエラーを返す
        if (!filter_var($_POST['email'], FILTER_VALIDATE_EMAIL)) {
            throw new \MyApp\Exception\InvalidEmail();
        }

        //  入力したデータ(パスワード)が形式じゃなかったらエラーを返す
        if (!preg_match('/\A[a-zA-Z0-9]+\z/', $_POST['password'])) {
            throw new \MyApp\Exception\InvalidPassword();
        }

        //  入力したデータ(アカウントID)が形式じゃなかったらエラーを返す
        if (!preg_match('/\A[a-zA-Z0-9]+\z/', $_POST['id'])) {
            throw new \MyApp\Exception\InvalidId();
        }
    }


とこんな感じにしました。そして、

catch (\MyApp\Exception\InvalidId $e){
    $this->setErrors('id', $e->getMessage());
}

が引っかかってほしいのに、

catch (\MyApp\Exception\InvalidEmail $e) {
    $this->setErrors('email', $e->getMessage());
}

で引っかかってしまうので、英数で入力してくださいというメッセージではなく、メールアドレスを正しく入力してください。というメッセージが表示するバグに遭遇しています。助けてください。
        

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • naoyama88

    2017/03/19 01:55

    実際に画面のフォームからどのような入力値を送信しているか具体的に提示していただけると回答も詳しくできるかとおもいます。

    キャンセル

  • 退会済みユーザー

    2017/03/19 17:29

    複数のユーザーから「やってほしいことだけを記載した丸投げの質問」という意見がありました
    「質問を編集する」ボタンから編集を行い、調査したこと・試したことを記入していただくと、回答が得られやすくなります。

回答 2

+1

複数のバリデーションを行っていると思いますが、
コードの順序的にメールアドレスのチェックから行っており、
メールアドレスに関する例外が先に投げられていますので、
IDのチェックまでたどり着いていません。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/03/19 17:26

    あーそうかもしれないです。いや、そうですね。
    どうしたら、次の検知まで行えますか?方法は色々ありますよね?

    キャンセル

check解決した方法

-3

//  投稿時処理(新規作成)
    protected function signuppostProcess() {
        // validate

        //  送られてきたトークンが本物かどうか調べる
        if (!isset($_POST['token']) || $_POST['token'] !== $_SESSION['token']) {
            echo "送信中エラー エラー番号T01";
            exit;
        }

        try{
            //  入力したデータ(メール)が形式じゃなかったらエラーを返す
            if (!filter_var($_POST['email'], FILTER_VALIDATE_EMAIL)) {
                throw new \MyApp\Exception\InvalidEmail();
            }
        }catch (\MyApp\Exception\InvalidEmail $e) {
            $this->setErrors('email', $e->getMessage());
        }

        try{
            //  入力したデータ(パスワード)が形式じゃなかったらエラーを返す
            if (!preg_match('/\A[a-zA-Z0-9]+\z/', $_POST['password'])) {
                throw new \MyApp\Exception\InvalidPassword();
            }
        }catch (\MyApp\Exception\InvalidPassword $e) {
            $this->setErrors('password', $e->getMessage());
        }

        try{
            //  入力したデータ(アカウントID)が形式じゃなかったらエラーを返す
            if (!preg_match('/\A[a-zA-Z0-9]+\z/', $_POST['id'])) {
                throw new \MyApp\Exception\InvalidId();
            }
        }catch (\MyApp\Exception\InvalidId $e){
            $this->setErrors('id', $e->getMessage());
        }

        /*
        try {
            //  送られてきたトークンが本物かどうか調べる
            if (!isset($_POST['token']) || $_POST['token'] !== $_SESSION['token']) {
                echo "送信中エラー エラー番号T01";
                exit;
            }

            //  入力したデータ(メール)が形式じゃなかったらエラーを返す
            if (!filter_var($_POST['email'], FILTER_VALIDATE_EMAIL)) {
                throw new \MyApp\Exception\InvalidEmail();
            }

            //  入力したデータ(パスワード)が形式じゃなかったらエラーを返す
            if (!preg_match('/\A[a-zA-Z0-9]+\z/', $_POST['password'])) {
                throw new \MyApp\Exception\InvalidPassword();
            }

            //  入力したデータ(アカウントID)が形式じゃなかったらエラーを返す
            if (!preg_match('/\A[a-zA-Z0-9]+\z/', $_POST['id'])) {
                throw new \MyApp\Exception\InvalidId();
            }
        } catch (\MyApp\Exception\InvalidEmail $e) {
            $this->setErrors('email', $e->getMessage());
        } catch (\MyApp\Exception\InvalidPassword $e) {
            $this->setErrors('password', $e->getMessage());
        }catch (\MyApp\Exception\InvalidId $e){
            $this->setErrors('id', $e->getMessage());
        }
        */


コメントアウトしてるところを書き直しました。すると、いくつかのエラーを同時に出す事ができました。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

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

  • ただいまの回答率 90.52%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • PHP

    20323questions

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