从 empty() 函数调用 __isset() 魔术方法，魔术方法返回 true 但实际上它应该返回 false

Question

class Base
{
  private $firstName;
  private $lastName;

  function setName($firstName, $lastName){
    $this->firstName = $firstName;
    $this->lastName = $lastName;
  }

  function __isset($property){
    return empty($this->$property);
  }

}

$ob = new Base();
$ob->setName("Hello", "world");

if (empty($ob->firstName)) {
  echo "exist";
} else {
  echo "not exist";
}

// Output: exist

我不明白为什么输出是exist。

魔术方法返回true，但实际上它应该返回false，因为我已经设置了属性。

Answer 1

嗯。我不得不承认这也让我有点吃惊。

首先，让我们把重点放在一边：

• 使用empty() 和isset() 可能很危险，而且不是一个好习惯。由于这些语言结构不是类型安全的，并且在需要时强制使用值，因此结果可能会导致意外错误。使用更具描述性和类型安全的选项通常更安全。将默认值设置为空字符串甚至null，并与之比较，等等。

• 您问题中的代码看起来不太实用，我无法想象有很多这样的用例。通常情况下，使用魔法方法并不是一个好主意，而使用类型化属性和/或类型化访问器（get/set 方法）是一个好主意。我猜您正在尝试创建一个父类，您将在其中添加一些其他继承类将利用的魔术逻辑，这很好......但我并不特别喜欢。

• 您的__isset() 实现存在固有缺陷，因为它在属性上调用empty()。想一想：当empty() 返回false（属性非为空）时，__isset() 也返回false。与您所期望的相反。修复很简单，只需返回相反的内容：return !empty($this->$property);。

但是，即使您应用该修复程序，您的代码也不会按预期工作。这就是我发现有点令人惊讶的一点......

问题是，是的，在不可访问的属性上调用 empty() 将调用魔术方法 __isset()（如果已定义）。

如果__isset() 返回false，empty() 返回真。如果__isset() 返回true，则检查__get() 的值是否为“假”（例如，空字符串、0、'0'、null、false 或空数组）全部返回true)。但如果魔法__get() 不存在，则该值无法访问，empty() 返回true。

因此，您还需要定义魔术访问器__get()。你的课程，“固定”，相当于like this：

class Base
{
  private $firstName;
  private $lastName;

  function setName($firstName, $lastName){
    $this->firstName = $firstName;
    $this->lastName = $lastName;
  }

  function __isset($property){
    return !empty($this->$property);
  }
  
  function __get($property) {
      return $this->$property;
  }

}

$ob = new Base();
$ob->setName("Hello", "world");

if(empty($ob->firstName)){
  echo "empty";
}
else{
  echo "not empty";
}

请注意，您需要两种方法。只定义魔法吸气剂也会失败，因为empty() 首先调用魔法__isset()，然后才（如果它返回true）魔法吸气剂。

Answer 2

查看您用于字段的访问修饰符，两者都设置为 private = 不能直接从类外部访问它，为此您将需要所谓的 getter -simple explanation：

class Base{
  private $firstName;
  private $lastName;

  function setName($firstName, $lastName){
    $this->firstName = $firstName;
    $this->lastName = $lastName;
  }
  
  function getFirstName (){
      return $this->firstName;
  }

  function getLastName (){
      return $this->lastName;
  }

  function __isset($property){
    return empty($this->$property);
  }

}

$ob = new Base();
$ob->setName("Hello", "world");

if(empty($ob->getFirstName())){
  echo "exist";
}
else{
  echo "not exist";
}