我是 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;
}
}
问题是你在构造函数中返回一个对象。这不是构造函数的工作方式。
我有两个选择给你:
-
创建返回 NormalUser 实例的静态方法(而不是使用构造函数):
// In your NormalUser class. static function createNormalUser($loginarray) { $email = $loginarray['email']; $pass = $loginarray['pass']; return parent::login($email,$pass); }
-
或者你坚持使用构造函数,但不返回对象实例,而是将其属性分配给
$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
行为的猜测是正确的,那应该会产生你想要的结果。