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

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

ただいまの
回答率

89.99%

登録判定処理がうまくいきません

解決済

回答 3

投稿 編集

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

Company_

score 10

実現したいこと

ユーザ登録成功した際に"登録が完了しました。あなたのユーザ名は${name}で、パスワードは${pass}です"
を表示させたいです

発生している問題

画面が真っ白で何も表示されない

試してみたこと

判定処理を何度か書き換えてみましたが、メッセージが表示されませんでした
登録処理ではエラーは起きていません

該当のコード

<?php
require_once('db_connection.php');

class User_registaration extends Db_Connection
{
    public function __construct()
      {
        parent::__construct();
      }

    //ユーザ情報登録
    public function user_registaration($name, $pass)
      {
      /*if(!isset($error))
        {*/
      //登録処理
            $sql = "INSERT INTO ip_d_user_information(infouser_user_name, infouser_password) VALUES (?, ?)";
            $stmt = $this->link->prepare($sql);
            $stmt->execute(array($name, password_hash($pass, PASSWORD_DEFAULT)));

      //判定処理
            $sql = "SELECT * FROM ip_d_user_information ORDER BY DESC LIMIT 1";
            $stmt = $this->link->query($sql);
            $data = $stmt->fetch(PDO::FETCH_ASSOC);
            if($name == $data['infouser_user_name'] && $pass == $data['infouser_password'])
              {
                $success = "登録が完了しました。あなたのユーザ名は${name}で、パスワードは${pass}です";
                return $success;
              }
        }
}

$username = $_POST["username"];
$password = $_POST["password"];
$password2 = $_POST["password2"];

$user = new User_registaration;

$success = $user->user_registaration($username, $password);

追記のコード

<?php

require_once('Smarty.class.php');
require_once('../file_class/login_authentication.php');
require_once('../file_class/user_registration.php');

$smarty = new Smarty();

$smarty->template_dir = '../templates';
$smarty->compile_dir  = '../templates_c';
$smarty->config_dir   = '../configs/';
$smarty->cache_dir    = '../cache/';

session_start();

// メッセージの初期化
$errorMessage = "";
$signUpMessage = "";

$smarty->assign('signUpMessage', $success);

$smarty->display('signup.tpl');
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • m6u

    2018/10/18 11:02

    〈画面が真っ白で何も表示されない〉エラーログにエラーメッセージなど残っていないかを点検するのが先。

    キャンセル

回答 3

+1

判定処理を何度か書き換えてみましたが、メッセージが表示されませんでした

はい、結果を変数に入れているだけなので、表示されません

最後にecho $success;とするなど、何らかの形で表示させましょう。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/10/18 10:31 編集

    $successは
    このファイルを呼び出してsmarty->assignを使用し表示させようとしてしました
    判定式はこのままであっていますでしょうか

    キャンセル

checkベストアンサー

0

いろいろ突っ込みどころがありそうですが
$_POSTの参照はそろそろやめたほうがよいかと

$username = filter_input(INPUT_POST,"username");
$password = filter_input(INPUT_POST,"password");
$password2 = filter_input(INPUT_POST,"password2");
  • その上で、$passwordと$password2の比較をしていないけど大丈夫ですか?

  • INSERT構文すでにuserが登録済みのときエラーになりますが大丈夫ですか?
    IGNOREなどつけてエラー回避

 $sql = "INSERT IGNORE INTO ip_d_user_information(infouser_user_name, infouser_password) VALUES (?, ?)";
  • select文発行していますがwhere句がないので全ユーザーの情報が抜けますが大丈夫ですか?
  $sql = "SELECT * FROM ip_d_user_information WHERE infouser_user_name=?";


※ただし、上記INSERTが失敗した場合=すでにユーザーがいる場合かってにすでに登録済みの
データを抜き取ることになるのでPDOStatement::rowCountなどをつかった対処が必要

$data['infouser_password']

ハッシュされたパスを参照してなにか意味があるのでしょうか?
そもそも登録後にパスワードを表示するという仕組み自体ダメです

  • $stmt->fetchは1行しかデータを取ってこないのでLIMIT 1しなくても大丈夫

なおPDOで処理しているならtry-catchで例外を拾ってください

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/10/18 11:25

    ・filter_input(INPUT_POST,"username");
     このやり方は知りませんでした 調べてみます
    提示していただいた内容を基にコードを修正したところ表示されました
    これから指摘を受けた部分を直してみます
    ありがとうございました

    キャンセル

0

判定処理のorder byでカラム指定してませんが問題ありませんか?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/10/18 10:58

    カラム指定したところ真っ白な画面ではなくなりました
    ですが、メッセージは表示されませんでした
    カラム指定はidで行いました

    キャンセル

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

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