1

ログインクラスとログインを含むlogin.phpを処理します。私はindex.phpにlogin.phpを含めています

検証が機能しない理由を理解できません。コードは次のとおりです。

login.php

<?php 
include_once('Database.php');

class login extends Database{
    protected $_db;

    public $_error;
    public $_password;
    public $_email;

    public function __construct(){
        $this->_db = new Database('localhost','root','','kupon') or $this->_error = 'Could not connect to database';
    }

    public function validate($email, $password){
        $query = $this->_db->query("SELECT * FROM users WHERE email='$this->_email' AND password ='$this->_password'");
        $rows = $this->_db->numrows();

        if($rows == 1){
            $result = mysql_fetch_assoc($query);
            if ($results['email'] == $email && $results['password'] == $password);
                return true;
        } else{
            $this->_error =  '<p> User Does not exists</p>';
        }
    }

}

$login = new login();


if (isset($_POST['email']) && isset($_POST['password'])){
    $email = $_POST['email'];
    $password = $_POST['password'];

    if($login->validate($email, $password))
        echo 'your in';
}

?>
                <form action="index.php" method="post" class="form">
                    <p class="email">
                        <input type="text" name="email" /> :דואר אלקטרוני</br>
                    </p>
                    <p class="password">
                        <input type="password" name="password" /> :סיסמא</br>
                    </p>
                    <p class="submit">  
                        <input type="submit" value="היכנס" />  
                    </p>  
                </form>

そして私はそれをindex.phpに含めるだけです

            <?php
                include_once ('php/ooplogin.php');
            ?>

ここで何が欠けていますか？

4

1 に答える 1

1

ここでユーザーを検証します：

$query = $this->_db->query("SELECT * FROM users WHERE email='$this->_email' AND password ='$this->_password'");

そして、あなたはここでまったく同じことをしようとしています：

  if ($results['email'] == $email && $results['password'] == $password);
                return true;

関数は次のようになります。

 public function validate($email, $password){
        // you should escape $email and $password to prevent SQL injection!!!
        $query = $this->_db->query("SELECT * FROM users WHERE email='$email' AND passwoird='$password'");
        $rows = $this->_db->numrows();
        return !!$rows;
  }

そして、そこにはセキュリティの問題があります。この記事を読んでください：

そして、私はあなたがここで何を成し遂げようとしているのかわかりません：

class login extends Database{
public function __construct(){
     $this->_db = new Database(...

Databaseクラスを拡張する場合は、次のようにします。

class login extends Database{
  public function __construct() {
    parent::__construct('localhost','root','','kupon') or $this->_error = 'Could not connect to database';
  }
  public function foobar() {
     $query = $this->query("SELECT * FROM foo");
  }
于 2012-08-25T10:50:30.953 に答える