对象未按预期启动 PHP OOP


object not initiating as intended PHP OOP

我是 php 和 OOP 的新手,所以决定将我现有的网站变成 oo php 以获得一些经验。这是我目前面临的问题。

我有一个从用户扩展的类普通用户。User 类内部是一个登录方法,该方法返回一个包含 Users 类属性数组的对象。

class NormalUser extends User {
public function __construct($loginarray){
    $email = $loginarray['email'];
    $pass = $loginarray['pass'];
    var_dump ( parent::login($email,$pass) );
    //return parent::login($email,$pass)
}
}

做var_dump(如上所述)时,我得到

布尔(假) 对象(用户)#8 (4){ ["user_id"]=> 字符串(1) "1" ["first_name"]=> 字符串(7) "梅兰妮" ["last_name"]=> 字符串(6) "Janson" ["user_level"]=> 字符串(1) "1" }

然而,在我的登录名.php页面中调用代码后:

$postdata = User::mysqli_array_escape($_POST);      
    $email = $postdata['email'];
    $pass = $postdata['pass'];
    $userstart = new NormalUser($postdata);
    var_dump($userstart);

我认为将$userstart调用为新的 NormalUser 对象会返回正确的变量,但是当我执行var_dump(如上所述)时,我收到以下内容:

对象(普通用户)#5 (4){ ["user_id"]=> NULL ["first_name"]=> NULL ["last_name"]=> NULL ["user_level"]=> NULL }

我不明白我哪里做错了,如果有人能够指出我正确的方向,我将不胜感激。

谢谢。

已编辑:根据要求添加了用户.php登录方法

    public static function login($email, $pass) {
    global $database;
    $sql = "SELECT user_id, first_name, last_name, user_level FROM users WHERE (email='$email' AND pass=SHA1('$pass')) AND active IS NULL LIMIT 1";
    $results = self::find_by_sql($sql);
    //var_dump($results);
    if (!empty($results)) {
        return array_shift($results);
    } else {
        return false;
    }
}

问题是你在构造函数中返回一个对象。这不是构造函数的工作方式。

我有两个选择给你:

  1. 创建返回 NormalUser 实例的静态方法(而不是使用构造函数):

    // In your NormalUser class.
    static function createNormalUser($loginarray) {
      $email = $loginarray['email'];
      $pass = $loginarray['pass'];
      return parent::login($email,$pass);
    }
    
  2. 或者你坚持使用构造函数,但不返回对象实例,而是将其属性分配给$this实例:

    function __construct($loginarray) {
      $email = $loginarray['email'];
      $pass = $loginarray['pass'];
      $temp = parent::login($email,$pass);
      $this->user_id = $temp->user_id;
      // etc...
    }
    

new NormalUser()创建一个NormalUser对象,该对象与User::login()返回的User对象完全不同。User对象会立即丢失,因为您没有将其分配给任何内容。

一种解决方案是手动分配每个字段:

class NormalUser extends User {
 public function __construct($loginarray){
  $email = $loginarray['email'];
  $pass = $loginarray['pass'];
  if($base = parent::login($email,$pass)) {
   $fields = array('user_id', 'first_name', 'last_name', 'user_level');
   foreach($fields as $fld) {
    $this->$fld = $base->$fld;
   }
  }
 }
}

但这对我来说是不对的。另一种选择是:

$email = $postdata['email'];
$pass = $postdata['pass'];
$userstart = NormalUser::login($postdata);
var_dump($userstart);

如果我对你find_by_sql行为的猜测是正确的,那应该会产生你想要的结果。