0

私は自分自身を OOP に入れようとしていますが、変数のスコープに問題があります。

セッション配列を除いてすべてが機能し、すべてが正しく投稿されます。

私の「セッションテーブル」とその要素は宣言されていますが、その変数は未定義のままです。次のエラー メッセージが表示されます。

Notice: 未定義の変数: 行
Notice: 非オブジェクトのプロパティを取得しようとしています

にアクセスするにはどうすればよい$rowですか?

これは、メソッドを含むクラスのコードです。

$db = db::connect();

class auth {
   protected $login;
   protected $password;
   protected $email;

   public function setLogin($login) {
      $this->login = $login;
   }

   public function setPassword($password) {
      $this->password = $password;
   }

   public function login($fields, $table, $col_login, $col_password) {
      $query = Db::getInstance()->prepare('SELECT ' . $fields . ' FROM ' . $table . ' WHERE ' . $col_login . ' = :login AND ' . $col_password . ' = :password');

      $query->bindValue(   ':login',    $this->login, PDO::PARAM_STR);
      $query->bindValue(':password', $this->password, PDO::PARAM_STR);
      $query->execute();

      if ($query->rowCount() > 0) {  
         $row = $query->fetch(PDO::FETCH_OBJ);

         echo '<pre>';
         print_r($row->u_login);
         echo '</pre>';

         return true;
      }
      else {
         return false;
      }

      $query->closeCursor();
   }    
}

これが私のフォームコードです。これは、クラス メソッドを呼び出す場所です。

<?php 
   session_start();
   if (isset($_POST['login_submit'])) {
      if (!empty($_POST['login']) && !empty($_POST['password'])) {
         $auth = new auth();
         $auth->setLogin($_POST['login']);
         $auth->setPassword(sha1($_POST['password']));

         if ($auth->login('u_login,u_password,u_email,u_id_level', 'users', 'u_login', 'u_password')) {                 
            $_SESSION['back_office'] = array(
                                             'login' => $row->u_login,   // Error, $row is undefined
                                             'level' => $row->u_level,
                                             'email' => $row->u_email    
                                            );
         }
         else {
            message::showError('Compte non reconnu');
         }
      }
      else {
         message::showError('Veuillez remplir tous les champs');
      }
   }
?>

<form action="test.php" name="loginform" method="post">
   <input type="text" name="login" />
   <input type="password" name="password" />
   <input type="submit" name="login_submit" value="Se connecter" />
</form>
4

3 に答える 3

0

認証関数では、行を返す必要があります

if($query->rowCount() > 0){  
            $row = $query->fetch(PDO::FETCH_OBJ);                
            return $row;
        }else{
            return false;
        }

呼び出しで、戻り値をloginto に割り当てます$row

if (($row = $auth->login('u_login,u_password,u_email,u_id_level', 'users', 'u_login', 'u_password'))) {

于 2013-07-20T09:56:07.750 に答える
0

$rowあなたは代わりに戻らなければなりませんreturn true

$row = $query->fetch(PDO::FETCH_OBJ);                
echo '<pre>';
print_r($row->u_login);
echo '</pre>';
return $row; // return true;

そして、あなたはこのようにチェックします

if($row = $auth->login('u_login,u_password,u_email,u_id_level', 
                       'users', 'u_login', 'u_password') !== false)

次に、値を設定できます$row

$_SESSION['back_office'] = array(
     'login' => $row->u_login,
     'level' => $row->u_level,
     'email' => $row->u_email    
);
于 2013-07-20T09:59:02.237 に答える
0

$row はグローバル変数ではないため、外部からアクセスすることはできません。

その間、それをグローバルにするのは悪いことです。

たとえば、$authenticated_user という変数を入れることをお勧めします。

そのように：

class auth {
    // ...
    protected $authenticated_user;
    //...
    public function login( // ... ) {
    // ...
       $authenticated_user = $query->fetch(PDO::FETCH_OBJ); 
    // ...
    }

    public getUser()
    { return $authenticated_user; }
}

あなたの認証呼び出し部分で：

$row = $auth->getUser();

それでいいはずです。

于 2013-07-20T10:04:22.930 に答える