类实例化失败

Question

如果在实例化过程中指定了无效参数，则我的对象将无法正常退出时会遇到一些问题。 我觉得这只是我需要重新关注的一个小语法问题。 任何帮助都超过了赞赏。

class bib {
   protected $bibid;
   public function __construct($new_bibid) {
      if(!$this->bibid = $this->validate_bibid($new_bibid)) {
        echo 'me';
        return false;
      }
      //carry on with other stuff if valid bibid
    }

    private static function validate_bibid($test_bibid) {
       //call this method every time you get a bibid from the user
       if(!is_int($test_bibid)) {
            return false;
       }
       return (int)$test_bibid;
    }
 }

请注意，我在其中有一个“ echo me”行，以表明它实际上是在返回false。 我在PHP中称呼它的方式如下：

if(!$bib=new bib('612e436')) {
    echo 'Error Message';
} else {
    //do what I intend to do with the object
}

这从上方输出me，但随后继续执行else块，执行我打算对有效对象进行的操作。

谁能在其中发现我做错的任何事情？

谢谢！

Answer 1

我在这段代码中看到了几个问题。

• 首先，我认为您想执行以下操作：

  $ myBib = new bib（）; if（$ myBib-> validate_bibid（'612e436'））{..做东西..}

  （或类似的东西）

  请记住，__ construct不是正常方法。 这是一个构造函数，它不应返回任何内容。 它已经隐式返回对您创建的新实例的引用。

• 其次，您的validate_bibid返回布尔值或整数。 您不会对此立即遇到问题，但是我个人不喜欢这种样式。

• 第三，您已经声明了受保护成员$ bibid，但是您没有在任何地方设置或引用它。 我希望例如在构造函数中进行设置。 之后，您可以不带任何参数地调用validate_bibid。

这段代码显然使您感到困惑，因为它具有一些怪异的构造，因此无法正常运行。 我建议从头开始重新思考和重写这一部分。

更新：

另一个问题：

我认为这行代码不符合您的想法：

 if(!$this->bibid = $this->validate_bibid($new_bibid)) {

您可能是这个意思：

 if(!$this->bibid == $this->validate_bibid($new_bibid)) {

 // Or even better:

 if($this->bibid <> $this->validate_bibid($new_bibid)) {

Answer 2

您无法在PHP中返回构造函数-该对象仍会正常创建。

您可以使用工厂或类似的东西。

if(!$bib = Bib_Factory::loadValidBib('612e436')){
    echo 'Error Message';
} else {
    //do what I intend to do with the object
}

或者在构造函数中引发异常，并使用try catch代替if语句。

Answer 3

当然，您需要与==进行比较，而不是=是赋值操作。